From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Subject: Re: [PATCH 2/2] edac: add support for Amazon's Annapurna Labs EDAC Date: Thu, 13 Jun 2019 09:57:33 +1000 Message-ID: <28912f8d0a39de519653472a3686fbd7317d18d3.camel@kernel.crashing.org> References: <1ae5e7a3464f9d8e16b112cd371957ea20472864.camel@kernel.crashing.org> <68446361fd1e742b284555b96b638fe6b5218b8b.camel@kernel.crashing.org> <20190611115651.GD31772@zn.tnic> <6df5a17bb1c900dc69b991171e55632f40d9426f.camel@kernel.crashing.org> <20190612034813.GA32652@zn.tnic> <08bd58dc0045670223f8d3bbc8be774505bd3ddf.camel@kernel.crashing.org> <20190612074242.53a4cf56@coco.lan> <20190612110039.GH32652@zn.tnic> <20190612084213.4fb9e054@coco.lan> <7705227ea831793cc9e45af32e0da8f5547cb14d.camel@kernel.crashing.org> <20190612122504.GI32652@zn.tnic> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20190612122504.GI32652@zn.tnic> Sender: linux-kernel-owner@vger.kernel.org To: Borislav Petkov Cc: Mauro Carvalho Chehab , James Morse , "Hawa, Hanna" , "robh+dt@kernel.org" , "Woodhouse, David" , "paulmck@linux.ibm.com" , "mark.rutland@arm.com" , "gregkh@linuxfoundation.org" , "davem@davemloft.net" , "nicolas.ferre@microchip.com" , "devicetree@vger.kernel.org" , "Shenhar, Talel" , "linux-kernel@vger.kernel.org" , "Chocron, Jonathan" , "Krupnik, Ronen" , "linux-edac@vger.kernel.org" , Hanoch, Uri List-Id: devicetree@vger.kernel.org On Wed, 2019-06-12 at 14:25 +0200, Borislav Petkov wrote: > > But for the main case that really needs to be in the kernel, which is > > DRAM, the recovery can usually be contained to the MC driver anyway. > > Right, if that is enough to handle the error properly. > > The memory-failure.c example I gave before is the error reporting > mechanism (x86 MCA) calling into the mm subsystem to poison and isolate > page frames which are known to contain errors. So you have two things > talking to each other. And none of them is an EDAC driver... I mean yes, the network drivers talk to the network stack, or even the memory allocator :-) I still don't see how that requires a big platform coordinator... Ben.