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

Re: [Xen-devel] [PATCH] libxl: ignore run_hotplug_scripts setting when backend_domid!=0



On Thu, Jul 04, 2013 at 03:05:54PM +0100, George Dunlap wrote:
> On 04/07/13 14:51, Wei Liu wrote:
> >On Thu, Jul 04, 2013 at 03:23:51PM +0200, Roger Pau Monné wrote:
> >>On 04/07/13 14:40, George Dunlap wrote:
> >>>On 04/07/13 12:39, Roger Pau Monné wrote:
> >>>>On 04/07/13 12:42, George Dunlap wrote:
> >>>>>On Tue, Apr 9, 2013 at 9:49 AM, Marek Marczykowski
> >>>>><marmarek@xxxxxxxxxxxxxxxxxxxxxx> wrote:
> >>>>>>Instead of forcing the user to change run_hotplug_scripts setting
> >>>>>>globally, just ignore it for backends outside of dom0.
> >>>>>>
> >>>>>>Signed-off-by: Marek Marczykowski <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
> >>>>>This seems like a good idea -- Roger / Ian, any thoughts?
> >>>>We already have this behaviour in libxl (see
> >>>>libxl_device.c:device_hotplug):
> >>>>
> >>>>/*
> >>>>   * If device is attached from a driver domain don't try to execute
> >>>>   * hotplug scripts
> >>>>   */
> >>>>if (aodev->dev->backend_domid != LIBXL_TOOLSTACK_DOMID)
> >>>>      goto out;
> >>>>
> >>>>This was introduced by commit 05bfd984dfe7014f1f5ea1133608b9bab589c120
> >What Marek's patch does is effectively Roger's commit 05bfd984 plus some
> >network specific bits. N.B. this patch predates 05bfd984.
> >
> >>>But you still get an error on domain creation if you actually set a
> >>>non-dom0 backend and run_hotplug_scripts is set; so you can't actually
> >>>run in "mixed mode" anyway.  This patch should, I think, enable you to
> >>>have libxl run the scripts in dom0, but udev run them in domu.
> >Long time ago Konrad sent me an email about running network backend in
> >another domain. The first thing to do is to disable hotplug script
> >globally.
> 
> Sure, that's easy.  The problem is, there's no reason to disable it
> globally -- you can run udev in the domnet, but have libxl run the
> scripts in dom0.
> 
> I'll try just taking out that check and seeing what happens...
> 

I'm not suggesting we disable it globally. I'm suggesting disabling
hotplug script makes domnet work (whether globally or locally).


Wei.

>  -George

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


 


Rackspace

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