diff options
author | Badari Pulavarty <pbadari@gmail.com> | 2007-04-13 11:13:42 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-04-16 21:09:18 -0400 |
commit | 6f29e35e2d4cdbc3e8785982314e54ec5df4ad37 (patch) | |
tree | 60b1e8cba4f69a5a1e24fa32cfde38f944f986b3 | |
parent | 1d00e832f0d65039abdebf83054b323a9aa05687 (diff) |
cache_k8_northbridges() overflows beyond allocation
cache_k8_northbridges() is storing config values to incorrect locations
(in flush_words) and also its overflowing beyond the allocation, causing
slab verification failures.
Signed-off-by: Badari Pulavarty <pbadari@us.ibm.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | arch/x86_64/kernel/k8.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86_64/kernel/k8.c b/arch/x86_64/kernel/k8.c index 6416682d33d0..bc11b32e8b4d 100644 --- a/arch/x86_64/kernel/k8.c +++ b/arch/x86_64/kernel/k8.c | |||
@@ -61,8 +61,8 @@ int cache_k8_northbridges(void) | |||
61 | dev = NULL; | 61 | dev = NULL; |
62 | i = 0; | 62 | i = 0; |
63 | while ((dev = next_k8_northbridge(dev)) != NULL) { | 63 | while ((dev = next_k8_northbridge(dev)) != NULL) { |
64 | k8_northbridges[i++] = dev; | 64 | k8_northbridges[i] = dev; |
65 | pci_read_config_dword(dev, 0x9c, &flush_words[i]); | 65 | pci_read_config_dword(dev, 0x9c, &flush_words[i++]); |
66 | } | 66 | } |
67 | k8_northbridges[i] = NULL; | 67 | k8_northbridges[i] = NULL; |
68 | return 0; | 68 | return 0; |