From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id A3D32A874 for ; Tue, 1 Mar 2016 18:24:39 +0100 (CET) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga101.jf.intel.com with ESMTP; 01 Mar 2016 09:24:38 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,524,1449561600"; d="scan'208";a="924465553" Received: from irsmsx151.ger.corp.intel.com ([163.33.192.59]) by orsmga002.jf.intel.com with ESMTP; 01 Mar 2016 09:24:37 -0800 Received: from irsmsx101.ger.corp.intel.com ([169.254.1.157]) by IRSMSX151.ger.corp.intel.com ([169.254.4.72]) with mapi id 14.03.0248.002; Tue, 1 Mar 2016 17:24:37 +0000 From: "Trahe, Fiona" To: "Jain, Deepak K" , "dev@dpdk.org" Thread-Topic: [PATCH] app/test: fix qat autotest failure Thread-Index: AQHRc9MKJw3qu/mRokOhPOj150lcZJ9E1kBg Date: Tue, 1 Mar 2016 17:24:36 +0000 Message-ID: <348A99DA5F5B7549AA880327E580B43588F8646F@IRSMSX101.ger.corp.intel.com> References: <1456851316-22234-1-git-send-email-deepak.k.jain@intel.com> In-Reply-To: <1456851316-22234-1-git-send-email-deepak.k.jain@intel.com> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYTliNjkyYTAtNWY0OC00Njk5LWE4MTctYjNkYzE2OTc0MzA1IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6IlJLMzZTVnhFcmxYK2xYbElnT3pTXC8rRU1oYmh6d3p5UXhOVEpUSG9qTTBZPSJ9 x-ctpclassification: CTP_IC x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH] app/test: fix qat autotest failure X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 01 Mar 2016 17:24:40 -0000 > -----Original Message----- > From: Jain, Deepak K > Sent: Tuesday, March 01, 2016 4:55 PM > To: dev@dpdk.org > Cc: Griffin, John; Trahe, Fiona; Jain, Deepak K > Subject: [PATCH] app/test: fix qat autotest failure >=20 > This patch fix the QAT autotest failure when run for multiple times. > it was caused as mbuf was not freed. >=20 > Fixes: 202d375c60b (app/test: add cryptodev unit and performance tests) >=20 > This patch depends on following patch: > cryptodev API changes > http://dpdk.org/ml/archives/dev/2016-February/034212.html >=20 > Signed-off-by: Deepak Kumar JAIN > --- > app/test/test_cryptodev.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) >=20 > diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c index > 208fc14..acba98a 100644 > --- a/app/test/test_cryptodev.c > +++ b/app/test/test_cryptodev.c > @@ -1845,6 +1845,18 @@ test_multi_session(void) > sessions[i], ut_params, ts_params), > "Failed to perform decrypt on request " > "number %u.", i); > + /* free crypto operation structure */ > + if (ut_params->op) > + rte_crypto_op_free(ut_params->op); > + > + /* > + * free mbuf - both obuf and ibuf are usually the same, > + * but rte copes even if we call free twice > + */ > + if (ut_params->obuf) { > + rte_pktmbuf_free(ut_params->obuf); > + ut_params->obuf =3D 0; > + } > } >=20 > /* Next session create should fail */ > -- > 2.1.0 Acked-by: Fiona Trahe