Re: [PATCH v6 4/9] KVM: selftests: Add VMGEXIT helper

[Pratik Rajesh Sampat]

On 2/11/25 7:59 PM, Sean Christopherson wrote:
> On Mon, Feb 03, 2025, Pratik R. Sampat wrote:
>> Abstract rep vmmcall coded into the VMGEXIT helper for the sev
>> library.
>>
>> No functional change intended.
>>
>> Reviewed-by: Pankaj Gupta <pankaj.gupta@xxxxxxx>
>> Tested-by: Srikanth Aithal <sraithal@xxxxxxx>
>> Signed-off-by: Pratik R. Sampat <prsampat@xxxxxxx>
>> ---
>> v5..v6:
>>
>> * Collected tags from Pankaj and Srikanth.
>> ---
>>  tools/testing/selftests/kvm/include/x86/sev.h    | 2 ++
>>  tools/testing/selftests/kvm/x86/sev_smoke_test.c | 2 +-
>>  2 files changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/tools/testing/selftests/kvm/include/x86/sev.h b/tools/testing/selftests/kvm/include/x86/sev.h
>> index 82c11c81a956..e7df5d0987f6 100644
>> --- a/tools/testing/selftests/kvm/include/x86/sev.h
>> +++ b/tools/testing/selftests/kvm/include/x86/sev.h
>> @@ -27,6 +27,8 @@ enum sev_guest_state {
>>  
>>  #define GHCB_MSR_TERM_REQ	0x100
>>  
>> +#define VMGEXIT()		{ __asm__ __volatile__("rep; vmmcall"); }
> 
> Please make this a proper inline function, there's no reason to use a macro.

Sure, I can make this an inline function.