From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:58741) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TbRaH-0001Wi-VA for qemu-devel@nongnu.org; Thu, 22 Nov 2012 02:52:31 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TbRaG-0005h0-QR for qemu-devel@nongnu.org; Thu, 22 Nov 2012 02:52:29 -0500 Received: from mx1.redhat.com ([209.132.183.28]:37091) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TbRaG-0005gu-Gw for qemu-devel@nongnu.org; Thu, 22 Nov 2012 02:52:28 -0500 Message-ID: <50ADD9B2.30301@redhat.com> Date: Thu, 22 Nov 2012 08:52:18 +0100 From: Gerd Hoffmann MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 00/14] Add Q35 base support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jason Baron Cc: agraf@suse.de, alex.williamson@redhat.com, mst@redhat.com, jan.kiszka@siemens.com, armbru@redhat.com, qemu-devel@nongnu.org, blauwirbel@gmail.com, yamahata@valinux.co.jp, juzhang@redhat.com, kevin@koconnor.net, anthony@codemonkey.ws, gsomlo@gmail.com, mkletzan@redhat.com, pbonzini@redhat.com, lcapitulino@redhat.com, afaerber@suse.de On 11/14/12 21:53, Jason Baron wrote: > Hi, > > These patches are intened to give us a base set of patches for Q35 upon which > to build. The major change in this series is to add the memory controller hub, > or 'mch' as proper member of the q35 host structure. This change refactors the > code a bit, and moves more intialization out of pc_q35.c and into q35.c. We > probably could go further and introduce a generic north bridge class and make > the 'mch' a child of it, but I'm not proposing that kind of change this late in > the development cycle. Tested the patches while hacking on q35 seabios support last days. > I've also dropped as many non-essential bits as possible, such as if=ahci from > the patch series. Patches 13-14 allow the use of the '-L' option to specify > the directory of the q35 dsdt table. These aren't strictly necessary. As one > could pass: > > -bios /root/seabios/seabios/out/bios.bin > -acpitable file=/root/seabios/seabios/out/q35-acpi-dsdt.aml > > But its rather cumbersome, and including the automatic load of the dsdt table, > is the future direction. That is, once the seabios bits are included we can > pull the q35 dsdt table into the qemu tree, and avoid extra options altogether. I'd prefer to not include patches 13+14. Patch 13 is not needed long-term for q35, it is just a temporary testing convinience. Having a search path instead of a single directory might be a useful feature independent of q35 support, but I think for this we should simply allow to explicitly specify a search path, i.e. something like -L "/path/to/dir/one:/path/to/dir/two". Patch 14 should wait until we've merged seabios with q35 support. It also needs to be fixed to allow overriding the acpi table via command line, i.e. only automatically load the default dsdt in case no dsdt was specified via -acpitable. Patches 1-12 are fine (and they still apply to master). cheers, Gerd