From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 206F838EB for ; Wed, 28 Sep 2016 02:45:23 +0200 (CEST) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga105.fm.intel.com with ESMTP; 27 Sep 2016 17:45:23 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.30,407,1470726000"; d="scan'208";a="1046738813" Received: from irsmsx151.ger.corp.intel.com ([163.33.192.59]) by fmsmga001.fm.intel.com with ESMTP; 27 Sep 2016 17:45:22 -0700 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.164]) by IRSMSX151.ger.corp.intel.com ([169.254.4.226]) with mapi id 14.03.0248.002; Wed, 28 Sep 2016 01:45:21 +0100 From: "De Lara Guarch, Pablo" To: "Kusztal, ArkadiuszX" , "dev@dpdk.org" CC: "Trahe, Fiona" , "Jain, Deepak K" , "Griffin, John" Thread-Topic: [PATCH v3] app/test: add AES GCM performance test to cryptodev Thread-Index: AQHSGMYCEduxt35ZfkGvKVWOjjA7bqCOEGzA Date: Wed, 28 Sep 2016 00:45:21 +0000 Message-ID: References: <1474984172-11838-1-git-send-email-arkadiuszx.kusztal@intel.com> In-Reply-To: <1474984172-11838-1-git-send-email-arkadiuszx.kusztal@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNjk5NzAzZTMtZjAxYy00MTNkLTg1N2MtNTFiMjI5MDMyZGNmIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6ImVVQUJvMXRuUW9oenNMXC9QVzhGQXBXRlVMU205MWhoNjloVlJuU0poSVNZPSJ9 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 v3] app/test: add AES GCM performance test to cryptodev 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: Wed, 28 Sep 2016 00:45:24 -0000 Hi Arek, > -----Original Message----- > From: Kusztal, ArkadiuszX > Sent: Tuesday, September 27, 2016 6:50 AM > To: dev@dpdk.org > Cc: Trahe, Fiona; Jain, Deepak K; De Lara Guarch, Pablo; Griffin, John; K= usztal, > ArkadiuszX > Subject: [PATCH v3] app/test: add AES GCM performance test to cryptodev >=20 > This patch adds AES Galois Counter Mode performance test case > for cryptodev QAT and AESNI GCM. Test is performed with different > buffer sizes, burst size of 32 and 128b key. Test vectors > are placed in app/test/test_cryptodev_perf_vectors.h file. >=20 > Signed-off-by: Arek Kusztal > --- > This patch depends on the following patches/patchsets: >=20 > "app/test: fix linkage scope of gmac plaintext array" > (http://dpdk.org/dev/patchwork/patch/16023/) >=20 > v2: > * Fixed 32b build problem in test_cryptodev_perf.c > v3: > * Added multiple test cases possibility > * Added AESNI GCM test suite > --- > app/test/test_cryptodev_gcm_test_vectors.h | 632 > +++++++++++++++++++++++++++++ > app/test/test_cryptodev_perf.c | 482 +++++++++++++++++++++- > 2 files changed, 1111 insertions(+), 3 deletions(-) >=20 > diff --git a/app/test/test_cryptodev_perf.c b/app/test/test_cryptodev_per= f.c > index 20713d4..3bb6f34 100644 > --- a/app/test/test_cryptodev_perf.c > +++ b/app/test/test_cryptodev_perf.c > @@ -41,6 +41,7 @@ [...] > +static int > +perf_AES_GCM(uint8_t dev_id, uint16_t queue_id, > + struct perf_test_params *pparams, uint32_t test_ops) > +{ [...] > + if (burst_dequeued =3D=3D 0) > + failed_polls++; > + else { > + processed +=3D burst_dequeued; > + > + for (m =3D 0; m < burst_dequeued; m++) { > + if (test_ops) { > + uint16_t iv_pad_len =3D > ALIGN_POW2_ROUNDUP > + (pparams->symmetric_op->iv_len, > 16); > + uint8_t *pkt =3D rte_pktmbuf_mtod( > + proc_ops[m]->sym->m_src, > + uint8_t *); > + > + TEST_ASSERT_BUFFERS_ARE_EQUAL( > + pparams->symmetric_op->c_data, > + pkt + iv_pad_len + > + pparams->symmetric_op->aad_len, > + pparams->symmetric_op->c_len, > + "GCM Ciphertext data not as > expected"); > + > + TEST_ASSERT_BUFFERS_ARE_EQUAL( > + pparams->symmetric_op->t_data, > + pkt + iv_pad_len + > + pparams->symmetric_op->aad_len + > + pparams->symmetric_op->c_len, > + pparams->symmetric_op->t_len, > + "GCM MAC data not as expected"); > + > + } > + rte_crypto_op_free(proc_ops[m]); > + } Please, check the indentation here. Checkpatch complained here. Thanks, Pablo > + } > + } > + > + tsc_end =3D rte_rdtsc_precise(); > + > + double ops_s =3D ((double)processed / (tsc_end - tsc_start)) > + * rte_get_tsc_hz(); > + double throughput =3D (ops_s * pparams->symmetric_op->p_len * 8) > + / 1000000000;