cvs commit: src/sys/i386/i386 pmap.c
rwatson at FreeBSD.org
Tue Nov 9 13:05:04 GMT 2004
On Tue, 9 Nov 2004, Robert Watson wrote:
> > I've tried changing the store_rel() to just do a simple store since writes are
> > ordered on x86, but benchmarks on SMP showed that it actually hurt. However,
> > it would probably be good to at least do that for UP. The current patch to
> > do it for all kernels is:
Interestingly, I've now run through some more "macro" benchmarks. I saw a
couple of percent improvement on UP from the change, but indeed, I saw a
slight decrease in performance for the rapid packet send benchmark on SMP.
So I guess my recommendation is to get this in the tree for UP, and see if
we can figure out why it's having the slow-down effect on SMP.
Robert N M Watson FreeBSD Core Team, TrustedBSD Projects
robert at fledge.watson.org Principal Research Scientist, McAfee Research
More information about the cvs-src