[v2] symmetric crypto, set message field to zero in SGL case

Message ID 20190612173141.67491-1-gosia.bakota@intel.com (mailing list archive)
State Accepted, archived
Delegated to: akhil goyal
Headers
Series [v2] symmetric crypto, set message field to zero in SGL case |

Checks

Context Check Description
ci/Intel-compilation fail Compilation issues
ci/intel-Performance-Testing success Performance Testing PASS
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/checkpatch success coding style OK

Commit Message

Gosia Bakota June 12, 2019, 5:31 p.m. UTC
  Fixes: e09231eaa2af ("crypto/qat: add SGL capability")
Cc: Arek Kusztal <arkadiuszx.kusztal@intel.com>
Cc: stable@dpdk.org

Signed-off-by: Gosia Bakota <gosia.bakota@intel.com>
---
v2 changes
 -remove file mode change
 
 drivers/crypto/qat/qat_sym.c | 2 ++
 1 file changed, 2 insertions(+)
  

Comments

Fiona Trahe June 12, 2019, 4:53 p.m. UTC | #1
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Gosia Bakota
> Sent: Wednesday, June 12, 2019 6:32 PM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH v2] symmetric crypto, set message field to zero in SGL case
> 
> Fixes: e09231eaa2af ("crypto/qat: add SGL capability")
> Cc: Arek Kusztal <arkadiuszx.kusztal@intel.com>
> Cc: stable@dpdk.org
> 
> Signed-off-by: Gosia Bakota <gosia.bakota@intel.com>
Acked-by: Fiona Trahe <fiona.trahe@intel.com>
  
Akhil Goyal June 19, 2019, 2:47 p.m. UTC | #2
> > Fixes: e09231eaa2af ("crypto/qat: add SGL capability")
> > Cc: Arek Kusztal <arkadiuszx.kusztal@intel.com>
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Gosia Bakota <gosia.bakota@intel.com>
> Acked-by: Fiona Trahe <fiona.trahe@intel.com>

No need to add " Cc: Arek Kusztal <arkadiuszx.kusztal@intel.com>" in commit message.
Also changes the subject as that of your v1. Please take care of this in future.

Applied to dpdk-next-crypto

Thanks
  

Patch

diff --git a/drivers/crypto/qat/qat_sym.c b/drivers/crypto/qat/qat_sym.c
index 8801ca567..7515a55d0 100644
--- a/drivers/crypto/qat/qat_sym.c
+++ b/drivers/crypto/qat/qat_sym.c
@@ -530,6 +530,8 @@  qat_sym_build_request(void *in_op, uint8_t *out_msg,
 			qat_req->comn_mid.dest_data_addr =
 					cookie->qat_sgl_dst_phys_addr;
 		}
+		qat_req->comn_mid.src_length = 0;
+		qat_req->comn_mid.dst_length = 0;
 	} else {
 		qat_req->comn_mid.src_data_addr = src_buf_start;
 		qat_req->comn_mid.dest_data_addr = dst_buf_start;