From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Arvind R <arvino55@gmail.com>
Cc: linux-edac@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 3.6.6 0/3] i82975x_edac: fatal bug fixed
Date: Thu, 22 Nov 2012 09:08:13 -0200 [thread overview]
Message-ID: <20121122090813.603edd55@redhat.com> (raw)
In-Reply-To: <CAA_L+UPNOLg=ZuZofc-uhybXKuBO37XBSGa2vDLYZ7wLkgAb=A@mail.gmail.com>
Hi Arvind,
Em Thu, 22 Nov 2012 14:52:35 +0530
Arvind R <arvino55@gmail.com> escreveu:
> Subject: [PATCH 3.6.6 0/3] i82975x_edac: fatal bug fixed
>
> This patchset fixes fatal crash caused by i82975x_edac driver.
> It consists of 3 parts:
> 1. Move the debug code to conform to the edac_core standard
> instead of using hardcoded ifdefs. Also add some more dram
> info outputs in the debug output.
> 2. Cleanup the long lines and other format errors found. Use
> defines for numeric constants used repeatedly in code.
> 3. Fix the crash caused by wrong memory_layer initialisation
> and dimm_label initialisation. Almost total, but cleaner
> rewrite of csrow_info initialisation. And a terser error
> reporting function.
> The patch has been applied to linux-3.6.6 and tested on
> on Asus P5WDG2 WS Pro mobo with 7 memory configurations,
> 3 symmetric and 4 assymmetric:
> Symmetric:
> 1. 2 ECC DIMMS, one on slot A1 and other on slot B1
> 2. 2 ECC DIMMS, one on slot A2 and other on slot B2
> 3. 4 ECC DIMMS, mobo fully populated.
> Assymmetric:
> 1. 1 ECC DIMM on A1
> 2. 1 ECC DIMM on B1
> 3. 2 ECC DIMMS on A1 and B2
> 4. 3 ECC DIMMS on A1, B2, B2
> Size, label values in /sys/devices/system/edac/mc/mc0 found
> consistent.
This patch series conflict with the one I did fixing the same issues:
http://git.kernel.org/?p=linux/kernel/git/mchehab/linux-edac.git;a=shortlog;h=refs/heads/linux_next
Could you please rebase it on the top of it?
Thanks,
Mauro
>
> Tested-by: Arvind R. <arvino55@gmail.com>
> Signed-off-by: Arvind R. <arvino55@gmail.com>
> ---
> drivers/edac/i82975x_edac.c | 361 +++++++++++++++++-----------------
> 1 file changed, 185 insertions(+), 176 deletions(-)
--
Regards,
Mauro
prev parent reply other threads:[~2012-11-22 18:38 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-22 9:22 [PATCH 3.6.6 0/3] i82975x_edac: fatal bug fixed Arvind R
2012-11-22 11:08 ` Mauro Carvalho Chehab [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20121122090813.603edd55@redhat.com \
--to=mchehab@redhat.com \
--cc=arvino55@gmail.com \
--cc=linux-edac@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.