[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig |
Date: |
Tue, 12 Jun 2018 14:00:05 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Gerd Hoffmann <address@hidden> writes:
> Hi,
>
>> > Now let's review the three commands:
>> >
>> > * Gerd, why does "info usbhost" have no QMP equivalent?
>
> Works only when running qemu directly, in the libvirt sandbox qemu
> hasn't the permissions needed to scan the host usb bus so that would be
> rather pointless ...
I don't think this meets either of the two criteria:
* I meets "makes no sense in QMP" only if QMP implies "can't scan host
USB bus". Libvirt implies it, but QMP doesn't imply libvirt; it's its
most important, not its sole user.
* It meets "of use only for human users" only if we're convinced it's of
no use to programs.
To avoid speculation and endless arguments about what could or could
not be of use, we've always stuck to "when in doubt, assume it could
be of use".
"Libvirt can't use it" falls short.
"Any management application worth anything would deny QEMU the
capability to scan the USB host bus, and thus wouldn't be able to use
it" is exactly the argument we intended to avoid.
QMP falling short of completeness in relatively unimportant ways like
this one isn't exactly terrible. The most serious effect is probably
serving as a bad example that leads to further arguments like this one.
These are well worth avoiding, though.
- [Qemu-devel] [PATCH v3 0/7] Reenable hmp for preconfig mode, Dr. David Alan Gilbert (git), 2018/06/08
- [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Dr. David Alan Gilbert (git), 2018/06/08
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Markus Armbruster, 2018/06/11
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Dr. David Alan Gilbert, 2018/06/11
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Gerd Hoffmann, 2018/06/12
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Dr. David Alan Gilbert, 2018/06/12
- [Qemu-devel] Abandon our QMP first policy? (was: [PATCH v3 5/7] hmp: Add info commands for preconfig), Markus Armbruster, 2018/06/15
- Re: [Qemu-devel] Abandon our QMP first policy? (was: [PATCH v3 5/7] hmp: Add info commands for preconfig), Dr. David Alan Gilbert, 2018/06/15
- Re: [Qemu-devel] Abandon our QMP first policy? (was: [PATCH v3 5/7] hmp: Add info commands for preconfig), Eduardo Habkost, 2018/06/15
- Re: [Qemu-devel] Abandon our QMP first policy? (was: [PATCH v3 5/7] hmp: Add info commands for preconfig), Gerd Hoffmann, 2018/06/18
- Re: [Qemu-devel] Abandon our QMP first policy? (was: [PATCH v3 5/7] hmp: Add info commands for preconfig), Dr. David Alan Gilbert, 2018/06/20
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Markus Armbruster, 2018/06/12
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Dr. David Alan Gilbert, 2018/06/12
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Eduardo Habkost, 2018/06/13
- Re: [Qemu-devel] [PATCH v3 5/7] hmp: Add info commands for preconfig, Daniel P . Berrangé, 2018/06/13