Re: [PATCH v7 06/20] x86/virt/tdx: Shut down TDX module in case of error

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "peterz@xxxxxxxxxxxxx" <peterz@xxxxxxxxxxxxx>, "Hansen, Dave" <dave.hansen@xxxxxxxxx>
Subject: Re: [PATCH v7 06/20] x86/virt/tdx: Shut down TDX module in case of error
From: "Huang, Kai" <kai.huang@xxxxxxxxx>
Date: Wed, 23 Nov 2022 01:24:04 +0000
Accept-language: en-US
Cc: "kvm@xxxxxxxxxxxxxxx" <kvm@xxxxxxxxxxxxxxx>, "Luck, Tony" <tony.luck@xxxxxxxxx>, "bagasdotme@xxxxxxxxx" <bagasdotme@xxxxxxxxx>, "ak@xxxxxxxxxxxxxxx" <ak@xxxxxxxxxxxxxxx>, "Wysocki, Rafael J" <rafael.j.wysocki@xxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "Christopherson,, Sean" <seanjc@xxxxxxxxxx>, "Chatre, Reinette" <reinette.chatre@xxxxxxxxx>, "pbonzini@xxxxxxxxxx" <pbonzini@xxxxxxxxxx>, "linux-mm@xxxxxxxxx" <linux-mm@xxxxxxxxx>, "Yamahata, Isaku" <isaku.yamahata@xxxxxxxxx>, "kirill.shutemov@xxxxxxxxxxxxxxx" <kirill.shutemov@xxxxxxxxxxxxxxx>, "Shahar, Sagi" <sagis@xxxxxxxxxx>, "imammedo@xxxxxxxxxx" <imammedo@xxxxxxxxxx>, "Gao, Chao" <chao.gao@xxxxxxxxx>, "Brown, Len" <len.brown@xxxxxxxxx>, "sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx" <sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx>, "Huang, Ying" <ying.huang@xxxxxxxxx>, "Williams, Dan J" <dan.j.williams@xxxxxxxxx>
In-reply-to: <Y30fjcRlXiWFhnym@hirez.programming.kicks-ass.net>
References: <cover.1668988357.git.kai.huang@intel.com> <48505089b645019a734d85c2c29f3c8ae2dbd6bd.1668988357.git.kai.huang@intel.com> <bea49f7f-8411-662f-9fa8-97c98f61ce8b@intel.com> <Y30fjcRlXiWFhnym@hirez.programming.kicks-ass.net>
User-agent: Evolution 3.44.4 (3.44.4-2.fc36)

On Tue, 2022-11-22 at 20:14 +0100, Peter Zijlstra wrote:
> On Tue, Nov 22, 2022 at 10:57:52AM -0800, Dave Hansen wrote:
> 
> > To me, this starts to veer way too far into internal implementation details.
> > 
> > 	Issue the TDH.SYS.LP.SHUTDOWN SEAMCALL on all BIOS-enabled CPUs
> > 	to shut down the TDX module.
> 
> We really need to let go of the whole 'all BIOS-enabled CPUs' thing.

As replied in another email I'll follow up with TDX module guys to see whether
we can remove/ease such limitation.

Thanks!

References:
- [PATCH v7 00/20] TDX host kernel support
  - From: Kai Huang
- [PATCH v7 06/20] x86/virt/tdx: Shut down TDX module in case of error
  - From: Kai Huang
- Re: [PATCH v7 06/20] x86/virt/tdx: Shut down TDX module in case of error
  - From: Dave Hansen
- Re: [PATCH v7 06/20] x86/virt/tdx: Shut down TDX module in case of error
  - From: Peter Zijlstra

Prev by Date: Re: [PATCH v7 06/20] x86/virt/tdx: Shut down TDX module in case of error
Next by Date: Re: [PATCH] mm: memcg: fix stale protection of reclaim target memcg
Previous by thread: Re: [PATCH v7 06/20] x86/virt/tdx: Shut down TDX module in case of error
Next by thread: [PATCH v7 07/20] x86/virt/tdx: Do TDX module global initialization
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]