Re: [PATCH 0/2] hw/acpi: bump MADT to revision 5

[Ani Sinha]

On Thu, 30 Mar 2023, Michael S. Tsirkin wrote:

> On Thu, Mar 30, 2023 at 01:06:36PM +0530, Ani Sinha wrote:
> >
> >
> > On Wed, 29 Mar 2023, Michael S. Tsirkin wrote:
> >
> > > On Wed, Mar 29, 2023 at 08:14:37AM -0500, Eric DeVolder wrote:
> > > >
> > > >
> > > > On 3/29/23 00:19, Michael S. Tsirkin wrote:
> > > > > Hmm I don't think we can reasonably make such a change for 8.0.
> > > > > Seems too risky.
> > > > > Also, I feel we want to have an internal (with "x-" prefix") flag to
> > > > > revert to old behaviour, in case of breakage on some guests.  and 
> > > > > maybe
> > > > > we want to keep old revision for old machine types.
> > > > Ok, what option name, for keeping old behavior, would you like?
> > >
> > > Don't much care. x-madt-rev?
> > >
> > > > >
> > > > >
> > > > > On Tue, Mar 28, 2023 at 11:59:24AM -0400, Eric DeVolder wrote:
> > > > > > The following Linux kernel change broke CPU hotplug for MADT 
> > > > > > revision
> > > > > > less than 5.
> > > > > >
> > > > > >   commit e2869bd7af60 ("x86/acpi/boot: Do not register processors 
> > > > > > that cannot be onlined for x2APIC")
> > > > >
> > > > > Presumably it's being fixed? Link to discussion? Patch fixing that in
> > > > > Linux?
> > > >
> > > > https://lore.kernel.org/linux-acpi/20230327191026.3454-1-eric.devolder@oracle.com/T/#t
> > >
> > > Great! Maybe stick a Link: tag in the commit log.
> >
> > If the original bug is in the kernel and kernel upstream has accepted both
> > your fix and Mario's patch on the acpi revision mess, I see no urgency to
> > fix this in QEMU.
> >
> > Maybe we can address this in the 8.1 development window.
>
> Why "maybe"? Eric is working on a patch I don't see why we won't
> address it.
>

I am not opposed to addressing it. I just think it might be too risky for
8.0 given that we are already past the hard feature freeze date.