Re: [tip:x86/apic] x86/acpi: Introduce persistent storage for cpuid <-> apicid mapping

From: Dou Liyang
Date: Sat Oct 08 2016 - 00:17:49 EST


Hi tglx,

At 10/07/2016 09:07 PM, Thomas Gleixner wrote:
On Thu, 6 Oct 2016, Dou Liyang wrote:

+ if (processor->id >= 255) {
+ ++disabled_cpus;

Incrementing disabled_cpus here is simply wrong because 0xff is an invalid
local APIC id. So we can simply return -EINVAL and be done with it.


Yes, It is.

+ return -EINVAL;

Thanks,

tglx



Thanks,

Dou