From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Date: Mon, 23 Mar 2020 20:00:50 +0000 Subject: Re: [PATCH v3 4/9] KVM: VMX: Configure runtime hooks using vmx_x86_ops Message-Id: List-Id: References: <20200321202603.19355-1-sean.j.christopherson@intel.com> <20200321202603.19355-5-sean.j.christopherson@intel.com> <87ftdz9ryn.fsf@vitty.brq.redhat.com> In-Reply-To: <87ftdz9ryn.fsf@vitty.brq.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Vitaly Kuznetsov , Sean Christopherson Cc: Paul Mackerras , Christian Borntraeger , Janosch Frank , David Hildenbrand , Cornelia Huck , Wanpeng Li , Jim Mattson , Joerg Roedel , Marc Zyngier , James Morse , Julien Thierry , Suzuki K Poulose , linux-mips@vger.kernel.org, kvm@vger.kernel.org, kvm-ppc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, kvmarm@lists.cs.columbia.edu, linux-kernel@vger.kernel.org On 23/03/20 13:27, Vitaly Kuznetsov wrote: >> - kvm_x86_ops->check_nested_events = vmx_check_nested_events; >> - kvm_x86_ops->get_nested_state = vmx_get_nested_state; >> - kvm_x86_ops->set_nested_state = vmx_set_nested_state; >> - kvm_x86_ops->get_vmcs12_pages = nested_get_vmcs12_pages; >> - kvm_x86_ops->nested_enable_evmcs = nested_enable_evmcs; >> - kvm_x86_ops->nested_get_evmcs_version = nested_get_evmcs_version; >> + ops->check_nested_events = vmx_check_nested_events; >> + ops->get_nested_state = vmx_get_nested_state; >> + ops->set_nested_state = vmx_set_nested_state; >> + ops->get_vmcs12_pages = nested_get_vmcs12_pages; >> + ops->nested_enable_evmcs = nested_enable_evmcs; >> + ops->nested_get_evmcs_version = nested_get_evmcs_version; > > A lazy guy like me would appreciate 'ops' -> 'vmx_x86_ops' rename as it > would make 'git grep vmx_x86_ops' output more complete. > I would prefer even more a kvm_x86_ops.nested struct but I would be okay with a separate patch. Paolo