diff options
author | Andi Kleen <andi@firstfloor.org> | 2008-02-10 19:35:20 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-11 12:20:50 -0500 |
commit | 1f07e988290fc45932f5028c9e2a862c37a57336 (patch) | |
tree | db0a20e2ead6a865af786e09e90ce1159a277c9e /include | |
parent | 0c0d61ca93d111c521182c0909e478fa709e05c6 (diff) |
Prevent IDE boot ops on NUMA system
Without this patch a Opteron test system here oopses at boot with
current git.
Calling to_pci_dev() on a NULL pointer gives a negative value so the
following NULL pointer check never triggers and then an illegal address
is referenced. Check the unadjusted original device pointer for NULL
instead.
Signed-off-by: Andi Kleen <ak@suse.de>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/ide.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/ide.h b/include/linux/ide.h index 23fad89292df..a3b69c10d667 100644 --- a/include/linux/ide.h +++ b/include/linux/ide.h | |||
@@ -1295,7 +1295,7 @@ static inline void ide_dump_identify(u8 *id) | |||
1295 | static inline int hwif_to_node(ide_hwif_t *hwif) | 1295 | static inline int hwif_to_node(ide_hwif_t *hwif) |
1296 | { | 1296 | { |
1297 | struct pci_dev *dev = to_pci_dev(hwif->dev); | 1297 | struct pci_dev *dev = to_pci_dev(hwif->dev); |
1298 | return dev ? pcibus_to_node(dev->bus) : -1; | 1298 | return hwif->dev ? pcibus_to_node(dev->bus) : -1; |
1299 | } | 1299 | } |
1300 | 1300 | ||
1301 | static inline ide_drive_t *ide_get_paired_drive(ide_drive_t *drive) | 1301 | static inline ide_drive_t *ide_get_paired_drive(ide_drive_t *drive) |