Re: [PATCH] KVM: X86: Make fpu allocation a common function

From: Vitaly Kuznetsov
Date: Tue Oct 15 2019 - 10:37:02 EST


Paolo Bonzini <pbonzini@xxxxxxxxxx> writes:

> On 15/10/19 12:53, Vitaly Kuznetsov wrote:
>> A very theoretical question: why do we have 'struct vcpu' embedded in
>> vcpu_vmx/vcpu_svm and not the other way around (e.g. in a union)? That
>> would've allowed us to allocate memory in common code and then fill in
>> vendor-specific details in .create_vcpu().
>
> Probably "because it's always been like that" is the most accurate answer.
>

OK, so let me make my question a bit less theoretical: would you be in
favor of changing the status quo? :-)

--
Vitaly