Date: Fri, 15 May 2009 09:22:05 +0400 From: Stanislav Sedov <stas@FreeBSD.org> To: "M. Warner Losh" <imp@bsdimp.com> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: svn commit: r192027 - head/sys/arm/at91 Message-ID: <20090515092205.6f6d06fa.stas@FreeBSD.org> In-Reply-To: <20090514.213712.-404854369.imp@bsdimp.com> References: <200905122114.n4CLEag9033208@svn.freebsd.org> <20090514.213712.-404854369.imp@bsdimp.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 14 May 2009 21:37:12 -0600 (MDT) "M. Warner Losh" <imp@bsdimp.com> mentioned: > In message: <200905122114.n4CLEag9033208@svn.freebsd.org> > Stanislav Sedov <stas@FreeBSD.org> writes: > : @@ -926,6 +937,7 @@ atestart_locked(struct ifnet *ifp) > : * tell the hardware to xmit the packet. > : */ > : WR4(sc, ETH_TAR, segs[0].ds_addr); > : + BARRIER(sc, ETH_TAR, 8, BUS_SPACE_BARRIER_WRITE); > : WR4(sc, ETH_TCR, segs[0].ds_len); > > Why is a barrier needed here? > Writing the TCR register triggers the transmit, so it had to be written strongly after the TAR register. That's why I added the barrier here. -- Stanislav Sedov ST4096-RIPE !DSPAM:4a0cfbe4994291152178454!
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20090515092205.6f6d06fa.stas>