[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 18/22] hmp: synchronize cpu state for lapic info
From: |
Woodhouse, David |
Subject: |
Re: [PATCH v4 18/22] hmp: synchronize cpu state for lapic info |
Date: |
Thu, 26 Oct 2023 16:50:09 +0000 |
> From: Dongli Zhang <dongli.zhang@oracle.com>
>
> 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.
>
> Cc: Joe Jin <joe.jin@oracle.com>
> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
> Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
Reviewed-by: David Woodhouse <dwmw@amazon.co.uk>
I spent a while staring at stale data from 'info lapic 1' this week
before realising. This fix would have been nice.
smime.p7s
Description: S/MIME cryptographic signature
Amazon Development Centre (London) Ltd.Registered in England and Wales with registration number 04543232 with its registered office at 1 Principal Place, Worship Street, London EC2A 2FA, United Kingdom.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v4 18/22] hmp: synchronize cpu state for lapic info,
Woodhouse, David <=