From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50329) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XWTGM-0000Vs-3D for qemu-devel@nongnu.org; Tue, 23 Sep 2014 12:48:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XWTGF-0001Mn-LN for qemu-devel@nongnu.org; Tue, 23 Sep 2014 12:48:26 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54622) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XWTGF-0001JR-Cz for qemu-devel@nongnu.org; Tue, 23 Sep 2014 12:48:19 -0400 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s8NGmDQE025172 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Tue, 23 Sep 2014 12:48:13 -0400 From: John Snow Date: Tue, 23 Sep 2014 12:47:59 -0400 Message-Id: <1411490885-29782-1-git-send-email-jsnow@redhat.com> Subject: [Qemu-devel] [PATCH 0/6] Q35: Implement -cdrom/-hda sugar List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, John Snow , armbru@redhat.com, stefanha@redhat.com, mst@redhat.com The Q35 board initialization does not currently bother to look for any drives added by the various syntactical sugar shorthands to be added to the AHCI HBA. These include -hda through -hdd, -cdrom, and -drive if=ide shorthands. An obstacle to having implemented this sooner is debate over whether or not to add an additional interface type, and how to manage the different units-per-bus mappings of various HBA implementations. This patch series: (1) Does not add IF_AHCI, but reuses IF_IDE (2) Allows the if_max_devs table to be overridden (3) Adds this override to the Q35 board type. (4) Finally, adds implementation to Q35 initialization. Other changes from RFC2: - Re-uses ide_drive_get instead of ahci_drive_get - Adds units-per-bus property to all Q35 machines - Changes orphan scanning to exclude IF_NONE and automatically added drives - Renames 'units-per-idebus' to 'units-per-default-bus' And allows override of any one IF type (block_default) John Snow (6): blockdev: Orphaned drive search blockdev: Allow overriding if_max_dev property pc/vl: Add units-per-default-bus property ide: Update ide_drive_get to be HBA agnostic qtest/bios-tables: Correct Q35 command line q35/ahci: Pick up -cdrom and -hda options blockdev.c | 56 ++++++++++++++++++++++++++++++++++++++++++++++- hw/i386/pc.c | 1 + hw/i386/pc_q35.c | 6 ++++- hw/ide/ahci.c | 15 +++++++++++++ hw/ide/ahci.h | 2 ++ hw/ide/core.c | 12 +++++----- include/hw/boards.h | 2 ++ include/sysemu/blockdev.h | 5 +++++ tests/bios-tables-test.c | 10 ++++----- vl.c | 20 ++++++++++++++++- 10 files changed, 115 insertions(+), 14 deletions(-) -- 1.9.3