From: Andrew Morton <akpm@digeo.com>
To: Dave Jones <davej@codemonkey.org.uk>
Cc: William Lee Irwin III <wli@holomorphy.com>,
Bill Davidsen <davidsen@tmr.com>,
Aaron Lehmann <aaronl@vitelus.com>,
Con Kolivas <conman@kolivas.net>,
linux kernel mailing list <linux-kernel@vger.kernel.org>
Subject: Re: [BENCHMARK] 2.5.47{-mm1} with contest
Date: Thu, 21 Nov 2002 09:21:47 -0800 [thread overview]
Message-ID: <3DDD162B.BAC88F94@digeo.com> (raw)
In-Reply-To: 20021121132014.GC9883@suse.de
Dave Jones wrote:
>
> On Wed, Nov 20, 2002 at 10:54:40PM -0800, Andrew Morton wrote:
> > > I think this merits some investigation. I, for one, am a big user of
> > > SIGIO in userspace C programs...
> > OK, got it back to 119000. Each signal was calling copy_*_user 24 times.
> > This gets it down to six.
>
> Good eyes. But.. this also applies to 2.4 (which should also then
> get faster). So the gap between 2.4 & 2.5 must be somewhere else ?
But 2.4 already inlines the usercopy functions. With this benchmark,
the cost of the function call is visible. Same with the dir_rtn_1
test - it is performing zillions of 3, 7, 10-byte copies into userspace.
The usercopy functions got themselves optimised for large copies and
cache footprint. Maybe we should inline them again. Maybe it doesn't
matter much.
> Also maybe we can do something about that multiple memcpy in copy_fpu_fxsave()
> In fact, that looks a bit fishy. We copy 10 bytes each memcpy, but
> advance the to ptr 5 bytes each iteration. What gives here ?
>
We'd buy a bit by arranging for the in-kernel copy of the fp state
to have the same layout as the hardware. That way it can be done in
a single big, fast, well-aligned slurp. But for some reason that code has
to convert into and out of a different representation.
But the real low-hanging fruit here is the observation that the
test application doesn't use floating point!!!
Maybe we need to take an fp trap now and then to "poll" the application
to see if it is still using float.
next prev parent reply other threads:[~2002-11-21 17:14 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-11-11 23:31 [BENCHMARK] 2.5.47{-mm1} with contest Con Kolivas
2002-11-12 0:09 ` Andrew Morton
2002-11-12 1:51 ` Con Kolivas
2002-11-12 2:07 ` mark walters
2002-11-12 2:18 ` Con Kolivas
2002-11-12 8:52 ` Giuliano Pochini
2002-11-12 9:20 ` Jens Axboe
2002-11-12 9:40 ` Con Kolivas
2002-11-12 3:04 ` Aaron Lehmann
2002-11-12 11:04 ` Andrew Morton
2002-11-12 14:20 ` Aaron Lehmann
2002-11-12 14:23 ` Jens Axboe
2002-11-12 20:37 ` Bill Davidsen
2002-11-12 20:59 ` Andrew Morton
2002-11-20 23:02 ` Andrew Morton
2002-11-21 0:08 ` William Lee Irwin III
2002-11-21 1:11 ` Alan Cox
2002-11-21 6:54 ` Andrew Morton
2002-11-21 13:20 ` Dave Jones
2002-11-21 17:21 ` Andrew Morton [this message]
2002-11-21 17:26 ` William Lee Irwin III
2002-11-21 18:18 ` Dave Jones
2002-11-21 18:25 ` Bill Davidsen
2002-11-21 14:00 ` Dave Jones
2002-11-21 19:59 ` Denis Vlasenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3DDD162B.BAC88F94@digeo.com \
--to=akpm@digeo.com \
--cc=aaronl@vitelus.com \
--cc=conman@kolivas.net \
--cc=davej@codemonkey.org.uk \
--cc=davidsen@tmr.com \
--cc=linux-kernel@vger.kernel.org \
--cc=wli@holomorphy.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox