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