iosapic.c NULL pointer deref

In the 2.4.0-test8 and -test9 IA-64 patch, the file arch/ia64/dig/iosapic.c, function pci_for_each_dev(), the bridge pointer can be NULL, but gets dereferenced anyhow, causing a nice tombstone. I see this with an AMI MegaRAID 471 card that happens to have two bridges on it. Putting in tests against NULL, that routine doesn't blow up any longer, but interrupts are not properly assigned to that card then, and eventually all SCSI reads/writes time out.

I'm happy to work with whomever is responsible for this code to have it properly assign interrupts to these cards.

Thanks,
Matt

Matt,

This code was put in specifically for AMI MegaRaid controller and bridge cards. Mike from Intel did the patch. Anyway, I am working on the IOSAPIC code to fix a problem and would like to fix this problem also. Please let me know if you have any suggestion.

Thanks,
Asit