From owner-cvs-src@FreeBSD.ORG  Sun Jul 11 23:13:14 2004
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id BB37216A4CE; Sun, 11 Jul 2004 23:13:14 +0000 (GMT)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id B2BB443D2D; Sun, 11 Jul 2004 23:13:14 +0000 (GMT)
	(envelope-from rwatson@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.12.11/8.12.11) with ESMTP id i6BNDE8W090407;
	Sun, 11 Jul 2004 23:13:14 GMT
	(envelope-from rwatson@repoman.freebsd.org)
Received: (from rwatson@localhost)
	by repoman.freebsd.org (8.12.11/8.12.11/Submit) id i6BNDEaw090406;
	Sun, 11 Jul 2004 23:13:14 GMT
	(envelope-from rwatson)
Message-Id: <200407112313.i6BNDEaw090406@repoman.freebsd.org>
From: Robert Watson <rwatson@FreeBSD.org>
Date: Sun, 11 Jul 2004 23:13:14 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/kern uipc_socket.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 11 Jul 2004 23:13:14 -0000

rwatson     2004-07-11 23:13:14 UTC

  FreeBSD src repository

  Modified files:
    sys/kern             uipc_socket.c 
  Log:
  Use sockbuf_pushsync() to synchronize stack and socket buffer state
  in soreceive() after removing an MT_SONAME mbuf from the head of the
  socket buffer.
  
  When processing MT_CONTROL mbufs in soreceive(), first remove all of
  the MT_CONTROL mbufs from the head of the socket buffer to a local
  mbuf chain, then feed them into dom_externalize() as a set, which
  both avoids thrashing the socket buffer lock when handling multiple
  control mbufs, and also avoids races with other threads acting on
  the socket buffer when the socket buffer mutex is released to enter
  the externalize code.  Existing races that might occur if the protocol
  externalize method blocked during processing have also been closed.
  
  Now that we synchronize socket buffer and stack state following
  modifications to the socket buffer, turn the manual synchronization
  that previously followed control mbuf processing with a set of
  assertions.  This can eventually be removed.
  
  The soreceive() code is now substantially more MPSAFE.
  
  Revision  Changes    Path
  1.201     +47 -34    src/sys/kern/uipc_socket.c