Re: [PATCH v1 3/5] KVM: SEV: Provide support for SNP_EXTENDED_GUEST_REQUEST NAE event

From: Sean Christopherson
Date: Mon Jun 24 2024 - 11:02:27 EST


On Mon, Jun 24, 2024, Tom Lendacky wrote:
> On 6/22/24 15:28, Carlos Bilbao wrote:
> >> +/*
> >> + * As per GHCB spec (see "SNP Extended Guest Request"), the certificate table
> >> + * is terminated by 24-bytes of zeroes.
> >> + */
> >> +static const u8 empty_certs_table[24];
> >
> >
> > Should this be:
> > staticconstu8 empty_certs_table[24] = { 0};
>
> Statics are always initialized to zero, so not necessary.

Heh, the whole thing is unnecessary.

/*
* As per GHCB spec (see "SNP Extended Guest Request"), the certificate
* table is terminated by 24 bytes of zeroes.
*/
if (data_npages && kvm_clear_guest(kvm, data_gpa, 24))
goto abort_request;