Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1584143

Re: [PATCH -v1.1] x86: Optimize clear_page()

From Borislav Petkov <bp@alien8.de>
Newsgroups linux.kernel
Subject Re: [PATCH -v1.1] x86: Optimize clear_page()
Date 2017-02-19 14:50 +0100
Message-ID <tcA2d-7ZK-7@gated-at.bofh.it> (permalink)
References <t932N-1TR-11@gated-at.bofh.it> <tb5MR-7Et-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wed, Feb 15, 2017 at 12:19:27PM +0100, Borislav Petkov wrote:
> Here's a fixed version of the breakage the 0day bot reported:
> 
> ---
> From: Borislav Petkov <bp@suse.de>
> Date: Thu, 9 Feb 2017 01:34:49 +0100
> Subject: [PATCH -v1.1] x86: Optimize clear_page()
> 
> Currently, we CALL clear_page() which then JMPs to the proper function
> chosen by the alternatives.
> 
> What we should do instead is CALL the proper function directly. (This
> was something Ingo suggested a while ago). So let's do that.
> 
> Measuring our favourite kernel build workload shows that there are no
> significant changes in performance.

...

> Signed-off-by: Borislav Petkov <bp@suse.de>
> [ Forgot to add clobbers. ]
> Reported-by: kernel test robot <fengguang.wu@intel.com>

Fengguang just confirmed too:

"Yes it works fine for 300 boots.

Tested-by: Fengguang Wu <fengguang.wu@intel.com>"

-- 
Regards/Gruss,
    Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

[PATCH -v1.1] x86: Optimize clear_page() Borislav Petkov <bp@alien8.de> - 2017-02-15 12:20 +0100
  Re: [PATCH -v1.1] x86: Optimize clear_page() Borislav Petkov <bp@alien8.de> - 2017-02-19 14:50 +0100

csiph-web