qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: aliguori@us.ibm.com, agraf@suse.de,
	Li Zhang <zhlcindy@linux.vnet.ibm.com>,
	qemu-devel@nongnu.org, Jens Freimann <jfrei@linux.vnet.ibm.com>,
	Li Zhang <zhlcindy@gmail.com>,
	Eugene Dvurechenski <jno@linux.vnet.ibm.com>,
	qemu-ppc@nongnu.org, afaerber@suse.de,
	david@gibson.dropbear.id.au
Subject: Re: [Qemu-devel] [RFC] usb and s390
Date: Tue, 07 Aug 2012 08:20:18 -0600	[thread overview]
Message-ID: <50212422.4040407@redhat.com> (raw)
In-Reply-To: <1344341985-10065-1-git-send-email-borntraeger@de.ibm.com>

[-- Attachment #1: Type: text/plain, Size: 1423 bytes --]

On 08/07/2012 06:19 AM, Christian Borntraeger wrote:
> 2nd attempt, with mailing lists on cc....
> This patch reminded us of another thing that we have been working on.
> Here is a patch that sits in my patch queue and  disables usb for s390
> We have not pushed that patch yet, since several libvirt versions will always 
> specify -usb, which will then break. 

You are correct that older libvirt always specified -usb; but with the
upcoming libvirt 0.10.0, we have fixed things to now allow the user to
request starting without -usb.

> 
> So are there any opinions about handling the usb option for platforms that
> actually dont support it?

Libvirt should probably learn which platforms don't support it and
automatically omit the -usb without user intervention on those
platforms, but at least we have the fallback to explicitly avoid the
-usb now.  Knowing which platforms support usb might be easier if there
were some QMP query-* command that could be used to learn whether a
particular machine type supports it.  That is, since this series is
adding the new syntax '-machine type=pseries,usb=off', libvirt should be
able to query '-machine type=pseries,?' to see what options the given
machine has, and preferably through a QMP interface rather than through
the command line.

-- 
Eric Blake   eblake@redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 620 bytes --]

  parent reply	other threads:[~2012-08-07 14:20 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-07  2:41 [Qemu-devel] [Qemu-ppc][PATCH v7 0/3] Add USB enablement and VGA enablement on sPAPR Li Zhang
2012-08-07  2:41 ` [Qemu-devel] [Qemu-ppc][PATCH v7 1/3] Add USB option in machine options Li Zhang
2012-08-07 12:19   ` [Qemu-devel] [RFC] usb and s390 Christian Borntraeger
2012-08-07 12:50     ` Li Zhang
2012-08-07 14:20     ` Eric Blake [this message]
2012-08-07 12:19   ` [Qemu-devel] [RFC/PATCH 1/1] USB code fenced for s390 Christian Borntraeger
2012-08-07 12:26     ` Peter Maydell
2012-08-07 20:22       ` Blue Swirl
2012-08-08  6:29         ` Christian Borntraeger
2012-08-07 12:57     ` Andreas Färber
2012-08-07 12:59     ` Li Zhang
2012-08-14 10:39   ` [Qemu-devel] [Qemu-ppc][PATCH v7 1/3] Add USB option in machine options Alexander Graf
2012-08-14 14:59     ` Li Zhang
2012-08-14 10:46   ` Alexander Graf
2012-08-14 15:01     ` Li Zhang
2012-08-14 23:10       ` [Qemu-devel] [Qemu-ppc] [PATCH " David Gibson
2012-08-14 23:41         ` Alexander Graf
2012-08-15  0:09         ` Benjamin Herrenschmidt
2012-08-15  1:24           ` Li Zhang
2012-08-15  1:47             ` David Gibson
2012-08-15  2:50               ` Li Zhang
2012-08-15  2:57                 ` David Gibson
2012-08-15  5:44                   ` Li Zhang
2012-08-15 11:13                     ` David Gibson
2012-08-15 14:17                       ` Li Zhang
2012-08-07  2:41 ` [Qemu-devel] [Qemu-ppc][PATCH v7 2/3] Add one new file vga-pci.h and cleanup on all platforms Li Zhang
2012-08-14 11:04   ` Alexander Graf
2012-08-07  2:42 ` [Qemu-devel] [Qemu-ppc][PATCH v7 3/3] spapr: Add support for -vga option Li Zhang
2012-08-14 11:04   ` Alexander Graf
2012-08-14 12:04     ` Benjamin Herrenschmidt
2012-08-14 12:13       ` David Gibson
2012-08-14 15:13         ` Li Zhang
2012-08-14 17:19           ` Li Zhang
2012-08-14 17:52             ` Li Zhang
2012-08-14 23:11               ` [Qemu-devel] [Qemu-ppc] [PATCH " David Gibson
2012-08-14 15:07     ` [Qemu-devel] [Qemu-ppc][PATCH " Li Zhang
2012-08-07 15:05 ` [Qemu-devel] [Qemu-ppc][PATCH v7 0/3] Add USB enablement and VGA enablement on sPAPR Li Zhang
2012-08-14 11:04   ` Alexander Graf

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=50212422.4040407@redhat.com \
    --to=eblake@redhat.com \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=borntraeger@de.ibm.com \
    --cc=david@gibson.dropbear.id.au \
    --cc=jfrei@linux.vnet.ibm.com \
    --cc=jno@linux.vnet.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=zhlcindy@gmail.com \
    --cc=zhlcindy@linux.vnet.ibm.com \
    /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).