[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 5/7] x86/vtx: Rename arch_vmx_struct to vmx_vcpu



> From: Andrew Cooper [mailto:andrew.cooper3@xxxxxxxxxx]
> Sent: Thursday, August 30, 2018 11:47 PM
> 
> On 30/08/18 15:54, Jan Beulich wrote:
> >>>> On 28.08.18 at 19:39, <andrew.cooper3@xxxxxxxxxx> wrote:
> >> The suffix and prefix are redundant, and the name is curiously odd.
> Rename it
> >> to vmx_vcpu to be consistent with all the other similar structures.
> >>
> >> No functional change.
> >>
> >> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> >> ---
> >> CC: Jan Beulich <JBeulich@xxxxxxxx>
> >> CC: Wei Liu <wei.liu2@xxxxxxxxxx>
> >> CC: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> >> CC: Jun Nakajima <jun.nakajima@xxxxxxxxx>
> >> CC: Kevin Tian <kevin.tian@xxxxxxxxx>
> >>
> >> Some of the local pointers are named arch_vmx.  I'm open to renaming
> them to
> >> just vmx (like all the other local pointers) if people are happy with the
> >> additional patch delta.
> > I'd be fine with that. With or without
> > Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
> 
> TBH, I was hoping for a comment from Kevin on this question.
> 
> Given that the net diffstat including the pointer renames is:
> 
> andrewcoop@andrewcoop:/local/xen.git/xen$ git d HEAD^ --stat
>  xen/arch/x86/hvm/vmx/vmcs.c        | 44
> ++++++++++++++++++++++----------------------
>  xen/arch/x86/hvm/vmx/vmx.c         |  4 ++--
>  xen/include/asm-x86/hvm/vcpu.h     |  2 +-
>  xen/include/asm-x86/hvm/vmx/vmcs.h |  2 +-
>  4 files changed, 26 insertions(+), 26 deletions(-)
> 
> I've decided to go ahead and do them, to improve the eventual code
> consistency.

yes, please go ahead. I didn't note that open earlier.

Thanks
Kevin
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.