Re: [PATCH v10 03/11] x86/cpufeatures: Add TDX Guest CPU feature

[Sathyanarayanan Kuppuswamy]

On 10/18/21 2:59 PM, Borislav Petkov wrote:
> On Mon, Oct 18, 2021 at 02:05:10PM -0700, Sathyanarayanan Kuppuswamy wrote:
> > Any consensus on this?
> Well, you can simply not put any comment there now because this patchset
> doesn't add those debug options.
>
> The patch which adds them, should add that comment explaining why that
> order needs to be the way it is.

For now I will remove the comment as you have suggested.

> In general, I get the impression from review that you add stuff for
> future patchsets which only confuses reviewers and instead, you should
> simply not do that but do only the required changes, only for the
> current aspect of functionality being added.

I will check the patch list again and re-organize the patch set, If I
find any such changes.

> > I think SME code also talks about future use case in its comment.
> No, this has nothing to do with a future use case: you should look at
> what that function does and then read that comment again.

Sorry, I will check the details correctly.

>
--
Sathyanarayanan Kuppuswamy
Linux Kernel Developer