[net-next 01/11] net/mlx5e: Extract code that queues neigh update work into function

[Saeed Mahameed]

From: Vlad Buslov <vla...@mellanox.com>

As a preparation for following refactoring that removes rtnl lock
dependency from neigh hash entry handlers, extract code that enqueues neigh
update work into standalone function. This commit doesn't change
functionality.

Signed-off-by: Vlad Buslov <vla...@mellanox.com>
Reviewed-by: Jianbo Liu <jian...@mellanox.com>
Reviewed-by: Roi Dayan <r...@mellanox.com>
Signed-off-by: Saeed Mahameed <sae...@mellanox.com>
---
 .../net/ethernet/mellanox/mlx5/core/en_rep.c  | 38 +++++++++++--------
 1 file changed, 23 insertions(+), 15 deletions(-)

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_rep.c 
b/drivers/net/ethernet/mellanox/mlx5/core/en_rep.c
index 7ce5cb6e527e..85a503f0423b 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/en_rep.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en_rep.c
@@ -821,6 +821,28 @@ static int mlx5e_nic_rep_netdevice_event(struct 
notifier_block *nb,
        return NOTIFY_OK;
 }
 
+static void
+mlx5e_rep_queue_neigh_update_work(struct mlx5e_priv *priv,
+                                 struct mlx5e_neigh_hash_entry *nhe,
+                                 struct neighbour *n)
+{
+       /* Take a reference to ensure the neighbour and mlx5 encap
+        * entry won't be destructed until we drop the reference in
+        * delayed work.
+        */
+       neigh_hold(n);
+
+       /* This assignment is valid as long as the the neigh reference
+        * is taken
+        */
+       nhe->n = n;
+
+       if (!queue_work(priv->wq, &nhe->neigh_update_work)) {
+               mlx5e_rep_neigh_entry_release(nhe);
+               neigh_release(n);
+       }
+}
+
 static struct mlx5e_neigh_hash_entry *
 mlx5e_rep_neigh_entry_lookup(struct mlx5e_priv *priv,
                             struct mlx5e_neigh *m_neigh);
@@ -864,22 +886,8 @@ static int mlx5e_rep_netevent_event(struct notifier_block 
*nb,
                        return NOTIFY_DONE;
                }
 
-               /* This assignment is valid as long as the the neigh reference
-                * is taken
-                */
-               nhe->n = n;
-
-               /* Take a reference to ensure the neighbour and mlx5 encap
-                * entry won't be destructed until we drop the reference in
-                * delayed work.
-                */
-               neigh_hold(n);
                mlx5e_rep_neigh_entry_hold(nhe);
-
-               if (!queue_work(priv->wq, &nhe->neigh_update_work)) {
-                       mlx5e_rep_neigh_entry_release(nhe);
-                       neigh_release(n);
-               }
+               mlx5e_rep_queue_neigh_update_work(priv, nhe, n);
                spin_unlock_bh(&neigh_update->encap_lock);
                break;
 
-- 
2.21.0