[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/5] socket: Add a reconnect option.
From: |
Ian Molton |
Subject: |
Re: [Qemu-devel] [PATCH 2/5] socket: Add a reconnect option. |
Date: |
Mon, 01 Feb 2010 22:44:55 +0000 |
User-agent: |
Mozilla-Thunderbird 2.0.0.22 (X11/20091109) |
Anthony Liguori wrote:
> I went back and looked at the last series and found my feedback. I had
> suggested that instead of automatically reconnecting, a mechanism should
> be added for a user to initiate a reconnect.
This sounds useful
> Additionally, we should emit events upon disconnect through QMP (now
> that we have that functionality).
This also.
> The main reason I dislike automatic reconnecting is that there is no
> correct way to handle the period of time while the VM is disconnected.
This is fine for egd protocol, the guest will just run low on entropy in
the meantime. Nothing should break.
> Auto reconnecting is implementing a policy to handle the failure within
> QEMU which is not universally the correct choice. This isn't so bad
> except for the fact that you aren't providing the mechanisms for users
> to implement other policies which means they're stuck with this
> particular policy.
Perhaps this feature could be added if needed in future? It seems a bit
ambitious to get all this 'right' with no use cases to test against.
-Ian
- [Qemu-devel] [PATCH 5/5] Build fix (missing header), (continued)
- [Qemu-devel] [PATCH 5/5] Build fix (missing header), Ian Molton, 2010/02/01
- Re: [Qemu-devel] [PATCH 4/5] virtio: Add virtio-rng driver, Anthony Liguori, 2010/02/01
- Re: [Qemu-devel] [PATCH 4/5] virtio: Add virtio-rng driver, Gerd Hoffmann, 2010/02/01
- Re: [Qemu-devel] [PATCH 4/5] virtio: Add virtio-rng driver, Anthony Liguori, 2010/02/01
- Re: [Qemu-devel] [PATCH 4/5] virtio: Add virtio-rng driver, Ian Molton, 2010/02/01
- Re: [Qemu-devel] [PATCH 4/5] virtio: Add virtio-rng driver, Ian Molton, 2010/02/01
- Re: [Qemu-devel] [PATCH 4/5] virtio: Add virtio-rng driver, Ian Molton, 2010/02/09
- Re: [Qemu-devel] [PATCH 2/5] socket: Add a reconnect option., Anthony Liguori, 2010/02/01
- Re: [Qemu-devel] [PATCH 2/5] socket: Add a reconnect option., Luiz Capitulino, 2010/02/01
- Re: [Qemu-devel] [PATCH 2/5] socket: Add a reconnect option., Anthony Liguori, 2010/02/01
- Re: [Qemu-devel] [PATCH 2/5] socket: Add a reconnect option.,
Ian Molton <=
- Re: [Qemu-devel] [PATCH 2/5] socket: Add a reconnect option., Anthony Liguori, 2010/02/01
- Re: [Qemu-devel] [PATCH 2/5] socket: Add a reconnect option., Ian Molton, 2010/02/02