From: Aravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>
To: <bp@suse.de>, <tglx@linutronix.de>, <mingo@redhat.com>, <hpa@zytor.com>
Cc: <x86@kernel.org>, <linux-kernel@vger.kernel.org>,
Aravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>
Subject: [PATCH V2 0/3] Updates to mce_amd_inj module
Date: Fri, 25 Sep 2015 09:20:48 -0500 [thread overview]
Message-ID: <1443190851-2172-1-git-send-email-Aravind.Gopalakrishnan@amd.com> (raw)
This patchset is mostly a resend of earlier patches which got acceptance
into maintainer's tree but did not make it into upstream.
Original versions of patches 2 and 3 of this patchset-
a. http://marc.info/?l=linux-edac&m=143327679901253&w=2
b. http://marc.info/?l=linux-edac&m=143392472509250&w=2
Changes from original versions-
Patch 2: Earlier version did not include the update to README file.
That was because README attribute was introduced in separate
patch (http://marc.info/?l=linux-edac&m=143327679301249&w=2)
But subsequently, only parts of it made it upstream.
Patch 3: NBCFG register offset definition exists already in
drivers/edac/amd64_edac.h which was included in earlier version.
But any dependency with drivers/edac/ was removed when the file
moved to arch/x86/ras. So, I have redefined it here.
I can move it to a different place if required.
With changes to the location of the module, I am adapting the patches
on top of latest tip.
Patch 1: Abort write file operation on invalid input
Patch 2: Extend flags attribute to accept values of 'df', 'th'.
These values will be used to trigger deferred error/threshold
error apic interrupts respectively.
Patch 3: Modify injection mechanism for bank 4 errors. Since they are
logged or reported only on NBC, we make sure that we inject on
the correct core here.
Changes from V1 (per Boris' comments)
- Instead of ignoring unwanted characters, return early on wrong input
from user as this way, we can save a copy from user input for bad values.
- With above change, modified patch title and commit message a bit.
- Use 2 letter strings to indicate error injection for Deferred/Threshold
error interrupts too.
Aravind Gopalakrishnan (3):
RAS, mce_amd_inj: Return early on invalid input
RAS, mce_amd_inj: Add capability to trigger apic interrupts
RAS, mce_amd_inj: Inject errors on NBC for bank 4 errors
arch/x86/ras/mce_amd_inj.c | 109 ++++++++++++++++++++++++++++++++++++++++++---
1 file changed, 102 insertions(+), 7 deletions(-)
--
2.4.0
next reply other threads:[~2015-09-25 19:17 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-25 14:20 Aravind Gopalakrishnan [this message]
2015-09-25 14:20 ` [PATCH V2 1/3] RAS, mce_amd_inj: Return early on invalid input Aravind Gopalakrishnan
2015-09-25 14:20 ` [PATCH V2 2/3] RAS, mce_amd_inj: Add capability to trigger apic interrupts Aravind Gopalakrishnan
2015-09-25 14:20 ` [PATCH V2 3/3] RAS, mce_amd_inj: Inject errors on NBC for bank 4 errors Aravind Gopalakrishnan
2015-09-28 9:06 ` [PATCH V2 0/3] Updates to mce_amd_inj module Borislav Petkov
2015-10-12 9:22 [PATCH 0/6] tip-queue 2015-10-12 Borislav Petkov
2015-10-12 9:22 ` [PATCH 1/6] x86/mce: Don't clear shared banks on Intel when offlining CPUs Borislav Petkov
2015-10-12 9:22 ` [PATCH 2/6] x86/ras/mce_amd_inj: Return early on invalid input Borislav Petkov
2015-10-12 14:31 ` [tip:ras/core] " tip-bot for Aravind Gopalakrishnan
2015-10-12 9:22 ` [PATCH 3/6] x86/ras/mce_amd_inj: Trigger deferred and thresholding errors interrupts Borislav Petkov
2015-10-12 14:31 ` [tip:ras/core] " tip-bot for Aravind Gopalakrishnan
2015-10-12 9:22 ` [PATCH 4/6] x86/ras/mce_amd_inj: Inject bank 4 errors on the NBC Borislav Petkov
2015-10-12 14:31 ` [tip:ras/core] " tip-bot for Aravind Gopalakrishnan
2015-10-12 9:22 ` [PATCH 5/6] x86/microcode/amd: Extract current patch level read to a function Borislav Petkov
2015-10-12 14:32 ` [tip:ras/core] " tip-bot for Borislav Petkov
2015-10-12 9:22 ` [PATCH 6/6] x86/microcode/amd: Do not overwrite final patch levels Borislav Petkov
2015-10-12 14:32 ` [tip:ras/core] " tip-bot for Borislav Petkov
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=1443190851-2172-1-git-send-email-Aravind.Gopalakrishnan@amd.com \
--to=aravind.gopalakrishnan@amd.com \
--cc=bp@suse.de \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=tglx@linutronix.de \
--cc=x86@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome