From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id 7E8CD7F54 for ; Mon, 19 Jan 2015 03:03:52 -0600 (CST) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay1.corp.sgi.com (Postfix) with ESMTP id 6B5F48F8033 for ; Mon, 19 Jan 2015 01:03:52 -0800 (PST) Received: from bombadil.infradead.org ([198.137.202.9]) by cuda.sgi.com with ESMTP id DTgoqPyikO3v2dSu (version=TLSv1 cipher=AES256-SHA bits=256 verify=NO) for ; Mon, 19 Jan 2015 01:03:48 -0800 (PST) Date: Mon, 19 Jan 2015 01:03:47 -0800 From: Christoph Hellwig Subject: Re: [PATCH 05/17] ocfs2: Use generic helpers for quotaon and quotaoff Message-ID: <20150119090347.GE27474@infradead.org> References: <1421412471-4747-1-git-send-email-jack@suse.cz> <1421412471-4747-6-git-send-email-jack@suse.cz> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1421412471-4747-6-git-send-email-jack@suse.cz> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Jan Kara Cc: linux-fsdevel@vger.kernel.org, cluster-devel@redhat.com, ocfs2-devel@oss.oracle.com, xfs@oss.sgi.com On Fri, Jan 16, 2015 at 01:47:39PM +0100, Jan Kara wrote: > Ocfs2 can just use the generic helpers provided by quota code for > turning quotas on and off when quota files are stored as system inodes. > The only difference is the feature test in ocfs2_quota_on() and that is > covered by dquot_quota_enable() checking whether usage tracking is > enabled (which can happen only if the filesystem has the quota feature > set). > > Signed-off-by: Jan Kara > --- > fs/ocfs2/super.c | 32 +------------------------------- > 1 file changed, 1 insertion(+), 31 deletions(-) > > diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c > index 83723179e1ec..706c71c2955d 100644 > --- a/fs/ocfs2/super.c > +++ b/fs/ocfs2/super.c > @@ -1000,36 +1000,6 @@ static void ocfs2_disable_quotas(struct ocfs2_super *osb) > } > } > > -/* Handle quota on quotactl */ > -static int ocfs2_quota_on(struct super_block *sb, int type, int format_id) > -{ > - unsigned int feature[OCFS2_MAXQUOTAS] = { > - OCFS2_FEATURE_RO_COMPAT_USRQUOTA, > - OCFS2_FEATURE_RO_COMPAT_GRPQUOTA}; > - > - if (!OCFS2_HAS_RO_COMPAT_FEATURE(sb, feature[type])) > - return -EINVAL; Where are we doing this feature check now? _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs