Received-SPF: pass (google.com: domain of linux-kernel+bounces-170468-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <3e77e370-5b30-4417-93e3-7e2cb7ed22fb@intel.com>
Date: Tue, 7 May 2024 10:13:45 +1200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 5/5] x86/virt/tdx: Export global metadata read
 infrastructure
To: Dave Hansen <dave.hansen@intel.com>, <linux-kernel@vger.kernel.org>,
	<kvm@vger.kernel.org>
CC: <x86@kernel.org>, <kirill.shutemov@linux.intel.com>,
	<peterz@infradead.org>, <tglx@linutronix.de>, <bp@alien8.de>,
	<mingo@redhat.com>, <hpa@zytor.com>, <seanjc@google.com>,
	<pbonzini@redhat.com>, <isaku.yamahata@intel.com>, <jgross@suse.com>
References: <cover.1709288433.git.kai.huang@intel.com>
 <ec9fc9f1d45348ddfc73362ddfb310cc5f129646.1709288433.git.kai.huang@intel.com>
 <fd159b26-b631-43f2-8f89-23d8d719fe80@intel.com>
Content-Language: en-US
From: "Huang, Kai" <kai.huang@intel.com>
In-Reply-To: <fd159b26-b631-43f2-8f89-23d8d719fe80@intel.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QXV3K1JaOTBBL0ZYOVYwVnRVTmZKbjVqS0hsSFBPQ08vYmpFWGFUeHpFWlQ4?=
 =?utf-8?B?Z2NaRGpzazMxK0RyWnJnbFJ6SytZMGcwME9QZkREcWwxNW55R3hpdWtRYXkw?=
 =?utf-8?B?UGQ0MHQ5VEwwYzRuNUNBUktwMm10N05KYWZRNWpMaXRlb0E0YjFrRkl6cEU1?=
 =?utf-8?B?S3prVEY5aXVqWXJiU0lIYStuUDRKVUEyNXpvMDNpL0VNdjcxYXZyb0tLYzBV?=
 =?utf-8?B?dE5jaU5GelM3Q1lTRjllOUZyL1lRd3oySER3YmZEOTV6ZlptM3dvWkZtQkJE?=
 =?utf-8?B?UmRXWHkxVVV1enI1Z1J0WXp1MEdabzZhdHNQTzRPMk4wc2VSbTdLWFpuRW9k?=
 =?utf-8?B?cDlzMnlwRVc0aGw2S3MxTHVOVEt5WHRKVER0cmd1Q0E5djh3QXBCUHdwU2xV?=
 =?utf-8?B?UDRUcksrN01LNkU4RnRUWTNJdGtQK1YrakgrZ3hCS1VwZGllbmVlcUxpMm5J?=
 =?utf-8?B?YjZuaFQzNVJ0UVBER3lRa3VGd0lnWlZHRnFDVEdEZkJhd0FpR3R2Mnp3QXVh?=
 =?utf-8?B?a1BlSzEvSE9PMnRsRHhaTFIrKzhHOFZyOWN5bjIrUUttTUFjODFGVytEd2h3?=
 =?utf-8?B?WC95RUt6eHpEb0wwQXFYOFg1RDFWYm5LY2VNcWRlUU5sZUJiNGJQZ1hDZi8w?=
 =?utf-8?B?RnROSmg3djFBNVdaQmN4Y0VDOWhzYWcvN2RMZUxxcDBTWmxHV0dmQy9KZCtI?=
 =?utf-8?B?UEt0RTdmZC9DNUVPWk1PejNaL0s0bTZzdmxpamxaRDlBY3NSSlFiWjk0SzI5?=
 =?utf-8?B?RitXSkJCNWsyVGxjRlBkTWY2RjBGUFowQ3hrZUl6ZFEzT0FTUm5oOWVITldw?=
 =?utf-8?B?aHVoMUdNcm96Q2t4R2YxbENucDdoVlBZaUhxeDNpbS9UbW04ZzFycjFjZVlK?=
 =?utf-8?B?NEl6MkxJUStwYWJpbUF1NkZFcVFsZGgxYW1zV00rTDZNR1RZMTNMOEoveG9P?=
 =?utf-8?B?cG5WWUZLQmxQNGZRZm56eHMzQ0tMNVZSTWY1SHdEMnlWSFN4Qlgvb3hpbjht?=
 =?utf-8?B?R0FMbzBsNlRYRnVHYUUzUk9MUkQ0ZjVrRnFjRENNbDAzVEpEanpYc1FTZ1FD?=
 =?utf-8?B?UXg5VHA3S2dZNzV2RTNZY3l4RVQ3UEU1MGZ0SXR3YitCWjh0K0VvZFZlN1Nw?=
 =?utf-8?B?ZTBON1BUMy9VM1FzTDRnbUF2WlZSalRNOGhWQi9ySy9ZWEV5M2lMSk5BRU9M?=
 =?utf-8?B?TXEvWFFBb2x0ZU5tdG1uWWorOFVoYnBlemVRS09iWE9kUjFjcjlaaDkxMlhn?=
 =?utf-8?B?NXJCMnVZT2lCd0JXZ3diakU5VTl1Wmo3cmo3eE51SWoyNkhObUpYODNoQVFJ?=
 =?utf-8?B?NmlQMkdURkw2Tk9Sa3RyQmZ5eEE5Z3RKNWhRZ1Fvc0hISXB4OUMzZVBTQ1VH?=
 =?utf-8?B?cUFiUUZGNDBVZ2xmSGFRbmVoSUhBakc0US9sUC91SGZvOTArRFFJZXhTWjIv?=
 =?utf-8?B?clZIT1Iwa1ZlSDk5cGk5SWwzUDZCU3J6TFMvdlgyY0VEVTdjTFFKYWZKcE9n?=
 =?utf-8?B?ZW8wbm1GT2o0SU1sMWMvQzh5T0pvM014Y09ZY1RtTUxpbmliRHVHU0h4QWw0?=
 =?utf-8?B?TW1ZZHNzZzhMVkhDY2Z3NHRNM1QxS2wrcmpjaDVMMFFRandoYVJvcUNhc3l4?=
 =?utf-8?B?SFpjdUtscWVLVzQvVDJ3WWVjK2d0bUZHYmszQWIxRDNON0RGMGdSdGRBV1hG?=
 =?utf-8?B?Y043TnlpWGVLTUw0Qk10Qkp0ZzBock9pdXIyN0dhdUxrSytxZm9MVklyS05J?=
 =?utf-8?B?QlRPVUJBYjk0TlQvQ09VWHdSTXZDcmxMSUUrbFNtVjdzVDFCRUpqTzNlZnJP?=
 =?utf-8?B?ckplbVVJUk5PNXh3QWlMR1AxNUJjVkhRdEF5MFpTWlNzdmwyN2wzWHVPSTVa?=
 =?utf-8?B?aDRad0E4dlNqeFgyeForRVRlaTdaa3hrL2tWNnBOdFViTFhheVFwZ3pXbnR2?=
 =?utf-8?B?MmxEUWtYUU41VWtyQzNaM294VzRDOGRDc1FtSWpyN1ZKcGNhVHpUTDN6V0xT?=
 =?utf-8?B?bXRya1U4WGpDMVRFQ3VQdFVnN21XNTNIWVNpSXl4V3NYaCtzby8vOWVxc3lm?=
 =?utf-8?B?M1B6UzgyRG9MYkxxeU82UDBHa1RnSFRRMTFsSDFWSVBpVE4wem0xb2k2bXRG?=
 =?utf-8?Q?dn7KiCSs5CL0XAVMX4+SqP6Ug?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 6c9d7440-a1d1-4d06-8fa7-08dc6e19d160
X-MS-Exchange-CrossTenant-AuthSource: BL1PR11MB5978.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 May 2024 22:13:54.7876
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 98DaZGAPB/2twicskTElw0IGXqMoZqs3leyIK/EtaQxHFM0VLZwf+9jfGnF6YorIm5wBG6CK+ZCX7GW5EaAh6g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7580
X-OriginatorOrg: intel.com


On 7/05/2024 3:43 am, Dave Hansen wrote:
> On 3/1/24 03:20, Kai Huang wrote:
>> KVM will need to read a bunch of non-TDMR related metadata to create and
>> run TDX guests.  Export the metadata read infrastructure for KVM to use.
> 
> All of this hinges on how KVM ends up using this infrastructure.
> 
> I want to see the whole picture before _any_ of this gets applied.
> These 5 patches only vaguely hint at that bigger picture.
> 
> I know folks want to draw some kind of line and say "x86" over here and
> "kvm" over there.  But that inclination really doesn't help anyone get
> that big picture view.

Understood.  I'll work with Rick on this and get back to you guys.

Thanks for the feedback!