On 01/25/2013 07:07 AM, Michal Privoznik wrote: > On 25.01.2013 14:55, Markus Armbruster wrote: >> Michal Privoznik <[email protected]> writes: >> >>> Currently, we are using 'tray_open' in QMP and 'tray-open' in >>> HMP. However, the QMP documentation was mistakenly using the >>> HMP version. >> >> Could you point me to where we're using "tray_open"? >> > > qemu.git $ git grep -npC8 tray_open qapi-schema.json >
> qapi-schema.json-705-{ 'type': 'BlockInfo',
> qapi-schema.json-706- 'data': {'device': 'str', 'type': 'str', 'removable':
> 'bool',
> qapi-schema.json-707- 'locked': 'bool', '*inserted':
> 'BlockDeviceInfo',
> qapi-schema.json:708: '*tray_open': 'bool', '*io-status':
> 'BlockDeviceIoStatus',
> qapi-schema.json-709- '*dirty': 'BlockDirtyInfo' } }
And since existing qemu versions output tray_open, we would be breaking
API if we fixed the QMP to return tray-open (libvirt could cope with
that API break by checking both spellings, but that feels dirty).
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
