[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCHv2 2/2] tap: mark fd handler as device
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] Re: [PATCHv2 2/2] tap: mark fd handler as device |
Date: |
Mon, 15 Nov 2010 17:18:17 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Nov 15, 2010 at 08:55:07AM -0600, Anthony Liguori wrote:
> On 11/15/2010 08:52 AM, Juan Quintela wrote:
> >"Michael S. Tsirkin"<address@hidden> wrote:
> >>There's no reason for tap to run when VM is stopped.
> >>If we let it, it confuses the bridge on TX
> >>and corrupts DMA memory on RX.
> >>
> >>Signed-off-by: Michael S. Tsirkin<address@hidden>
> >once here, what handlers make sense to run while stopped?
> >/me can think of the normal console, non live migration, loadvm and not
> >much more. Perhaps it is easier to just move the other way around?
>
> I'm not sure I concur that this is really a problem.
> Semantically, I don't think that stop has to imply that the guest
> memory no longer changes.
>
> Regards,
>
> Anthony Liguori
>
> >Later, Juan.
Well, I do not really know about vmstop that is not for migration.
For most vmstop calls are for migration. And there, the problems are very
real.
First, it's not just memory. At least for network transmit, sending out
packets with the same MAC from two locations is a problem. See?
For memory, it is much worse: any memory changes can either get
discarded or not. This breaks consistency guarantees that guest relies
upon. Imagine virtio index getting updated but content not being
updated. See?
--
MST
- [Qemu-devel] [PATCHv2 0/2] migration: stop dma while VM is stopped, Michael S. Tsirkin, 2010/11/04
- [Qemu-devel] [PATCHv2 1/2] char: separate device and system fd handlers, Michael S. Tsirkin, 2010/11/04
- [Qemu-devel] [PATCHv2 2/2] tap: mark fd handler as device, Michael S. Tsirkin, 2010/11/04
- [Qemu-devel] Re: [PATCHv2 2/2] tap: mark fd handler as device, Juan Quintela, 2010/11/15
- Re: [Qemu-devel] Re: [PATCHv2 2/2] tap: mark fd handler as device, Stefan Hajnoczi, 2010/11/15
- Re: [Qemu-devel] Re: [PATCHv2 2/2] tap: mark fd handler as device, Anthony Liguori, 2010/11/15
- Re: [Qemu-devel] Re: [PATCHv2 2/2] tap: mark fd handler as device, Michael S. Tsirkin, 2010/11/15
- Re: [Qemu-devel] Re: [PATCHv2 2/2] tap: mark fd handler as device, Anthony Liguori, 2010/11/15
- [Qemu-devel] Re: [PATCHv2 2/2] tap: mark fd handler as device, Michael S. Tsirkin, 2010/11/15