Hi, My hp.com address is dead. This is my new address. Luckily, I just happened to be scanning LKML and thought "who would be crazy enough to be touching cpqphp?" :) * Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>: > Jiri Slaby <jslaby@xxxxxxx> wrote: > > Hi, > > > > we have a system where there is a pci hotplug class device to be handled > > by cpqphp, but it is not a bridge. But in cpqhpc_probe there is: > > struct pci_bus *bus; > > ... > > bus = pdev->subordinate; > > ... > > bus->max_bus_speed = PCI_SPEED_66MHz_PCIX; > > > > But as it is not a bridge, subordinate is NULL and the kernel crashes. > > > > Any idea what would be a correct fix here? > > > > The bugzilla entry is at: > > https://bugzilla.novell.com/show_bug.cgi?id=609338 > > I don't think we have anyone actively working on CPQHPC these days. > Seems like the simple patch would be to check whether pdev->subordinate > or bus exists before using it... Have you poked around for specs on > this at all? I think Greg/Jesse's suggestion is correct - just return if it's not a bridge. I managed to find an ancient machine that actually had this hardware but never had time to work on it, and then switched jobs. As far as I know, the hardware is still there, with my old group. But I seriously think it's just time to rip this driver out of the tree. If anyone is actually using this driver to do hotplug, I will personally buy that person a full half-barrel of Colorado's finest microbrew of choice in order to numb the pain of making poor career decisions. Hell, we can drink it together, during which time I'll hack on the driver. The code surely can't get any worse. The occasional bug report shows that it's just more of a maintenance burden than anything else. /ac -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
