Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754266AbbHCQWb (ORCPT ); Mon, 3 Aug 2015 12:22:31 -0400 Received: from mail-wi0-f170.google.com ([209.85.212.170]:33937 "EHLO mail-wi0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754172AbbHCQW1 (ORCPT ); Mon, 3 Aug 2015 12:22:27 -0400 Date: Mon, 3 Aug 2015 17:22:23 +0100 From: Matt Fleming To: "Jonathan (Zhixiong) Zhang" Cc: Matt Fleming , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, ying.huang@intel.com, tony.luck@gmail.com, fu.wei@linaro.org, al.stone@linaro.org, bp@alien8.de, rjw@rjwysocki.net, catalin.marinas@arm.com, will.deacon@arm.com, hanjun.guo@linaro.org, linux-efi@vger.kernel.org, linux-kernel@vger.kernel.org, linaro-acpi@lists.linaro.org, timur@codeaurora.org Subject: Re: [PATCH V9 2/5] x86: acpi: implement arch_apei_get_mem_attributes() Message-ID: <20150803162223.GP2725@codeblueprint.co.uk> References: <1438292109-4170-1-git-send-email-zjzhang@codeaurora.org> <1438292109-4170-3-git-send-email-zjzhang@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1438292109-4170-3-git-send-email-zjzhang@codeaurora.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3587 Lines: 83 On Thu, 30 Jul, at 02:35:06PM, Jonathan (Zhixiong) Zhang wrote: > From: "Jonathan (Zhixiong) Zhang" > > ... to allow arch specific implementation of getting page > protection type associated with a physical address. > > On x86, we currently have no way to lookup the EFI memory map > attributes for a region in a consistent way because the > memmap is discarded after efi_free_boot_services(). So if > you call efi_mem_attributes() during boot and at runtime, > you could theoretically see different attributes. > > Since we are yet to see any x86 platforms that require > anything other than PAGE_KERNEL (some arm64 platforms > require the equivalent of PAGE_KERNEL_NOCACHE), return that > until we know differently. > > Signed-off-by: Jonathan (Zhixiong) Zhang > --- > arch/x86/kernel/acpi/apei.c | 19 +++++++++++++++++++ > include/acpi/apei.h | 1 + > 2 files changed, 20 insertions(+) > > diff --git a/arch/x86/kernel/acpi/apei.c b/arch/x86/kernel/acpi/apei.c > index c280df6b2aa2..675bd46c4e17 100644 > --- a/arch/x86/kernel/acpi/apei.c > +++ b/arch/x86/kernel/acpi/apei.c > @@ -60,3 +60,22 @@ void arch_apei_flush_tlb_one(unsigned long addr) > { > __flush_tlb_one(addr); > } > + > +static inline pgprot_t arch_apei_get_mem_attribute(phys_addr_t addr) > +{ > + > + /* > + * We currently have no way to lookup the EFI memory map > + * attributes for a region in a consistent way because the > + * memmap is discarded after efi_free_boot_services(). So if > + * you call efi_mem_attributes() during boot and at runtime, > + * you could theoretically see different attributes. > + * > + * Since we are yet to see any x86 platforms that require > + * anything other than PAGE_KERNEL (some arm64 platforms > + * require the equivalent of PAGE_KERNEL_NOCACHE), return that > + * until we know differently. > + */ > + > + return __pgprot(PAGE_KERNEL); > +} > diff --git a/include/acpi/apei.h b/include/acpi/apei.h > index 284801ac7042..64a12ce9880b 100644 > --- a/include/acpi/apei.h > +++ b/include/acpi/apei.h > @@ -46,6 +46,7 @@ int erst_clear(u64 record_id); > int arch_apei_enable_cmcff(struct acpi_hest_header *hest_hdr, void *data); > void arch_apei_report_mem_error(int sev, struct cper_sec_mem_err *mem_err); > void arch_apei_flush_tlb_one(unsigned long addr); > +pgprot_t arch_apei_get_mem_attribute(phys_addr_t addr); > > #endif > #endif This doesn't compile :( /home/matt/src/kernels/efi/arch/x86/kernel/acpi/apei.c:64:24: error: static declaration of ‘arch_apei_get_mem_attribute’ follows non-static declaration static inline pgprot_t arch_apei_get_mem_attribute(phys_addr_t addr) ^ In file included from /home/matt/src/kernels/efi/arch/x86/kernel/acpi/apei.c:15:0: /home/matt/src/kernels/efi/include/acpi/apei.h:48:10: note: previous declaration of ‘arch_apei_get_mem_attribute’ was here pgprot_t arch_apei_get_mem_attribute(phys_addr_t addr); ^ /home/matt/src/kernels/efi/arch/x86/kernel/acpi/apei.c: In function ‘arch_apei_get_mem_attribute’: /home/matt/src/kernels/efi/arch/x86/kernel/acpi/apei.c:80:2: error: incompatible types when initializing type ‘long unsigned int’ using type ‘pgprot_t’ return __pgprot(PAGE_KERNEL); -- Matt Fleming, Intel Open Source Technology Center -- 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/