public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@qumranet.com>
To: "Daniel P. Berrange" <berrange@redhat.com>
Cc: Marcelo Tosatti <marcelo@kvack.org>,
	kvm-devel@lists.sourceforge.net,
	Glauber Costa <gcosta@redhat.com>
Subject: Re: [patch 23/23] QEMU/KVM: device hot-remove
Date: Wed, 05 Mar 2008 08:28:46 +0200	[thread overview]
Message-ID: <47CE3D9E.3030709@qumranet.com> (raw)
In-Reply-To: <20080304200629.GR6713@redhat.com>

Daniel P. Berrange wrote:
>> (qemu) info block
>> ide0-hd0: type=hd removable=0 file=/root/images/marcelo5.img ro=0 drv=raw
>> ide1-cd0: type=cdrom removable=1 locked=0 [not inserted]
>> floppy0: type=floppy removable=1 locked=0 [not inserted]
>> sd0: type=floppy removable=1 locked=0 [not inserted]
>> scsi0-hd0: type=hd removable=0 file=/tmp/bigfile ro=0 drv=raw
>> scsi0-hd1: type=hd removable=0 file=/tmp/bigfile.2 ro=0 drv=raw
>>
>> (qemu) info network
>> VLAN 0 devices:
>>   tap: ifname=tap0 setup_script=qemu-ifup-tap0
>>   rtl8139 pci macaddr=52:54:00:12:34:56
>>     
>
> This is utterly horrible for a human to parse & use if they're using the
> QEMU monitor, let alone something that libvirt could parse. In fact this
> doesn't let you map between the network device & pci device if there is
> more than one device added because 'info pci' doesn't show the MAC address
> info, and 'info network' does not show any PCI device number info - the
> same for disks.
>
>   

We need a machine friendly protocol for libvirt and other management 
tools.  Versioned commands (with some backward compatibility), command 
discovery, and command/response tagging so you can associate an async 
reply to the command that triggered it, and quoting so that strings with 
spaces and other special chars are properly supported.  But how the 
information is presented is orthogonal to what information is presented.

btw, the qemu command line parses something fairly similar, I don't see 
why libvirt should have problems with it.  It wouldn't be fun to code, 
but is doable.

-- 
Do not meddle in the internals of kernels, for they are subtle and quick to panic.


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

  reply	other threads:[~2008-03-05  6:28 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-04 18:34 [patch 00/23] [RFC] QEMU/KVM ACPI PCI hotplug Marcelo Tosatti
2008-03-04 18:34 ` [patch 01/23] QEMU/KVM: add PCI IRQ routing information up to slot 32 Marcelo Tosatti
2008-03-05  5:40   ` Avi Kivity
2008-03-04 18:34 ` [patch 02/23] QEMU/KVM: add devices to represent PCI slots with _EJ0 method Marcelo Tosatti
2008-03-04 18:34 ` [patch 03/23] QEMU/KVM: add OperationRegion and GPE handler for add/removal notification Marcelo Tosatti
2008-03-04 18:34 ` [patch 04/23] QEMU/KVM: add pci_find_bus Marcelo Tosatti
2008-03-04 18:34 ` [patch 05/23] QEMU/KVM: return PCIDevice on net device init and record devfn Marcelo Tosatti
2008-03-04 18:34 ` [patch 06/23] QEMU/KVM: pci hotplug GPE support Marcelo Tosatti
2008-03-04 18:34 ` [patch 07/23] QEMU/KVM: dynamic drive/drive_opt index allocation Marcelo Tosatti
2008-03-04 18:34 ` [patch 08/23] QEMU/KVM: dynamic nic info " Marcelo Tosatti
2008-03-04 18:34 ` [patch 09/23] QEMU/KVM: drive removal support Marcelo Tosatti
2008-03-04 18:34 ` [patch 10/23] QEMU/KVM: record devfn on block driver instance Marcelo Tosatti
2008-03-04 18:34 ` [patch 11/23] QEMU/KVM: move drives_opt for external use Marcelo Tosatti
2008-03-04 18:34 ` [patch 12/23] QEMU/KVM: net/drive add/remove tweaks Marcelo Tosatti
2008-03-04 18:34 ` [patch 13/23] QEMU/KVM: add net_client_uninit Marcelo Tosatti
2008-03-04 18:34 ` [patch 14/23] QEMU/KVM: device hot-add Marcelo Tosatti
2008-03-04 19:14   ` Daniel P. Berrange
2008-03-04 19:30     ` Anthony Liguori
2008-03-04 19:53       ` Daniel P. Berrange
2008-03-04 21:44         ` Itamar Heim
2008-03-05  5:50   ` Avi Kivity
2008-03-11 14:18     ` Marcelo Tosatti
2008-03-04 18:34 ` [patch 15/23] QEMU/KVM: add pci_find_device Marcelo Tosatti
2008-03-04 18:34 ` [patch 16/23] QEMU/KVM: add cpu_unregister_io_memory and make io mem table index dynamic Marcelo Tosatti
2008-03-04 18:34 ` [patch 17/23] QEMU/KVM: notify _EJ0 through _SEJ OperationRegion Marcelo Tosatti
2008-03-04 18:34 ` [patch 18/23] QEMU/KVM: handle SEJ notifications Marcelo Tosatti
2008-03-04 18:34 ` [patch 19/23] QEMU/KVM: add qemu_free_irqs Marcelo Tosatti
2008-03-04 18:34 ` [patch 20/23] QEMU/KVM: add pci_unregister_device Marcelo Tosatti
2008-03-04 18:34 ` [patch 21/23] QEMU/KVM: LSI SCSI unregister callback Marcelo Tosatti
2008-03-04 18:34 ` [patch 22/23] QEMU/KVM: zero ioport_opaque on isa_unassign_ioport Marcelo Tosatti
2008-03-04 18:34 ` [patch 23/23] QEMU/KVM: device hot-remove Marcelo Tosatti
2008-03-04 19:18   ` Daniel P. Berrange
2008-03-04 19:32     ` Anthony Liguori
2008-03-04 20:00       ` Marcelo Tosatti
2008-03-04 20:06         ` Daniel P. Berrange
2008-03-05  6:28           ` Avi Kivity [this message]
2008-03-05  5:46       ` Avi Kivity
2008-03-05  6:47   ` Avi Kivity
2008-03-05  6:00 ` [patch 00/23] [RFC] QEMU/KVM ACPI PCI hotplug Avi Kivity

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=47CE3D9E.3030709@qumranet.com \
    --to=avi@qumranet.com \
    --cc=berrange@redhat.com \
    --cc=gcosta@redhat.com \
    --cc=kvm-devel@lists.sourceforge.net \
    --cc=marcelo@kvack.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