On Fri, 4 Dec 2009, Dmitry Torokhov wrote: > > Maybe reference counting is inelegant; it depends on your point of > > view. Can you think of a more elegant way to make sure that a pointer > > isn't stale? > > Yes, just say "no" to device_create() and friends. device_create() wasn't used in the case Oliver is discussing. > Embed device structure in > yours, You can't do that when the device structure wasn't created by your driver. > be mindful of lifetime rules and only use "your" device (i.e device > bound to your driver). What do you mean by "use"? In Oliver's case he wasn't using the device, he was using the device structure. (Maybe that's what you meant.) And he wanted to use it at a time when it wasn't bound to his driver, because userspace still had an open file reference to it. There isn't really any way around this. > This way, as long as your refcount your instance you > can rest assured the device structure is there as well. I rather think that a simple device_get() and device_put() is easier than trying to follow a bunch of rules, especially in cases where they don't apply! :-) Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
