Re: [Xen-devel] [PATCH V2] x86, mce: Consolidate AMD mcheck initialization

[Aravind Gopalakrishnan <aravind.gopalakrishnan@xxxxxxx>]

On 5/28/2014 5:02 AM, Egger, Christoph wrote:
> On 27.05.14 17:14, Aravind Gopalakrishnan wrote:
> > amd_k8.c did a lot of common work and very little K8
> > specific work. So merge init functions of amd_f10.c and
> > amd_k8.c and move it into the common amd_mcheck_init
> > handler. With that done, there is not much left in either
> > files, so fold all code into just one file - mce_amd.c
> >
> > While at it, update the comments regarding documentation
> > with correct URL's and revision numbers.
> >
> > Also, update copyright info.
> >
> > Changes in V2:
> >      - Fix coding style issues
>  From the functional perspective this patch is fine.
> Regarding coding style, make Jan happy, then you have my
>
> Acked-by: Christoph Egger <chegger@xxxxxxxxx>
Thanks.

I'll address the issues Jan pointed out in the earlier reply and re-send.
-Aravind.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel