From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:52391)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1Sn2vP-0007SF-Kg
	for qemu-devel@nongnu.org; Fri, 06 Jul 2012 03:26:00 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1Sn2vN-0001Rl-Qj
	for qemu-devel@nongnu.org; Fri, 06 Jul 2012 03:25:59 -0400
Received: from mail-bk0-f45.google.com ([209.85.214.45]:64466)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1Sn2vN-0001Qt-K0
	for qemu-devel@nongnu.org; Fri, 06 Jul 2012 03:25:57 -0400
Received: by bkcji1 with SMTP id ji1so708577bkc.4
	for <qemu-devel@nongnu.org>; Fri, 06 Jul 2012 00:25:55 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <CAEgOgz6q9Qg8JBzHbVpZ2n_FqgJCqnoKLLF+BF2-4Z778m2VKA@mail.gmail.com>
References: <1341507652-22155-1-git-send-email-peter.maydell@linaro.org>
	<1341507652-22155-4-git-send-email-peter.maydell@linaro.org>
	<CAEgOgz6q9Qg8JBzHbVpZ2n_FqgJCqnoKLLF+BF2-4Z778m2VKA@mail.gmail.com>
Date: Fri, 6 Jul 2012 08:25:55 +0100
Message-ID: <CAFEAcA8FjahxFKaXL5WUQcSXs8bPwHowEbkmt1dtsw77Xn0tAw@mail.gmail.com>
From: Peter Maydell <peter.maydell@linaro.org>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [Qemu-devel] [PATCH 3/6] hw/arm_boot.c: Check for RAM sizes
	exceeding ATAGS capacity
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Crosthwaite <peter.crosthwaite@petalogix.com>
Cc: qemu-devel@nongnu.org, patches@linaro.org

On 6 July 2012 03:05, Peter Crosthwaite <peter.crosthwaite@petalogix.com> wrote:
> On Fri, Jul 6, 2012 at 3:00 AM, Peter Maydell <peter.maydell@linaro.org> wrote:
>> +            if (info->ram_size >= (1ULL << 32)) {
>> +                fprintf(stderr, "qemu: RAM size must be less than 4GB to boot"
>> +                        " Linux kernel without device tree\n");
>
> Error message is a bit weird. Shouldnt it be "RAM size must be less
> than 4GB to boot Linux kernel with ATAG command line". DTB shouldnt be
> the only way to boot with >4GB.

I'm trying to steer the user toward the way to solve their problem,
which is to use a device tree. I expect less than 1 QEMU user in 100
has any idea what ATAGS are.

> DTB shouldnt be the only way to boot with >4GB.

I'm not sure what you mean here -- we have two boot methods and
only DTB handles large RAM sizes. Or do you mean machines with eg
flash where you could boot by putting a bootloader in flash?

-- PMM