Re: [PATCH] net: core: Clean code style issues in `neighbour.c`

[Jakub Kicinski]

On Fri, 5 Feb 2021 14:28:21 +0800 Zheng Yongjun wrote:
> Do code format alignment to clean code style issues.
> 
> Signed-off-by: Zheng Yongjun 

I don't think these changes make the code substantively more readable.
At the same time they make the history harder to follow and may cause
backporting failures.

Unless area maintainers disagree I'm not applying this patch, sorry.

[PATCH] net: core: Clean code style issues in `neighbour.c`

[Zheng Yongjun]

Do code format alignment to clean code style issues.

Signed-off-by: Zheng Yongjun 
---
 net/core/neighbour.c | 17 -
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/net/core/neighbour.c b/net/core/neighbour.c
index 9500d28a43b0..a742c918a09b 100644
--- a/net/core/neighbour.c
+++ b/net/core/neighbour.c
@@ -299,7 +299,7 @@ static void neigh_flush_dev(struct neigh_table *tbl, struct 
net_device *dev,
struct neighbour __rcu **np = >hash_buckets[i];
 
while ((n = rcu_dereference_protected(*np,
-   lockdep_is_held(>lock))) != NULL) {
+ 
lockdep_is_held(>lock))) != NULL) {
if (dev && n->dev != dev) {
np = >next;
continue;
@@ -309,7 +309,7 @@ static void neigh_flush_dev(struct neigh_table *tbl, struct 
net_device *dev,
continue;
}
rcu_assign_pointer(*np,
-  rcu_dereference_protected(n->next,
+  rcu_dereference_protected(n->next,
lockdep_is_held(>lock)));
write_lock(>lock);
neigh_del_timer(n);
@@ -634,7 +634,7 @@ static struct neighbour *___neigh_create(struct neigh_table 
*tbl,
lockdep_is_held(>lock));
 n1 != NULL;
 n1 = rcu_dereference_protected(n1->next,
-   lockdep_is_held(>lock))) {
+   lockdep_is_held(>lock))) {
if (dev == n1->dev && !memcmp(n1->primary_key, n->primary_key, 
key_len)) {
if (want_ref)
neigh_hold(n1);
@@ -962,7 +962,7 @@ static void neigh_periodic_work(struct work_struct *work)
 * BASE_REACHABLE_TIME.
 */
queue_delayed_work(system_power_efficient_wq, >gc_work,
- NEIGH_VAR(>parms, BASE_REACHABLE_TIME) >> 1);
+  NEIGH_VAR(>parms, BASE_REACHABLE_TIME) >> 1);
write_unlock_bh(>lock);
 }
 
@@ -1620,8 +1620,7 @@ struct neigh_parms *neigh_parms_alloc(struct net_device 
*dev,
if (p) {
p->tbl= tbl;
refcount_set(>refcnt, 1);
-   p->reachable_time =
-   neigh_rand_reach_time(NEIGH_VAR(p, 
BASE_REACHABLE_TIME));
+   p->reachable_time = neigh_rand_reach_time(NEIGH_VAR(p, 
BASE_REACHABLE_TIME));
dev_hold(dev);
p->dev = dev;
write_pnet(>net, net);
@@ -1693,7 +1692,7 @@ void neigh_table_init(int index, struct neigh_table *tbl)
 
 #ifdef CONFIG_PROC_FS
if (!proc_create_seq_data(tbl->id, 0, init_net.proc_net_stat,
- _stat_seq_ops, tbl))
+ _stat_seq_ops, tbl))
panic("cannot create neighbour proc dir entry");
 #endif
 
@@ -1714,10 +1713,10 @@ void neigh_table_init(int index, struct neigh_table 
*tbl)
rwlock_init(>lock);
INIT_DEFERRABLE_WORK(>gc_work, neigh_periodic_work);
queue_delayed_work(system_power_efficient_wq, >gc_work,
-   tbl->parms.reachable_time);
+  tbl->parms.reachable_time);
timer_setup(>proxy_timer, neigh_proxy_process, 0);
skb_queue_head_init_class(>proxy_queue,
-   _table_proxy_queue_class);
+ _table_proxy_queue_class);
 
tbl->last_flush = now;
tbl->last_rand  = now + tbl->parms.reachable_time * 20;
-- 
2.22.0