[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor device
From: |
Jan Kiszka |
Subject: |
Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2) |
Date: |
Tue, 14 Apr 2009 11:29:30 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Daniel P. Berrange wrote:
> On Tue, Apr 14, 2009 at 12:15:23PM +0300, Avi Kivity wrote:
>> Daniel P. Berrange wrote:
>>> Yes indeed its a little crazy :-) As anthony mentioned if libvirt were
>>> able to be notified of changes a user makes in the monitor, there's no
>>> reason we could not allow end users to access the monitor of a VM
>>> libvirt is managing. We just need to make sure libvirt doesn't miss
>>> changes like attaching or detaching block devices, etc, because that'll
>>> cause crash/data loss later when libvirt migrates or does save/restore,
>>> etc because it'll launch QEMU with wrong args
>>>
>> You still have an inherent race here.
>>
>> user: plug in disk
>> libvirt: start migration, still without disk
>> qemu: libvirt, a disk has been plugged in.
>
> That is true, but we'd still be considering direct monitor access to
> be a 'expert' user mode of use. If they wish to shoot themselves in
> the foot by triggering a migration at same time they are hotplugging
> I'm fine if their whole leg gets blown away.
...while there is also nothing that speaks against blocking any device
hot-plugging while migration is ongoing. Independent of if there is some
management app involved or the user himself plays with multiple monitors.
Jan
--
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), (continued)
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Anthony Liguori, 2009/04/09
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Avi Kivity, 2009/04/09
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Anthony Liguori, 2009/04/09
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Avi Kivity, 2009/04/11
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Anthony Liguori, 2009/04/11
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Avi Kivity, 2009/04/11
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Jamie Lokier, 2009/04/12
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Daniel P. Berrange, 2009/04/14
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Avi Kivity, 2009/04/14
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Daniel P. Berrange, 2009/04/14
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2),
Jan Kiszka <=
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Avi Kivity, 2009/04/14
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Avi Kivity, 2009/04/14
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Jamie Lokier, 2009/04/14
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Jamie Lokier, 2009/04/14
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Avi Kivity, 2009/04/16
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Zachary Amsden, 2009/04/11
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Zachary Amsden, 2009/04/12
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Gerd Hoffmann, 2009/04/14
- Re: [libvirt] Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Jamie Lokier, 2009/04/14
- Re: [Qemu-devel] [PATCH 1/6] Allow multiple monitor devices (v2), Avi Kivity, 2009/04/11