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

Re: [Xen-devel] [PATCH V2] tools/libxc, xen/x86: Added xc_set_mem_access_multi()



On Sep 2, 2016 09:03, "Jan Beulich" <JBeulich@xxxxxxxx> wrote:
>
> >>> On 02.09.16 at 16:50, <tamas@xxxxxxxxxxxxx> wrote:
> > On Sep 2, 2016 05:45, "Jan Beulich" <JBeulich@xxxxxxxx> wrote:
> >>
> >> >>> On 02.09.16 at 13:18, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
> >> > On 09/02/2016 01:02 PM, Jan Beulich wrote:
> >> >>>>> On 02.09.16 at 10:51, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
> >> >>> Changes since V1 / RFC:
> >> >>>  - Renamed xc_set_mem_access_sparse() to xc_set_mem_access_multi(),
> >> >>>    and XENMEM_access_op_set_access_sparse to
> >> >>>    XENMEM_access_op_set_access_multi.
> >> >>>  - Renamed the 'nr' parameter to 'size'.
> >> >>
> >> >> Why?
> >> >
> >> > Tamas suggested it, size sounded more intuitive to him. I have no
> >> > problem with either nr or size.
> >>
> >> Size to me means something in bytes, which clearly isn't the case
> >> here. There's not even support for other then 4k pages so far.
> >
> > Lets make it array_size then to clarify?
>
> What's wrong with "nr", matching the other (existing) function?
>

IMHO it's too generic. So either a more descriptive name or a comment is warranted to explain the inputs.

Tamas

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

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