Re: [PATCH v4 01/24] x86/sev: Separate MSR and GHCB based snp_cpuid() via a callback

[Tom Lendacky <thomas.lendacky@amd.com>]

On 7/11/25 15:59, Borislav Petkov wrote:
> On Wed, Jul 09, 2025 at 10:12:48AM -0500, Tom Lendacky wrote:
>> Not sure the renaming makes it read any easier or say anything more. It
>> does add extra changes to the diff that have to be read through, though,
>> so I don't think it is beneficial.
> 
> So it really comes natural to split them into a msr_prot and a ghcb_prot
> variant. If we added a separate patch ontop that does only the renaming, then
> that would probably be more churn than necessary.

Right, they already are though:

  __sev_cpuid_hv_msr() and __sev_cpuid_hv_ghcb()

the first one meaning that the hypervisor is being called using the msr
protocol and the second one meaning that the hypervisor is being called
using the ghcb protocol.

That's why I made the comment. Just changing

  __sev_cpuid_hv_msr() to __sev_cpuid_msr_prot()

isn't saying anything more in my opinion.

Thanks,
Tom

> 
>> Maybe rename this parameter to snp_cpuid or snp_cpuid_fn or similar,
>> because it can be very confusing to see "cpuid" on its own like this.a
> 
> Yeah, that's a good point - snp_cpuid_fn clearly states that it is a function
> pointer and not *the* cpuid() function.
> 
> Thx.
>