[dpdk-stable] patch 'compress/qat: fix checksum on decompression' has been queued to stable release 18.08.1

Kevin Traynor ktraynor at redhat.com
Wed Nov 21 17:04:38 CET 2018


Hi,

FYI, your patch has been queued to stable release 18.08.1

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 11/26/18. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the patch applied
to the branch. If the code is different (ie: not only metadata diffs), due for example to
a change in context or macro names, please double check it.

Thanks.

Kevin Traynor

---
>From 4d5adf65e272f0a2e147c72cf763cfc29b613ded Mon Sep 17 00:00:00 2001
From: Fiona Trahe <fiona.trahe at intel.com>
Date: Sat, 22 Sep 2018 14:25:37 +0100
Subject: [PATCH] compress/qat: fix checksum on decompression

[ upstream commit 7586c5787375b3947d7d12c91aa99ac4ff58203d ]

Checksum was always 0 on QAT decompression due to
incorrect use of union variable.

Fixes: 6a7ea14819e9 ("compress/qat: add xform processing")

Signed-off-by: Fiona Trahe <fiona.trahe at intel.com>
Acked-by: Tomasz Jozwiak <tomaszx.jozwiak at intel.com>
---
 drivers/compress/qat/qat_comp.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/compress/qat/qat_comp.c b/drivers/compress/qat/qat_comp.c
index 38c8a5b82..6f1f2dc25 100644
--- a/drivers/compress/qat/qat_comp.c
+++ b/drivers/compress/qat/qat_comp.c
@@ -346,15 +346,14 @@ qat_comp_private_xform_create(struct rte_compressdev *dev,
 		  ((xform->compress.deflate.huffman == RTE_COMP_HUFFMAN_DEFAULT)
 				   && qat->interm_buff_mz == NULL))
-
 			qat_xform->qat_comp_request_type =
 					QAT_COMP_REQUEST_FIXED_COMP_STATELESS;
 
+		qat_xform->checksum_type = xform->compress.chksum;
 
 	} else {
 		qat_xform->qat_comp_request_type = QAT_COMP_REQUEST_DECOMPRESS;
+		qat_xform->checksum_type = xform->decompress.chksum;
 	}
 
-	qat_xform->checksum_type = xform->compress.chksum;
-
 	if (qat_comp_create_templates(qat_xform, qat->interm_buff_mz, xform)) {
 		QAT_LOG(ERR, "QAT: Problem with setting compression");
-- 
2.19.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2018-11-21 15:59:14.824493803 +0000
+++ 0048-compress-qat-fix-checksum-on-decompression.patch	2018-11-21 15:59:13.000000000 +0000
@@ -1,13 +1,14 @@
-From 7586c5787375b3947d7d12c91aa99ac4ff58203d Mon Sep 17 00:00:00 2001
+From 4d5adf65e272f0a2e147c72cf763cfc29b613ded Mon Sep 17 00:00:00 2001
 From: Fiona Trahe <fiona.trahe at intel.com>
 Date: Sat, 22 Sep 2018 14:25:37 +0100
 Subject: [PATCH] compress/qat: fix checksum on decompression
 
+[ upstream commit 7586c5787375b3947d7d12c91aa99ac4ff58203d ]
+
 Checksum was always 0 on QAT decompression due to
 incorrect use of union variable.
 
 Fixes: 6a7ea14819e9 ("compress/qat: add xform processing")
-Cc: stable at dpdk.org
 
 Signed-off-by: Fiona Trahe <fiona.trahe at intel.com>
 Acked-by: Tomasz Jozwiak <tomaszx.jozwiak at intel.com>


More information about the stable mailing list