Message ID | 20240827100407.3914090-3-lihongbo22@huawei.com (mailing list archive) |
---|---|
State | Superseded |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | make use of the helper macro LIST_HEAD() | expand |
On Tue, Aug 27, 2024 at 06:04:04PM +0800, Hongbo Li wrote: > list_head can be initialized automatically with LIST_HEAD() > instead of calling INIT_LIST_HEAD(). Here we can simplify > the code. > > Signed-off-by: Hongbo Li <lihongbo22@huawei.com> > --- > net/tipc/socket.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/net/tipc/socket.c b/net/tipc/socket.c > index 1a0cd06f0eae..9d30e362392c 100644 > --- a/net/tipc/socket.c > +++ b/net/tipc/socket.c > @@ -1009,12 +1009,11 @@ static int tipc_send_group_anycast(struct socket *sock, struct msghdr *m, > struct tipc_member *mbr = NULL; > struct net *net = sock_net(sk); > u32 node, port, exclude; > - struct list_head dsts; > int lookups = 0; > int dstcnt, rc; > bool cong; > + LIST_HEAD(dsts); nit: Please place this new line where the old dsts line was, in order to preserve, within the context of this hunk, reverse xmas tree order - longest line to shortest - for local variable declarations. > > - INIT_LIST_HEAD(&dsts); > ua->sa.type = msg_nametype(hdr); > ua->scope = msg_lookup_scope(hdr); > > @@ -1161,10 +1160,9 @@ static int tipc_send_group_mcast(struct socket *sock, struct msghdr *m, > struct tipc_group *grp = tsk->group; > struct tipc_msg *hdr = &tsk->phdr; > struct net *net = sock_net(sk); > - struct list_head dsts; > u32 dstcnt, exclude; > + LIST_HEAD(dsts); > > - INIT_LIST_HEAD(&dsts); > ua->sa.type = msg_nametype(hdr); > ua->scope = msg_lookup_scope(hdr); > exclude = tipc_group_exclude(grp);
On 2024/8/28 0:57, Simon Horman wrote: > On Tue, Aug 27, 2024 at 06:04:04PM +0800, Hongbo Li wrote: >> list_head can be initialized automatically with LIST_HEAD() >> instead of calling INIT_LIST_HEAD(). Here we can simplify >> the code. >> >> Signed-off-by: Hongbo Li <lihongbo22@huawei.com> >> --- >> net/tipc/socket.c | 6 ++---- >> 1 file changed, 2 insertions(+), 4 deletions(-) >> >> diff --git a/net/tipc/socket.c b/net/tipc/socket.c >> index 1a0cd06f0eae..9d30e362392c 100644 >> --- a/net/tipc/socket.c >> +++ b/net/tipc/socket.c >> @@ -1009,12 +1009,11 @@ static int tipc_send_group_anycast(struct socket *sock, struct msghdr *m, >> struct tipc_member *mbr = NULL; >> struct net *net = sock_net(sk); >> u32 node, port, exclude; >> - struct list_head dsts; >> int lookups = 0; >> int dstcnt, rc; >> bool cong; >> + LIST_HEAD(dsts); > > nit: Please place this new line where the old dsts line was, > in order to preserve, within the context of this hunk, > reverse xmas tree order - longest line to shortest - ok, I've learned, the reverse xmas tree order. Thank you! Thanks, Hongbo > for local variable declarations. > >> >> - INIT_LIST_HEAD(&dsts); >> ua->sa.type = msg_nametype(hdr); >> ua->scope = msg_lookup_scope(hdr); >> >> @@ -1161,10 +1160,9 @@ static int tipc_send_group_mcast(struct socket *sock, struct msghdr *m, >> struct tipc_group *grp = tsk->group; >> struct tipc_msg *hdr = &tsk->phdr; >> struct net *net = sock_net(sk); >> - struct list_head dsts; >> u32 dstcnt, exclude; >> + LIST_HEAD(dsts); >> >> - INIT_LIST_HEAD(&dsts); >> ua->sa.type = msg_nametype(hdr); >> ua->scope = msg_lookup_scope(hdr); >> exclude = tipc_group_exclude(grp); >
diff --git a/net/tipc/socket.c b/net/tipc/socket.c index 1a0cd06f0eae..9d30e362392c 100644 --- a/net/tipc/socket.c +++ b/net/tipc/socket.c @@ -1009,12 +1009,11 @@ static int tipc_send_group_anycast(struct socket *sock, struct msghdr *m, struct tipc_member *mbr = NULL; struct net *net = sock_net(sk); u32 node, port, exclude; - struct list_head dsts; int lookups = 0; int dstcnt, rc; bool cong; + LIST_HEAD(dsts); - INIT_LIST_HEAD(&dsts); ua->sa.type = msg_nametype(hdr); ua->scope = msg_lookup_scope(hdr); @@ -1161,10 +1160,9 @@ static int tipc_send_group_mcast(struct socket *sock, struct msghdr *m, struct tipc_group *grp = tsk->group; struct tipc_msg *hdr = &tsk->phdr; struct net *net = sock_net(sk); - struct list_head dsts; u32 dstcnt, exclude; + LIST_HEAD(dsts); - INIT_LIST_HEAD(&dsts); ua->sa.type = msg_nametype(hdr); ua->scope = msg_lookup_scope(hdr); exclude = tipc_group_exclude(grp);
list_head can be initialized automatically with LIST_HEAD() instead of calling INIT_LIST_HEAD(). Here we can simplify the code. Signed-off-by: Hongbo Li <lihongbo22@huawei.com> --- net/tipc/socket.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)