]> git.hungrycats.org Git - linux/commit
[PATCH] Prevent PCI driver registration failure oopsing
authorRussell King <rmk+lkml@arm.linux.org.uk>
Thu, 29 Jan 2004 06:26:16 +0000 (22:26 -0800)
committerLinus Torvalds <torvalds@home.osdl.org>
Thu, 29 Jan 2004 06:26:16 +0000 (22:26 -0800)
commitb5849a353834172c4e11b494aa5df5c7bd37bd55
treef5f63686f4fdd784b642239d0495def3ba471348
parent0dc316dee192a2e178d52b2a2324fbe18ed46f1c
[PATCH] Prevent PCI driver registration failure oopsing

Greg,

As discussed about six or so months ago, we agreed to hold off this
patch until fairly late, due to its ability to catch duplicate PCI
driver names.  Please note that I haven't attempted to reproduce the
problem with recent kernels, and that all ARM kernel patches released
since then have had this patch in.

I'm guessing this will actually be 2.6.1 material since it probably
doesn't show for PCI drivers which are part of the kernel tree.

If pci_register_driver fails, the register the PCI driver structure
will not be registered with the driver model.  pci_register_driver
returns with negative value, and we then attempt to unregister the
driver structure.  This leads to an oops in the driver model.

The driver model does not return the number of devices it successfully
bound the driver to, and neither does pci_register_driver() return
this information.

Therefore, all of the code below is redundant.

(There's a little redundancy left in drivers/pci/pci-driver.c but it
is harmless unlike this block.)
include/linux/pci.h