i cant see anything except some stupid debug code touching that memory. it is reading it though. shouldnt we sync it for READs at the bottom of rx_intr too?
dlg On 22/01/2013, at 9:49 AM, David Hill <dh...@mindcry.org> wrote: > I think this should be POSTREAD in bnx_rx_intr. This matches > FreeBSD. > > Index: if_bnx.c > =================================================================== > RCS file: /cvs/src/sys/dev/pci/if_bnx.c,v > retrieving revision 1.100 > diff -N -u -p if_bnx.c > --- if_bnx.c 13 Jan 2013 05:45:10 -0000 1.100 > +++ if_bnx.c 21 Jan 2013 23:46:38 -0000 > @@ -4323,7 +4323,7 @@ bnx_rx_intr(struct bnx_softc *sc) > bus_dmamap_sync(sc->bnx_dmatag, > sc->rx_bd_chain_map[i], 0, > sc->rx_bd_chain_map[i]->dm_mapsize, > - BUS_DMASYNC_POSTWRITE); > + BUS_DMASYNC_POSTREAD); > > /* Get the hardware's view of the RX consumer index. */ > hw_cons = sc->hw_rx_cons = sblk->status_rx_quick_consumer_index0;
