Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753001AbbHCL1G (ORCPT ); Mon, 3 Aug 2015 07:27:06 -0400 Received: from foss.arm.com ([217.140.101.70]:53089 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752143AbbHCL1E (ORCPT ); Mon, 3 Aug 2015 07:27:04 -0400 Date: Mon, 3 Aug 2015 12:26:58 +0100 From: Will Deacon To: "Jonathan (Zhixiong) Zhang" Cc: Matt Fleming , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "x86@kernel.org" , "tony.luck@gmail.com" , "ard.biesheuvel@linaro.org" , "fu.wei@linaro.org" , "al.stone@linaro.org" , "bp@alien8.de" , "rjw@rjwysocki.net" , "lenb@kernel.org" , "ying.huang@intel.com" , Catalin Marinas , "leif.lindholm@linaro.org" , "hanjun.guo@linaro.org" , "linux-efi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-acpi@vger.kernel.org" , "linux-arm-msm@vger.kernel.org" , "linaro-acpi@lists.linaro.org" , "vgandhi@codeaurora.org" , "timur@codeaurora.org" Subject: Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map Message-ID: <20150803112658.GE10501@arm.com> References: <1438292109-4170-1-git-send-email-zjzhang@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1438292109-4170-1-git-send-email-zjzhang@codeaurora.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1530 Lines: 35 Hi Jonathan, On Thu, Jul 30, 2015 at 10:35:04PM +0100, Jonathan (Zhixiong) Zhang wrote: > From: "Jonathan (Zhixiong) Zhang" > > On a platform with APEI (ACPI Platform Error Interface) enabled, firmware > updates a memory region with hardware error record using nocache > attribute. When OS reads the region, since it maps the region with > cacahed attribute even though EFI memory map defines this region as > uncached, OS gets stale data and errorneously reports there is no new > HW error. > > When ghes driver maps the memory region, it uses the cache attribute > according to EFI memory map, if EFI memory map feature is enabled > at runtime. > > Since both arch/x86 and arch/ia64 implemented architecture agnostic EFI > memory map attribue lookup function efi_memattributes(), the code is > moved from arch/x86 into EFI subsystem and is declared as __weak; archs > other than ia64 should not override the default implementation. > > V9: > 1. Rebased to arm64-upstream-14543 of arm64/master. > 2. Match strict MM type in arch_apei_get_mem_attribute(). I guess this is all going via Matt's tree? I'm happy to take the new memory type in arch/arm64/ if there's nothing currently queued, but I suspect it makes more sense for it to stay together. Will -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/