On Fri, Dec 16, 2022, Yu Zhang wrote: > KVM already has a 'GPA_INVALID' defined as (~(gpa_t)0) in > kvm_types.h, and it is used by ARM and X86 xen code. We do > not need a specific definition of 'INVALID_GPA' for X86. > > Instead of using the common 'GPA_INVALID' for X86, replace > the definition of 'GPA_INVALID' with 'INVALID_GPA', and > change the users of 'GPA_INVALID', so that the diff can be > smaller. Also because the name 'INVALID_GPA' tells the user > we are using an invalid GPA, while the name 'GPA_INVALID' > is emphasizing the GPA is an invalid one. > > Also, add definition of 'INVALID_GFN' because it is more > proper than 'INVALID_GPA' for GFN variables. Please wrap closer to ~75 chars, ~60 is too aggressive. > No functional change intended. > > Signed-off-by: Yu Zhang <yu.c.zhang@xxxxxxxxxxxxxxx> > Reviewed-by: Paul Durrant <paul@xxxxxxx> > --- Reviewed-by: Sean Christopherson <seanjc@xxxxxxxxxx>