Re: [PATCH 1/2] MCE, AMD: Make MC2 decoding part of amd_decoder_opsas well

From: Jacob Shin
Date: Mon Dec 17 2012 - 16:11:21 EST


On Mon, Dec 17, 2012 at 10:08:02PM +0100, Borislav Petkov wrote:
> On Mon, Dec 17, 2012 at 12:59:00PM -0800, Joe Perches wrote:
> > The new pr_emerg is effectively a termination of the
> > previous one. There's no need for 2 consecutive uses
> > of pr_emerg(HW_ERR ...) here.
>
> Actually, in decode_mc5_mce(), the second pr_emerg needs to stay like
> what it is now.
>
> So Jacob, please leave those changes out, I'll sort them out later.

Okay, will do, I'll change fix it in my changes then. But I'll wait
for your feedback tomorrow, before sending out V2.

Thanks!

-Jacob

>
> Thanks.
>
> --
> Regards/Gruss,
> Boris.
>
> Sent from a fat crate under my desk. Formatting is fine.
> --
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/