On 8/27/21 1:13 PM, Borislav Petkov wrote:

On Fri, Aug 27, 2021 at 01:07:59PM -0500, Brijesh Singh wrote:

quoted

Okay, works for me. The main reason why I choose the enum was to not
expose the GHCB exit code to the driver

Why does that matter?

Those definitions are present in <asm/xxx>. Somewhere I read said that
if possible a new drivers should avoid including the <asm/xxx>. This is
one of the motivation to create a new file to provide the selective
information.

Lemme ask my question again:

Is SNP available on something which is !x86, all of a sudden?

Nope

Why would you want to compile that driver on anything but x86?

Nobody should compile it for !x86. If my read about including <asm/xx>
in driver is wrong then I do not see any need for creating new header
file. I can drop the header file inclusion in next update.

thanks