Thread (183 messages) 183 messages, 11 authors, 2022-01-29

Re: [PATCH v8 27/40] x86/boot: Add Confidential Computing type to setup_data

From: Dave Hansen <hidden>
Date: 2021-12-13 15:08:14
Also in: linux-coco, linux-efi, linux-mm, lkml, platform-driver-x86

On 12/13/21 6:49 AM, Brijesh Singh wrote:
quoted
I was more concerned that this structure could change sizes if it were
compiled on 32-bit versus 64-bit code.  For kernel ABIs, we try not to
do that.

Is this somehow OK when talking to firmware?  Or can a 32-bit OS and
64-bit firmware never interact?
For SNP, both the firmware and OS need to be 64-bit. IIRC, both the
Linux and OVMF do not enable the memory encryption for the 32-bit.
Could you please make the structure's size invariant?  That's great if
there's no problem in today's implementation, but it's best no to leave
little land mines like this around.  Let's say someone copies your code
as an example of something that interacts with a firmware table a few
years or months down the road.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help