On Fri, 2004-04-02 at 03:43, Mike Anderson wrote: > I have looked at the sd issue off and on due to the previous open race > reported by Alan Stern. While the window can be narrowed inside SCSI you > need help for the calling subsystem to know when it is OK to cleanup and > your routine will not be called anymore. A similar problem also showed > up in the tear down the host directory entry in /proc/scsi but was only > fixed up so far due to its depreciated status. > > http://marc.theaimsgroup.com/?t=105545175900001&r=1&w=2 > > I believe as indicated above that all cross subsystem registrations need > a release / put callback. This would allow the release chain to be > called from block -> ULD -> scsi core -> LLDD. > > Recently I have not been spending the proper time looking at this, but > last look it appeared that we needed to add a release / put method call > to the gendisk disk_release routine. The release function or object to do > the put on would need to be set prior to the call to add_disk.
Actually, I can't believe this problem to be local entirely to SCSI. So, a simpler mechanism (and more globally useful) might be to have a two phase driver release in sysfs: - the current ->remove would stay where it is (as a notify on device_del). On receving this the driver begins clean up enough to drop any internal references to the device it is holding. - then introduce a ->release which is called as part of dropping the last device reference where the driver cleans up any resources the driver was keeping to service the removed but not released device. Then, we'd obviously not call unregister_cdrom and kfree the scsi_cd structure until ->release time. James ------------------------------------------------------- This SF.Net email is sponsored by: IBM Linux Tutorials Free Linux tutorial presented by Daniel Robbins, President and CEO of GenToo technologies. Learn everything from fundamentals to system administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click _______________________________________________ [EMAIL PROTECTED] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
