From owner-freebsd-current@FreeBSD.ORG Tue Jan 16 19:02:47 2007 Return-Path: X-Original-To: freebsd-current@freebsd.org Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id CF32316A47C for ; Tue, 16 Jan 2007 19:02:47 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: from nf-out-0910.google.com (nf-out-0910.google.com [64.233.182.189]) by mx1.freebsd.org (Postfix) with ESMTP id 2CA1613C45B for ; Tue, 16 Jan 2007 19:02:46 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: by nf-out-0910.google.com with SMTP id k27so2481077nfc for ; Tue, 16 Jan 2007 11:02:45 -0800 (PST) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references:x-google-sender-auth; b=YZE0tLgPRXi+hfCE3w1kfRdpT8GDWp3BAzGffR6YFMLIM34rfy7wI6OrtglSL0Kj/GiQVHkwoFuumYaaNBZjNeW1r0D9xNsLUCro2NCqmZfNADKCoyZBLqsosSDLwe29d0qhmMJD05HcZ4Ign0pFMmEwBEPRGRlAQKJ/RpiJzTo= Received: by 10.49.80.12 with SMTP id h12mr3173846nfl.1168974160335; Tue, 16 Jan 2007 11:02:40 -0800 (PST) Received: by 10.48.238.9 with HTTP; Tue, 16 Jan 2007 11:02:40 -0800 (PST) Message-ID: <3bbf2fe10701161102w5a094dfge7faf641ab1a0425@mail.gmail.com> Date: Tue, 16 Jan 2007 20:02:40 +0100 From: "Attilio Rao" Sender: asmrookie@gmail.com To: "Nick Evans" In-Reply-To: <20070116131836.0681a51d@pleiades.nextvenue.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline 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> <20070116122925.5cdb1ded@pleiades.nextvenue.com> <3bbf2fe10701160951x1cb4cd75pd843b453a8f4a96@mail.gmail.com> <20070116131836.0681a51d@pleiades.nextvenue.com> X-Google-Sender-Auth: 2259c4ab2e877946 Cc: Kip Macy , freebsd-current@freebsd.org, Suleiman Souhlal , Pawel Jakub Dawidek , freebsd-arch@freebsd.org Subject: Re: [PATCH] Mantaining turnstile aligned to 128 bytes in i386 CPUs X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Jan 2007 19:02:48 -0000 2007/1/16, Nick Evans : > On Tue, 16 Jan 2007 18:51:30 +0100 > "Attilio Rao" wrote: > > > 2007/1/16, Nick Evans : > > > 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" > > > > > > Attilio, > > > > > > What class of Xeon do you need this tested on, P3 or the newer P4+ stuff? > > > I have a few systems here I can test this on including a quad P3-Xeon box > > > that I've been testing Jeff's ULE 2.0 on. Do you have specific extra test > > > points or are the before and after for the (non)preemption cases > > > sufficient? I can also give you access to the systems if that is easier. > > > > Hi Nick, > > thanks a lot for responsivness and help. > > I think that P3 Xeon alredy should show some speedup, in particular I > > would see some tests on the P3-Xeon quad. > > It would be enough reproduce the test I did (using the same options > > file and starting ebizzy for 1-2 times before the results gathering). > > I hope you are suitable for doing this benchmarks alone (FreeBSD + > > university + job don't leave me too much time ATM :(( ). > > > > Thanks a lot for your efforts, > > Attilio > > > > > > -- > > Peace can only be achieved by understanding - A. Einstein > > > Yea, we should be able to handle benchmarking this if you can help me > through the first hurdle. Do you have a version that compiles cleanly on > -CURRENT? The version I dug up off lkml via google complains: > > root@current[13:10]# make > gcc -Wall -lpthread -o ebizzy ebizzy.c > In file included from ebizzy.c:43: > /usr/include/malloc.h:3:2: #error " has been replaced by " > ebizzy.c: In function `read_options': > ebizzy.c:212: warning: implicit declaration of function `mallopt' > ebizzy.c:212: error: `M_MMAP_MAX' undeclared (first use in this function) > ebizzy.c:212: error: (Each undeclared identifier is reported only once > ebizzy.c:212: error: for each function it appears in.) > ebizzy.c: In function `alloc_mem': > ebizzy.c:224: error: `MAP_ANONYMOUS' undeclared (first use in this function) > *** Error code 1 > > Stop in /root/ebizzy. > > > I didn't see an updated version listed in our archives anywhere. Since ebizzy doesn't compile natively on FreeBSD try this patch (it disables the -m option, but it doesn't matter currently): http://users.gufi.org/~rookie/works/patches/ts-sq/ebizzy.diff Attilio -- Peace can only be achieved by understanding - A. Einstein