Re: [PATCH dovetail v5 0/2] Fix KVM guest support of dovetail

[Philippe Gerum <rpm@xenomai.org>]

Florian Bezdeka <florian.bezdeka@siemens.com> writes:

> Hi all, 
>
> this is hopefully the final version of the dovetail KVM guest support
> enablement. Dovetail failed to boot without patch 1 and crashed without
> patch 2 under high IO load scenarios.
>
> Tested with LOCKDEP / PROVE_LOCKING enabled. Stress-Tests passed on two
> different hypervisors, namely KVM and VirtualBox.
>
> All dovetail versions (down to 5.10) are affected. Backporting would be
> required on my end, I need 6.1 to be fixed.
>
> This patches apply down to 5.15, 5.10 would need manual backporting. I
> can look into that once this series is accepted for 6.9.
>
> CC: Jan Kiszka <jan.kiszka@siemens.com>
> CC: Philippe Gerum <rpm@xenomai.org>
>
> Signed-off-by: Florian Bezdeka <florian.bezdeka@siemens.com>
> ---
> Changes in v5:
> - Patch 2: safe_halt() -> native_safe_halt()
> - Link to v4: https://lore.kernel.org/r/20240531-flo-fix-kvm-guest-for-6-9-v4-0-77471c9473d8@siemens.com
>
> Changes in v4:
> - Patch 2: Sligthly changed the patch description 
> - Patch 2: Do not stall the inband stage in task context (handle Linux
>   IRQ context and task context the same way)
> - Link to v3: https://lore.kernel.org/r/20240531-flo-fix-kvm-guest-for-6-9-v3-0-ebcf517500a3@siemens.com
>
> Changes in v3:
> - Removed RFC tag
> - Took care of review comments from Jan (patch 2) 
> - Link to v2: https://lore.kernel.org/r/20240531-flo-fix-kvm-guest-for-6-9-v2-0-5534fcae4eb5@siemens.com
>
> Changes in v2:
> - Drop patch one 
> - Now patch 1: Handle async page faults inband, rework patch subject
> - Now patch 2: Fix kvm_wait(), PV spinlocks
> - Link to v1: https://lore.kernel.org/r/20240531-flo-fix-kvm-guest-for-6-9-v1-0-1c895c256f8d@siemens.com
>
> ---
> Florian Bezdeka (2):
>       KVM: x86: dovetail: Move handling async page faults to the inband stage
>       KVM: x86: dovetail: Allow kvm_wait() to be called from any context
>
>  arch/x86/include/asm/idtentry.h |  2 +-
>  arch/x86/kernel/irq_pipeline.c  |  6 ++++++
>  arch/x86/kernel/kvm.c           | 29 ++++++++++++++++++++++-------
>  3 files changed, 29 insertions(+), 8 deletions(-)
> ---
> base-commit: 17865a05b459df6e7e7e38018b15b29a96df49af
> change-id: 20240531-flo-fix-kvm-guest-for-6-9-5ef61aac08ad
>
> Best regards,

Merged, thanks.

-- 
Philippe.