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 /arch | |
| 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>
Diffstat (limited to 'arch')
| -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; |
