>>> On 05.04.17 at 19:32, <andrew.coop...@citrix.com> wrote: > The function hvm_translate_linear_addr() translates a virtual address to a > linear address, not a linear address to a physical address. Correct its name. > > Both hvm_translate_virtual_addr() and hvmemul_virtual_to_linear() return a > linear address, but a parameter name of paddr is easily confused with paddr_t. > Rename it to linear, to clearly identify the address space, and for > consistency with hvm_virtual_to_linear_addr(). > > No functional change. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Reviewed-by: Paul Durrant <paul.durr...@citrix.com> > --- > CC: Jan Beulich <jbeul...@suse.com> > CC: Tim Deegan <t...@xen.org> > CC: Julien Grall <julien.gr...@arm.com> > > v2: > * Alter commit message, as there is disagreement as to the intended meaning > of paddr. Whatever the intended meaning, it is currently confusing against > paddr_t.
And as said on irc - you could have kept my R-b despite not doing the originally requested parameter name change, so here it is again: Reviewed-by: Jan Beulich <jbeul...@suse.com> Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
