Re: [PATCH RFC v3 03/21] ACPI: processor: Register CPUs that are online, but not described in the DSDT

Russell King (Oracle)
2024-01-29T15:32:55Z

On Mon, Jan 29, 2024 at 01:03:54PM +0000, Jonathan Cameron wrote:
> I poked this on x86 - it only applies with hotplug enabled anyway so
> same result as doing the hotplug later - All possible Processor() entries
> already exist in DSDT. Hence this isn't the source of the mysterious
> broken configuration.
>
> If anyone does poke this path, the old discussion between James
> and Salil provides some instructions (mostly the thread is about
> another issue).
> https://op-lists.linaro.org/archives/list/linaro-open-discussions@op-lists.linaro.org/thread/DNAGB2FB5ALVLV2BYWYOCLKGNF77PNXS/
>
> Also on x86 a test involving smp 2,max-cpus=4 and adding cpu-id 3
> (so skipping 2) doesn't boot. (this is without Salil's QEMU patches).
> I guess there are some well known rules in there that I don't know about
> and QEMU isn't preventing people shooting themselves in the foot.
>
> As I'm concerned, drop this patch.
> If there are platforms out there doing this wrong they'll surface once
> we get this into more test farms (so linux-next). If we need this
> 'fix' we can apply it when we have a problem firmware to point at.

Now dropped.

--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!