[Top][All Lists]

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

Re: [Qemu-devel] [PATCH] qemu-xen: make use of xenstore relative paths

From: Roger Pau Monné
Subject: Re: [Qemu-devel] [PATCH] qemu-xen: make use of xenstore relative paths
Date: Thu, 26 Sep 2013 19:20:31 +0200
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/20130801 Thunderbird/17.0.8

On 26/09/13 18:46, Anthony PERARD wrote:
> On Wed, Sep 18, 2013 at 09:50:58PM +0200, Roger Pau Monne wrote:
>> Qemu has several hardcoded xenstore paths that are only valid on Dom0.
>> Attempts to launch a Qemu instance (to act as a userspace backend for
>> PV disks) will fail because Qemu is not able to access those paths
>> when running on a domain different than Dom0.
>> Instead make the xenstore paths relative to the domain where Qemu is
>> actually running.
>> Signed-off-by: Roger Pau Monné <address@hidden>
>> Cc: address@hidden
>> Cc: Anthony PERARD <address@hidden>
>> Cc: Stefano Stabellini <address@hidden>
> This look fine. One issue with the patch: the file xen_backend.c have
> been moved to hw/xen/xen_backend.c.

Thanks, this is based on the stable Qemu version in Xen tree, I should
have done the change on top of the main qemu.git repo.

> I've also tryied it in a stubdomain, and it does not boot anymore
> because the qemu in the stubdom can not read the state.  I have tried
> again without the change in xen-all.c, and the stubdom does not complain
> anymore. So in the change in xenstore_record_dm_state() needed as well?

Yes, if we run a Qemu instance inside a driver domain it wouldn't make
much sense IMHO to write the state of that Qemu instance on a xenstore
path that belongs to the Dom0, and also we would need to give the driver
domain permissions to write on a xenstore path that's inside the Dom0
xenstore path, which doesn't seem like a good idea.

To make Qemu work on a domain different than Dom0 you will also need the
following patch from my driver domain series:


If not the guest is unable to create the device-model/<domid>/state
xenstore entry. For stubdomains would it be really hard to change the
Dom0 to check for /local/domain/<stubdom_id>/device-model/<domid>/state
instead of /local/domain/0/device-model/<domid>/state?

reply via email to

[Prev in Thread] Current Thread [Next in Thread]