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

Re: [Xen-devel] [PATCH] tools/hotplug/Linux/init.d/: added other xen kernel modules on xencommons start



>>> On 03.08.12 at 13:08, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> wrote:
> Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/hotplug/Linux/init.d/: 
> added other xen kernel modules on xencommons start"):
>> On Fri, 2012-06-08 at 15:07 +0100, Ian Jackson wrote:
>> > Fabio Fantoni writes ("[Xen-devel] [PATCH] tools/hotplug/Linux/init.d/: 
> added other xen kernel modules on xencommons start"):
>> > > tools/hotplug/Linux/init.d/: added other xen kernel modules on 
>> > > xencommons start
>> > 
>> > This looks at least harmless to me.
>> > 
>> > I'm surprised, however, that these things aren't loaded automatically.
>> > For example, shouldn't the xenbus driver's enumeration automatically
>> > load blkback too ?
>> 
>> Yes it should, there is autoloading stuff for all the backends.
>> 
>> Not sure about gntalloc. I suspect not.
>> 
>> > Having said that, I'm inclined to apply this unless someone 
>> > explains that it's a bad idea.
> 
> I have applied it.
> 
> But we should still try to fix the upstream kernels not to need it.

And just like for the respective pending blktap item, it should
- be made sure this gets backed out again after 4.2, making sure
  the loading happens either automatically or when the module is
  in fact needed, and
- not exclusively try the pv-ops kernel's module names.

Jan


_______________________________________________
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®.