Re: [patch 5/3] fastboot: sync the async execution before late_initcall and move level 6s (sync) first

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Simon Arlott
Date: Wednesday, July 30, 2008 - 11:25 am

On 30/07/08 15:08, Alan Stern wrote:

With fastboot:
162 ohci_hcd_mod_init+0x0/0xa6
167 pcie_portdrv_init+0x0/0x4d
182 saa7134_init+0x0/0x4a
205 ehci_hcd_init+0x0/0x8b
299 snd_usb_audio_init+0x0/0x3d
557 e1000_init_module+0x0/0x88
1227 amd74xx_ide_init+0x0/0x1b
2306 nv_init+0x0/0x1b

Without fastboot:
103 ehci_hcd_init+0x0/0x8b
113 raid5_init+0x0/0x3e
127 pci_iommu_init+0x0/0x17
148 ohci_hcd_mod_init+0x0/0xa4
183 saa7134_init+0x0/0x4a
297 snd_usb_audio_init+0x0/0x3d
557 e1000_init_module+0x0/0x88
1227 amd74xx_ide_init+0x0/0x1b
2303 nv_init+0x0/0x1b
2859 usblp_init+0x0/0x1b

Boot log attached.

It looks like usb device driver init requires it to immediately block and 
wait for all devices to have completed init - so regardless of where we 
put the usb/ directory in the initcall order, it will always wait a while 
because the drivers will be immediately after the hcd init... perhaps 
if we moved the hcd init to before net/, sata/, ide/ etc. (all the things 
that take time themselves) but left usb device drivers to the end it 
would actually get the benefit of that separate thread.

I don't have the time to rework how usb/ is linked in order to try that, 
but moving usb/ to before net/ and sata/ while making all the usb device 
drivers that get used be late initcalls could be used to test it.

-- 
Simon Arlott
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [patch 5/3] fastboot: sync the async execution before ..., Arjan van de Ven, (Sun Jul 20, 2:50 pm)
Re: [patch 5/3] fastboot: sync the async execution before ..., Arjan van de Ven, (Tue Jul 29, 2:04 pm)
Re: [patch 5/3] fastboot: sync the async execution before ..., Arjan van de Ven, (Tue Jul 29, 2:21 pm)
Re: [patch 5/3] fastboot: sync the async execution before ..., Simon Arlott, (Wed Jul 30, 11:25 am)
[PATCH RFC] USB: Add HCD fastboot, Simon Arlott, (Wed Aug 6, 11:40 am)
Re: [PATCH RFC] USB: Add HCD fastboot, Alan Stern, (Wed Aug 6, 12:11 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Simon Arlott, (Wed Aug 6, 12:20 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Greg KH, (Wed Aug 6, 12:29 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Alan Stern, (Wed Aug 6, 12:49 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Arjan van de Ven, (Wed Aug 6, 12:56 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Simon Arlott, (Wed Aug 6, 1:07 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Alan Stern, (Wed Aug 6, 1:09 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Arjan van de Ven, (Wed Aug 6, 1:17 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Alan Stern, (Wed Aug 6, 1:26 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Alan Stern, (Wed Aug 6, 1:27 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Simon Arlott, (Wed Aug 6, 2:49 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Alan Stern, (Wed Aug 6, 3:34 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Simon Arlott, (Wed Aug 6, 3:53 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, David Brownell, (Wed Aug 6, 8:29 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, David Brownell, (Wed Aug 6, 8:34 pm)
Re: [PATCH RFC] USB: Add HCD fastboot, Emanoil Kotsev, (Thu Aug 7, 2:28 am)
Re: [PATCH RFC] USB: Add HCD fastboot, Alan Stern, (Thu Aug 7, 7:14 am)
Re: [PATCH RFC] USB: Add HCD fastboot, Alan Stern, (Thu Aug 7, 9:47 am)
Re: [PATCH RFC] USB: Add HCD fastboot, Rene Herman, (Fri Aug 8, 2:24 am)
Re: [PATCH RFC] USB: Add HCD fastboot, Simon Arlott, (Fri Aug 8, 4:29 am)
Re: [PATCH RFC] USB: Add HCD fastboot, Rene Herman, (Fri Aug 8, 7:30 am)