Path: csiph.com!goblin2!goblin1!goblin.stu.neva.ru!eternal-september.org!feeder.eternal-september.org!newsfeed.Update.UU.SE!news.Update.UU.SE!itgate.net!bofh.it!news.nic.it!robomod From: "Zhang, Jonathan Zhixiong" Newsgroups: linux.kernel Subject: Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map Date: Thu, 06 Aug 2015 15:50:01 +0200 Message-ID: References: X-Original-To: Matt Fleming User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: robomod@news.nic.it List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Approved: robomod@news.nic.it Lines: 34 Organization: linux.* mail to news gateway X-Original-Cc: Borislav Petkov , Will Deacon , 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" , "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" X-Original-Date: Thu, 6 Aug 2015 06:47:23 -0700 X-Original-Message-ID: <55C3656B.5090808@codeaurora.org> X-Original-References: <1438292109-4170-1-git-send-email-zjzhang@codeaurora.org> <20150803112658.GE10501@arm.com> <20150803162354.GQ2725@codeblueprint.co.uk> <20150804042552.GG31787@nazgul.tnic> <55C0DD30.9070800@codeaurora.org> <20150805092125.GB4332@codeblueprint.co.uk> <55C232BC.1010107@codeaurora.org> <20150805161000.GE4332@codeblueprint.co.uk> <20150805161325.GF4332@codeblueprint.co.uk> X-Original-Sender: linux-kernel-owner@vger.kernel.org Xref: csiph.com linux.kernel:1201733 On 8/5/2015 9:13 AM, Matt Fleming wrote: > On Wed, 05 Aug, at 05:10:00PM, Matt Fleming wrote: >> On Wed, 05 Aug, at 08:58:52AM, Zhang, Jonathan Zhixiong wrote: >>> >>> Yes, absolutely. It has been in my mind, sorry I was tied up in the last >>> few days. I wonder what I may missed, I certainly do x86 build test with >>> versions of the patch set. >>> >>> I will try today and report back. >> >> No problem. >> >> I'd suggest moving the implementation into include/acpi/apei.h and >> leaving it as static inline instead of implementing it out of line in >> arch/x86/kernel/acpi/apei.c. > > Yeah, I didn't mean include/acpi/apei.h but the architecture-specific > equivalent. > Thank you Matt. It builds well on x86 with default configuration, but not if APEI feature is turned on. This is my fault. I just sent a new version of the patch set out. -- Jonathan (Zhixiong) Zhang The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project -- 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/