Re: [PATCH 1/2] wire up ->open for directories

[Dave Chinner <david@xxxxxxxxxxxxx>]

On Sun, Oct 26, 2008 at 04:34:37PM -0400, Christoph Hellwig wrote:
> Currently there's no ->open method set for directories on XFS.  That
> means we don't perform any check for opening too large directories
> without O_LARGEFILE, we don't check for shut down filesystems, and we
> don't actually do the readahead for the first block in the directory.
> 
> Instead of just setting the directories open routine to xfs_file_open
> we merge the shutdown check directly into xfs_file_open and create
> a new xfs_dir_open that first calls xfs_file_open and then performs
> the readahead for block 0.

Looks good to me.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx