From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Egger Subject: Re: [PATCH] MCE: consolidate AMD initialization Date: Mon, 29 Oct 2012 11:03:15 +0100 Message-ID: <508E5463.1040100@amd.com> References: <508A490B.4050407@amd.com> <508A4935.6090801@amd.com> <508E5EF702000078000A5039@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <508E5EF702000078000A5039@nat28.tlf.novell.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: Jan Beulich Cc: "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org On 10/29/12 10:48, Jan Beulich wrote: >>>> On 26.10.12 at 10:26, Christoph Egger wrote: >> On 10/26/12 10:25, Christoph Egger wrote: >>> >>> Move AMD specific initialization to AMD files. >>> >>> Signed-off-by: Christoph Egger > > Let's do this properly: There's no K7 supporting 64-bit mode afaict, > so rather than moving around the call to amd_k7_mcheck_init() > can't we just drop it and the whole (inconsistently named) k7.c file? I think it is better to apply this first and then remove k7 to simplify backporting if needed/wanted. > Also (not in this patch of course), I'd prefer mce_amd_quirks.c > to get merged into mce_amd.c now that we have the latter. After some thinking is there some good reason to do this? Christoph -- ---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