Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751256AbdIPLXc (ORCPT ); Sat, 16 Sep 2017 07:23:32 -0400 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:46630 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751185AbdIPLXb (ORCPT ); Sat, 16 Sep 2017 07:23:31 -0400 X-IronPort-AV: E=Sophos;i="5.42,402,1500933600"; d="scan'208";a="237719602" Date: Sat, 16 Sep 2017 13:23:28 +0200 (CEST) From: Julia Lawall X-X-Sender: jll@hadrien To: Haneen Mohammed cc: outreachy-kernel , Oleg Drokin , Andreas Dilger , James Simmons , Greg Kroah-Hartman , lustre-devel@lists.lustre.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [Outreachy kernel] [PATCH] staging: lustre: lnet: Replace list_for_each with list_for_each_entry In-Reply-To: <20170916004148.GA25693@Haneen> Message-ID: References: <20170916004148.GA25693@Haneen> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3128 Lines: 99 On Fri, 15 Sep 2017, Haneen Mohammed wrote: > Replace use of the combination of list_for_each and list_entry > with list_for_each_entry to simplify the code and remove variables > that are used only in list_for_each. > Issue found and corrected using Coccinelle script: > > @r@ > expression head, member, e; > type T1, T2, T3; > iterator name list_for_each, list_for_each_entry; > identifier pos, var; > @@ > > -T1 *pos; > ...when!=pos=e; > > -list_for_each(pos, head) > +list_for_each_entry(var, head, member) > { > ...when!=pos=e; > when!=T3 *var; > -var = list_entry(pos, T2, member); > ...when!=pos=e; > } > ...when!=pos=e; Actually, one could consider that there should be when != pos, not when != pos=e, because you need that there are no references at all to pos to be able to delete it. But it's true that if pos is not initialized then there should be no other kinds of references either. The other possibility for initialization would be eg f(...,&pos,...) You could be suspicious of &pos in general. But anyway there is no &pos in this code. julia > Signed-off-by: Haneen Mohammed Acked-by: Julia Lawall > --- > drivers/staging/lustre/lnet/lnet/router.c | 9 ++------- > 1 file changed, 2 insertions(+), 7 deletions(-) > > diff --git a/drivers/staging/lustre/lnet/lnet/router.c b/drivers/staging/lustre/lnet/lnet/router.c > index 3df101b..b8eba33 100644 > --- a/drivers/staging/lustre/lnet/lnet/router.c > +++ b/drivers/staging/lustre/lnet/lnet/router.c > @@ -222,15 +222,12 @@ struct lnet_remotenet * > lnet_find_net_locked(__u32 net) > { > struct lnet_remotenet *rnet; > - struct list_head *tmp; > struct list_head *rn_list; > > LASSERT(!the_lnet.ln_shutdown); > > rn_list = lnet_net2rnethash(net); > - list_for_each(tmp, rn_list) { > - rnet = list_entry(tmp, struct lnet_remotenet, lrn_list); > - > + list_for_each_entry(rnet, rn_list, lrn_list) { > if (rnet->lrn_net == net) > return rnet; > } > @@ -243,7 +240,6 @@ static void lnet_shuffle_seed(void) > __u32 lnd_type, seed[2]; > struct timespec64 ts; > struct lnet_ni *ni; > - struct list_head *tmp; > > if (seeded) > return; > @@ -254,8 +250,7 @@ static void lnet_shuffle_seed(void) > * Nodes with small feet have little entropy > * the NID for this node gives the most entropy in the low bits > */ > - list_for_each(tmp, &the_lnet.ln_nis) { > - ni = list_entry(tmp, struct lnet_ni, ni_list); > + list_for_each_entry(ni, &the_lnet.ln_nis, ni_list) { > lnd_type = LNET_NETTYP(LNET_NIDNET(ni->ni_nid)); > > if (lnd_type != LOLND) > -- > 2.7.4 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > To post to this group, send email to outreachy-kernel@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20170916004148.GA25693%40Haneen. > For more options, visit https://groups.google.com/d/optout. >