compress/qat: fixed data-plane return from QAT PMD

Message ID 1554740188-12738-1-git-send-email-fiona.trahe@intel.com (mailing list archive)
State Accepted, archived
Delegated to: akhil goyal
Headers
Series compress/qat: fixed data-plane return from QAT PMD |

Checks

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

Commit Message

Fiona Trahe April 8, 2019, 4:16 p.m. UTC
  Fixes: 62ada2182a46 ("compress/qat: add dynamic sgl allocation")

Signed-off-by: Fiona Trahe <fiona.trahe@intel.com>
---
 drivers/compress/qat/qat_comp.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)
  

Comments

Tomasz Jozwiak April 9, 2019, 7:16 a.m. UTC | #1
> -----Original Message-----
> From: Trahe, Fiona
> Sent: Monday, April 8, 2019 6:16 PM
> To: dev@dpdk.org
> Cc: akhil.goyal@nxp.com; Jozwiak, TomaszX <tomaszx.jozwiak@intel.com>;
> Trahe, Fiona <fiona.trahe@intel.com>
> Subject: [PATCH] compress/qat: fixed data-plane return from QAT PMD
> 
> Fixes: 62ada2182a46 ("compress/qat: add dynamic sgl allocation")
> 
> Signed-off-by: Fiona Trahe <fiona.trahe@intel.com>
> ---


Acked-by: Tomasz Jozwiak <tomaszx.jozwiak@intel.com>
  

Patch

diff --git a/drivers/compress/qat/qat_comp.c b/drivers/compress/qat/qat_comp.c
index 2fadb02..dd0fe1b 100644
--- a/drivers/compress/qat/qat_comp.c
+++ b/drivers/compress/qat/qat_comp.c
@@ -93,7 +93,7 @@ 
 				QAT_DP_LOG(ERR, "QAT PMD can't allocate memory"
 					   " for %d elements of SGL",
 					   op->m_src->nb_segs);
-				op->status = RTE_COMP_OP_STATUS_INVALID_ARGS;
+				op->status = RTE_COMP_OP_STATUS_ERROR;
 				return -ENOMEM;
 			}
 			/* new SGL is valid now */
@@ -128,8 +128,8 @@ 
 				QAT_DP_LOG(ERR, "QAT PMD can't allocate memory"
 					   " for %d elements of SGL",
 					   op->m_dst->nb_segs);
-				op->status = RTE_COMP_OP_STATUS_INVALID_ARGS;
-				return -EINVAL;
+				op->status = RTE_COMP_OP_STATUS_ERROR;
+				return -ENOMEM;
 			}
 			/* new SGL is valid now */
 			cookie->qat_sgl_dst_d = (struct qat_sgl *)tmp;