Re: [PATCH v3 3/7] sysctl: Make topologyinfo and numainfo sysctls a little more efficient

[Boris Ostrovsky <boris.ostrovsky%oracle.com@localhost>]

On 02/16/2015 09:08 AM, Dario Faggioli wrote:
> On Mon, 2015-02-09 at 15:04 -0500, Boris Ostrovsky wrote:
> > Currently both of these sysctls make a copy to userspace for each index of
> > various query arrays. We should try to copy whole arrays instead.
> >
> > This requires some changes in sysctl's public data structure, thus bump
> > interface version.
> >
> > Report topology for all present (not just online) cpus.
> >
> > Rename xen_sysctl_topologyinfo and XEN_SYSCTL_topologyinfo to reflect the fact
> > that these are used for CPU topology. Subsequent patch will add support for
> > PCI topology sysctl.
> >
> > Clarify some somments in sysctl.h.
> >
> > Signed-off-by: Boris Ostrovsky <boris.ostrovsky%oracle.com@localhost>
> AFAIUI, this patch will change, so I'll look at it again in next
> version. What I've seen here seemed all good to me.

Jan and Andrew don't want to make changes that I describe in the first 
paragraph so that will probably be different in the next version.

-boris