lkml.org 
[lkml]   [2004]   [May]   [8]   [last100]   RSS Feed
Views: [more markup]   [less markup]   [headers]   [forward]  
 
Messages in this thread
/
DateSat, 8 May 2004 03:41:29 -0700
FromAndrew Morton <>
SubjectRe: dentry bloat.
viro@parcelfarce.linux.theplanet.co.uk wrote:
>
> On Sat, May 08, 2004 at 03:11:59AM -0700, Andrew Morton wrote:
> > Andrew Morton <akpm@osdl.org> wrote:
> > >
> > > Here be another patch.
> > 
> > 
> > Can't help myself!
> > 
> > 
> > - d_vfs_flags can be removed - just use d_flags.  It looks like someone
> >   added d_vfs_flags with the intent of doing bitops on it, but all
> >   modifications are under dcache_lock.  4 bytes saved.
> 
> Bzzert.  ->d_vfs_flags modifications are under dcache_lock; ->d_flags ones
> are *not* - they are up to whatever filesystem code happens to use them.

yup, I hunted down the rest.

> > - Pack things so that dentry.d_name.len and dentry.d_flags occupy the same
> >   word.  4 bytes saved.
> 
> d_name.len is accessed on very hot paths.  In particular, we are looking
> at it while traversing hash chains and we do that without dcache_lock
> (see callers of ->d_compare()).

That's OK.  The d_move() d_movecount and locking logic takes care of that. 
But the ushorts are a bit dopey - I'll give those 4 bytes back.

-
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/

\
 
 \ /
  Last update: 2005-03-22 13:03    [from the cache]
©2003-2008