From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54039) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1anhrZ-0006sv-5B for qemu-devel@nongnu.org; Wed, 06 Apr 2016 03:26:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1anhrV-00066d-Uo for qemu-devel@nongnu.org; Wed, 06 Apr 2016 03:26:53 -0400 Received: from mx1.redhat.com ([209.132.183.28]:51691) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1anhrV-000651-OQ for qemu-devel@nongnu.org; Wed, 06 Apr 2016 03:26:49 -0400 References: From: Thomas Huth Message-ID: <5704BA35.1050501@redhat.com> Date: Wed, 6 Apr 2016 09:26:45 +0200 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/3] slirp: deliver received TCP RSTs to the guest List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: steven@steven676.net, qemu-devel@nongnu.org Cc: "Edgar E. Iglesias" , Jan Kiszka , Samuel Thibault On 06.04.2016 02:13, steven@steven676.net wrote: > QEMU's user-mode networking does not currently pass received TCP RSTs to > guests, meaning that applications in guests hang if the remote server > rejects their network connections. This is particularly noticeable when > IPv6 is enabled, the guest is configured to prefer IPv6 and the remote > server rejects IPv6 connections (segment-data.zqtk.net is one example), > but the bug appears to be longstanding and affects TCP over IPv4 as > well. > > There are three short patches in this series. The first fixes a crash > which would be exposed by the last patch in the series. The second, > which fixes delivery of an RST interrupting an already-established TCP > connection, was submitted by Edgar Iglesias in 2008 and appears to have > been missed then. The last patch fixes the case where the remote end > sends RST in reply to our SYN (rejects our incoming connection attempt). > > Lightly tested on a Linux host with Linux and Windows 7 guests. > > Edgar E. Iglesias (1): > slirp: Propagate host TCP RST to the guest. > > Steven Luo (2): > slirp: don't crash when tcp_sockclosed() is called with a NULL tp > slirp: handle deferred ECONNREFUSED on non-blocking TCP sockets > > slirp/socket.c | 17 ++++++++++++++++- > slirp/tcp_input.c | 6 ++++++ > slirp/tcp_subr.c | 7 +++++-- > 3 files changed, 27 insertions(+), 3 deletions(-) FWIW, please make sure to CC: Samuel Thibault when posting slirp patches - he's the new (co-)maintainer of the slirp code now. Thanks, Thomas