linux-kernel - RE: [PATCH] ACPI: processor_idle: Skip dummy wait for processors based on the Zen microarchitecture

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <MN0PR12MB610110D90985366A4B952CCCE24E9@MN0PR12MB6101.namprd12.prod.outlook.com>
Date:   Thu, 22 Sep 2022 17:48:48 +0000
From:   "Limonciello, Mario" <Mario.Limonciello@....com>
To:     Dave Hansen <dave.hansen@...el.com>,
        "Nayak, K Prateek" <KPrateek.Nayak@....com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC:     "rafael@...nel.org" <rafael@...nel.org>,
        "lenb@...nel.org" <lenb@...nel.org>,
        "linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
        "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
        "dave.hansen@...ux.intel.com" <dave.hansen@...ux.intel.com>,
        "bp@...en8.de" <bp@...en8.de>,
        "tglx@...utronix.de" <tglx@...utronix.de>,
        "andi@...as.de" <andi@...as.de>, "puwen@...on.cn" <puwen@...on.cn>,
        "peterz@...radead.org" <peterz@...radead.org>,
        "rui.zhang@...el.com" <rui.zhang@...el.com>,
        "gpiccoli@...lia.com" <gpiccoli@...lia.com>,
        "daniel.lezcano@...aro.org" <daniel.lezcano@...aro.org>,
        "Narayan, Ananth" <Ananth.Narayan@....com>,
        "Shenoy, Gautham Ranjal" <gautham.shenoy@....com>,
        "Ong, Calvin" <Calvin.Ong@....com>,
        "stable@...r.kernel.org" <stable@...r.kernel.org>,
        "regressions@...ts.linux.dev" <regressions@...ts.linux.dev>
Subject: RE: [PATCH] ACPI: processor_idle: Skip dummy wait for processors
 based on the Zen microarchitecture

[Public]



> -----Original Message-----
> From: Dave Hansen <dave.hansen@...el.com>
> Sent: Thursday, September 22, 2022 12:02
> To: Nayak, K Prateek <KPrateek.Nayak@....com>; linux-
> kernel@...r.kernel.org
> Cc: rafael@...nel.org; lenb@...nel.org; linux-acpi@...r.kernel.org; linux-
> pm@...r.kernel.org; dave.hansen@...ux.intel.com; bp@...en8.de;
> tglx@...utronix.de; andi@...as.de; puwen@...on.cn; Limonciello, Mario
> <Mario.Limonciello@....com>; peterz@...radead.org;
> rui.zhang@...el.com; gpiccoli@...lia.com; daniel.lezcano@...aro.org;
> Narayan, Ananth <Ananth.Narayan@....com>; Shenoy, Gautham Ranjal
> <gautham.shenoy@....com>; Ong, Calvin <Calvin.Ong@....com>;
> stable@...r.kernel.org; regressions@...ts.linux.dev
> Subject: Re: [PATCH] ACPI: processor_idle: Skip dummy wait for processors
> based on the Zen microarchitecture
> 
> On 9/22/22 09:54, K Prateek Nayak wrote:
> >
> > On 9/22/2022 10:14 PM, Dave Hansen wrote:
> >> On 9/20/22 23:36, K Prateek Nayak wrote:
> >>> Cc: stable@...r.kernel.org
> >>> Cc: regressions@...ts.linux.dev
> >> *Is* this a regression?
> > On second thought, it is not a regression.
> > Will remove the tag on v2.
> 
> What were you planning for v2?
> 
> Rafael suggested something like the attached patch.  It's not nearly as
> fragile as the Zen check you proposed earlier.
> 
> Any testing or corrections on the commentary would be appreciated.

This seems reasonable to me.  Appreciate the suggestion.

Some small nits:
1) You reference inb() specifically in the commit message, but the code that is skipped is
actually inl().

2) The title says to limit it to old intel systems, but nothing about this actually enforces that.
It actually is limited to all Intel systems, but effectively won't be used on anything but new
ones because of intel_idle.

As an idea for #2 you could check for CONFIG_INTEL_IDLE in the Intel case and
if it's not defined show a pr_notice_once() type of message trying to tell people to use
Intel Idle instead for better performance.