On May 7, 2010, at 12:13 PM, Oleksandr Tymoshenko wrote:

   Proposed patch addresses hifn(4) problems on FreeBSD/mips. Current
implementation keeps some of the state information (indexes in
buffers, etc) in DMA-mapped memory and bus_dma code invalidates them
during sync operations. This fix moves data that doesn't belong to DMA
ring to softc structure.

Patch: http://people.freebsd.org/~gonzo/hifn.diff
Stats for original driver:
   http://people.freebsd.org/~gonzo/hifn.stats.orig.txt
Stats for patched version:
   http://people.freebsd.org/~gonzo/hifn.stats.patched.txt



The changes look fine and make sense (did something similar for some other drivers for when the dma data structures were mapped uncached). I can't see any performance change in your stats; but I'm just eyeballing the numbers side-by-side. Was this on x86? (where there should be zero difference) It would be good to present these numbers better (e.g. curves on the same graph, ministat output, etc).

        Sam

_______________________________________________
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"

Reply via email to