qat/compress: fix qat comp setup inter_buffers

Message ID 20190124150304.13828-1-marko.kovacevic@intel.com
State Accepted
Delegated to: akhil goyal
Headers show
Series
  • qat/compress: fix qat comp setup inter_buffers
Related show

Checks

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

Commit Message

Marko Kovacevic Jan. 24, 2019, 3:03 p.m.
This patch fixes qat comp setup inter_buffers
memory allocation size to a broader size
then just 2MB

Fixes: a124830a6f00 ("compress/qat: enable dynamic huffman encoding")
Cc: fiona.trahe@intel.com

Signed-off-by: Marko Kovacevic <marko.kovacevic@intel.com>
---
 drivers/compress/qat/qat_comp_pmd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Trahe, Fiona Jan. 30, 2019, 12:14 p.m. | #1
> -----Original Message-----
> From: Kovacevic, Marko
> Sent: Thursday, January 24, 2019 3:03 PM
> To: dev@dpdk.org
> Cc: stable@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>; ktraynor@redhat.com; Kovacevic, Marko
> <marko.kovacevic@intel.com>; Trahe, Fiona <fiona.trahe@intel.com>
> Subject: [PATCH] qat/compress: fix qat comp setup inter_buffers
> 
> This patch fixes qat comp setup inter_buffers
> memory allocation size to a broader size
> then just 2MB
> 
> Fixes: a124830a6f00 ("compress/qat: enable dynamic huffman encoding")
> Cc: fiona.trahe@intel.com
> 
> Signed-off-by: Marko Kovacevic <marko.kovacevic@intel.com>
Acked-by: <fiona.trahe@intel.com>
Akhil Goyal March 19, 2019, 2:02 p.m. | #2
On 1/30/2019 5:44 PM, Trahe, Fiona wrote:
>
>> -----Original Message-----
>> From: Kovacevic, Marko
>> Sent: Thursday, January 24, 2019 3:03 PM
>> To: dev@dpdk.org
>> Cc: stable@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>; ktraynor@redhat.com; Kovacevic, Marko
>> <marko.kovacevic@intel.com>; Trahe, Fiona <fiona.trahe@intel.com>
>> Subject: [PATCH] qat/compress: fix qat comp setup inter_buffers
>>
>> This patch fixes qat comp setup inter_buffers
>> memory allocation size to a broader size
>> then just 2MB
>>
>> Fixes: a124830a6f00 ("compress/qat: enable dynamic huffman encoding")
>> Cc: fiona.trahe@intel.com
>>
>> Signed-off-by: Marko Kovacevic <marko.kovacevic@intel.com>
> Acked-by: <fiona.trahe@intel.com>
Applied to dpdk-next-crypto

Thanks.

Patch

diff --git a/drivers/compress/qat/qat_comp_pmd.c b/drivers/compress/qat/qat_comp_pmd.c
index 27c8856..d45cd67 100644
--- a/drivers/compress/qat/qat_comp_pmd.c
+++ b/drivers/compress/qat/qat_comp_pmd.c
@@ -185,7 +185,7 @@  qat_comp_setup_inter_buffers(struct qat_comp_dev_private *comp_dev,
 
 	memzone = rte_memzone_reserve_aligned(inter_buff_mz_name, full_size,
 			comp_dev->compressdev->data->socket_id,
-			RTE_MEMZONE_2MB, QAT_64_BYTE_ALIGN);
+			RTE_MEMZONE_IOVA_CONTIG, QAT_64_BYTE_ALIGN);
 	if (memzone == NULL) {
 		QAT_LOG(ERR, "Can't allocate intermediate buffers"
 				" for device %s", comp_dev->qat_dev->name);