[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:17, "Razvan Cojocaru" <rcojocaru@xxxxxxxxxxxxxxx> wrote:
>
> On 09/02/2016 06:13 PM, Tamas K Lengyel wrote:
> > On Sep 2, 2016 09:03, "Jan Beulich" <JBeulich@xxxxxxxx
> > <mailto:JBeulich@xxxxxxxx>> wrote:
> >>
> >> >>> On 02.09.16 at 16:50, <tamas@xxxxxxxxxxxxx
> > <mailto:tamas@xxxxxxxxxxxxx>> wrote:
> >> > On Sep 2, 2016 05:45, "Jan Beulich" <JBeulich@xxxxxxxx
> > <mailto:JBeulich@xxxxxxxx>> wrote:
> >> >>
> >> >> >>> On 02.09.16 at 13:18, <rcojocaru@xxxxxxxxxxxxxxx
> > <mailto:rcojocaru@xxxxxxxxxxxxxxx>> wrote:
> >> >> > On 09/02/2016 01:02 PM, Jan Beulich wrote:
> >> >> >>>>> On 02.09.16 at 10:51, <rcojocaru@xxxxxxxxxxxxxxx
> > <mailto: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.
>
> If this satisfies everybody, I'll keep 'nr' and add a comment describing
> the function and parameters (which is a good thing anyway).
>
>

SGTM.

Thanks,
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®.