Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Mon, 4 Apr 2022 17:33:22 -0700
From:   Saeed Mahameed <saeed@kernel.org>
To:     Leon Romanovsky <leon@kernel.org>
Cc:     Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
        Leon Romanovsky <leonro@nvidia.com>,
        Jason Gunthorpe <jgg@nvidia.com>, linux-kernel@vger.kernel.org,
        linux-rdma@vger.kernel.org, netdev@vger.kernel.org,
        Saeed Mahameed <saeedm@nvidia.com>,
        Tariq Toukan <tariqt@nvidia.com>
Subject: Re: [PATCH mlx5-next 4/5] net/mlx5: Remove tls vs. ktls separation
 as it is the same
Message-ID: <20220405003322.afko7uo527w5j3zu@sx1>
References: <cover.1649073691.git.leonro@nvidia.com>
 <67e596599edcffb0de43f26551208dfd34ac777e.1649073691.git.leonro@nvidia.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Disposition: inline
In-Reply-To: <67e596599edcffb0de43f26551208dfd34ac777e.1649073691.git.leonro@nvidia.com>
Precedence: bulk

On 04 Apr 15:08, Leon Romanovsky wrote:
>From: Leon Romanovsky <leonro@nvidia.com>
>
>After removal FPGA TLS, we can remove tls->ktls indirection too,
>as it is the same thing.
>
>Reviewed-by: Tariq Toukan <tariqt@nvidia.com>
>Signed-off-by: Leon Romanovsky <leonro@nvidia.com>
>---
> .../net/ethernet/mellanox/mlx5/core/Makefile  |  2 +-
> .../ethernet/mellanox/mlx5/core/en/params.c   |  2 +-
> .../mellanox/mlx5/core/en_accel/en_accel.h    | 11 +--
> .../mellanox/mlx5/core/en_accel/ktls.c        | 22 ++++-
> .../mellanox/mlx5/core/en_accel/ktls.h        | 32 +++++++
> .../mellanox/mlx5/core/en_accel/ktls_rx.c     |  2 +-
> .../en_accel/{tls_stats.c => ktls_stats.c}    | 38 ++++-----
> .../mellanox/mlx5/core/en_accel/ktls_tx.c     | 18 +++-
> .../mellanox/mlx5/core/en_accel/ktls_txrx.h   | 28 +++++-
> .../mellanox/mlx5/core/en_accel/tls.c         | 70 ---------------
> .../mellanox/mlx5/core/en_accel/tls.h         | 85 -------------------
> .../mellanox/mlx5/core/en_accel/tls_rxtx.c    | 70 ---------------
> .../mellanox/mlx5/core/en_accel/tls_rxtx.h    | 85 -------------------
> .../net/ethernet/mellanox/mlx5/core/en_main.c |  8 +-
> .../net/ethernet/mellanox/mlx5/core/en_rx.c   |  5 +-
> .../ethernet/mellanox/mlx5/core/en_stats.c    |  8 +-
> 16 files changed, 130 insertions(+), 356 deletions(-)
> rename drivers/net/ethernet/mellanox/mlx5/core/en_accel/{tls_stats.c => ktls_stats.c} (76%)

Why not ktls_*.c => tls_*.c ? 

Since we now have one TLS implementation, it would've been easier to maybe
repurpose TLS to be KTLS only and avoid renaming every TLS to KTLS in all
functions and files.

So just keep tls.c and all mlx5_tls_xyz functions and implement ktls
directly in them, the renaming will be done only on the ktls implementation
part of the code rather than in every caller.

> delete mode 100644 drivers/net/ethernet/mellanox/mlx5/core/en_accel/tls.c
> delete mode 100644 drivers/net/ethernet/mellanox/mlx5/core/en_accel/tls.h
> delete mode 100644 drivers/net/ethernet/mellanox/mlx5/core/en_accel/tls_rxtx.c
> delete mode 100644 drivers/net/ethernet/mellanox/mlx5/core/en_accel/tls_rxtx.h
>