linux-kernel - Re: [PATCH v7 1/1] x86/acpi, x86/boot: Add multiprocessor wake-up support

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] [day] [month] [year] [list]

Date:   Fri, 17 Sep 2021 10:08:10 -0700
From:   "Kuppuswamy, Sathyanarayanan" 
        <sathyanarayanan.kuppuswamy@...ux.intel.com>
To:     Thomas Gleixner <tglx@...utronix.de>,
        Ingo Molnar <mingo@...hat.com>,
        Rafael J Wysocki <rjw@...ysocki.net>,
        H Peter Anvin <hpa@...or.com>,
        Peter Zijlstra <peterz@...radead.org>
Cc:     Len Brown <lenb@...nel.org>,
        Kuppuswamy Sathyanarayanan <knsathya@...nel.org>,
        Sean Christopherson <seanjc@...gle.com>,
        Andi Kleen <ak@...ux.intel.com>, x86@...nel.org,
        linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
        Sean Christopherson <sean.j.christopherson@...el.com>,
        "Rafael J . Wysocki" <rafael.j.wysocki@...el.com>
Subject: Re: [PATCH v7 1/1] x86/acpi, x86/boot: Add multiprocessor wake-up
 support

Hi x86 Maintainers,

On 8/17/21 6:55 AM, Kuppuswamy, Sathyanarayanan wrote:
> Hi,
> 
> On 7/19/21 9:25 PM, Kuppuswamy Sathyanarayanan wrote:
>> As per ACPI specification r6.4, sec 5.2.12.19, a new sub
>> structure – multiprocessor wake-up structure - is added to the
>> ACPI Multiple APIC Description Table (MADT) to describe the
>> information of the mailbox. If a platform firmware produces the
>> multiprocessor wake-up structure, then OS may use this new
>> mailbox-based mechanism to wake up the APs.
>>
>> Add ACPI MADT wake table parsing support for x86 platform and if
>> MADT wake table is present, update apic->wakeup_secondary_cpu with
>> new API which uses MADT wake mailbox to wake-up CPU.
>>
>> Co-developed-by: Sean Christopherson<sean.j.christopherson@...el.com>
>> Signed-off-by: Sean Christopherson<sean.j.christopherson@...el.com>
>> Reviewed-by: Andi Kleen<ak@...ux.intel.com>
>> Reviewed-by: Rafael J. Wysocki<rafael.j.wysocki@...el.com>
>> Signed-off-by: Kuppuswamy Sathyanarayanan<sathyanarayanan.kuppuswamy@...ux.intel.com>
>> ---
>>
>> Changes since v6:
>>   * Rebased on top of v5.14-rc1.
>>
>> Changes since v5:
>>   * None ( CCed ACPI list)
>>
>> Changes since v4:
>>   * Used smp_store_release() in place of WRITE_ONCE().
>>   * Addressed some checkpatch warnings.
>>
>> Changes since v3:
>>   * Removed acpi_mp_wake_mailbox_init() and moved init code to
>>     acpi_wakeup_cpu().
>>   * Removed redundant NULL pointer check for acpi_mp_wake_mailbox.
>>   * Added comments/debug prints as per Rafael's suggestion.
>>   * Removed MADT/SVKL ACPI patches from this patchset. It will be
>>     merged via ACPICA submission.
> 
> 
> Gentle ping! Any comments on this patch?

I think I have addressed all the comments raised by the reviewers. So, Can you
please consider queuing this patch for next merge window ?

> 

-- 
Sathyanarayanan Kuppuswamy
Linux Kernel Developer