qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: John Snow <jsnow@redhat.com>
To: qemu-devel@nongnu.org
Cc: jsnow@redhat.com, armbru@redhat.com
Subject: [Qemu-devel] [RFC v2 0/3] Q35/AHCI -cdrom/-hda desugaring
Date: Thu, 18 Sep 2014 13:59:03 -0400	[thread overview]
Message-ID: <1411063146-24058-1-git-send-email-jsnow@redhat.com> (raw)

This is an extremely rough/quick sketch of
a -cdrom/-hda desugaring fix for Q35/AHCI.

Before I spent any time on it, I wanted feedback
from Markus or anyone else who had concerns about
how this problem would get fixed.

This is, then, rough approach #2.

Highlights:
(1) Add a board property (instead of a HBA property, sigh)
    that defines how we should map (index, (bus,unit)).

(2) Modify drive_new to accept the MachineClass instead of
    the default interface type. This does not affect how
    default drives get added, because any over-rides to
    the "default type" get handled in options, so while
    it appears we have removed the type of default drives,
    we have not.

(3) Create helpers for AHCI to assist the Q35 board in
    populating the AHCI device with the IDE drives.

(4) Create a helper to whine at us for oversights and
    help bug reporters give us more meaningful information.

John Snow (3):
  blockdev: Add function to search for orphaned drives
  Add units-per-idebus property
  ahci: implement -cdrom and -hd[a-d]

 blockdev.c                | 29 +++++++++++++++++++++++++++--
 device-hotplug.c          |  2 +-
 hw/i386/pc_q35.c          |  6 +++++-
 hw/ide/ahci.c             | 31 +++++++++++++++++++++++++++++++
 hw/ide/ahci.h             |  3 +++
 include/hw/boards.h       |  3 ++-
 include/sysemu/blockdev.h |  3 ++-
 vl.c                      | 24 ++++++++++++++++--------
 8 files changed, 87 insertions(+), 14 deletions(-)

-- 
1.9.3

             reply	other threads:[~2014-09-18 17:59 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-18 17:59 John Snow [this message]
2014-09-18 17:59 ` [Qemu-devel] [RFC v2 1/3] blockdev: Add function to search for orphaned drives John Snow
2014-09-19  8:28   ` Markus Armbruster
2014-09-19 15:50     ` John Snow
2014-09-18 17:59 ` [Qemu-devel] [RFC v2 2/3] Add units-per-idebus property John Snow
2014-09-19  9:39   ` Markus Armbruster
2014-09-21  9:34     ` Marcel Apfelbaum
2014-09-22  7:51       ` Markus Armbruster
2014-09-18 17:59 ` [Qemu-devel] [RFC v2 3/3] ahci: implement -cdrom and -hd[a-d] John Snow
2014-09-19  9:49   ` Markus Armbruster
2014-09-19  9:53 ` [Qemu-devel] [RFC v2 0/3] Q35/AHCI -cdrom/-hda desugaring Markus Armbruster
2014-09-22 23:17   ` John Snow
2014-09-23  7:36     ` Markus Armbruster

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1411063146-24058-1-git-send-email-jsnow@redhat.com \
    --to=jsnow@redhat.com \
    --cc=armbru@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).