RE: [PATCH v6 0/6] Support TDX guests on Hyper-V

[Dexuan Cui]

> From: Dexuan Cui <decui@xxxxxxxxxxxxx>
> Sent: Thursday, May 4, 2023 3:54 PM
> To: ak@xxxxxxxxxxxxxxx; arnd@xxxxxxxx; bp@xxxxxxxxx;
> brijesh.singh@xxxxxxx; dan.j.williams@xxxxxxxxx;
> dave.hansen@xxxxxxxxxxxxxxx; Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>;
> hpa@xxxxxxxxx; jane.chu@xxxxxxxxxx; kirill.shutemov@xxxxxxxxxxxxxxx; KY
> Srinivasan <kys@xxxxxxxxxxxxx>; linux-arch@xxxxxxxxxxxxxxx;
> linux-hyperv@xxxxxxxxxxxxxxx; luto@xxxxxxxxxx; mingo@xxxxxxxxxx;
> peterz@xxxxxxxxxxxxx; rostedt@xxxxxxxxxxx;
> sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx; seanjc@xxxxxxxxxx;
> tglx@xxxxxxxxxxxxx; tony.luck@xxxxxxxxx; wei.liu@xxxxxxxxxx; x86@xxxxxxxxxx;
> Michael Kelley (LINUX) <mikelley@xxxxxxxxxxxxx>
> Cc: linux-kernel@xxxxxxxxxxxxxxx; Tianyu Lan <Tianyu.Lan@xxxxxxxxxxxxx>;
> Dexuan Cui <decui@xxxxxxxxxxxxx>
> Subject: [PATCH v6 0/6] Support TDX guests on Hyper-V
> 
> The patchset adds the Hyper-V specific code so that a TDX guest can run
> on Hyper-V. Please review.
> 
> The v6 patchset is based on today's mainline (a1fd058b07d5).
> 
> The v6 patchset addressed Michael's comments on patch 5:
>   Removed 2 unnecessary lines of messages from the commit log.
>   Fixed the error handling path for hv_synic_alloc()/free().
>   Printed the 'ret' in hv_synic_alloc()/free().
> 
> @Michael Kelley: Can you please review patch 5?

Thanks Michael for your Reviewed-by on patch 5.
 
> @x86 maintainers:
>   If the patches look good to you, can you please take patch 1 and 2
> into the tip tree?

Hi Dave and all, could you please take a look at the patchset?

The patchset can also be viewed here:
https://lwn.net/ml/linux-kernel/20230504225351.10765-1-decui%40microsoft.com/

> @Wei Liu: I think patch 3, 4, 5, 6 should go through the Hyper-V tree
> since they change the Hyper-V code.
> 
> If you want to view the patches on github, it is here:
> https://github.com/dcui/tdx/commits/decui/mainline/v6
> 
> FYI, v1-v5 are here:
>  <snipped> on 5/23/2023
>  
> Thanks,
> Dexuan
> 
> Dexuan Cui (6):
>   x86/tdx: Retry TDVMCALL_MAP_GPA() when needed
>   x86/tdx: Support vmalloc() for tdx_enc_status_changed()
>   x86/hyperv: Add hv_isolation_type_tdx() to detect TDX guests
>   x86/hyperv: Support hypercalls for TDX guests
>   Drivers: hv: vmbus: Support TDX guests
>   x86/hyperv: Fix serial console interrupts for TDX guests
> 
>  arch/x86/coco/tdx/tdx.c            | 122
> ++++++++++++++++++++++-------
>  arch/x86/hyperv/hv_apic.c          |   6 +-
>  arch/x86/hyperv/hv_init.c          |  27 ++++++-
>  arch/x86/hyperv/ivm.c              |  20 +++++
>  arch/x86/include/asm/hyperv-tlfs.h |   3 +-
>  arch/x86/include/asm/mshyperv.h    |  20 +++++
>  arch/x86/kernel/cpu/mshyperv.c     |  43 ++++++++++
>  drivers/hv/hv.c                    |  65 ++++++++++++++-
>  drivers/hv/hv_common.c             |  30 +++++++
>  include/asm-generic/mshyperv.h     |   1 +
>  10 files changed, 300 insertions(+), 37 deletions(-)

Thanks,
-- Dexuan