All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Borislav Petkov <bp@amd64.org>
Cc: Tony Luck <tony.luck@intel.com>,
	EDAC devel <linux-edac@vger.kernel.org>, X86-ML <x86@kernel.org>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/2] x86, MCE: Use MCE decoder notifier chain
Date: Thu, 04 Aug 2011 21:56:58 -0300	[thread overview]
Message-ID: <4E3B3FDA.1050706@redhat.com> (raw)
In-Reply-To: <20110802115853.GC15897@aftab>

Boris,

Em 02-08-2011 08:58, Borislav Petkov escreveu:
> Ping?
> 
> Mauro, any progress on those below?

Sorry for taking a long time on that. I'm testing it right now, together with
a few other patches for i7core.

I've added it (with a few conflict fixes) at:
	ssh://master.kernel.org/pub/scm/linux/kernel/git/mchehab/edac.git  linux_next

If everything works as before, I'll send it to Linus tomorrow, assuming that
linux-next will catch them.

Thanks,
Mauro

> 
> Thanks.
> 
> On Mon, Jul 18, 2011 at 10:24:44AM -0400, Borislav Petkov wrote:
>> From: Borislav Petkov <borislav.petkov@amd.com>
>>
>> Hi Mauro, Tony,
>>
>> the following two patches convert i7core_edac to use the MCE decoder
>> notifier chain, thus getting rid of the custom edac_mce hook. There
>> should be no functionality change following from this along with a nice
>> diffstat of removing ca 100 lines.
>>
>> The patches are ontop of the latest tip/ras/core and are available at
>>
>> git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git fix-mce-logging
>>
>> I'd like to ask you guys to please give them a run to make sure I
>> haven't broken i7core_edac.
>>
>> Thanks.
>>
>>  arch/x86/kernel/cpu/mcheck/mce.c |   18 ++++-------
>>  drivers/edac/Kconfig             |    3 --
>>  drivers/edac/Makefile            |    1 -
>>  drivers/edac/edac_mce.c          |   61 --------------------------------------
>>  drivers/edac/i7core_edac.c       |   51 +++++++++++++++----------------
>>  include/linux/edac_mce.h         |   31 -------------------
>>  6 files changed, 31 insertions(+), 134 deletions(-)
>>
>>
>>
> 


  reply	other threads:[~2011-08-05  0:58 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-18 14:24 [PATCH 0/2] x86, MCE: Use MCE decoder notifier chain Borislav Petkov
2011-07-18 14:24 ` [PATCH 1/2] x86, MCE: Use notifier chain only for MCE decoding Borislav Petkov
2011-07-18 14:24 ` [PATCH 2/2] i7core_edac: Drop the edac_mce facility Borislav Petkov
2011-08-02 11:58 ` [PATCH 0/2] x86, MCE: Use MCE decoder notifier chain Borislav Petkov
2011-08-05  0:56   ` Mauro Carvalho Chehab [this message]
2011-08-05  7:57     ` Borislav Petkov
2011-08-05 10:35       ` Ingo Molnar

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=4E3B3FDA.1050706@redhat.com \
    --to=mchehab@redhat.com \
    --cc=bp@amd64.org \
    --cc=linux-edac@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tony.luck@intel.com \
    --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 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.