From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35330) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XMVbO-0002jy-8g for qemu-devel@nongnu.org; Wed, 27 Aug 2014 01:17:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XMVbJ-0004Yq-72 for qemu-devel@nongnu.org; Wed, 27 Aug 2014 01:16:58 -0400 Received: from mx1.redhat.com ([209.132.183.28]:21162) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XMVbI-0004Yj-UJ for qemu-devel@nongnu.org; Wed, 27 Aug 2014 01:16:53 -0400 Message-ID: <53FD69B5.9010604@redhat.com> Date: Wed, 27 Aug 2014 13:16:37 +0800 From: Jason Wang MIME-Version: 1.0 References: <1409040377-12088-1-git-send-email-zhang.zhanghailiang@huawei.com> In-Reply-To: <1409040377-12088-1-git-send-email-zhang.zhanghailiang@huawei.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V4] net: Forbid dealing with packets when VM is not running List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: zhanghailiang , qemu-devel@nongnu.org Cc: peter.huangpeng@huawei.com, luonengjun@huawei.com, aliguori@amazon.com, stefanha@redhat.com, mst@redhat.com On 08/26/2014 04:06 PM, zhanghailiang wrote: > For all NICs(except virtio-net) emulated by qemu, > Such as e1000, rtl8139, pcnet and ne2k_pci, > Qemu can still receive packets when VM is not running. > > If this happened in *migration's* last PAUSE VM stage, but > before the end of the migration, the new receiving packets will possibly dirty > parts of RAM which has been cached in *iovec*(will be sent asynchronously) and > dirty parts of new RAM which will be missed. > This will lead serious network fault in VM. > > To avoid this, we forbid receiving packets in generic net code when > VM is not running. > > Bug reproduction steps: > (1) Start a VM which configured at least one NIC > (2) In VM, open several Terminal and do *Ping IP -i 0.1* > (3) Migrate the VM repeatedly between two Hosts > And the *PING* command in VM will very likely fail with message: > 'Destination HOST Unreachable', the NIC in VM will stay unavailable unless you > run 'service network restart' > > Signed-off-by: zhanghailiang > --- > v4: > - The action of flush queued packets is unnecessary, remove this. > (Thanks for the help of Jason Wang and Stefan). > > v3: > - change the 'vmstate' to 'vm_running' > > v2: > - remove the superfluous check of nc->received_disabled > --- > > net/net.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/net/net.c b/net/net.c > index 6d930ea..4cb92c0 100644 > --- a/net/net.c > +++ b/net/net.c > @@ -41,6 +41,7 @@ > #include "qapi-visit.h" > #include "qapi/opts-visitor.h" > #include "qapi/dealloc-visitor.h" > +#include "sysemu/sysemu.h" > > /* Net bridge is currently not supported for W32. */ > #if !defined(_WIN32) > @@ -452,6 +453,12 @@ void qemu_set_vnet_hdr_len(NetClientState *nc, int len) > > int qemu_can_send_packet(NetClientState *sender) > { > + int vm_running = runstate_is_running(); > + > + if (!vm_running) { > + return 0; > + } > + > if (!sender->peer) { > return 1; > } Reviewed-by: Jason Wang