From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753336AbcADJ7P (ORCPT ); Mon, 4 Jan 2016 04:59:15 -0500 Received: from mx2.suse.de ([195.135.220.15]:47679 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751818AbcADJ7M (ORCPT ); Mon, 4 Jan 2016 04:59:12 -0500 Date: Mon, 4 Jan 2016 10:59:21 +0100 From: Jan Kara To: Julia Lawall Cc: Jan Kara , kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, Mark Fasheh , Joel Becker , ocfs2-devel@oss.oracle.com Subject: Re: [PATCH] quota: constify qtree_fmt_operations structures Message-ID: <20160104095921.GA13014@quack.suse.cz> References: <1451634817-1903-1-git-send-email-Julia.Lawall@lip6.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1451634817-1903-1-git-send-email-Julia.Lawall@lip6.fr> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri 01-01-16 08:53:37, Julia Lawall wrote: > The qtree_fmt_operations structures are never modified, so declare them as > const. > > Done with the help of Coccinelle. > > Signed-off-by: Julia Lawall Thanks! I've merged the patch to my tree. Honza > > --- > fs/ocfs2/quota.h | 2 +- > fs/ocfs2/quota_global.c | 2 +- > fs/quota/quota_v2.c | 4 ++-- > include/linux/dqblk_qtree.h | 2 +- > 4 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/include/linux/dqblk_qtree.h b/include/linux/dqblk_qtree.h > index 82a1652..ff8b553 100644 > --- a/include/linux/dqblk_qtree.h > +++ b/include/linux/dqblk_qtree.h > @@ -34,7 +34,7 @@ struct qtree_mem_dqinfo { > unsigned int dqi_entry_size; /* Size of quota entry in quota file */ > unsigned int dqi_usable_bs; /* Space usable in block for quota data */ > unsigned int dqi_qtree_depth; /* Precomputed depth of quota tree */ > - struct qtree_fmt_operations *dqi_ops; /* Operations for entry manipulation */ > + const struct qtree_fmt_operations *dqi_ops; /* Operations for entry manipulation */ > }; > > int qtree_write_dquot(struct qtree_mem_dqinfo *info, struct dquot *dquot); > diff --git a/fs/quota/quota_v2.c b/fs/quota/quota_v2.c > index 2aa012a..ed85d4f 100644 > --- a/fs/quota/quota_v2.c > +++ b/fs/quota/quota_v2.c > @@ -30,13 +30,13 @@ static void v2r1_mem2diskdqb(void *dp, struct dquot *dquot); > static void v2r1_disk2memdqb(struct dquot *dquot, void *dp); > static int v2r1_is_id(void *dp, struct dquot *dquot); > > -static struct qtree_fmt_operations v2r0_qtree_ops = { > +static const struct qtree_fmt_operations v2r0_qtree_ops = { > .mem2disk_dqblk = v2r0_mem2diskdqb, > .disk2mem_dqblk = v2r0_disk2memdqb, > .is_id = v2r0_is_id, > }; > > -static struct qtree_fmt_operations v2r1_qtree_ops = { > +static const struct qtree_fmt_operations v2r1_qtree_ops = { > .mem2disk_dqblk = v2r1_mem2diskdqb, > .disk2mem_dqblk = v2r1_disk2memdqb, > .is_id = v2r1_is_id, > diff --git a/fs/ocfs2/quota.h b/fs/ocfs2/quota.h > index b6d5133..d153e6e 100644 > --- a/fs/ocfs2/quota.h > +++ b/fs/ocfs2/quota.h > @@ -82,7 +82,7 @@ struct ocfs2_quota_chunk { > extern struct kmem_cache *ocfs2_dquot_cachep; > extern struct kmem_cache *ocfs2_qf_chunk_cachep; > > -extern struct qtree_fmt_operations ocfs2_global_ops; > +extern const struct qtree_fmt_operations ocfs2_global_ops; > > struct ocfs2_quota_recovery *ocfs2_begin_quota_recovery( > struct ocfs2_super *osb, int slot_num); > diff --git a/fs/ocfs2/quota_global.c b/fs/ocfs2/quota_global.c > index 44df24b..f0c7ae1 100644 > --- a/fs/ocfs2/quota_global.c > +++ b/fs/ocfs2/quota_global.c > @@ -123,7 +123,7 @@ static int ocfs2_global_is_id(void *dp, struct dquot *dquot) > dquot->dq_id); > } > > -struct qtree_fmt_operations ocfs2_global_ops = { > +const struct qtree_fmt_operations ocfs2_global_ops = { > .mem2disk_dqblk = ocfs2_global_mem2diskdqb, > .disk2mem_dqblk = ocfs2_global_disk2memdqb, > .is_id = ocfs2_global_is_id, > > -- Jan Kara SUSE Labs, CR