Re: [PATCH 02/25] KVM: TDX: Define TDX architectural definitions

From: Paolo Bonzini
Date: Tue Sep 10 2024 - 12:21:52 EST


On 8/29/24 21:46, Edgecombe, Rick P wrote:
This leads to another topic that defining all the TDX structure in this
patch seems unfriendly for review. It seems better to put the
introduction of definition and its user in a single patch.

Yea.

I don't know, it's easier to check a single patch against the manual. I don't have any objection to leaving everything here instead of scattering it over multiple patches, in fact I think I prefer it.

Paolo