All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Layton <jlayton@poochiereds.net>
To: linux-fsdevel@vger.kernel.org
Subject: Re: [RFC][PATCH] ensure i_ino uniqueness in filesystems without permanent inode numbers (via lazy hashing)
Date: Wed, 29 Nov 2006 15:33:26 -0500	[thread overview]
Message-ID: <456DEE96.6060008@poochiereds.net> (raw)
In-Reply-To: <4569980E.4020804@redhat.com>

Jeff Layton wrote:
> Here is a different approach to the problem of i_ino uniqueness. Again,
> I'll refer to my original email and patch for a description of the 
> problem...
> 
> With this patch, I'm taking the approach of only assigning out an i_ino
> value when it's actually needed. This adds a lazy_getattr function. If
> i_ino is 0, then this does an iunique and hashes the inode before doing
> the actual getattr.
> 

Actually, after having a closer look at this, I don't think this turns 
out to be feasible either. Some filesystems (including pipefs) want an 
i_ino value early on for generating the qstr passed to d_alloc.

Since they want the value so early, there would be little benefit in 
attempting to delay assigning an i_ino value. It might be a win for some 
filesystems, but ones like pipefs and sockfs wouldn't be able to use 
this scheme, and those are the ones we're primarily concerned with 
performance-wise.

I'm going to plan to clean up my IDR patch and resubmit it, as I think 
it's probably the best scheme we have that seems to actually fix the 
problem.

Unless someone else has a better idea... :-)

-- Jeff


      reply	other threads:[~2006-11-29 20:33 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-26 13:35 [RFC][PATCH] ensure i_ino uniqueness in filesystems without permanent inode numbers (via lazy hashing) Jeff Layton
2006-11-29 20:33 ` Jeff Layton [this message]

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=456DEE96.6060008@poochiereds.net \
    --to=jlayton@poochiereds.net \
    --cc=linux-fsdevel@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.