Re: [Xen-devel] [PATCH v6 2/5] sysctl: Add sysctl interface for querying PCI topology

2015-04-07 Thread Daniel De Graaf
On 04/06/2015 06:12 PM, Boris Ostrovsky wrote: Signed-off-by: Boris Ostrovsky . Acked-by: Daniel De Graaf ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v6 2/5] sysctl: Add sysctl interface for querying PCI topology

2015-04-07 Thread Boris Ostrovsky
On 04/07/2015 12:09 PM, Andrew Cooper wrote: On 06/04/15 23:12, Boris Ostrovsky wrote: Signed-off-by: Boris Ostrovsky --- Changes in v6: * Dropped continuations, the sysctl now returns after 64 iteration if necessary Why? The caller should reasonably expect a single hypercall to complete, not

Re: [Xen-devel] [PATCH v6 2/5] sysctl: Add sysctl interface for querying PCI topology

2015-04-07 Thread Andrew Cooper
On 06/04/15 23:12, Boris Ostrovsky wrote: > Signed-off-by: Boris Ostrovsky > --- > > Changes in v6: > * Dropped continuations, the sysctl now returns after 64 iteration if > necessary Why? The caller should reasonably expect a single hypercall to complete, not to have to reissue repeated to get

[Xen-devel] [PATCH v6 2/5] sysctl: Add sysctl interface for querying PCI topology

2015-04-06 Thread Boris Ostrovsky
Signed-off-by: Boris Ostrovsky --- Changes in v6: * Dropped continuations, the sysctl now returns after 64 iteration if necessary * -ENODEV returned if device is not found * sysctl's first_dev is now expected to be used by userspace to continue the query * Added XSM hooks docs/misc/xsm-flask.t