From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:52317)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1XUtDG-0006KU-CN
	for qemu-devel@nongnu.org; Fri, 19 Sep 2014 04:06:51 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1XUtD7-0000AI-Av
	for qemu-devel@nongnu.org; Fri, 19 Sep 2014 04:06:42 -0400
Received: from mail-wg0-x22b.google.com ([2a00:1450:400c:c00::22b]:50207)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1XUtD7-0000A0-46
	for qemu-devel@nongnu.org; Fri, 19 Sep 2014 04:06:33 -0400
Received: by mail-wg0-f43.google.com with SMTP id x12so2034934wgg.14
	for <qemu-devel@nongnu.org>; Fri, 19 Sep 2014 01:06:27 -0700 (PDT)
Sender: Paolo Bonzini <paolo.bonzini@gmail.com>
Message-ID: <541BE3FE.2020108@redhat.com>
Date: Fri, 19 Sep 2014 10:06:22 +0200
From: Paolo Bonzini <pbonzini@redhat.com>
MIME-Version: 1.0
References: <1411057074-11157-1-git-send-email-pbonzini@redhat.com>
	<1411112184.24750.10.camel@nilsson.home.kraxel.org>
In-Reply-To: <1411112184.24750.10.camel@nilsson.home.kraxel.org>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH 0/6] pc: bring ACPI table size below to 2.0
 levels, try fixing -initrd for good
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: Gerd Hoffmann <kraxel@redhat.com>
Cc: jsnow@redhat.com, qemu-devel@nongnu.org, mst@redhat.com

Il 19/09/2014 09:36, Gerd Hoffmann ha scritto:
> Hmm.  That assumes we are running seabios, where we know how much memory
> we actually need.

Right.  However, note that this only affects one patch in the series
(patch 3).  Patches 1-2 are useful to unify Xen-specific behavior with
other hypervisors, and patches 4-6 are useful to future-proof ACPI table
sizes for migration.

> IMHO we should either really parse the memory map, or reserve more
> space.

I agree.  However, we need to cater for a fixed initrd loading address
until this is done.  And even after linuxboot.bin is improved, the older
version will be used when migrating for older QEMU machine types, so it
is important to have a decent fallback.

Paolo