From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kara Subject: Re: [PATCH]: Remove redundant underlying file system checking that the quota files resides from dquot_quota_on() Date: Mon, 23 Apr 2012 09:44:12 +0200 Message-ID: <20120423074412.GA6512@quack.suse.cz> References: <4F93A22D.300@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "linux-fsdevel@vger.kernel.org" , jack@suse.cz To: Jeff Liu Return-path: Received: from cantor2.suse.de ([195.135.220.15]:60423 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751690Ab2DWHo1 (ORCPT ); Mon, 23 Apr 2012 03:44:27 -0400 Content-Disposition: inline In-Reply-To: <4F93A22D.300@oracle.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: Hi, On Sun 22-04-12 14:16:13, Jeff Liu wrote: > When executing quotaon(8), all the file systems(ext3/ext4/reiserfs) > binded to the VFS quota framework have already ensured that the quota > files are resides on the same file system with the specified super block > before invoking dquot_quota_on(), looks like we can be safely remove it > from there IMHO. Not really, ext2 doesn't perform the check. Also ext3 or reiserfs when not using journalled quota don't do the check. So the test has to stay there. Is there any special reason why you would like to have it removed? Honza > --- > fs/quota/dquot.c | 11 ++++------- > 1 files changed, 4 insertions(+), 7 deletions(-) > > diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c > index 4674197..cb917a2 100644 > --- a/fs/quota/dquot.c > +++ b/fs/quota/dquot.c > @@ -2199,13 +2199,10 @@ int dquot_quota_on(struct super_block *sb, int type, int format_id, > int error = security_quota_on(path->dentry); > if (error) > return error; > - /* Quota file not on the same filesystem? */ > - if (path->dentry->d_sb != sb) > - error = -EXDEV; > - else > - error = vfs_load_quota_inode(path->dentry->d_inode, type, > - format_id, DQUOT_USAGE_ENABLED | > - DQUOT_LIMITS_ENABLED); > + > + error = vfs_load_quota_inode(path->dentry->d_inode, type, > + format_id, DQUOT_USAGE_ENABLED | > + DQUOT_LIMITS_ENABLED); > return error; > } > EXPORT_SYMBOL(dquot_quota_on); > -- > 1.7.9 -- Jan Kara SUSE Labs, CR