From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44774) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xb8yL-0006Db-KU for qemu-devel@nongnu.org; Mon, 06 Oct 2014 10:09:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xb8yD-0001yv-Sp for qemu-devel@nongnu.org; Mon, 06 Oct 2014 10:09:09 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60791) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xb8yD-0001yZ-M7 for qemu-devel@nongnu.org; Mon, 06 Oct 2014 10:09:01 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s96E90ZK025425 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Mon, 6 Oct 2014 10:09:01 -0400 Date: Mon, 6 Oct 2014 17:12:31 +0300 From: "Michael S. Tsirkin" Message-ID: <20141006141231.GA29955@redhat.com> References: <1411057074-11157-1-git-send-email-pbonzini@redhat.com> <20141002121140.GA25373@redhat.com> <542D5391.8080007@redhat.com> <20141002134137.GA26273@redhat.com> <542D5687.3050604@redhat.com> <20141002134905.GC26273@redhat.com> <54329C29.9070508@redhat.com> <20141006135253.GA29563@redhat.com> <54329F65.90101@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <54329F65.90101@redhat.com> Subject: Re: [Qemu-devel] [PATCH 0/6] pc: bring ACPI table size below to 2.0 levels, try fixing -initrd for good List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: jsnow@redhat.com, qemu-devel@nongnu.org On Mon, Oct 06, 2014 at 03:55:49PM +0200, Paolo Bonzini wrote: > Il 06/10/2014 15:52, Michael S. Tsirkin ha scritto: > > Maybe we should just modify ACPI and rom files in general to use > > something else, not RAM? > > It looked like a good fit initially so we went ahead with it, > > but these things are fairly small, so it's not a problem to > > migrate them as part of the device state. > > Yes, that would have been a good design too, but I'm not sure it's worth > changing it now. So far, it has certainly helped us; it both revealed > bugs (though a bit too late) and kept us honest. Bugs that surface too late is exactly what worries me. I'll try to look at how hard to implement the above change is. > These patches would be easy to revert from now till 2.2 release, they're > just an incremental improvement on top of 2.1.2. > > I'll post a v2 that includes the linuxboot changes to look at the memory > map, Pls keep linuxboot things a separate patchset. > and keep the 160k padding for the sake of old linuxboot option ROMs > being migrated to 2.2. > > Paolo I think we can drop this for 2.2 machine type. We can also drop the fw cfg file for new machine types. -- MST