[RFC PATCH 0/5] driver core: driver probe asynchronously
Hi, This patchset implements asynchronous driver probe for driver_register, and try to address the below problems about driver init during kernel boot: - help to solve some dependency problem during kernel boot (such as, request_firmware is called inside probe when driver is built in kernel[1]) The idea behind the patch is very simple: - seperate driver probe from driver_register and run this part in one standalone kernel thread context - so driver_register will become two parts: register the driver on the bus, and trigger to schedule a kernel thread to do the driver probe if autoprobe is set Fortunately, my OMAP4 based Pandaboard boots fine with the patchset, and looks it may work well. More or less, some problems might be triggered by these patchset, but it should be helpful and not a big deal: - the dependency problem may be found, and it either exposes the driver's probelm or help to improve the asynchronous probe approach - can use driver_register_sync to work around it In summary, there are at least two advantages about asynchronous driver probe: - speedup kernel boot when many drivers are built in kernel - make driver's probe() not need to consider running something asynchronously(such as, scsi scan, request_firmware_no_wait, ...), so easier to write drivers It should a very simple way to help to solve the problem[1], without any changes on current drivers which call request_firmware() in its probe(). BTW, the patchset doesn't solve the problem completely, and still some work is needed. [1], http://marc.info/?t=13467641612&r=1&w=2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC PATCH 0/5] driver core: driver probe asynchronously
On Sun, Sep 09, 2012 at 07:25:15AM +0800, Ming Lei wrote: > Hi, > > This patchset implements asynchronous driver probe for driver_register, > and try to address the below problems about driver init during > kernel boot: > - help to solve some dependency problem during kernel boot > (such as, request_firmware is called inside probe when driver > is built in kernel[1]) > > The idea behind the patch is very simple: > > - seperate driver probe from driver_register and run this part > in one standalone kernel thread context > > - so driver_register will become two parts: register the driver > on the bus, and trigger to schedule a kernel thread to do the > driver probe if autoprobe is set You do realize we tried this out about 5+ years ago, it caused major problems, no real benefit, and so we removed it (or maybe never did the final commit with it, it might never have hit Linus's tree due to all of the problems we found.) > Fortunately, my OMAP4 based Pandaboard boots fine with the patchset, and > looks it may work well. > > More or less, some problems might be triggered by these patchset, but > it should be helpful and not a big deal: > - the dependency problem may be found, and it either exposes > the driver's probelm or help to improve the asynchronous probe > approach > > - can use driver_register_sync to work around it > > In summary, there are at least two advantages about asynchronous driver > probe: > - speedup kernel boot when many drivers are built in kernel I would be really amazed if this is true in any measurable fashion. Again, we tried this, and it did seem to be a tiny bit faster, but in the end, wasn't worth it. > - make driver's probe() not need to consider running something > asynchronously(such as, scsi scan, request_firmware_no_wait, ...), > so easier to write drivers No, that's a bus issue, not a driver issue. Busses can do this today, if they want to, no problem at all. So, some busses have (maybe, maybe not, I can't remember, but I think ATA did), and that is the level you need to do this at, not at the driver core level, sorry. > It should a very simple way to help to solve the problem[1], without > any changes on current drivers which call request_firmware() in its > probe(). BTW, the patchset doesn't solve the problem completely, and > still some work is needed. As you state, this doesn't really solve the problem, and will cause a lot more (think about all the busses that aren't expecting to have their probe functions called at the same time as other probe functions are being called.) So while I applaud the effort, I can't accept this due to the past history of when it was tried before. sorry, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC PATCH 0/5] driver core: driver probe asynchronously
On Sat, 2012-09-08 at 19:57 -0700, Greg Kroah-Hartman wrote: > As you state, this doesn't really solve the problem, and will cause a > lot more (think about all the busses that aren't expecting to have > their > probe functions called at the same time as other probe functions are > being called.) > > So while I applaud the effort, I can't accept this due to the past > history of when it was tried before. Maybe a compromise would be to have ->probe() be called synchronously as done currently, but to add support for a specific -EAGAIN return from it, requiring a later asynchronous re-probe ? (Possibly requiring an explicit trigger) That would allow drivers who fail some kind of "oddball" dependency (the firmware load is such as case, there are a few others) to basically install some kind of notifier and try again later when the dependency has been fulfilled. That way existing drivers still behave synchronously, there is no breakage unless drivers are explicitly modified for async probing. Cheers, Ben. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC PATCH 0/5] driver core: driver probe asynchronously
At Sun, 09 Sep 2012 17:47:46 +1000, Benjamin Herrenschmidt wrote: > > On Sat, 2012-09-08 at 19:57 -0700, Greg Kroah-Hartman wrote: > > As you state, this doesn't really solve the problem, and will cause a > > lot more (think about all the busses that aren't expecting to have > > their > > probe functions called at the same time as other probe functions are > > being called.) > > > > So while I applaud the effort, I can't accept this due to the past > > history of when it was tried before. > > Maybe a compromise would be to have ->probe() be called synchronously as > done currently, but to add support for a specific -EAGAIN return from > it, requiring a later asynchronous re-probe ? (Possibly requiring an > explicit trigger) So it's something pretty similar like -EPROBE_DEFER, but the deferred probing is rather based on the standard async probe? I find it'd make sense, as the current deferred probing is somewhat hackish. Forcing all asynchronous right now looks risky, but selective way of easy async probing would be welcome. thanks, Takashi > That would allow drivers who fail some kind of "oddball" dependency (the > firmware load is such as case, there are a few others) to basically > install some kind of notifier and try again later when the dependency > has been fulfilled. > > That way existing drivers still behave synchronously, there is no > breakage unless drivers are explicitly modified for async probing. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC PATCH 0/5] driver core: driver probe asynchronously
On Sun, Sep 9, 2012 at 10:57 AM, Greg Kroah-Hartman wrote: > On Sun, Sep 09, 2012 at 07:25:15AM +0800, Ming Lei wrote: >> Hi, >> >> This patchset implements asynchronous driver probe for driver_register, >> and try to address the below problems about driver init during >> kernel boot: >> - help to solve some dependency problem during kernel boot >> (such as, request_firmware is called inside probe when driver >> is built in kernel[1]) >> >> The idea behind the patch is very simple: >> >> - seperate driver probe from driver_register and run this part >> in one standalone kernel thread context >> >> - so driver_register will become two parts: register the driver >> on the bus, and trigger to schedule a kernel thread to do the >> driver probe if autoprobe is set > > You do realize we tried this out about 5+ years ago, it caused major > problems, no real benefit, and so we removed it (or maybe never did the > final commit with it, it might never have hit Linus's tree due to all of > the problems we found.) Thanks for the information you provided. I guess you mean the commit 21c7f30b1d3f8a3de3128478daca3ce203fc8733 driver core: per-subsystem multithreaded probing which was reverted surely. IMO, this patch is different with the above commit: - the probe order is guaranteed to be that parent first, children second since both parent's lock and child's lock are held inside driver_attach. In the commit of 21c7f30b, only device's lock is held in the asynchronous kthread. - it should be supported already by hotplug buses: suppose all drivers in the bus have been loaded already, so the probe order depends on the order of device_add, which is an uncertain event and is random, so there should be no probe order dependency on hotplug buses. - the async probe is just triggered in driver_init(module_init) path, which is very similar with loading driver from module(looks no obvious probe order dependency for non-dependent moduels) I will study the original commit further and other non-hotplug buses to see if there are more dependency about probe order. But IMO, this patch does respect the basic probe order: parent first, children second. > >> Fortunately, my OMAP4 based Pandaboard boots fine with the patchset, and >> looks it may work well. >> >> More or less, some problems might be triggered by these patchset, but >> it should be helpful and not a big deal: >> - the dependency problem may be found, and it either exposes >> the driver's probelm or help to improve the asynchronous probe >> approach >> >> - can use driver_register_sync to work around it >> >> In summary, there are at least two advantages about asynchronous driver >> probe: >> - speedup kernel boot when many drivers are built in kernel > > I would be really amazed if this is true in any measurable fashion. > Again, we tried this, and it did seem to be a tiny bit faster, but in > the end, wasn't worth it. If we can make sure that the approach is doable, I will provide some test data. > >> - make driver's probe() not need to consider running something >> asynchronously(such as, scsi scan, request_firmware_no_wait, ...), >> so easier to write drivers > > No, that's a bus issue, not a driver issue. Busses can do this today, > if they want to, no problem at all. So, some busses have (maybe, maybe > not, I can't remember, but I think ATA did), and that is the level you > need to do this at, not at the driver core level, sorry. IMO, most of drivers have no probe order dependency, and that is my motivation to try it. From current async_schedule users, most of them are used inside driver probe, see below: arch/sh/drivers/pci/pcie-sh7786.c:572: async_schedule(sh7786_pcie_hwops->port_init_hw, port); drivers/acpi/battery.c:1109:async_schedule(acpi_battery_init_async, NULL); drivers/ata/libata-core.c:6118: async_schedule(async_port_probe, ap); drivers/base/power/main.c:694: async_schedule(async_resume, dev); drivers/base/power/main.c:1166: async_schedule(async_suspend, dev); drivers/block/floppy.c:4353:async_schedule(floppy_async_init, NULL); drivers/md/raid5.c:1478:async_schedule(async_run_ops, sh); drivers/regulator/core.c:2964: async_schedule_domain(regulator_bulk_enable_async, drivers/s390/block/dasd.c:3011: async_schedule(dasd_generic_auto_online, cdev); drivers/scsi/libsas/sas_ata.c:857: async_schedule_domain(async_sas_ata_eh, dev, &async); drivers/scsi/scsi_scan.c:1875: async_schedule(do_scan_async, data); drivers/scsi/sd.c:2734: async_schedule_domain(sd_probe_async, sdkp, &scsi_sd_probe_domain); sound/soc/soc-dapm.c:1656: async_schedule_domain(dapm_pre_sequence_async, d, sound/soc/soc-dapm.c:1670: async_schedule_domain(dapm_post_sequence_async,