From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: Possible regression in next-20150323 due to "ARM, arm64: kvm: get rid of the bounce page" Date: Tue, 31 Mar 2015 11:58:21 -0700 Message-ID: <7hbnj99epe.fsf@deeprootsystems.com> References: <20150324004537.GA24816@verge.net.au> <20150324161358.GA694@kahuna> <20150326003939.GA25368@verge.net.au> <20150326133631.GB2805@arm.com> <20150327002554.GA5527@verge.net.au> <20150327100612.GB1562@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150327100612.GB1562@arm.com> (Will Deacon's message of "Fri, 27 Mar 2015 10:06:12 +0000") List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Will Deacon , Ard Biesheuvel Cc: Nishanth Menon , Magnus Damm , Russell King - ARM Linux , Arnd Bergmann , "linux-sh@vger.kernel.org" , Marc Zyngier , Catalin Marinas , Tyler Baker , Simon Horman , "grygorii.strashko@linaro.org" , "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" List-Id: linux-omap@vger.kernel.org Will, Ard, Will Deacon writes: > On Fri, Mar 27, 2015 at 12:25:54AM +0000, Simon Horman wrote: >> On Thu, Mar 26, 2015 at 08:29:21AM -0700, Tyler Baker wrote: >> > On 26 March 2015 at 06:36, Will Deacon wrote: >> > > On Thu, Mar 26, 2015 at 12:39:39AM +0000, Simon Horman wrote: >> > >> On Tue, Mar 24, 2015 at 11:13:58AM -0500, Nishanth Menon wrote: >> > >> > I think we now have a new error: (seen with omap2plus_defconfig) >> > >> > on next-20150324 : >> > >> > ./arch/arm/kernel/vmlinux.lds:677: undefined symbol `__hyp_idmap_size' >> > >> > referenced in expression >> > >> > make: *** [vmlinux] Error 1 >> > >> >> > >> Thanks, I am seeing that too. >> > >> >> > >> My armchair suggestion is that the following should be reverted. >> > >> >> > >> e60a1fec44a2f ("ARM: kvm: implement replacement for ld's LOG2CEIL()") >> > >> 06f75a1f62000 ("ARM, arm64: kvm: get rid of the bounce page") >> > > >> > > Can you try again with the latest -next please? We've merged an additional >> > > patch aimed at sorting this out. Reverting isn't really an option, as >> > > there's an awful lot of code that depends on the bounce page removal. >> > >> > Here are the kernelci.org -next results[1], if you click the build >> > status you can dig down into the build failures. next-20150326 has now >> > hit a compiler bug, Arnd mentioned he was looking into this issue. >> >> I have confirmed that next-20150326 does not compile without >> the following reverted: >> >> 12eb3e833961 ("ARM: kvm: assert on HYP section boundaries not actual code size") >> e60a1fec44a2 ("ARM: kvm: implement replacement for ld's LOG2CEIL()") >> 06f75a1f6200 ("ARM, arm64: kvm: get rid of the bounce page") > > Thanks for testing this and sorry for the continued breakage. Which > toolchain did you say you were using? Ard has some more patches trying to > fix this, but none of our toolchains seem to tickle the issue. I've also tested on the default ARM toolchains available with ubuntu[1] Are there any updates on this issue? It ha broken most of the ARM defconfigs in linux-next[2], and since it's been broken for a week now, it is masking other types of issues that we can normally find via automated boot testing. Kevin [1] arm-linux-gnueabihf-gcc (Ubuntu/Linaro 4.8.2-16ubuntu4) 4.8.2 arm-linux-gnueabi-gcc (Ubuntu/Linaro 4.7.3-12ubuntu1) 4.7.3 [2] http://kernelci.org/job/next/ From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@kernel.org (Kevin Hilman) Date: Tue, 31 Mar 2015 11:58:21 -0700 Subject: Possible regression in next-20150323 due to "ARM, arm64: kvm: get rid of the bounce page" In-Reply-To: <20150327100612.GB1562@arm.com> (Will Deacon's message of "Fri, 27 Mar 2015 10:06:12 +0000") References: <20150324004537.GA24816@verge.net.au> <20150324161358.GA694@kahuna> <20150326003939.GA25368@verge.net.au> <20150326133631.GB2805@arm.com> <20150327002554.GA5527@verge.net.au> <20150327100612.GB1562@arm.com> Message-ID: <7hbnj99epe.fsf@deeprootsystems.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Will, Ard, Will Deacon writes: > On Fri, Mar 27, 2015 at 12:25:54AM +0000, Simon Horman wrote: >> On Thu, Mar 26, 2015 at 08:29:21AM -0700, Tyler Baker wrote: >> > On 26 March 2015 at 06:36, Will Deacon wrote: >> > > On Thu, Mar 26, 2015 at 12:39:39AM +0000, Simon Horman wrote: >> > >> On Tue, Mar 24, 2015 at 11:13:58AM -0500, Nishanth Menon wrote: >> > >> > I think we now have a new error: (seen with omap2plus_defconfig) >> > >> > on next-20150324 : >> > >> > ./arch/arm/kernel/vmlinux.lds:677: undefined symbol `__hyp_idmap_size' >> > >> > referenced in expression >> > >> > make: *** [vmlinux] Error 1 >> > >> >> > >> Thanks, I am seeing that too. >> > >> >> > >> My armchair suggestion is that the following should be reverted. >> > >> >> > >> e60a1fec44a2f ("ARM: kvm: implement replacement for ld's LOG2CEIL()") >> > >> 06f75a1f62000 ("ARM, arm64: kvm: get rid of the bounce page") >> > > >> > > Can you try again with the latest -next please? We've merged an additional >> > > patch aimed at sorting this out. Reverting isn't really an option, as >> > > there's an awful lot of code that depends on the bounce page removal. >> > >> > Here are the kernelci.org -next results[1], if you click the build >> > status you can dig down into the build failures. next-20150326 has now >> > hit a compiler bug, Arnd mentioned he was looking into this issue. >> >> I have confirmed that next-20150326 does not compile without >> the following reverted: >> >> 12eb3e833961 ("ARM: kvm: assert on HYP section boundaries not actual code size") >> e60a1fec44a2 ("ARM: kvm: implement replacement for ld's LOG2CEIL()") >> 06f75a1f6200 ("ARM, arm64: kvm: get rid of the bounce page") > > Thanks for testing this and sorry for the continued breakage. Which > toolchain did you say you were using? Ard has some more patches trying to > fix this, but none of our toolchains seem to tickle the issue. I've also tested on the default ARM toolchains available with ubuntu[1] Are there any updates on this issue? It ha broken most of the ARM defconfigs in linux-next[2], and since it's been broken for a week now, it is masking other types of issues that we can normally find via automated boot testing. Kevin [1] arm-linux-gnueabihf-gcc (Ubuntu/Linaro 4.8.2-16ubuntu4) 4.8.2 arm-linux-gnueabi-gcc (Ubuntu/Linaro 4.7.3-12ubuntu1) 4.7.3 [2] http://kernelci.org/job/next/