Re: [PATCH 6/8] amd64_edac: enforce synchronous probe

[Dmitry Torokhov <dmitry.torokhov@gmail.com>]

On Wed, Mar 18, 2015 at 03:51:41PM -0400, Tejun Heo wrote:
> Hello,
> 
> On Wed, Mar 18, 2015 at 12:36:22PM -0700, Dmitry Torokhov wrote:
> > Because they are not inherently problematic. I mean from the kernel POV
> > they work fine, the question is if your userspace can deal with them or
> > not. For example ChromeOS userspace is fine.
> 
> async already provides mechanisms to solve the above problem.  This
> doesn't have to be an either-or thing.  I still don't get why we
> aren't converting drivers properly over to async so that they still
> follow the ordering rules where necessary.  What's wrong with just
> blacklisting the ones which can't follow ordering rules for now and
> lifting the blacklist as they get fixed?  That'd provide a gradual
> transition path with the matching incentive for converting the drivers
> while not disturbing userland.

Tejun, I lost you here. Certainly you are not arguing for going through
the drivers one by one and making their module init code to engage
async_schedule to continue the device creation in link order (well,
sorta, since deferred probing already violates it).

Also, it is not only kernel that may not be prepared for asynchronous
probing, but userspace as well. And I do not think that we should be
working towards preserving the init order in the long run as more and
more bits become hot pluggable and we should be able to handle devices
come and go gracefully anyway.

Thanks.

-- 
Dmitry