From owner-freebsd-bugs@FreeBSD.ORG  Sun May 29 18:10:06 2011
Return-Path: <owner-freebsd-bugs@FreeBSD.ORG>
Delivered-To: freebsd-bugs@hub.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id B5A60106566B
	for <freebsd-bugs@hub.freebsd.org>;
	Sun, 29 May 2011 18:10:06 +0000 (UTC)
	(envelope-from gnats@FreeBSD.org)
Received: from freefall.freebsd.org (freefall.freebsd.org
	[IPv6:2001:4f8:fff6::28])
	by mx1.freebsd.org (Postfix) with ESMTP id 868548FC15
	for <freebsd-bugs@hub.freebsd.org>;
	Sun, 29 May 2011 18:10:06 +0000 (UTC)
Received: from freefall.freebsd.org (localhost [127.0.0.1])
	by freefall.freebsd.org (8.14.4/8.14.4) with ESMTP id p4TIA6iM081906
	for <freebsd-bugs@freefall.freebsd.org>; Sun, 29 May 2011 18:10:06 GMT
	(envelope-from gnats@freefall.freebsd.org)
Received: (from gnats@localhost)
	by freefall.freebsd.org (8.14.4/8.14.4/Submit) id p4TIA6IS081905;
	Sun, 29 May 2011 18:10:06 GMT (envelope-from gnats)
Date: Sun, 29 May 2011 18:10:06 GMT
Message-Id: <201105291810.p4TIA6IS081905@freefall.freebsd.org>
To: freebsd-bugs@FreeBSD.org
From: dfilter@FreeBSD.ORG (dfilter service)
Cc: 
Subject: Re: kern/154504: commit references a PR
X-BeenThere: freebsd-bugs@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: dfilter service <dfilter@FreeBSD.ORG>
List-Id: Bug reports <freebsd-bugs.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
	<mailto:freebsd-bugs-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-bugs>
List-Post: <mailto:freebsd-bugs@freebsd.org>
List-Help: <mailto:freebsd-bugs-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
	<mailto:freebsd-bugs-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 29 May 2011 18:10:06 -0000

The following reply was made to PR kern/154504; it has been noted by GNATS.

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/154504: commit references a PR
Date: Sun, 29 May 2011 18:01:06 +0000 (UTC)

 Author: trociny
 Date: Sun May 29 18:00:50 2011
 New Revision: 222454
 URL: http://svn.freebsd.org/changeset/base/222454
 
 Log:
   In soreceive_generic(), if MSG_WAITALL is set but the request is
   larger than the receive buffer, we have to receive in sections.
   When notifying the protocol that some data has been drained the
   lock is released for a moment. Returning we block waiting for the
   rest of data. There is a race, when data could arrive while the
   lock was released and then the connection stalls in sbwait.
   
   Fix this by checking for data before blocking and skip blocking
   if there are some.
   
   PR:		kern/154504
   Reported by:	Andrey Simonenko <simon@comsys.ntu-kpi.kiev.ua>
   Tested by:	Andrey Simonenko <simon@comsys.ntu-kpi.kiev.ua>
   Reviewed by:	rwatson
   Approved by:	kib (co-mentor)
   MFC after:	2 weeks
 
 Modified:
   head/sys/kern/uipc_socket.c
 
 Modified: head/sys/kern/uipc_socket.c
 ==============================================================================
 --- head/sys/kern/uipc_socket.c	Sun May 29 15:10:12 2011	(r222453)
 +++ head/sys/kern/uipc_socket.c	Sun May 29 18:00:50 2011	(r222454)
 @@ -1845,10 +1845,16 @@ dontblock:
  			}
  			SBLASTRECORDCHK(&so->so_rcv);
  			SBLASTMBUFCHK(&so->so_rcv);
 -			error = sbwait(&so->so_rcv);
 -			if (error) {
 -				SOCKBUF_UNLOCK(&so->so_rcv);
 -				goto release;
 +			/*
 +			 * We could receive some data while was notifying
 +			 * the protocol. Skip blocking in this case.
 +			 */
 +			if (so->so_rcv.sb_mb == NULL) {
 +				error = sbwait(&so->so_rcv);
 +				if (error) {
 +					SOCKBUF_UNLOCK(&so->so_rcv);
 +					goto release;
 +				}
  			}
  			m = so->so_rcv.sb_mb;
  			if (m != NULL)
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"