diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2013-09-09 09:23:19 +0300 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2013-10-07 19:09:34 +0200 |
| commit | 4f9bf7e2fb7035725c2899db0d00d77007d9113c (patch) | |
| tree | b17697e1c06d8bfaf80cc535152ec61a6a910088 /src/mainboard/amd/thatcher/mptable.c | |
| parent | 11b47801b2c3abc3bb8c523b722fafa1103bff45 (diff) | |
| download | coreboot-4f9bf7e2fb7035725c2899db0d00d77007d9113c.tar.gz coreboot-4f9bf7e2fb7035725c2899db0d00d77007d9113c.tar.bz2 coreboot-4f9bf7e2fb7035725c2899db0d00d77007d9113c.zip | |
AMD hudson yangtze: Fix corruption of a global ramstage variable
A late for loop may reference over the current array allocation
and corrupt an unrelated global variable. As a quick fix bumb the
size of the array allocation uniformly to 6.
We missed these boards for commit 9c7d73ca because the arrays
had been renamed.
Change-Id: Iff2f2a0090d9302576bc72195d2a3f6fa37ce29a
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: http://review.coreboot.org/3954
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Reviewed-by: Bruce Griffith <Bruce.Griffith@se-eng.com>
Diffstat (limited to 'src/mainboard/amd/thatcher/mptable.c')
| -rw-r--r-- | src/mainboard/amd/thatcher/mptable.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mainboard/amd/thatcher/mptable.c b/src/mainboard/amd/thatcher/mptable.c index 8253b2c9cfe8..10d2fea31a72 100644 --- a/src/mainboard/amd/thatcher/mptable.c +++ b/src/mainboard/amd/thatcher/mptable.c @@ -30,7 +30,7 @@ //-#define IO_APIC_ID CONFIG_MAX_PHYSICAL_CPUS + 1 #define IO_APIC_ID CONFIG_MAX_CPUS -extern u8 bus_hudson[3]; +extern u8 bus_hudson[6]; extern u32 bus_type[256]; extern u32 sbdn_hudson; |