From owner-freebsd-net@FreeBSD.ORG Mon Aug 25 19:02:10 2008 Return-Path: Delivered-To: net@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 679F41065683; Mon, 25 Aug 2008 19:02:10 +0000 (UTC) (envelope-from jhay@meraka.csir.co.za) Received: from zibbi.meraka.csir.co.za (zibbi.meraka.csir.co.za [IPv6:2001:4200:7000:2::1]) by mx1.freebsd.org (Postfix) with ESMTP id F15878FC08; Mon, 25 Aug 2008 19:02:08 +0000 (UTC) (envelope-from jhay@meraka.csir.co.za) Received: by zibbi.meraka.csir.co.za (Postfix, from userid 3973) id 6317C33CBC; Mon, 25 Aug 2008 21:02:07 +0200 (SAST) Date: Mon, 25 Aug 2008 21:02:07 +0200 From: John Hay To: gnn@FreeBSD.org Message-ID: <20080825190207.GA73478@zibbi.meraka.csir.co.za> References: <20080821203519.GA51534@onelab2.iet.unipi.it> <48AE23FF.9070009@FreeBSD.org> <48AF08B7.4090804@FreeBSD.org> <48AF330B.4010802@FreeBSD.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.1i Cc: Luigi Rizzo , "Bruce M. Simpson" , net@FreeBSD.org Subject: Re: Small patch to multicast code... X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 25 Aug 2008 19:02:10 -0000 On Fri, Aug 22, 2008 at 03:03:30PM -0700, gnn@FreeBSD.org wrote: > At Fri, 22 Aug 2008 22:43:39 +0100, > Bruce M. Simpson wrote: > > > > gnn@FreeBSD.org wrote: > > > Somehow the data that the device needs to do the proper checksum > > > offload is getting trashed here. Now, since it's clear we need a > > > writable packet structure so that we don't trash the original, I'm > > > wondering if the m_pullup() will be sufficient. > > > > > > > If it's serious enough to break UDP checksumming on the wire, perhaps we > > should just swallow the mbuf allocator heap churn and do the m_dup() for > > now, but slap in a big comment about why it's there. > > I think if none of us finds a better way before early next week that's > what I'll do so that this at least works in 7.1. I think we and others have been hitting this problem: kern/119635: [em] Bad UDP packet checksum with em(4) and rxcsum/txcsum enabled http://www.freebsd.org/cgi/query-pr.cgi?pr=119635 I'll try out your patch. John -- John Hay -- John.Hay@meraka.csir.co.za / jhay@FreeBSD.org