Commit 23532061 authored by Uladzislau Rezki (Sony)'s avatar Uladzislau Rezki (Sony) Committed by Joel Fernandes (Google)
Browse files

net/mlx5: Rename kfree_rcu() to kfree_rcu_mightsleep() 



The kfree_rcu() and kvfree_rcu() macros' single-argument forms are
deprecated.  Therefore switch to the new kfree_rcu_mightsleep() and
kvfree_rcu_mightsleep() variants. The goal is to avoid accidental use
of the single-argument forms, which can introduce functionality bugs in
atomic contexts and latency bugs in non-atomic contexts.

Cc: Ariel Levkovich <lariel@nvidia.com>
Cc: Saeed Mahameed <saeedm@nvidia.com>
Cc: Vlad Buslov <vladbu@nvidia.com>
Signed-off-by: default avatarUladzislau Rezki (Sony) <urezki@gmail.com>
Signed-off-by: default avatarPaul E. McKenney <paulmck@kernel.org>
Signed-off-by: default avatarJoel Fernandes (Google) <joel@joelfernandes.org>
parent aef3b8b8

drivers/net/ethernet/mellanox/mlx5/core/en/tc/int_port.c

+1 −1
Original line number Diff line number Diff line
@@ -242,7 +242,7 @@ mlx5e_int_port_remove(struct mlx5e_tc_int_port_priv *priv, 
		mlx5_del_flow_rules(int_port->rx_rule);

	mapping_remove(ctx, int_port->mapping);

	mlx5e_int_port_metadata_free(priv, int_port->match_metadata);

	kfree_rcu(int_port);

	kfree_rcu_mightsleep(int_port);

	priv->num_ports--;

}

drivers/net/ethernet/mellanox/mlx5/core/en_accel/macsec.c

+2 −2
Original line number Diff line number Diff line
@@ -670,7 +670,7 @@ static int mlx5e_macsec_del_txsa(struct macsec_context *ctx) 


	mlx5e_macsec_cleanup_sa(macsec, tx_sa, true);

	mlx5_destroy_encryption_key(macsec->mdev, tx_sa->enc_key_id);

	kfree_rcu(tx_sa);

	kfree_rcu_mightsleep(tx_sa);

	macsec_device->tx_sa[assoc_num] = NULL;



out:
@@ -849,7 +849,7 @@ static void macsec_del_rxsc_ctx(struct mlx5e_macsec *macsec, struct mlx5e_macsec 
	xa_erase(&macsec->sc_xarray, rx_sc->sc_xarray_element->fs_id);

	metadata_dst_free(rx_sc->md_dst);

	kfree(rx_sc->sc_xarray_element);

	kfree_rcu(rx_sc);

	kfree_rcu_mightsleep(rx_sc);

}



static int mlx5e_macsec_del_rxsc(struct macsec_context *ctx)