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 14:34:39 +0100 Message-ID: <508E85EF.9060908@amd.com> References: <508A490B.4050407@amd.com> <508A4935.6090801@amd.com> <508E5EF702000078000A5039@nat28.tlf.novell.com> <508E5463.1040100@amd.com> <508E670802000078000A5150@nat28.tlf.novell.com> <508E5B3C.6030801@amd.com> <508E6D2E02000078000A51AD@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <508E6D2E02000078000A51AD@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 11:49, Jan Beulich wrote: >>>> On 29.10.12 at 11:32, Christoph Egger wrote: >> On 10/29/12 11:22, Jan Beulich wrote: >>>>>> On 29.10.12 at 11:03, Christoph Egger wrote: >>>> 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. >>> >>> I'm not seeing these changes as backporting candidates. >> >> I had SLES in mind. >> >>>>> 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? >>> >>> Imo it had been there simply because there was no mce_amd.c at >>> the time it got introduced (and afaics it should nevertheless have >>> been named mce_amd.c from the beginning). >> >> I have a patch ready that removes k7 support which is on top of this >> init cleanup patch. >> I also have a patch which merges mce_amd_quirks into mce_amd.c on >> top of the k7 removal. >> In which order do you want them? > > Okay, if you got them done already, let's go with the order you > have. Patches sent. 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