From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH] Increase file size limit on FreeBSD UFS flavours Date: Thu, 17 May 2012 16:10:52 -0700 Message-ID: <20120517161052.d37cf8ae.akpm@linux-foundation.org> References: <4FAFBC30.4040305@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Evgeniy Dushistov To: Vladimir =?UTF-8?Q?'=CF=86-coder/phcoder'?= Serbinenko Return-path: In-Reply-To: <4FAFBC30.4040305@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Sun, 13 May 2012 15:50:40 +0200 Vladimir '__-coder/phcoder' Serbinenko wrote: > Increase the file size limit on FreeBSD UFS flavours. > > Signed-off-by: Vladimir Serbinenko > > diff --git a/fs/ufs/super.c b/fs/ufs/super.c > index ac8e279..7e80a68 100644 > --- a/fs/ufs/super.c > +++ b/fs/ufs/super.c > @@ -768,6 +768,7 @@ static int ufs_fill_super(struct super_block *sb, void *data, int silent) > uspi->s_fshift = 9; > uspi->s_sbsize = super_block_size = 1536; > uspi->s_sbbase = 0; > + sb->s_maxbytes = MAX_LFS_FILESIZE; > flags |= UFS_DE_44BSD | UFS_UID_44BSD | UFS_ST_44BSD | UFS_CG_44BSD; > break; > case UFS_MOUNT_UFSTYPE_UFS2: > @@ -778,6 +779,7 @@ static int ufs_fill_super(struct super_block *sb, void *data, int silent) > uspi->s_fshift = 9; > uspi->s_sbsize = super_block_size = 1536; > uspi->s_sbbase = 0; > + sb->s_maxbytes = MAX_LFS_FILESIZE; > flags |= UFS_TYPE_UFS2 | UFS_DE_44BSD | UFS_UID_44BSD | UFS_ST_44BSD | UFS_CG_44BSD; > break; > But MAX_NON_LFS exists for a reason. Before this change, the Linux UFS implementation had never had to deal with >4G files (I assume). Are you telling us that the fs has been thoroughly audited and tested with large files and that it all worked OK with no other changes?