On Tue, 2008-03-04 at 12:43 -0500, Alan Stern wrote: > On Tue, 4 Mar 2008, James Bottomley wrote: > > > If you want to see for yourself, it's fs/block_dev.c:do_open(). > > Thanks for the pointer. I see the subroutine is not exactly loaded > with comments... > > > > If you can suggest an alternative approach, I'll be glad to change the > > > patch. > > > > Well, since this is the 7th patch of an eight patch series, I don't > > really have the context for that. However, I assume it's all on the > > linux-usb list? If so, I'll add it to my list of things to look at. > > The patches before it probably won't give you any additional useful > context. The following patch might; that's the one affecting > usb-storage. OK, I looked through the patch series. It really looks like the in-kernel thing is the wrong approach. Why not export this want_persist flag via sysfs and then have hal and udev update it when they see a filesystem on an underlying usb device mounted (and zero it out on unmount)? That way you always have the exactly correct state and there's no need to go fishing for information in layers that are difficult to get to. James -- 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
