On Mon, Sep 8, 2014 at 6:29 PM, Tejun Heo <tj@xxxxxxxxxx> wrote: > On Mon, Sep 08, 2014 at 06:26:04PM -0700, Luis R. Rodriguez wrote: >> > Alternatively, add a module-generic param "async_probe" or whatever >> > and use that to switch the behavior should work too. I don't know >> > which way is better but either should work fine. >> >> I take it by this you meant a generic system-wide sysctl or kernel cmd >> line option to enable this for al drivers? > > Well, either global or per-insmod switch should work. There probably > are details that I haven't mentioned - e.g. probably global switch is > easier to backport and deploy to existing systems Yes a global sysctl solution might make it easier to backport. > - but as long as it > works I don't have fundmental objections either way. OK then one only concern I would have with this is that the presence of such a flag doesn't necessarily mean that all drivers on a system have been tested for asynch probe yet. I'd feel much more comfortable if this global flag allowed say specific drivers that *did* have such a bool enabled, for example. Then that would enable synchronous behaviour for the kernel by default, require the flag for enabling the new async feature but only for drivers that have been tested. That also still would not technically solve the issue of the current existence of the timeout, unless of course we wish to ask systemd to only make the timeout take effect *iff* the global sysctl flag / whatever was enabled. Luis -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
