Date: Sun, 30 Jul 2006 15:48:07 +0900 From: Pyun YongHyeon <pyunyh@gmail.com> To: Yar Tikhiy <yar@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, Pyun YongHyeon <yongari@FreeBSD.org> Subject: Re: cvs commit: src/sys/dev/em if_em.c Message-ID: <20060730064807.GA32731@cdnetworks.co.kr> In-Reply-To: <20060729205009.GA37970@freefall.freebsd.org> References: <200607270043.k6R0hY8g088353@repoman.freebsd.org> <20060729205009.GA37970@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Jul 29, 2006 at 08:50:09PM +0000, Yar Tikhiy wrote: > On Thu, Jul 27, 2006 at 12:43:34AM +0000, Pyun YongHyeon wrote: > > yongari 2006-07-27 00:43:34 UTC > > > > FreeBSD src repository > > > > Modified files: > > sys/dev/em if_em.c > > Log: > > Prepending an mbuf after loading a DMA map results in unexpected > > result. So, modify mbuf chains before loading a DMA map. > > > > Revision Changes Path > > 1.122 +28 -31 src/sys/dev/em/if_em.c > > Thanks a lot! Do you think this can fix kern/72933? > I can't sure it helps as the submitter reported the same issue on bge(4). Btw, I think the VLAN fixup code should use m_prepend(9) insatead of M_PREEND(9) because we don't know whether a new mbuf is allocated or not after M_PREPEND(9) call. -- Regards, Pyun YongHyeon
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060730064807.GA32731>