From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755551AbbBBNjl (ORCPT ); Mon, 2 Feb 2015 08:39:41 -0500 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:57233 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753471AbbBBNji (ORCPT ); Mon, 2 Feb 2015 08:39:38 -0500 Date: Mon, 2 Feb 2015 14:39:36 +0100 From: Pavel Machek To: Nicolas Pitre , Linus Torvalds Cc: Tony Lindgren , Pali =?iso-8859-1?Q?Roh=E1r?= , linux-omap , Linux Kernel Mailing List , Aaro Koskinen , Sebastian Reichel , Ivaylo Dimitrov , Russell King , linux-arm-kernel@lists.infradead.org, Laura Abbott , Russell King Subject: Re: N900 v3.19-rc5 arm atags_to_fdt.c is broken Message-ID: <20150202133936.GA26985@amd> References: <201501232239.55801@pali> <201501232336.36187@pali> <20150126161454.GE28663@atomide.com> <201501261723.10505@pali> <20150126170323.GF28663@atomide.com> <20150127133739.GB25306@amd> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 On Tue 2015-01-27 10:16:24, Nicolas Pitre wrote: > On Tue, 27 Jan 2015, Pavel Machek wrote: > > > > > > > > I would say, problem is because omap3-n900 binary DT is too large > > > > > > I agree. > > > > > > > OK if that's the case, then your patch makes sense to me. It also > > > > seems we can have the temporary stack be larger than the initial > > > > stack just for atags_to_fdt. > > > > > > The stack size isn't the issue, but rather its location. We need to > > > position it away from the DT data. The DT size is known and we could > > > use that, plus some room for the insertion of new data coming from the > > > ATAG conversion. > > > > > > Something like the following would be a more robust solution: > > > > Tested-by: Pavel Machek > > > > (Note, that in 3.19 dts for n900 got too big, so we are actually > > triggering old bugs. That means that this is a regression fix, and > > should go in ASAP). > > It is queued here: > http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=8294/1 Hmm, but it should be on kernel.org, not in private arm trees, as it fixes a regression. Russell, you are the ARM maintainer, can you push it to Linus? Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html