Re: [Xen-devel] [PATCH] xen: write information about supported backends

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Mon, 4 Apr 2016, Juergen Gross wrote:
> On 01/04/16 16:56, Stefano Stabellini wrote:
> > On Wed, 30 Mar 2016, Juergen Gross wrote:
> >> Add a Xenstore directory for each supported pv backend. This will allow
> >> Xen tools to decide which backend type to use in case there are
> >> multiple possibilities.
> >>
> >> The information is added under
> >> /local/domain/<backend-domid>/device-model/<domid>/backends
> >> before the "running" state is written to Xenstore. Using a directory
> >> for each directory enables us to add parameters for specific backends
> >> in the future.
> >>
> >> In order to reuse the Xenstore directory creation already present in
> >> hw/xen/xen_devconfig.c move the related functions to
> >> hw/xen/xen_backend.c where they fit better.
> > 
> > An interface change like this one, needs a new file under docs (in the
> > Xen repository) to document it.  Please add a reference to it, in the
> > commit message here.
> 
> Is the Xenstore path documentation enough? Or do you want something
> like docs/misc/qemu-deprivilege.txt to be added?

Something like qemu-deprivilege.txt would be acceptable.


> > In addition please make sure that this can work with QEMU running as
> > non-root. What are the permissions of the new xenstore directory?
> 
> Should work, as I'm using the same functions as any backend in qemu is
> using. The new directory is read-only for the qemu target domain (same
> as the backend directories).

All right, just make sure to have xen-qemuuser-shared or
xen-qemuuser-domid on your system when you test :-)

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