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

Re: [Xen-devel] [PATCH] libxl: disable usbredirection if spice is disabled

On Tue, 2014-05-27 at 17:01 +0200, Fabio Fantoni wrote:
> Now if usbredirection is enabled in domU's xl cfg is added also
> if spice is disabled and then usbredirection remain unused.
> This patch if usbredirection is enabled but spice not disable
> usbredirection and show a warning.
> Signed-off-by: Fabio Fantoni <fabio.fantoni@xxxxxxx>

Ack in principal but:
> +            LOG(WARN,"usbredirection is used only by spice, spice is 
> disabled"
> +            "therefore usbredirection is disabled.");

This misses a space between "disabled" and "therefore" and is also a
rather long line.

How about just "spice disabled, disabling usbredirection"?


Xen-devel mailing list



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