Re: [PATCH v3 1/1] hmp: synchronize cpu state for lapic info

[Thomas Huth]

On 02/11/2023 07.00, Markus Armbruster wrote:
> Dongli Zhang <dongli.zhang@oracle.com> writes:
>
> > Hi Juan,
> >
> > On 10/30/23 09:31, Juan Quintela wrote:
> > > Dongli Zhang <dongli.zhang@oracle.com> wrote:
> > > > While the default "info lapic" always synchronizes cpu state ...
> > > >
> > > > mon_get_cpu()
> > > > -> mon_get_cpu_sync(mon, true)
> > > >     -> cpu_synchronize_state(cpu)
> > > >        -> ioctl KVM_GET_LAPIC (taking KVM as example)
> > > >
> > > > ... the cpu state is not synchronized when the apic-id is available as
> > > > argument.
> > > >
> > > > The cpu state should be synchronized when apic-id is available. Otherwise
> > > > the "info lapic <apic-id>" always returns stale data.
> > > >
> > > > Reference:
> > > > https://urldefense.com/v3/__https://lore.kernel.org/all/20211028155457.967291-19-berrange@redhat.com/__;!!ACWV5N9M2RV99hQ!KOLfuCesLC4T6ka9bjf4x6ncC34GPK9pVvWwOJhbwSZw2fwp3Mxlakk0fnR-NCoqRPKOX7X4SOAxozQBC7VQ$
> > > >
> > > > Cc: Joe Jin <joe.jin@oracle.com>
> > > > Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
> > > > Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
> > > > Reviewed-by: David Woodhouse <dwmw@amazon.co.uk>
> > >
> > > Reviewed-by: Juan Quintela <quintela@redhat.com>
> > >
> > > But I wonder how I did get CC'd on this patch O:-)
> >
> > Thank you very much!
> >
> > This component does not have a maintainer. I just blindly cc all suggested
> > reviewers :), in order to get it reviewed and merged.
> >
> > get_maintainer.pl: No maintainers found, printing recent contributors.
> > get_maintainer.pl: Do not blindly cc: them on patches!  Use common sense.
>
> I posted a fix the other day:
>
>      Subject: [PATCH] MAINTAINERS: Fully cover target/i386/*
>      Message-ID: <20231030085336.2681386-1-armbru@redhat.com>
>      https://lore.kernel.org/all/20231030085336.2681386-1-armbru@redhat.com/

I wrote a slightly better patch a month ago already (since these files do 
not only belong to TCG, but also to the other accelerators):

 https://lore.kernel.org/qemu-devel/20230929134551.395438-1-thuth@redhat.com/

Unfortunately, it got ignored so far :-(

 Thomas