Messages in this thread | | | Date | Mon, 17 Dec 2012 15:11:24 -0600 | From | Jacob Shin <> | Subject | Re: [PATCH 1/2] MCE, AMD: Make MC2 decoding part of amd_decoder_ops as well |
| |
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. > -- >
| |