Bug 7942
Summary: | ohci1394 module broken and cannot be removed | ||
---|---|---|---|
Product: | Drivers | Reporter: | Robert Crocombe (rcrocomb) |
Component: | IEEE1394 | Assignee: | Stefan Richter (stefanr) |
Status: | CLOSED CODE_FIX | ||
Severity: | normal | ||
Priority: | P2 | ||
Hardware: | i386 | ||
OS: | Linux | ||
Kernel Version: | 2.6.20 | Subsystem: | |
Regression: | --- | Bisected commit-id: | |
Attachments: |
ieee1394: fix host device registering when nodemgr disabled
ieee1394: fix host device registering when nodemgr disabled |
Description
Robert Crocombe
2007-02-05 14:58:30 UTC
Could you try with this reverted? "ieee1394: nodemgr: fix deadlock in shutdown" http://git2.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=8252bbb1363b7fe963a3eb6f8a36da619a6f5a65 (That one was to fix bug 6706, alas there is now bug 7792... grrr.) Actually, that commit isn't in effect on your setup: "ieee1394: nodemgr and IRM functionality disabled" --- i.e. reverting it won't change anything AFAICS. Testing here on IA32, 2.6.20-rc6 + latest 1394 drivers: # modprobe ieee1394 disable_nodmgr=1 # modprobe ohci1394 -> spinlock lockup Could you try 2.6.19.2 plus a patch from http://me.in-berlin.de/~s5r6/linux1394/updates/2.6.19.y/ ? Either of v250 or v273 or OK for this purpose; v250 is almost identical to what is in 2.6.20. Did you use disable_nodemgr=1 under 2.6.19 too? Re comment #2: Hmm, maybe it _is_ effective: hpsb_alloc_host() adds a host device and host class device which, according to nodemgr_dev_template_host has .driver = &nodemgr_mid_layer_driver, However nodemgr_mid_layer_driver is registered with the driver core in init_ieee1394_nodemgr() which is not run if disable_nodemgr=1. Created attachment 10304 [details]
ieee1394: fix host device registering when nodemgr disabled
Appears to fix the issue for me.
Sorry, the regression was mostly my fault.
Created attachment 10306 [details]
ieee1394: fix host device registering when nodemgr disabled
will submit the fix to Linus and -stable soon patch committed to 2.6.20-git# and proposed for 2.6.20.1 |