From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Egger Subject: Re: [PATCH] MCE: consolidate AMD initialization Date: Fri, 26 Oct 2012 10:26:29 +0200 Message-ID: <508A4935.6090801@amd.com> References: <508A490B.4050407@amd.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------010209000300050807000308" Return-path: In-Reply-To: <508A490B.4050407@amd.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org --------------010209000300050807000308 Content-Type: text/plain; charset="ISO-8859-15" Content-Transfer-Encoding: 7bit On 10/26/12 10:25, Christoph Egger wrote: > > Move AMD specific initialization to AMD files. > > Signed-off-by: Christoph Egger > -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85689 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 --------------010209000300050807000308 Content-Type: text/plain; charset="us-ascii"; name="xen_mce_init.diff" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="xen_mce_init.diff" Content-Description: xen_mce_init.diff diff -r 7abb25095de0 xen/arch/x86/cpu/mcheck/mce.c --- a/xen/arch/x86/cpu/mcheck/mce.c Thu Oct 25 13:15:19 2012 +0200 +++ b/xen/arch/x86/cpu/mcheck/mce.c Fri Oct 26 08:50:17 2012 +0200 @@ -560,30 +560,6 @@ void mcheck_mca_clearbanks(struct mca_ba } } -static enum mcheck_type amd_mcheck_init(struct cpuinfo_x86 *ci) -{ - enum mcheck_type rc = mcheck_none; - - switch (ci->x86) { - case 6: - rc = amd_k7_mcheck_init(ci); - break; - - default: - /* Assume that machine check support is available. - * The minimum provided support is at least the K8. */ - case 0xf: - rc = amd_k8_mcheck_init(ci); - break; - - case 0x10 ... 0x17: - rc = amd_f10_mcheck_init(ci); - break; - } - - return rc; -} - /*check the existence of Machine Check*/ int mce_available(struct cpuinfo_x86 *c) { diff -r 7abb25095de0 xen/arch/x86/cpu/mcheck/mce.h --- a/xen/arch/x86/cpu/mcheck/mce.h Thu Oct 25 13:15:19 2012 +0200 +++ b/xen/arch/x86/cpu/mcheck/mce.h Fri Oct 26 08:50:17 2012 +0200 @@ -39,10 +39,7 @@ enum mcheck_type { }; /* Init functions */ -enum mcheck_type amd_k7_mcheck_init(struct cpuinfo_x86 *c); -enum mcheck_type amd_k8_mcheck_init(struct cpuinfo_x86 *c); -enum mcheck_type amd_f10_mcheck_init(struct cpuinfo_x86 *c); - +enum mcheck_type amd_mcheck_init(struct cpuinfo_x86 *c); enum mcheck_type intel_mcheck_init(struct cpuinfo_x86 *c, bool_t bsp); void intel_mcheck_timer(struct cpuinfo_x86 *c); diff -r 7abb25095de0 xen/arch/x86/cpu/mcheck/mce_amd.c --- a/xen/arch/x86/cpu/mcheck/mce_amd.c Thu Oct 25 13:15:19 2012 +0200 +++ b/xen/arch/x86/cpu/mcheck/mce_amd.c Fri Oct 26 08:50:17 2012 +0200 @@ -98,3 +98,28 @@ mc_amd_addrcheck(uint64_t status, uint64 BUG(); return 0; } + +enum mcheck_type +amd_mcheck_init(struct cpuinfo_x86 *ci) +{ + enum mcheck_type rc = mcheck_none; + + switch (ci->x86) { + case 6: + rc = amd_k7_mcheck_init(ci); + break; + + default: + /* Assume that machine check support is available. + * The minimum provided support is at least the K8. */ + case 0xf: + rc = amd_k8_mcheck_init(ci); + break; + + case 0x10 ... 0x17: + rc = amd_f10_mcheck_init(ci); + break; + } + + return rc; +} diff -r 7abb25095de0 xen/arch/x86/cpu/mcheck/mce_amd.h --- a/xen/arch/x86/cpu/mcheck/mce_amd.h Thu Oct 25 13:15:19 2012 +0200 +++ b/xen/arch/x86/cpu/mcheck/mce_amd.h Fri Oct 26 08:50:17 2012 +0200 @@ -1,6 +1,10 @@ #ifndef _MCHECK_AMD_H #define _MCHECK_AMD_H +enum mcheck_type amd_k7_mcheck_init(struct cpuinfo_x86 *c); +enum mcheck_type amd_k8_mcheck_init(struct cpuinfo_x86 *c); +enum mcheck_type amd_f10_mcheck_init(struct cpuinfo_x86 *c); + int mc_amd_recoverable_scan(uint64_t status); int mc_amd_addrcheck(uint64_t status, uint64_t misc, int addrtype); --------------010209000300050807000308 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --------------010209000300050807000308--