Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 16 Jan 2007 13:55:36 -0800
From:      "Kip Macy" <kip.macy@gmail.com>
To:        "Nick Evans" <nevans@talkpoint.com>
Cc:        Attilio Rao <attilio@freebsd.org>, freebsd-current@freebsd.org, freebsd-arch@freebsd.org
Subject:   Re: [PATCH] Mantaining turnstile aligned to 128 bytes in i386 CPUs
Message-ID:  <b1fa29170701161355lc021b90o35fa5f9acb5749d@mail.gmail.com>
In-Reply-To: <20070116154258.568e1aaf@pleiades.nextvenue.com>
References:  <3bbf2fe10607250813w8ff9e34pc505bf290e71758@mail.gmail.com> <3bbf2fe10607250814m1a476f09p2d962dedc0c99be1@mail.gmail.com> <200607251232.51230.jhb@freebsd.org> <3bbf2fe10607251004wf94e238xb5ea7a31c973817f@mail.gmail.com> <3bbf2fe10607261127p3f01a6c3w80027754f7d4e594@mail.gmail.com> <3bbf2fe10607281004o6727e976h19ee7e054876f914@mail.gmail.com> <3bbf2fe10701160851r79b04464m2cbdbb7f644b22b6@mail.gmail.com> <20070116154258.568e1aaf@pleiades.nextvenue.com>

next in thread | previous in thread | raw e-mail | index | archive | help
x86 pre-P4 had 32-byte cache lines. Thus older processors will not benefit.

 -Kip

On 1/16/07, Nick Evans <nevans@talkpoint.com> wrote:
> On Tue, 16 Jan 2007 17:51:03 +0100
> "Attilio Rao" <attilio@freebsd.org> wrote:
>
> > 2006/7/28, Attilio Rao <attilio@freebsd.org>:
> > >
> > > After some thinking, I think it's better using init/fini methods
> > > (since they hide the sizeof(struct turnstile) with size parameter).
> > >
> > > Feedbacks and comments are welcome:
> > > http://users.gufi.org/~rookie/works/patches/uma_sync_init.diff
> >
> > [CC'ed all the interested people]
> >
> > Even if a long time is passed I did some benchmarks based on ebizzy tool.
> > This program claims to reproduce a real httpd server behaviour and is
> > used into the Linux world for benchmarks, AFAIK.
> > I think that results of the comparison on this patch is very
> > interesting, and I think it worths a commit :)
> > I think that results can be even better on a Xeon machine (I had no
> > chance to reproduce this on some of these).
> > (Results taken in consideration have been measured after some starts,
> > in order to minimize caching differences).
> >
> > The patch:
> > http://users.gufi.org/~rookie/works/patches/ts-sq/ts-sq.diff
> >
> > The benchmark results:
> > http://users.gufi.org/~rookie/works/patches/ts-sq/ts-sq.benchmark
> >
> > The kernel options file:
> > http://users.gufi.org/~rookie/works/patches/ts-sq/CURRENT
> >
> > For any information, comment, etc. please feel free to contact me.
> >
> > Attilio
> >
> >
> > --
> > Peace can only be achieved by understanding - A. Einstein
> > _______________________________________________
> > freebsd-current@freebsd.org mailing list
> > http://lists.freebsd.org/mailman/listinfo/freebsd-current
> > To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
>
> Some preliminary results:
>
> PREEMPTION: 4BSD, Quad P3-Xeon, 2GB ram
>
> pre-patch
>
> 1.  176.36 real       703.75 user         0.01 sys
> 2.  176.73 real       704.34 user         0.03 sys
> 3.  176.49 real       703.72 user         0.04 sys
> 4.  175.81 real       701.36 user         0.03 sys
> 5.  176.57 real       700.98 user         0.02 sys
>
> post-patch
>
> 1.  179.17 real       714.39 user         0.01 sys
> 2.  178.33 real       711.50 user         0.04 sys
> 3.  178.32 real       711.04 user         0.03 sys
> 4.  177.34 real       707.51 user         0.03 sys
> 5.  178.25 real       710.17 user         0.03 sys
>
> _______________________________________________
> freebsd-current@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?b1fa29170701161355lc021b90o35fa5f9acb5749d>