From: William Lee Irwin III <wli@holomorphy.com>
To: Kirill Korotaev <kk@sw.ru>
Cc: Andrew Morton <akpm@osdl.org>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Invalidate_inodes can be very slow
Date: Mon, 13 Oct 2003 04:54:58 -0700 [thread overview]
Message-ID: <20031013115458.GI16158@holomorphy.com> (raw)
In-Reply-To: <200310131545.01779.kk@sw.ru>
William Lee Irwin III <wli@holomorphy.com> wrote:
>> Generally dcache_lock stands in front of inode_lock, even with the
>> current hashtable RCU code. inode_lock has been seen before in unusual
>> situations I don't remember offhand, though generally it's not #1.
>> The workloads used for, say, benchmark testing don't adequately model
>> situations like what you just mentioned (or a number of other real-life
>> usage cases), so per-sb inode_lock may be worth considering on a priori
>> grounds, though it would probably be better to actually set something
>> up to test that scenario.
On Mon, Oct 13, 2003 at 03:45:01PM +0400, Kirill Korotaev wrote:
> This patch can be tested quite easily. Main changes are in invalidate_list.
> This path can be triggered by umount/quota off.
> So I tested it the following way:
> 1. mounting/working/umounting partition (and turning quota on/off)
> 2. running memeater to call prune_icache when partition was mounted
> to test that lists are ok.
> All other places should be ok, since simple list_add/list_del is inserted.
Sorry if I was unclear, I had in mind SMP performance testing of mount
and unmount -heavy workloads, like uni setups with many automounted fs's,
not stability testing per se.
-- wli
next prev parent reply other threads:[~2003-10-13 11:51 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-10-13 9:18 Kirill Korotaev
2003-10-13 9:53 ` William Lee Irwin III
2003-10-13 10:46 ` Kirill Korotaev
2003-10-13 11:06 ` William Lee Irwin III
2003-10-13 11:08 ` Andrew Morton
2003-10-13 11:19 ` William Lee Irwin III
2003-10-13 11:45 ` Kirill Korotaev
2003-10-13 11:54 ` William Lee Irwin III [this message]
2003-10-13 12:02 ` Kirill Korotaev
2003-10-13 12:11 ` William Lee Irwin III
2003-10-13 12:21 ` Kirill Korotaev
2003-10-13 12:29 ` Nikita Danilov
2003-10-13 13:08 ` Kirill Korotaev
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=20031013115458.GI16158@holomorphy.com \
--to=wli@holomorphy.com \
--cc=akpm@osdl.org \
--cc=kk@sw.ru \
--cc=linux-kernel@vger.kernel.org \
/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
all inboxes | Powered by JetHome®