lkml.org 
[lkml]   [2014]   [Sep]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v1 5/5] driver-core: add driver asynchronous probe support
Hello, Luis.

On Mon, Sep 29, 2014 at 11:22:08PM +0200, Luis R. Rodriguez wrote:
> > > + /* For now lets avoid stupid bug reports */
> > > + if (!strcmp(bus->name, "pci") ||
> > > + !strcmp(bus->name, "pci_express") ||
> > > + !strcmp(bus->name, "hid") ||
> > > + !strcmp(bus->name, "sdio") ||
> > > + !strcmp(bus->name, "gameport") ||
> > > + !strcmp(bus->name, "mmc") ||
> > > + !strcmp(bus->name, "i2c") ||
> > > + !strcmp(bus->name, "platform") ||
> > > + !strcmp(bus->name, "usb"))
> > > + return true;
> >
> > Ugh... things like this tend to become permanent. Do we really need
> > this? And how are we gonna find out what's broken why w/o bug
> > reports?
>
> Yeah... well we have two options, one is have something like this to
> at least make it generally useful or remove this and let folks who
> care start fixing async for all modules. The downside to removing
> this is it makes async probe pretty much useless on most systems
> right now, it would mean systemd would have to probably consider
> the list above if they wanted to start using this without expecting
> systems to not work.

So, I'd much prefer blacklist approach if something like this is a
necessity. That way, we'd at least know what doesn't work.

Thanks.

--
tejun


\
 
 \ /
  Last update: 2014-09-29 23:41    [W:0.217 / U:0.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site