Thread (86 messages) 86 messages, 6 authors, 2021-07-05

Re: [PATCH Part1 RFC v3 03/22] x86/sev: Save the negotiated GHCB version

From: Borislav Petkov <bp@alien8.de>
Date: 2021-06-03 19:57:25
Also in: kvm, linux-coco, linux-crypto, linux-mm, lkml, platform-driver-x86

On Wed, Jun 02, 2021 at 09:03:57AM -0500, Brijesh Singh wrote:
+/*
+ * Since feature negotiation related variables are set early in the boot
+ * process they must reside in the .data section so as not to be zeroed
+ * out when the .bss section is later cleared.
+ */
From previous review:

 ...

   *
   * GHCB protocol version negotiated with the hypervisor.
   */

You need to document what this variable is used for so add a comment
over it please.
+static u16 ghcb_version __section(".data");
+
-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help