linux-kernel - Re: [LKP] [lkp] [locking/mutexes] cb4bbc457b: -40.0% unixbench.score

Open Source and information security mailing list archives

Message-ID: <87io2iih8w.fsf@yhuang-dev.intel.com>
Date:	Mon, 25 Jan 2016 09:23:59 +0800
From:	"Huang\, Ying" <ying.huang@...el.com>
To:	Davidlohr Bueso <dave@...olabs.net>
Cc:	Waiman Long <Waiman.Long@...com>,
	Peter Zijlstra <peterz@...radead.org>, <lkp@...org>,
	Will Deacon <will.deacon@....com>,
	LKML <linux-kernel@...r.kernel.org>,
	"Paul E. McKenney" <paulmck@...ibm.com>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Ingo Molnar <mingo@...hat.com>,
	Ding Tianhong <dingtianhong@...wei.com>,
	Thomas Gleixner <tglx@...utronix.de>, fengguang.wu@...el.com,
	Tim Chen <tim.c.chen@...ux.intel.com>,
	Jason Low <jason.low2@...com>
Subject: Re: [LKP] [lkp] [locking/mutexes] cb4bbc457b: -40.0% unixbench.score

Davidlohr Bueso <dave@...olabs.net> writes:

> On Fri, 22 Jan 2016, kernel test robot wrote:
>
>>FYI, we noticed the below changes on
>>
>>https://github.com/0day-ci/linux
>> Ding-Tianhong/locking-mutexes-don-t-spin-on-owner-when-wait-list-is-not-NULL/20160121-173317
>>commit cb4bbc457bfed6194ffab1b10c7be73b3f16ca2d ("locking/mutexes: don't spin on owner when wait list is not NULL.")
>
> I'm not sure why this would even be reported, as this patch has not been accepted
> or acked or nothin', by anyone.

Sorry for bothering.  The purpose is FYI as in the original report
email.  We test patches posted to LKML, if we found some changes related
to the patch, we will send out a report.  Hope the reviewer could
take that as information for his/her review if the report isn't totally
nonsense.

> In this particular case that raw performance drop
> is because spinning is pretty much disabled by Ding's change. Totally expected for
> the kind of workload unixbench triggers.

The report is just raw performance data, it still need people to explain
it.  Thanks a lot for your explanation.

> All this does is hurt lkml-searchability.

Sorry, I don't understand this.  You could still search the original
patch.  Could you explain a little?

Best Regards,
Huang, Ying

> Thanks,
> Davidlohr
> _______________________________________________
> LKP mailing list
> LKP@...ts.01.org
> https://lists.01.org/mailman/listinfo/lkp

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives