All of lore.kernel.org
 help / color / mirror / Atom feed
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


             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 \
    --subject='Re: [PATCH V2 0/3] Updates to mce_amd_inj module' \
    /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

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.