From owner-freebsd-arch@FreeBSD.ORG Tue Jan 16 21:24:05 2007 Return-Path: X-Original-To: freebsd-arch@freebsd.org Delivered-To: freebsd-arch@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 2481B16A415; Tue, 16 Jan 2007 21:24:05 +0000 (UTC) (envelope-from nevans@talkpoint.com) Received: from relay.talkpoint.com (relay.talkpoint.com [204.141.15.158]) by mx1.freebsd.org (Postfix) with ESMTP id DB31713C461; Tue, 16 Jan 2007 21:24:04 +0000 (UTC) (envelope-from nevans@talkpoint.com) Received: from ASSP-nospam ([127.0.0.1]) by relay.talkpoint.com with Microsoft SMTPSVC(5.0.2195.6713); Tue, 16 Jan 2007 16:24:03 -0500 Received: from 204.141.15.136 ([204.141.15.136] helo=postal.talkpoint.com) by ASSP-nospam ; 16 Jan 07 21:24:03 -0000 Received: from pleiades.nextvenue.com ([204.141.15.194]) by postal.talkpoint.com with SMTP (Microsoft Exchange Internet Mail Service Version 5.5.2653.13) id ZN3V72DX; Tue, 16 Jan 2007 16:24:03 -0500 Date: Tue, 16 Jan 2007 16:24:03 -0500 From: Nick Evans To: John Baldwin Message-ID: <20070116162403.70e2ec8a@pleiades.nextvenue.com> In-Reply-To: <200701161607.43725.jhb@freebsd.org> References: <3bbf2fe10607250813w8ff9e34pc505bf290e71758@mail.gmail.com> <3bbf2fe10701160851r79b04464m2cbdbb7f644b22b6@mail.gmail.com> <20070116154258.568e1aaf@pleiades.nextvenue.com> <200701161607.43725.jhb@freebsd.org> X-Mailer: Sylpheed-Claws 2.4.0 (GTK+ 2.8.20; i386-portbld-freebsd6.1) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 16 Jan 2007 21:24:04.0023 (UTC) FILETIME=[A5B33470:01C739B4] Cc: Attilio Rao , McKnight , Justin, freebsd-arch@freebsd.org Subject: Re: [PATCH] Mantaining turnstile aligned to 128 bytes in i386 CPUs X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Jan 2007 21:24:05 -0000 On Tue, 16 Jan 2007 16:07:43 -0500 John Baldwin wrote: > On Tuesday 16 January 2007 15:42, Nick Evans wrote: > > On Tue, 16 Jan 2007 17:51:03 +0100 > > "Attilio Rao" wrote: > > > > > 2006/7/28, Attilio Rao : > > > > > > > > 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 > > What did you use to do your benchmark? Also, have you tried adjusting > UMA_ALIGN_SYNC (maybe use 64 - 1)? > > -- > John Baldwin Tested with ebizzy, default runtime options, no WITNESS or INVARIANTS. I haven't tried variations on UMA_ALIGN_SYNC, that's next on the list. I also have a single core P4-Xeon and dual core Presler 915 that I'm going to test this with as soon as they're done building world with the latest -CURRENT. When ULE 2.0 shakes out a bit more I plan on testing with that too. I'll post results on my other hardware sometime tonight. Nick