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

Re: [PATCH] allow dd=URL to update storage drivers



On Fri, 2008-07-11 at 18:02 -0500, Jeff Bastian wrote:
> On Fri, 11 Jul 2008, Jeremy Katz wrote:
> > The problem is that you could be loading your driver disk off of
> > something that comes from a storage driver (which comment in the code
> > aside, is supported now).  By not loading storage drivers, you then
> > don't have the driver for where you may be grabbing your driver disk.
> 
> Good point.
> 
> I've modified the patch to only turn on LOADER_FLAGS_NOSTORAGE if 
> loaderData.ddsrc matches 'nfs:', 'ftp://', or 'http://'.  It will still 
> load the storage drivers if using 'cdrom' or 'path:'
> 
> Will that work?

It feels like the sort of kludge that we're going to hate forever... but
it will probably work.

Of course, you haven't at all addressed the question of "this doesn't
apply to master".  

Jeremy


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