From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 905E3A0350; Wed, 1 Jul 2020 08:19:21 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 86D501BFB5; Wed, 1 Jul 2020 08:19:21 +0200 (CEST) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id AB0271BEE6 for ; Wed, 1 Jul 2020 08:19:19 +0200 (CEST) IronPort-SDR: hid5M28q0YpZs6+QsXCHq+L3m1VpZIZ9Zn2vsa7EFDCsCkEgwr71tBY60u2+4Jy9W+gG93keE7 sR+Nb2u18/9A== X-IronPort-AV: E=McAfee;i="6000,8403,9668"; a="207989195" X-IronPort-AV: E=Sophos;i="5.75,299,1589266800"; d="scan'208";a="207989195" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Jun 2020 23:19:18 -0700 IronPort-SDR: F2jqzxJchIcBfhZ39Z1y/ZveUsBR2rboEoBuvj0uD4vahnX/qg5YCS3S7p7oBpFDL0PjpfLa/Q EAGkYYbpTGrA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,299,1589266800"; d="scan'208";a="312587529" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga008.jf.intel.com with ESMTP; 30 Jun 2020 23:19:18 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 30 Jun 2020 23:19:17 -0700 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.85]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.208]) with mapi id 14.03.0439.000; Wed, 1 Jul 2020 14:19:15 +0800 From: "Tu, Lijuan" To: "Jiang, YuX" , "dts@dpdk.org" CC: "Jiang, YuX" Thread-Topic: [dts] [PATCH V1] tests/crypto_perf_cryptodev_perf, cryptodev_common: optimize cryptodev test script Thread-Index: AQHWT0z+KWdlkcp4506ErBxlSp4wM6jyQHHQ Date: Wed, 1 Jul 2020 06:19:15 +0000 Message-ID: <8CE3E05A3F976642AAB0F4675D0AD20E0BC68139@SHSMSX101.ccr.corp.intel.com> References: <1593568769-343083-1-git-send-email-yux.jiang@intel.com> In-Reply-To: <1593568769-343083-1-git-send-email-yux.jiang@intel.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="iso-2022-jp" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dts] [PATCH V1] tests/crypto_perf_cryptodev_perf, cryptodev_common: optimize cryptodev test script X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dts-bounces@dpdk.org Sender: "dts" Applied ,thanks > -----Original Message----- > From: dts On Behalf Of Jiang, YuX > Sent: 2020=1B$BG/=1B(B7=1B$B7n=1B(B1=1B$BF|=1B(B 9:59 > To: dts@dpdk.org > Cc: Jiang, YuX > Subject: [dts] [PATCH V1] tests/crypto_perf_cryptodev_perf, > cryptodev_common: optimize cryptodev test script >=20 > From: "Jiang,yu" >=20 > optimize case's result which cryptodev perf does not support. >=20 > Signed-off-by: Jiang,yu > --- > tests/TestSuite_crypto_perf_cryptodev_perf.py | 8 +++++--- > tests/cryptodev_common.py | 4 ++-- > 2 files changed, 7 insertions(+), 5 deletions(-) >=20 > diff --git a/tests/TestSuite_crypto_perf_cryptodev_perf.py > b/tests/TestSuite_crypto_perf_cryptodev_perf.py > index fc8f502..e1780f5 100644 > --- a/tests/TestSuite_crypto_perf_cryptodev_perf.py > +++ b/tests/TestSuite_crypto_perf_cryptodev_perf.py > @@ -230,13 +230,13 @@ class PerfTestsCryptodev(TestCase): >=20 > def _run_crypto_perf(self): > if cc.is_test_skip(self): > - return > + return "skip" >=20 > self.c_num, self.t_num =3D self._get_core_and_thread_num() > devices =3D self._get_crypto_device(self.t_num) > if not devices: > self.logger.info("can not get device or unsupported, skip.") > - return > + return "skip" >=20 > eal_opt_str =3D cc.get_eal_opt_str(self, devices) > crypto_perf_opt_str =3D self._get_crypto_perf_opt_str() @@ -391,= 7 +391,9 > @@ class PerfTestsCryptodev(TestCase): >=20 > def _run_crypto_perf_throughput(self): > results =3D self._run_crypto_perf() > - > + if results =3D=3D "skip": > + return > + self.verify(results, "test results is none, Test Failed") > stats_results =3D self._stat_results_by_buf_size(results) > json_result =3D [] >=20 > diff --git a/tests/cryptodev_common.py b/tests/cryptodev_common.py index > 832b3bf..23c7cd9 100644 > --- a/tests/cryptodev_common.py > +++ b/tests/cryptodev_common.py > @@ -138,7 +138,7 @@ def get_qat_devices(test_case, cpm_num=3DNone, > num=3D1): > cpm_num =3D len(test_case.dev.keys()) > n, dev_list =3D 0, [] > if cpm_num > len(test_case.dev.keys()): > - self.logger.warning("QAT card only {} cpm, but {} required".form= at( > + test_case.logger.warning("QAT card only {} cpm, but {} > + required".format( > len(test_case.dev), cpm_num)) > return [] > for i in range(num): > @@ -148,7 +148,7 @@ def get_qat_devices(test_case, cpm_num=3DNone, > num=3D1): > if i < len(test_case.dev[cpm]): > dev_list.append(test_case.dev[cpm][i]) > else: > - self.logger.warning("not enough vf in cpm: {}".format(cp= m)) > + test_case.logger.warning("not enough vf in cpm: > + {}".format(cpm)) > n +=3D 1 > return dev_list >=20 > -- > 2.7.4