From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60923) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dBdEQ-0003zq-7U for qemu-devel@nongnu.org; Fri, 19 May 2017 04:25:55 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dBdEM-0003DD-2L for qemu-devel@nongnu.org; Fri, 19 May 2017 04:25:54 -0400 Received: from mx1.redhat.com ([209.132.183.28]:41098) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dBdEL-0003D6-S0 for qemu-devel@nongnu.org; Fri, 19 May 2017 04:25:49 -0400 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 7975F8124F for ; Fri, 19 May 2017 08:25:48 +0000 (UTC) Date: Fri, 19 May 2017 09:25:38 +0100 From: "Daniel P. Berrange" Message-ID: <20170519082538.GD4912@redhat.com> Reply-To: "Daniel P. Berrange" References: <1495176212-14446-1-git-send-email-peterx@redhat.com> <1495176212-14446-2-git-send-email-peterx@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1495176212-14446-2-git-send-email-peterx@redhat.com> Subject: Re: [Qemu-devel] [PATCH RFC 1/6] io: only allow return path for socket typed List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Xu Cc: qemu-devel@nongnu.org, Juan Quintela , "Dr . David Alan Gilbert" On Fri, May 19, 2017 at 02:43:27PM +0800, Peter Xu wrote: > We don't really have a return path for the other types yet. Let's check > this when .get_return_path() is called. > > For this, we introduce a new feature bit, and set it up only for socket > typed IO channels. > > This will help detect earlier failure for postcopy, e.g., logically > speaking postcopy cannot work with "exec:". Before this patch, when we > try to migrate with "migrate -d exec:cat>out", we'll hang the system. > With this patch, we'll get: > > (qemu) migrate -d exec:cat>out > Unable to open return-path for postcopy This is wrong - post-copy migration *can* work with exec: - it just entirely depends on what command you are running. Your example ran a command which is unidirectional, but if you ran 'exec:socat ...' you would have a fully bidirectional channel. Actually the channel is always bi-directional, but 'cat' simply won't ever send data back to QEMU. If QEMU hangs when the other end doesn't send data back, that actually seems like a potentially serious bug in migration code. Even if using the normal 'tcp' migration protocol, if the target QEMU server hangs and fails to send data to QEMU on the return path, the source QEMU must never hang. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|