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

Re: [Xen-devel] [PATCH] libxl: adjust point of backend name resolution

On Tue, 2013-04-30 at 16:03 +0100, Eric Shelton wrote:
> Resolution of a backend name to a domid needs to happen a little earlier
> in some cases.
> For example, if a domU is specified as a backend for a
> disk and, as previously written, libxl__device_disk_setdefault() calls
> libxl__resolve_domid() last, then disk->backend_domid still equals
> LIBXL_TOOLSTACK_DOMID when libxl__device_disk_set_backend() is called.
> This results in libxl__device_disk_set_backend() making an incorrect
> attempt to validate the target by calling stat() on a file on dom0,
> resulting in ERROR_INVAL (see libxl_device.c lines 239-248), which
> prevents creation of the frontend domain.
> Likewise, libxl__device_nic_setdefault() previously made use of
> nic->backend_domid before it was set.
> Signed-off-by: Eric Shelton <eshelton@xxxxxxxxx>

Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

> Cc: Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>

Daniel, any comments?

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.