RE: [PATCH] RDMA/siw: use crypto_shash_digest() in siw_qp_prepare_tx()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Eric Biggers <ebiggers@xxxxxxxxxx>
> Sent: Sunday, October 29, 2023 5:59 AM
> To: Bernard Metzler <BMT@xxxxxxxxxxxxxx>; Jason Gunthorpe <jgg@xxxxxxxx>;
> Leon Romanovsky <leon@xxxxxxxxxx>; linux-rdma@xxxxxxxxxxxxxxx
> Cc: linux-crypto@xxxxxxxxxxxxxxx
> Subject: [EXTERNAL] [PATCH] RDMA/siw: use crypto_shash_digest() in
> siw_qp_prepare_tx()
> 
> From: Eric Biggers <ebiggers@xxxxxxxxxx>
> 
> Simplify siw_qp_prepare_tx() by using crypto_shash_digest() instead of
> an init+update+final sequence.  This should also improve performance.
> 
> Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx>
> ---
>  drivers/infiniband/sw/siw/siw_qp_tx.c | 12 ++++--------
>  1 file changed, 4 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/infiniband/sw/siw/siw_qp_tx.c
> b/drivers/infiniband/sw/siw/siw_qp_tx.c
> index 60b6a4135961..5b390f08f1cd 100644
> --- a/drivers/infiniband/sw/siw/siw_qp_tx.c
> +++ b/drivers/infiniband/sw/siw/siw_qp_tx.c
> @@ -242,28 +242,24 @@ static int siw_qp_prepare_tx(struct siw_iwarp_tx
> *c_tx)
>  				c_tx->pkt.c_untagged.ddp_mo = 0;
>  			else
>  				c_tx->pkt.c_tagged.ddp_to =
>  					cpu_to_be64(wqe->sqe.raddr);
>  		}
> 
>  		*(u32 *)crc = 0;
>  		/*
>  		 * Do complete CRC if enabled and short packet
>  		 */
> -		if (c_tx->mpa_crc_hd) {
> -			crypto_shash_init(c_tx->mpa_crc_hd);
> -			if (crypto_shash_update(c_tx->mpa_crc_hd,
> -						(u8 *)&c_tx->pkt,
> -						c_tx->ctrl_len))
> -				return -EINVAL;
> -			crypto_shash_final(c_tx->mpa_crc_hd, (u8 *)crc);
> -		}
> +		if (c_tx->mpa_crc_hd &&
> +		    crypto_shash_digest(c_tx->mpa_crc_hd, (u8 *)&c_tx->pkt,
> +					c_tx->ctrl_len, (u8 *)crc) != 0)
> +			return -EINVAL;
>  		c_tx->ctrl_len += MPA_CRC_SIZE;
> 
>  		return PKT_COMPLETE;
>  	}
>  	c_tx->ctrl_len += MPA_CRC_SIZE;
>  	c_tx->sge_idx = 0;
>  	c_tx->sge_off = 0;
>  	c_tx->pbl_idx = 0;
> 
>  	/*
> 
> base-commit: 2af9b20dbb39f6ebf9b9b6c090271594627d818e
> --
> 2.42.0
Thank you Eric, looks good to me!

Acked-by: Bernard Metzler <bmt@xxxxxxxxxxxxxx>




[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]
  Powered by Linux