Re: [PATCH] make growfs check device size limits too

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Thu, Apr 26, 2007 at 04:30:14PM +1000, Nathan Scott wrote:
> On the mount path we check for a superblock that describes a filesystem
> to large for the running kernel to handle.  This catches the case of an
> attempt to mount a >16TB filesystem on i386 (where we are limited by the
> page->index size, for XFS metadata buffers in xfs_buf.c).
> 
> This patch makes similar checks on the growfs code paths for regular and
> realtime growth, else we can end up with filesystem corruption, it would
> seem (from #xfs chatter).  Untested patch follows; probably better to do
> this as a macro, in a header, and call that in each place...?

Yeah, the check should probably we in one place only.  Given that's it's
only used in slow pathes a function would probably do it.