Re: [PATCH v8 07/16] x86/sev: Move and reorganize sev guest request api
From: Borislav Petkov
Date: Mon Apr 22 2024 - 09:15:36 EST
On Thu, Feb 15, 2024 at 05:01:19PM +0530, Nikunj A Dadhania wrote:
> Subject: Re: [PATCH v8 07/16] x86/sev: Move and reorganize sev guest request api
s/api/API/g
Please check your whole patchset for proper naming/abbreviations
spelling, etc.
Another one: s/sev/SEV/g and so on. You should know the drill by now.
> For enabling Secure TSC, SEV-SNP guests need to communicate with the
> AMD Security Processor early during boot. Many of the required
> functions are implemented in the sev-guest driver and therefore not
> available at early boot. Move the required functions and provide
> API to the sev guest driver for sending guest message and vmpck
> routines.
Patches which move and reorganize must always be split: first patch(es):
you do *purely* *mechanical* move without any code changes. Then you do
the code changes ontop so that a reviewer can have a chance of seeing
what you're doing.
Thx.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette