Messages in this thread | | | Date | Fri, 17 May 2024 17:58:58 +0200 | From | Borislav Petkov <> | Subject | Re: [PATCH v4 04/15] x86/sev: Check for the presence of an SVSM in the SNP Secrets page |
| |
On Thu, May 02, 2024 at 10:29:02AM -0500, Tom Lendacky wrote: > PAGE_ALIGNED and IS_ALIGNED are from two separate header files (mm.h and > align.h) which seems like a lot of extra changes for just one check.
No, pls put them in a single shared/mm.h header. And no, those are not a lot of extra changes - those are changes which are moving the code in the right direction and we do them sooner rather than later, otherwise they'd pile up and we'll never be able to find time to do them - sev.c movement attempt case-in-point.
> Not sure I agree. I'd prefer to keep the comment here because it is > specific to this rmpadjust() call. See below.
Just don't replicate some versions of the same comment all over the place. Do one big comment which explains which RMPADJUST has to do with VMPL levels - perhaps over the insn - and then refer to it from the other places after adding the specific explanations for them.
> Right. Not sure about the "cannot", more like "must not." The specification > states that the guest should run at a VMPL other than 0. If an SVSM starts > the guest at VMPL0, then the SVSM would not be protected from guest.
Yeah, well, you do terminate the guest if it is running at VMPL 0 *in* the presence of a SVSM so it is a "must not". Ok.
-- Regards/Gruss, Boris.
https://people.kernel.org/tglx/notes-about-netiquette
| |