Date: Fri, 22 Aug 2008 15:03:30 -0700 From: gnn@FreeBSD.org To: "Bruce M. Simpson" <bms@FreeBSD.org> Cc: Luigi Rizzo <rizzo@iet.unipi.it>, net@FreeBSD.org Subject: Re: Small patch to multicast code... Message-ID: <m28wuohfm5.wl%gnn@neville-neil.com> In-Reply-To: <48AF330B.4010802@FreeBSD.org> References: <m27iaa6v43.wl%gnn@neville-neil.com> <20080821203519.GA51534@onelab2.iet.unipi.it> <m23aky6ncl.wl%gnn@neville-neil.com> <48AE23FF.9070009@FreeBSD.org> <m2tzdd6j36.wl%gnn@neville-neil.com> <48AF08B7.4090804@FreeBSD.org> <m2fxowhgq8.wl%gnn@neville-neil.com> <48AF330B.4010802@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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. Best, George
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?m28wuohfm5.wl%gnn>