Received-SPF: pass (google.com: domain of linux-kernel+bounces-198140-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Sun, 2 Jun 2024 09:51:25 +0300
From: Leon Romanovsky <leon@kernel.org>
To: Bharat Bhushan <bbhushan2@marvell.com>
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
	sgoutham@marvell.com, gakula@marvell.com, sbhatta@marvell.com,
	hkelam@marvell.com, davem@davemloft.net, edumazet@google.com,
	kuba@kernel.org, pabeni@redhat.com, jerinj@marvell.com,
	lcherian@marvell.com, richardcochran@gmail.com
Subject: Re: [net-next,v3 6/8] cn10k-ipsec: Process inline ipsec transmit
 offload
Message-ID: <20240602065125.GH3884@unreal>
References: <20240528135349.932669-1-bbhushan2@marvell.com>
 <20240528135349.932669-7-bbhushan2@marvell.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240528135349.932669-7-bbhushan2@marvell.com>

On Tue, May 28, 2024 at 07:23:47PM +0530, Bharat Bhushan wrote:
> Prepare and submit crypto hardware (CPT) instruction for
> outbound inline ipsec crypto mode offload. The CPT instruction
> have authentication offset, IV offset and encapsulation offset
> in input packet. Also provide SA context pointer which have
> details about algo, keys, salt etc. Crypto hardware encrypt,
> authenticate and provide the ESP packet to networking hardware.
> 
> Signed-off-by: Bharat Bhushan <bbhushan2@marvell.com>
> ---
>  .../marvell/octeontx2/nic/cn10k_ipsec.c       | 224 ++++++++++++++++++
>  .../marvell/octeontx2/nic/cn10k_ipsec.h       |  40 ++++
>  .../marvell/octeontx2/nic/otx2_common.c       |  23 ++
>  .../marvell/octeontx2/nic/otx2_common.h       |   3 +
>  .../ethernet/marvell/octeontx2/nic/otx2_pf.c  |   2 +
>  .../marvell/octeontx2/nic/otx2_txrx.c         |  33 ++-
>  .../marvell/octeontx2/nic/otx2_txrx.h         |   3 +
>  7 files changed, 325 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/ethernet/marvell/octeontx2/nic/cn10k_ipsec.c b/drivers/net/ethernet/marvell/octeontx2/nic/cn10k_ipsec.c
> index 136aebe2a007..1974fda2e0d3 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/nic/cn10k_ipsec.c
> +++ b/drivers/net/ethernet/marvell/octeontx2/nic/cn10k_ipsec.c
> @@ -7,8 +7,11 @@
>  #include <net/xfrm.h>
>  #include <linux/netdevice.h>
>  #include <linux/bitfield.h>
> +#include <crypto/aead.h>
> +#include <crypto/gcm.h>
>  
>  #include "otx2_common.h"
> +#include "otx2_struct.h"
>  #include "cn10k_ipsec.h"
>  
>  static bool is_dev_support_inline_ipsec(struct pci_dev *pdev)
> @@ -843,3 +846,224 @@ void cn10k_ipsec_clean(struct otx2_nic *pf)
>  	cn10k_outb_cpt_clean(pf);
>  }
>  EXPORT_SYMBOL(cn10k_ipsec_clean);

<...>

> +bool cn10k_ipsec_transmit(struct otx2_nic *pf, struct netdev_queue *txq,
> +			  struct otx2_snd_queue *sq, struct sk_buff *skb,
> +			  int num_segs, int size)
> +{
> +	struct cpt_ctx_info_s *sa_info;
> +	struct cpt_inst_s inst;
> +	struct cpt_res_s *res;
> +	struct xfrm_state *x;
> +	dma_addr_t dptr_iova;
> +	struct sec_path *sp;
> +	u8 encap_offset;
> +	u8 auth_offset;
> +	u8 gthr_size;
> +	u8 iv_offset;
> +	u16 dlen;
> +
> +	/* Check for Inline IPSEC enabled */
> +	if (!(pf->flags & OTX2_FLAG_INLINE_IPSEC_ENABLED)) {
> +		netdev_err(pf->netdev, "Ipsec not enabled, drop packet\n");

<...>

> +		netdev_err(pf->netdev, "%s: no xfrm state len = %d\n",
> +			   __func__, sp->len);

<...>

> +		netdev_err(pf->netdev, "no xfrm_input_state()\n");

<...>

> +		netdev_err(pf->netdev, "un supported offload mode %d\n",
> +			   x->props.mode);

<...>

> +		netdev_err(pf->netdev, "Invalid IP header, ip-length zero\n");

<...>

> +		netdev_err(pf->netdev, "Invalid SA conext\n");

All these prints are in datapath and can be triggered by network
packets. These and RX prints need to be deleted.

Thanks