Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <129a2714-bad3-6f91-d841-51121bcc6e8d@intel.com>
Date: Wed, 10 Nov 2021 09:54:27 +0800
From: Yin Fengwei <fengwei.yin@...el.com>
To: Kees Cook <keescook@...omium.org>, kernel test robot
	<oliver.sang@...el.com>
CC: Mickaël Salaün <mic@...ikod.net>,
	<lkp@...ts.01.org>, <lkp@...el.com>, <ying.huang@...el.com>,
	<feng.tang@...el.com>, <zhengjun.xing@...ux.intel.com>, Al Viro
	<viro@...iv.linux.org.uk>, Andrew Morton <akpm@...ux-foundation.org>, "Aleksa
 Sarai" <cyphar@...har.com>, Andy Lutomirski <luto@...nel.org>, Arnd Bergmann
	<arnd@...db.de>, Casey Schaufler <casey@...aufler-ca.com>, Christian Brauner
	<christian.brauner@...ntu.com>, Christian Heimes <christian@...hon.org>,
	Deven Bowers <deven.desai@...ux.microsoft.com>, Dmitry Vyukov
	<dvyukov@...gle.com>, Eric Biggers <ebiggers@...nel.org>, Eric Chiang
	<ericchiang@...gle.com>, Florian Weimer <fweimer@...hat.com>, "Geert
 Uytterhoeven" <geert@...ux-m68k.org>, James Morris <jmorris@...ei.org>, "Jan
 Kara" <jack@...e.cz>, Jann Horn <jannh@...gle.com>, Jonathan Corbet
	<corbet@....net>, Lakshmi Ramasubramanian <nramas@...ux.microsoft.com>,
	"Madhavan T . Venkataraman" <madvenka@...ux.microsoft.com>, Matthew Garrett
	<mjg59@...gle.com>, Matthew Wilcox <willy@...radead.org>, Miklos Szeredi
	<mszeredi@...hat.com>, Mimi Zohar <zohar@...ux.ibm.com>, Paul Moore
	<paul@...l-moore.com>, Philippe Trébuchet
	<philippe.trebuchet@....gouv.fr>, Scott Shell <scottsh@...rosoft.com>, "Shuah
 Khan" <shuah@...nel.org>, Steve Dower <steve.dower@...hon.org>, Steve Grubb
	<sgrubb@...hat.com>, Thibaut Sautereau <thibaut.sautereau@....gouv.fr>,
	Vincent Strubel <vincent.strubel@....gouv.fr>,
	<kernel-hardening@...ts.openwall.com>, <linux-api@...r.kernel.org>,
	<linux-fsdevel@...r.kernel.org>, <linux-integrity@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>, <linux-security-module@...r.kernel.org>,
	Mickaël Salaün <mic@...ux.microsoft.com>
Subject: Re: [fs] a0918006f9: netperf.Throughput_tps -11.6% regression

Hi Kees,

On 11/10/21 1:21 AM, Kees Cook wrote:
>>     555600            -0.1%     555305        netperf.Throughput_total_tps
>>      34725            -0.1%      34706        netperf.Throughput_tps
>>
>>
>> Fengwei also helped review these results and commented:
>> I suppose these three CPUs have different cache policy. It also could be
>> related with netperf throughput testing.
> Does moving the syscall implementation somewhere else change things?
> That's a _huge_ performance change for something that isn't even called.
> What's going on here?

We just tried to do trick change to make sure the new function doesn't
make other kernel function address changed. But didn't try to move around
the new function even it's not called. We could try to move it around to
see the impact to netperf throughput.


We tried the original patch (without change to make sure no kernel function
address patch) on other box. As report, the regression are different on
different CPUs like:
       -11.6% vs -5.7% vs 0.1%

So my guess is that the different CPUs on these test box have different
cache policies which cause different performance impact when kernel
functions/data address are different. 

Yes. This is strange. We don't know exact reason. Need deeper investigation
on this.


Regards
Yin, Fengwei

> 
> -Kees
> 

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.