On Tue, Aug 08, 2023 at 05:18:03PM +0200, Simon Horman wrote: > On Mon, Aug 07, 2023 at 01:44:18PM +0300, Leon Romanovsky wrote: > > From: Patrisious Haddad <phaddad@xxxxxxxxxx> > > > > Add steering table in RDMA_TX domain, to forward MACsec traffic > > to MACsec crypto table in NIC domain. > > The tables are created in a lazy manner when the first TX SA is > > being created, and destroyed upon the destruction of the last SA. > > > > Signed-off-by: Patrisious Haddad <phaddad@xxxxxxxxxx> > > Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxx> > > --- > > .../mellanox/mlx5/core/lib/macsec_fs.c | 46 ++++++++++++++++++- > > 1 file changed, 45 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/lib/macsec_fs.c b/drivers/net/ethernet/mellanox/mlx5/core/lib/macsec_fs.c > > index d39ca7c66542..15e7ea3ed79f 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/lib/macsec_fs.c > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/lib/macsec_fs.c > > @@ -95,6 +95,8 @@ struct mlx5_macsec_tx { > > struct ida tx_halloc; > > > > struct mlx5_macsec_tables tables; > > + > > + struct mlx5_flow_table *ft_rdma_tx; > > }; > > > > struct mlx5_macsec_rx_rule { > > @@ -173,6 +175,9 @@ static void macsec_fs_tx_destroy(struct mlx5_macsec_fs *macsec_fs) > > struct mlx5_macsec_tx *tx_fs = macsec_fs->tx_fs; > > struct mlx5_macsec_tables *tx_tables; > > > > + if (mlx5_is_macsec_roce_supported(macsec_fs->mdev)) > > + mlx5_destroy_flow_table(tx_fs->ft_rdma_tx); > > Hi Patrisious and Leon, > > mlx5_is_macsec_roce_supported() is used here, but it doesn't seem > to be added until a later in this series. It is my fault, I reordered patches prior to submission and didn't compile them separately. Thanks > > ...