[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



On 5/29/2014 11:26 AM, Jan Beulich wrote:
Aravind Gopalakrishnan <aravind.gopalakrishnan@xxxxxxx> 05/28/14 5:55 PM >>>
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
Changes in V3:
     >- More coding style fixes.
Signed-off-by: Aravind Gopalakrishnan <aravind.gopalakrishnan@xxxxxxx>
Looks like you lost Christoph's ack - everything else looks fine now.


Yeah.was unsure of adding it just in case I missed something and had to spin another patch.
Figured if everything is OK, then you could add it while committing :)

Thanks,
-Aravind.


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.