Re: [CFT][PATCH] ramfs/tmpfs readdir()

From: Trond Myklebust (trond.myklebust@fys.uio.no)
Date: Mon Nov 05 2001 - 08:05:00 EST


>>>>> " " == Manfred Spraul <manfred@colorfullife.com> writes:

> Content-Type: text/plain; charset=us-ascii
> Content-Transfer-Encoding: 7bit

>>
>> Note that other filesystems would already enjoy having a
>> d_offset in the dentry: it allows for various other
>> optimizations (ie making "unlink()" a O(1) operation, by not
>> having to search the directory).
>>
> The dentry structure already contains 2 members for filesystem
> use (d_time and d_fsdata), is a third member really required?

It could be useful. For NFS, we're also looking for 64-bits in which
to store the parent directory's mtime. This is needed in order to
improve the dentry revalidation heuristics.

Ultimately, though, allowing the filesystem to allocate dentries
itself on its own slab might be a preferable manner to accomodate the
need for these private fields. IOW to allow one to allocate

   struct nfs_dentry {
        struct dentry dentry;
        ... private fields ....
   };

Cheers,
  Trond
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Wed Nov 07 2001 - 21:00:26 EST