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

Commit Message

Zheng Yongjun Feb. 5, 2021, 6:28 a.m. UTC

Do code format alignment to clean code style issues.

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

Comments

Jakub Kicinski Feb. 6, 2021, 9 p.m. UTC | #1

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 <[e-mail]>

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

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 = &nht->hash_buckets[i];
 
 		while ((n = rcu_dereference_protected(*np,
-					lockdep_is_held(&tbl->lock))) != NULL) {
+						      lockdep_is_held(&tbl->lock))) != NULL) {
 			if (dev && n->dev != dev) {
 				np = &n->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(&tbl->lock)));
 			write_lock(&n->lock);
 			neigh_del_timer(n);
@@ -634,7 +634,7 @@  static struct neighbour *___neigh_create(struct neigh_table *tbl,
 					    lockdep_is_held(&tbl->lock));
 	     n1 != NULL;
 	     n1 = rcu_dereference_protected(n1->next,
-			lockdep_is_held(&tbl->lock))) {
+					    lockdep_is_held(&tbl->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, &tbl->gc_work,
-			      NEIGH_VAR(&tbl->parms, BASE_REACHABLE_TIME) >> 1);
+			   NEIGH_VAR(&tbl->parms, BASE_REACHABLE_TIME) >> 1);
 	write_unlock_bh(&tbl->lock);
 }
 
@@ -1620,8 +1620,7 @@  struct neigh_parms *neigh_parms_alloc(struct net_device *dev,
 	if (p) {
 		p->tbl		  = tbl;
 		refcount_set(&p->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(&p->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,
-			      &neigh_stat_seq_ops, tbl))
+				  &neigh_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(&tbl->lock);
 	INIT_DEFERRABLE_WORK(&tbl->gc_work, neigh_periodic_work);
 	queue_delayed_work(system_power_efficient_wq, &tbl->gc_work,
-			tbl->parms.reachable_time);
+			   tbl->parms.reachable_time);
 	timer_setup(&tbl->proxy_timer, neigh_proxy_process, 0);
 	skb_queue_head_init_class(&tbl->proxy_queue,
-			&neigh_table_proxy_queue_class);
+				  &neigh_table_proxy_queue_class);
 
 	tbl->last_flush = now;
 	tbl->last_rand	= now + tbl->parms.reachable_time * 20;