From owner-p4-projects@FreeBSD.ORG  Thu Dec  7 12:47:18 2006
Return-Path: <owner-p4-projects@FreeBSD.ORG>
X-Original-To: p4-projects@freebsd.org
Delivered-To: p4-projects@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 32767)
	id 0BC4016A4A7; Thu,  7 Dec 2006 12:47:18 +0000 (UTC)
X-Original-To: perforce@FreeBSD.org
Delivered-To: perforce@FreeBSD.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id DCCD916A40F;
	Thu,  7 Dec 2006 12:47:17 +0000 (UTC)
	(envelope-from glebius@FreeBSD.org)
Received: from cell.sick.ru (cell.sick.ru [217.72.144.68])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 837B643FD6;
	Thu,  7 Dec 2006 12:40:24 +0000 (GMT)
	(envelope-from glebius@FreeBSD.org)
Received: from cell.sick.ru (glebius@localhost [127.0.0.1])
	by cell.sick.ru (8.13.4/8.13.3) with ESMTP id kB7CfDN8081626
	(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO);
	Thu, 7 Dec 2006 15:41:13 +0300 (MSK)
	(envelope-from glebius@FreeBSD.org)
Received: (from glebius@localhost)
	by cell.sick.ru (8.13.4/8.13.1/Submit) id kB7CfDvv081625;
	Thu, 7 Dec 2006 15:41:13 +0300 (MSK)
	(envelope-from glebius@FreeBSD.org)
X-Authentication-Warning: cell.sick.ru: glebius set sender to
	glebius@FreeBSD.org using -f
Date: Thu, 7 Dec 2006 15:41:12 +0300
From: Gleb Smirnoff <glebius@FreeBSD.org>
To: Andre Oppermann <andre@FreeBSD.org>
Message-ID: <20061207124112.GW32700@FreeBSD.org>
References: <200612062319.kB6NJgsq031755@repoman.freebsd.org>
	<20061207110225.GU32700@FreeBSD.org> <4578070A.2030609@freebsd.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
In-Reply-To: <4578070A.2030609@freebsd.org>
User-Agent: Mutt/1.5.6i
Cc: Perforce Change Reviews <perforce@FreeBSD.org>,
	Paolo Pisati <piso@FreeBSD.org>
Subject: Re: PERFORCE change 111230 for review
X-BeenThere: p4-projects@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: p4 projects tree changes <p4-projects.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/p4-projects>
List-Post: <mailto:p4-projects@freebsd.org>
List-Help: <mailto:p4-projects-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 07 Dec 2006 12:47:18 -0000

On Thu, Dec 07, 2006 at 01:20:26PM +0100, Andre Oppermann wrote:
A> Gleb Smirnoff wrote:
A> >On Wed, Dec 06, 2006 at 11:19:42PM +0000, Paolo Pisati wrote:
A> >P> http://perforce.freebsd.org/chv.cgi?CH=111230
A> >P> 
A> >P> Change 111230 by piso@piso_newluxor on 2006/12/06 23:19:06
A> >P> 
A> >P> 	To workaround a problem when an mbuf is bigger than MCLBYTES
A> >P> 	(and thus not handled by m_megapullup()), introduce & use
A> >P> 	m_jumbo16pullup() that uses 16k jumbo size cluster.
A> >P> 	
A> >P> 	This problem only showed up on traffic generated on a box with 
A> >P> 	a nic tso enabled.
A> >
A> >Paolo,
A> >
A> >this isn't a fix. Another application will do write(,, 16k + 1) and
A> >m_jumbo16pullup() will fail again. Please backout it, it is a hack.
A> >
A> >We need to fix TSO in such way that real packets, that will be
A> >transmitted to wire, will be passed to pfil handlers.
A> 
A> That is not possible.

ATM this should be at least documented behavior. And a solution should
be thought, because pfil must see real packets, not their precursors.

-- 
Totus tuus, Glebius.
GLEBIUS-RIPN GLEB-RIPE