On Fri, 2 May 2008, Matthew Wilcox wrote: > On Fri, May 02, 2008 at 05:19:40PM +0100, Ricardo Martins wrote: > > There are 14 drivers in drivers/isdn/hisax/ still using calls to > > pci_find_device() instead of the pci_get_device(). From what I gather, a > > simple sed 's|pci_find_device|pci_get_device|' is enough to fix this, > > since both functions use the same input and give the same output. > > Is this correct? > > It's never correct. If it were that easy (a) there wouldn't be a rename > in the first place and (b) it would have been done with a shell-script > already. > > pci_get_device increments a refcount on the pci_dev. pci_find_device > does not. Your patch must include calls to pci_dev_put() *when > appropriate*. Note, for example, that pci_get_device decrements the > refcount on the device passed in. I looked at this briefly at one point. To my recollection, many of the files that use pci_find_device store the result in a global variable. So one would have to think carefully about the useful lifetime of the value of that variable to know where to put the puts. julia -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
