diff options
| author | Borislav Petkov <borislav.petkov@amd.com> | 2011-04-08 15:05:21 +0200 |
|---|---|---|
| committer | Borislav Petkov <borislav.petkov@amd.com> | 2011-04-26 16:18:12 +0200 |
| commit | f030ddfb3752df36bb73285353374fc04feabb80 (patch) | |
| tree | 1b8f21ba1b29b36dcb748366b4fdcad2754c68f4 /drivers | |
| parent | 4949603a6fabf3a54cbd7be6df1681789abfca7d (diff) | |
| download | lwn-f030ddfb3752df36bb73285353374fc04feabb80.tar.gz lwn-f030ddfb3752df36bb73285353374fc04feabb80.zip | |
amd64_edac: Remove node interleave warning
This warning was wrongfully added for a normal condition - intlvsel
actually selects the destination node when node interleaving is enabled
and it is not a mismatch. For a detailed example, see section 2.8.10.2
"Node Interleaving" in F10h BKDG.
Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/edac/amd64_edac.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index 4b4071ee2f5c..601142a0738a 100644 --- a/drivers/edac/amd64_edac.c +++ b/drivers/edac/amd64_edac.c @@ -1401,12 +1401,8 @@ static int f1x_match_to_this_node(struct amd64_pvt *pvt, unsigned range, return -EINVAL; } - if (intlv_en && - (intlv_sel != ((sys_addr >> 12) & intlv_en))) { - amd64_warn("Botched intlv bits, en: 0x%x, sel: 0x%x\n", - intlv_en, intlv_sel); + if (intlv_en && (intlv_sel != ((sys_addr >> 12) & intlv_en))) return -EINVAL; - } sys_addr = f1x_swap_interleaved_region(pvt, sys_addr); |