From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============9197688042165971430==" MIME-Version: 1.0 From: Davidlohr Bueso To: lkp@lists.01.org Subject: Re: [locking/mutexes] cb4bbc457b: -40.0% unixbench.score Date: Thu, 21 Jan 2016 18:48:01 -0800 Message-ID: <20160122024801.GA23224@linux-uzut.site> In-Reply-To: <874me6jql0.fsf@yhuang-dev.intel.com> List-Id: --===============9197688042165971430== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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 s= pin on owner when wait list is not NULL.") I'm not sure why this would even be reported, as this patch has not been ac= cepted or acked or nothin', by anyone. In this particular case that raw performanc= e drop is because spinning is pretty much disabled by Ding's change. Totally expec= ted for the kind of workload unixbench triggers. All this does is hurt lkml-searchability. Thanks, Davidlohr --===============9197688042165971430==-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752321AbcAVCsS (ORCPT ); Thu, 21 Jan 2016 21:48:18 -0500 Received: from mx2.suse.de ([195.135.220.15]:38994 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751562AbcAVCsN (ORCPT ); Thu, 21 Jan 2016 21:48:13 -0500 Date: Thu, 21 Jan 2016 18:48:01 -0800 From: Davidlohr Bueso To: kernel test robot Cc: Ding Tianhong , lkp@01.org, LKML , Waiman Long , Tim Chen , Jason Low , Will Deacon , Thomas Gleixner , "Paul E. McKenney" , Linus Torvalds , Peter Zijlstra , Ingo Molnar , 0day robot Subject: Re: [lkp] [locking/mutexes] cb4bbc457b: -40.0% unixbench.score Message-ID: <20160122024801.GA23224@linux-uzut.site> References: <874me6jql0.fsf@yhuang-dev.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: <874me6jql0.fsf@yhuang-dev.intel.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. 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. All this does is hurt lkml-searchability. Thanks, Davidlohr