From: "Dieter Nützel" <Dieter.Nuetzel@hamburg.de>
To: Andrea Arcangeli <andrea@suse.de>
Cc: Robert Love <rml@tech9.net>,
Roger Larsson <roger.larsson@norran.net>,
linux-kernel@vger.kernel.org,
ReiserFS List <reiserfs-list@namesys.com>
Subject: Re: [PATCH] Preemption Latency Measurement Tool
Date: Thu, 20 Sep 2001 09:57:50 +0200 [thread overview]
Message-ID: <20010920075736Z274357-761+10744@vger.kernel.org> (raw)
In-Reply-To: <1000939458.3853.17.camel@phantasy> <20010920063143.424BD1E41A@Cantor.suse.de> <20010920084131.C1629@athlon.random>
In-Reply-To: <20010920084131.C1629@athlon.random>
Am Donnerstag, 20. September 2001 08:41 schrieb Andrea Arcangeli:
> Those inodes lines reminded me one thing, you may want to give it a try:
>
> --- 2.4.10pre12aa1/fs/inode.c.~1~ Thu Sep 20 01:44:07 2001
> +++ 2.4.10pre12aa1/fs/inode.c Thu Sep 20 08:37:33 2001
> @@ -295,6 +295,12 @@
> * so we have to start looking from the list head.
> */
> tmp = head;
> +
> + if (unlikely(current->need_resched)) {
> + spin_unlock(&inode_lock);
> + schedule();
> + spin_lock(&inode_lock);
> + }
> }
> }
>
You've forgotten a one liner.
#include <linux/locks.h>
+#include <linux/compiler.h>
But this is not enough. Even with reniced artsd (-20).
Some shorter hiccups (0.5~1 sec).
-Dieter
dbench 16 (without renice)
Throughput 31.0483 MB/sec (NB=38.8104 MB/sec 310.483 MBit/sec)
7.270u 29.290s 1:09.03 52.9% 0+0k 0+0io 511pf+0w
load: 1034
Worst 20 latency times of 4261 measured in this period.
usec cause mask start line/file address end line/file
4617 reacqBKL 0 1375/sched.c c0114d94 1381/sched.c
2890 spin_lock 1 1376/sched.c c0114db3 697/sched.c
2072 BKL 1 533/inode.c c016d9cd 697/sched.c
2062 BKL 1 1302/inode.c c016f359 52/inode.c
2039 BKL 1 1302/inode.c c016f359 697/sched.c
1908 BKL 0 1302/inode.c c016f359 929/namei.c
1870 BKL 0 1302/inode.c c016f359 1381/sched.c
1859 spin_lock 0 547/sched.c c0112fe4 697/sched.c
1834 BKL 0 30/inode.c c016ce51 1381/sched.c
1834 BKL 1 1302/inode.c c016f359 1380/sched.c
1833 BKL 1 1437/namei.c c014c42f 697/sched.c
1831 BKL 1 452/exit.c c011af61 697/sched.c
1820 BKL 1 1437/namei.c c014c42f 842/inode.c
1797 BKL 0 1302/inode.c c016f359 842/inode.c
1741 spin_lock 0 547/sched.c c0112fe4 1380/sched.c
1696 reacqBKL 1 1375/sched.c c0114d94 697/sched.c
1690 BKL 1 1437/namei.c c014c42f 1380/sched.c
1652 BKL 1 533/inode.c c016d9cd 1380/sched.c
1648 BKL 1 1870/namei.c c014d420 1381/sched.c
1643 BKL 0 927/namei.c c014b2bf 1381/sched.c
dbench 16 (artsd renice -20)
Throughput 32.8102 MB/sec (NB=41.0127 MB/sec 328.102 MBit/sec)
7.490u 28.570s 1:05.38 55.1% 0+0k 0+0io 511pf+0w
load: 963
Worst 20 latency times of 4649 measured in this period.
usec cause mask start line/file address end line/file
4828 spin_lock 1 1376/sched.c c0114db3 697/sched.c
2446 spin_lock 1 1376/sched.c c0114db3 1380/sched.c
2237 reacqBKL 1 1375/sched.c c0114d94 697/sched.c
2205 reacqBKL 1 1375/sched.c c0114d94 1381/sched.c
2205 BKL 1 1302/inode.c c016f359 1380/sched.c
2076 BKL 0 1302/inode.c c016f359 1381/sched.c
2065 spin_lock 0 547/sched.c c0112fe4 697/sched.c
2048 BKL 0 30/inode.c c016ce51 697/sched.c
2040 BKL 1 452/exit.c c011af61 697/sched.c
1982 BKL 0 1437/namei.c c014c42f 1381/sched.c
1953 BKL 0 1437/namei.c c014c42f 1380/sched.c
1942 BKL 1 1302/inode.c c016f359 697/sched.c
1847 BKL 0 1302/inode.c c016f359 842/inode.c
1761 BKL 0 1437/namei.c c014c42f 697/sched.c
1734 reacqBKL 1 1375/sched.c c0114d94 52/inode.c
1708 BKL 0 30/inode.c c016ce51 1306/inode.c
1705 BKL 0 927/namei.c c014b2bf 1381/sched.c
1688 BKL 0 1302/inode.c c016f359 929/namei.c
1679 BKL 1 1437/namei.c c014c42f 842/inode.c
1648 BKL 1 1302/inode.c c016f359 1439/namei.c
dbench 48 looks like this (without renice)
Throughput 24.5736 MB/sec (NB=30.717 MB/sec 245.736 MBit/sec)
23.250u 89.760s 4:18.85 43.6% 0+0k 0+0io 1311pf+0w
load: 4429
Worst 20 latency times of 8033 measured in this period.
usec cause mask start line/file address end line/file
10856 spin_lock 1 1376/sched.c c0114db3 697/sched.c
10705 BKL 1 1302/inode.c c016f359 697/sched.c
10577 spin_lock 1 1376/sched.c c0114db3 303/namei.c
9427 spin_lock 1 547/sched.c c0112fe4 697/sched.c
8526 reacqBKL 1 1375/sched.c c0114d94 697/sched.c
4492 reacqBKL 1 1375/sched.c c0114d94 1381/sched.c
4171 BKL 1 1302/inode.c c016f359 1381/sched.c
3902 reacqBKL 0 1375/sched.c c0114d94 1306/inode.c
3376 spin_lock 0 1376/sched.c c0114db3 1380/sched.c
3132 BKL 0 1302/inode.c c016f359 1380/sched.c
3096 spin_lock 1 547/sched.c c0112fe4 1380/sched.c
2808 BKL 0 30/inode.c c016ce51 1381/sched.c
2807 spin_lock 0 547/sched.c c0112fe4 1381/sched.c
2782 BKL 0 452/exit.c c011af61 1380/sched.c
2631 spin_lock 0 483/dcache.c c0153efa 520/dcache.c
2533 BKL 0 533/inode.c c016d9cd 1380/sched.c
2489 BKL 0 927/namei.c c014b2bf 1380/sched.c
2389 BKL 1 452/exit.c c011af61 52/inode.c
2369 BKL 1 1302/inode.c c016f359 842/inode.c
2327 BKL 1 30/inode.c c016ce51 1380/sched.c
next prev parent reply other threads:[~2001-09-20 7:58 UTC|newest]
Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-09-19 22:44 [PATCH] Preemption Latency Measurement Tool Robert Love
2001-09-20 1:40 ` Ignacio Vazquez-Abrams
2001-09-20 2:23 ` safemode
2001-09-20 1:13 ` David Lang
2001-09-20 2:57 ` Robert Love
2001-09-20 2:38 ` Robert Love
2001-09-20 6:31 ` Dieter Nützel
2001-09-20 6:31 ` Dieter Nützel
2001-09-20 20:27 ` Robert Love
[not found] ` <200109202111.f8KLBgG16833@zero.tech9.net>
2001-09-20 22:09 ` [PATCH] Preemption patch 2.4.9-ac12 Robert Love
[not found] ` <20010920063143.424BD1E41A@Cantor.suse.de>
2001-09-20 6:41 ` [PATCH] Preemption Latency Measurement Tool Andrea Arcangeli
2001-09-20 7:57 ` Dieter Nützel [this message]
[not found] ` <20010920075751.6CA791E6B2@Cantor.suse.de>
2001-09-20 8:21 ` Andrea Arcangeli
2001-09-20 20:13 ` george anzinger
2001-09-20 20:38 ` Randy.Dunlap
2001-09-20 21:10 ` Robert Love
2001-09-20 21:35 ` Dieter Nützel
2001-09-20 22:03 ` Oliver Xymoron
2001-09-20 22:51 ` Dieter Nützel
[not found] ` <200109202252.f8KMqLG17327@zero.tech9.net>
2001-09-21 3:17 ` Robert Love
2001-09-21 15:48 ` george anzinger
2001-09-22 21:09 ` Dieter Nützel
2001-09-22 23:40 ` safemode
2001-09-22 23:46 ` Dieter Nützel
2001-09-23 0:15 ` safemode
[not found] ` <200109222340.BAA37547@blipp.internet5.net>
2001-09-23 0:38 ` Roger Larsson
2001-09-23 1:42 ` safemode
2001-09-23 3:02 ` Robert Love
2001-09-23 16:43 ` Roger Larsson
2001-09-23 0:42 ` Dieter Nützel
[not found] ` <200109222341.f8MNfnG25152@zero.tech9.net>
2001-09-23 2:50 ` Robert Love
2001-09-23 3:14 ` george anzinger
2001-09-23 4:06 ` Dieter Nützel
[not found] ` <200109222347.f8MNlMG25157@zero.tech9.net>
2001-09-23 2:54 ` Robert Love
2001-09-27 0:02 ` [reiserfs-list] " Dieter Nützel
[not found] ` <200109230016.f8N0G6G25222@zero.tech9.net>
2001-09-23 2:58 ` Robert Love
[not found] ` <200109222120.f8MLKYG24859@zero.tech9.net>
2001-09-23 2:44 ` Robert Love
[not found] ` <200109200757.JAA60995@blipp.internet5.net>
2001-09-20 17:37 ` Roger Larsson
2001-09-20 21:29 ` Robert Love
2001-09-20 21:53 ` Dieter Nützel
[not found] ` <200109200758.f8K7wEG13675@zero.tech9.net>
2001-09-20 21:09 ` Robert Love
2001-09-20 20:01 ` Tobias Diedrich
2001-09-20 22:01 ` Robert Love
2001-09-22 3:57 ` Andre Pang
2001-09-22 6:10 ` Robert Love
2001-09-22 7:22 ` Andre Pang
2001-09-23 3:18 ` george anzinger
2001-09-23 3:21 ` Robert Love
2001-09-23 7:05 ` Robert Love
2001-09-23 12:03 ` Andre Pang
2001-09-23 18:31 ` Robert Love
2001-09-22 12:56 ` ksoftirqd? (Was: Re: [PATCH] Preemption Latency Measurement Tool) Roger Larsson
2001-09-22 13:14 ` Andrea Arcangeli
2001-09-22 20:51 ` Roger Larsson
2001-09-22 21:33 ` Andrea Arcangeli
[not found] <200109202253.RAA21082@waste.org>
2001-09-20 23:15 ` [PATCH] Preemption Latency Measurement Tool Oliver Xymoron
2001-09-21 0:42 ` Roger Larsson
2001-09-21 1:03 ` Alan Cox
2001-09-21 1:22 ` Andrea Arcangeli
2001-09-21 1:51 ` Rik van Riel
2001-09-21 1:38 ` Roger Larsson
2001-09-21 1:53 ` Roger Larsson
2001-09-21 2:08 ` Roger Larsson
2001-09-21 2:29 ` Rik van Riel
2001-09-21 16:24 ` Jussi Laako
2001-09-21 16:36 ` Alan Cox
2001-09-21 18:46 ` Thomas Sailer
2001-09-22 10:30 ` Jussi Laako
2001-09-21 16:18 ` Stefan Westerfeld
2001-09-21 20:18 ` Dieter Nützel
[not found] ` <200109212018.f8LKImG21229@zero.tech9.net>
2001-09-21 21:47 ` Robert Love
-- strict thread matches above, loose matches on Subject: below --
2002-04-09 5:23 [PATCH] preemption latency measurement tool Robert Love
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=20010920075736Z274357-761+10744@vger.kernel.org \
--to=dieter.nuetzel@hamburg.de \
--cc=andrea@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=reiserfs-list@namesys.com \
--cc=rml@tech9.net \
--cc=roger.larsson@norran.net \
/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