On Mon, Aug 31, 2020 at 07:09:37PM +0200, Borislav Petkov wrote:
> On Mon, Aug 24, 2020 at 10:55:04AM +0200, Joerg Roedel wrote:
> > + /* Check if AP Jump Table is non-zero and page-aligned */
> > + if (!jump_table_addr || jump_table_addr & ~PAGE_MASK)
> > + return 0;
>
> I think you need to return !0 here so that the panic() below fires with
> a modified message:
>
> panic("Failed to get/update SEV-ES AP Jump Table");
>
> or are we gonna boot an UP guest still?
Right, this needs a !0 return value. This code runs once on the boot CPU
before the kernel starts the secondary CPUs, so a panic() is the right
thing to do if this fails.
Regards,
Joerg
