xfs
[Top] [All Lists]

Re: [REVIEW 1/2] Case insensitive support for XFS - kernel patch

To: Barry Naujok <bnaujok@xxxxxxx>
Subject: Re: [REVIEW 1/2] Case insensitive support for XFS - kernel patch
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Wed, 23 Jan 2008 06:55:22 +0000
Cc: Eric Sandeen <sandeen@xxxxxxxxxxx>, "xfs@xxxxxxxxxxx" <xfs@xxxxxxxxxxx>, xfs-dev <xfs-dev@xxxxxxx>
In-reply-to: <op.t49g2olx3jf8g2@pc-bnaujok.melbourne.sgi.com>
References: <op.t43zfc073jf8g2@pc-bnaujok.melbourne.sgi.com> <47918927.2000603@sandeen.net> <op.t49g2olx3jf8g2@pc-bnaujok.melbourne.sgi.com>
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.17 (2007-11-01)
I'm currently sick in my bed, so just a very quick glance over the
patch and a few comments:

 - no need to export the new inode ops, they're not used by the quota
   or dmapi methods.
 - any chance you could rebase this ontop of my various vnode removals,
   this should make some things a little cleaner.
 - there's a lot of hair dcache internals in here, so please run this
   patch past linux-fsdevel@xxxxxxxxxxxxxxx to get some broader review.


<Prev in Thread] Current Thread [Next in Thread>