linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@infradead.org>
To: Barry Naujok <bnaujok@sgi.com>
Cc: Christoph Hellwig <hch@infradead.org>,
	xfs@oss.sgi.com, linux-fsdevel@vger.kernel.org, aia21@cantab.net
Subject: Re: [PATCH 3/4] XFS: Return case-insensitive match for dentry cache
Date: Thu, 15 May 2008 00:57:00 -0400	[thread overview]
Message-ID: <20080515045700.GA4328@infradead.org> (raw)
In-Reply-To: <op.ua4wa7t03jf8g2@pc-bnaujok.melbourne.sgi.com>

On Wed, May 14, 2008 at 05:55:45PM +1000, Barry Naujok wrote:
> Not quite sure if this is the right test, but I did 1000 creates on
> a brand new filesystem with and without ci on my SATA drive, both
> sustained almost 600 creates per second.
>
> I believe creates would be the worst case scenario for not adding
> negative dentries?

No, negative dentries shouldn't have any effect on that.  negative
entries help to optimize away lookups.  E.g. thing of the PATH variable
and say your shell is not in the first directory listed there.  Having
a negative dentry for it means that you don't have to do a lookup in
the first directories everytime someone wants to use the shell.

  reply	other threads:[~2008-05-15  4:57 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-13  7:57 [PATCH 0/4] XFS: Case-insensitive support - ASCII only Barry Naujok
2008-05-13  7:57 ` [PATCH 1/4] XFS: Name operation vector for hash and compare Barry Naujok
2008-05-13  8:31   ` Christoph Hellwig
2008-05-13  7:57 ` [PATCH 2/4] XFS: add op_flags field and helpers to xfs_da_args Barry Naujok
2008-05-13  8:34   ` Christoph Hellwig
2008-05-14  6:12     ` Barry Naujok
2008-05-13  7:57 ` [PATCH 3/4] XFS: Return case-insensitive match for dentry cache Barry Naujok
2008-05-13  8:57   ` Christoph Hellwig
2008-05-14  6:15     ` Barry Naujok
2008-05-14  7:55     ` Barry Naujok
2008-05-15  4:57       ` Christoph Hellwig [this message]
2008-05-15  5:14         ` Barry Naujok
2008-05-15 13:43           ` Anton Altaparmakov
2008-05-15 14:11             ` Christoph Hellwig
2008-05-16  0:30               ` Barry Naujok
2008-05-16  7:25               ` Anton Altaparmakov
2008-05-20  2:24             ` Barry Naujok
2008-05-20 18:23               ` Christoph Hellwig
2008-05-20 20:50                 ` Sunil Mushran
2008-05-13  7:57 ` [PATCH 4/4] XFS: ASCII case-insensitive support Barry Naujok
2008-05-13  8:58   ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2008-05-14  7:52 [PATCH 0/4] XFS: ASCII case-insensitivity support Barry Naujok
2008-05-14  7:52 ` [PATCH 3/4] XFS: Return case-insensitive match for dentry cache Barry Naujok

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=20080515045700.GA4328@infradead.org \
    --to=hch@infradead.org \
    --cc=aia21@cantab.net \
    --cc=bnaujok@sgi.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=xfs@oss.sgi.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;
as well as URLs for NNTP newsgroup(s).