From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757458Ab0JYQj0 (ORCPT ); Mon, 25 Oct 2010 12:39:26 -0400 Received: from www.tglx.de ([62.245.132.106]:35975 "EHLO www.tglx.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755923Ab0JYQjY (ORCPT ); Mon, 25 Oct 2010 12:39:24 -0400 Date: Mon, 25 Oct 2010 18:38:40 +0200 (CEST) From: Thomas Gleixner To: Andi Kleen cc: Huang Ying , Ingo Molnar , Len Brown , "linux-kernel@vger.kernel.org" , "linux-acpi@vger.kernel.org" , Borislav Petkov , "H. Peter Anvin" , Don Zickus , Linus Torvalds , Andrew Morton , Mauro Carvalho Chehab , Tony Luck Subject: Re: [NAK] Re: [PATCH -v2 9/9] ACPI, APEI, Generic Hardware Error Source POLL/IRQ/NMI notification type support In-Reply-To: <20101025091913.GA17622@basil.fritz.box> Message-ID: References: <1287992610-14996-1-git-send-email-ying.huang@intel.com> <1287992610-14996-10-git-send-email-ying.huang@intel.com> <20101025084553.GA27119@elte.hu> <1287997112.2862.322.camel@yhuang-dev> <20101025091913.GA17622@basil.fritz.box> User-Agent: Alpine 2.00 (LFD 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 25 Oct 2010, Andi Kleen wrote: > > > Sigh, please integrate all this into EDAC (drivers/edac/) properly, instead of > > > turning it into YET ANOTHER hardware vendor special hw-errors thing. We can do > > > better than this. EDAC is almost there: it has support for Nehalem, AMD, a couple > > > of older chips. > > > > I think APEI (ACPI Platform Error Interface) is another driver. Why > > integrate two drivers? > > Yes they're solving quite different problems from EDAC with different > interfaces and for different devices in the ACPI space. > > The earlier nack seems to be based on a lot of confusion on what the code > does. Errm. That patch series carries a lot of other weird stuff including a new "memory allocator", a new ioremap implementation private to the acpi code and new character device driver for hardware error reporting. > Subject: [PATCH -v2 5/9] Hardware error device core > > Hardware error device is a kind of device which can report hardware > errors. The examples of hardware error device include APEI GHES, PCIe > AER, etc. > > Hardware error device core in this patch provides common services for > various hardware error devices. But it does not even make an attempt to explain why this error reporting cannot be done via the existing interfaces and why they can't be extended to fit your needs. What's so special about APEI GHES and PCIe AER that we need another incompatible "just fits your needs" ABI which makes tooling folks deal with another completely different interface ? The only explanation I have is that you are simply not willing to work with others and this is just another proof of a repeating problem. Thanks, tglx