linux-kernel - Re: [PATCH 1/2] x86/sev/vc: fix efi runtime instruction emulation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250529230233.GEaDjniaXGlxAU0NzA@fat_crate.local>
Date: Fri, 30 May 2025 01:02:33 +0200
From: Borislav Petkov <bp@...en8.de>
To: Gerd Hoffmann <kraxel@...hat.com>
Cc: Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar <mingo@...hat.com>,
	Dave Hansen <dave.hansen@...ux.intel.com>, x86@...nel.org,
	"H. Peter Anvin" <hpa@...or.com>, kvm@...r.kernel.org,
	linux-coco@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] x86/sev/vc: fix efi runtime instruction emulation

On Wed, May 28, 2025 at 09:38:24AM +0200, Gerd Hoffmann wrote:
> Use case is coconut-svsm providing an uefi variable store and edk2
> runtime code doing svsm protocol calls to send requests to the svsm
> variable store.  edk2 needs a caa page mapping and a working rdmsr
> instruction for that.
> 
> Another less critical but useful case is edk2 debug logging to qemu
> debugcon port.  That needs a working cpuid instruction because edk2
> uses that to figure whenever sev is active and adapt ioport access
> accordingly.

Yeah, I'd like for those justifications be in the commit messages please.

> > We'd like to add them to our test pile.
> 
> That is a bit difficult right now because there are a number of pieces
> which need to fall into place before this is easily testable.  You need:
> 
>  * host kernel with vmplanes patch series (for snp vmpl support).
>  * coconut svsm with uefi variable store patches.
>  * edk2 patches so it talks to svsm for variable access.
>  * igvm support patches for qemu.
> 
> Hope I didn't forgot something ...

So why are you sending those for the kernel now is so many other things are
still moving?

What if something in those things change? Then you need to touch those
again...

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette