ACK: [SRU][O/N:linux-intel][PATCH v2 0/3] Can't use graphics=vnc on TDVM

[Kevin Becker]

On Fri, Mar 7, 2025 at 7:33 AM Thibault Ferrante
<thibault.ferrante at canonical.com> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/2101083
>
> [ Impact ]
>
>  * if a TDVM is created with vnc graphics enabled, the VM itself crashes, and the following entries are seen in the logs:
>
> 2024-07-11T09:19:19.469114Z qemu-system-x86_64: Failed to get registers: Input/output error 2024-07-11 09:19:19.853+0000: shutting down, reason=crashed
>
> [ Fix ]
>
>  * Backport patch 25-27 from upstream submission:
> https://lore.kernel.org/all/20250226195529.2314580-26-pbonzini@redhat.com/
>
> [ Test Plan ]
>
>  * Create a Noble (24.04) Host and Guest setup following https://github.com/canonical/tdx/
>  * Create a VM with graphics type as vnc.
>  * Power the VM on.
>  * Check for the qemu logs to find the aforementioned messages.
>
> [ Where problems could occur ]
>
>  * KVM regression
>
> [ Other Info ]
>
>  * Built and tested by intel on targeted hardware.
>  * Original bug report : https://github.com/canonical/tdx/issues/202
>
> v2:
>  * Fixed wrong patch numbering.
>
> Thibault Ferrante (3):
>   UBUNTU: SAUCE: KVM: x86/mmu: Add parameter "kvm" to
>     kvm_mmu_page_ad_need_write_protect()
>   UBUNTU: SAUCE: KVM: x86: Make cpu_dirty_log_size a per-VM value
> Yan Zhao (1):
>   UBUNTU: SAUCE: KVM: Add parameter "kvm" to kvm_cpu_dirty_log_size()
>     and its callers
>
>  arch/x86/include/asm/kvm_host.h | 11 ++++++-----
>  arch/x86/kvm/mmu/mmu.c          |  6 +++---
>  arch/x86/kvm/mmu/mmu_internal.h |  5 +++--
>  arch/x86/kvm/mmu/spte.c         |  2 +-
>  arch/x86/kvm/mmu/tdp_mmu.c      | 12 ++++++------
>  arch/x86/kvm/vmx/main.c         |  1 -
>  arch/x86/kvm/vmx/vmx.c          |  5 ++---
>  arch/x86/kvm/x86.c              |  6 +++---
>  include/linux/kvm_dirty_ring.h  | 11 ++++++-----
>  virt/kvm/dirty_ring.c           | 11 ++++++-----
>  virt/kvm/kvm_main.c             |  4 ++--
>  11 files changed, 38 insertions(+), 36 deletions(-)
>
> --

Acked-by: Kevin Becker <kevin.becker at canonical.com>