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 405723421 for ; Fri, 23 Nov 2018 16:12:21 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 23 Nov 2018 07:12:20 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,270,1539673200"; d="scan'208";a="110726268" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga001.jf.intel.com with ESMTP; 23 Nov 2018 07:12:20 -0800 Received: from fmsmsx154.amr.corp.intel.com (10.18.116.70) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.408.0; Fri, 23 Nov 2018 07:12:19 -0800 Received: from HASMSX110.ger.corp.intel.com (10.184.198.28) by FMSMSX154.amr.corp.intel.com (10.18.116.70) with Microsoft SMTP Server (TLS) id 14.3.408.0; Fri, 23 Nov 2018 07:12:18 -0800 Received: from hasmsx112.ger.corp.intel.com ([169.254.11.97]) by HASMSX110.ger.corp.intel.com ([169.254.6.157]) with mapi id 14.03.0415.000; Fri, 23 Nov 2018 17:12:16 +0200 From: "Jozwiak, TomaszX" To: "Varghese, Vipin" , "dev@dpdk.org" , "Trahe, Fiona" , "Shally.Verma@cavium.com" , "akhil.goyal@nxp.com" Thread-Topic: [dpdk-dev] [PATCH v4 3/5] doc/guides/tools: add doc files Thread-Index: AQHUgzjF7w29xyWllE+SFeOfsdAju6VdUn4AgAAjvQA= Date: Fri, 23 Nov 2018 15:12:15 +0000 Message-ID: References: <1542978409-28668-1-git-send-email-tomaszx.jozwiak@intel.com> <1542983269-30093-1-git-send-email-tomaszx.jozwiak@intel.com> <1542983269-30093-4-git-send-email-tomaszx.jozwiak@intel.com> <4C9E0AB70F954A408CC4ADDBF0F8FA7D4D2C1AF9@BGSMSX101.gar.corp.intel.com> In-Reply-To: <4C9E0AB70F954A408CC4ADDBF0F8FA7D4D2C1AF9@BGSMSX101.gar.corp.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [10.103.104.46] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v4 3/5] doc/guides/tools: add doc files X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Nov 2018 15:12:21 -0000 > -----Original Message----- > From: Varghese, Vipin > Sent: Friday, November 23, 2018 4:00 PM > To: Jozwiak, TomaszX ; dev@dpdk.org; Trahe, > Fiona ; Jozwiak, TomaszX > ; Shally.Verma@cavium.com; > akhil.goyal@nxp.com > Subject: RE: [dpdk-dev] [PATCH v4 3/5] doc/guides/tools: add doc files >=20 > Apologies Tomasz, I shared my comments in v3. sharing for v4 >=20 > > Crypto performance test application > > M: Declan Doherty > > F: app/test-crypto-perf/ > > diff --git a/doc/guides/rel_notes/release_18_11.rst > > b/doc/guides/rel_notes/release_18_11.rst > > index 32ff0e5..d44cf30 100644 > > --- a/doc/guides/rel_notes/release_18_11.rst > > +++ b/doc/guides/rel_notes/release_18_11.rst > > @@ -328,6 +328,11 @@ New Features > > additional command-line parameter values from the > "DPDK_TEST_PARAMS" > > environment variable to make this application easier to use. > > > > +* **Added a compression performance test tool.** >=20 > It is mentioned as test application above, here test tool. Should we make= this > synced? Will sync >=20 > > > > API Changes > > ----------- > > diff --git a/doc/guides/tools/comp_perf.rst > > b/doc/guides/tools/comp_perf.rst new file mode 100644 index > > 0000000..1428348 > > --- /dev/null > > +++ b/doc/guides/tools/comp_perf.rst > > @@ -0,0 +1,75 @@ > > +.. SPDX-License-Identifier: BSD-3-Clause > > + Copyright(c) 2018 Intel Corporation. > > + > > +dpdk-test-compress-perf Application > > +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D >=20 > Should we remove extra '=3D'? It's ok. Please take a look at console output or count the chars=20 >=20 > > + > > +The ``dpdk-test-compress-perf`` tool is a Data Plane Development Kit > > +(DPDK) utility that allows measuring performance parameters of PMDs >=20 > Should we keep this as test application? Or is this an utility like pdump= or > procinfo? >=20 > > + > > + > > +Limitations > > +~~~~~~~~~~~ >=20 > Extra '~'? No. It's ok. Please take a look at console output or count the chars >=20 > > + > > +* Only supports the fixed compression and stateless operation. >=20 > Is 'stateful' not supported a limitation rather than supporting stateless= ? In this initial version we support only stateless operation. Will be update= d in the future versions. >=20 > > + > > +Command line options > > +-------------------- > > + >=20 > Missing '-' No. It's ok. Please take a look at console output or count the chars >=20 > > +Compiling the Application > > +------------------------- > > + > > +**Step 1: PMD setting** > > + > > +The ``dpdk-test-compress-perf`` tool depends on compression device > > +drivers PMD which can be disabled by default in the build > > +configuration file > > ``common_base``. > > +The compression device drivers PMD which should be tested can be > > +enabled > > by setting:: > > + > > + CONFIG_RTE_LIBRTE_PMD_ISAL=3Dy >=20 > Does this test application other PMD? If not can we cover this in limitat= ion? Should be eg.: >=20 > > + > > + > > +Running the Application > > +----------------------- > > + > > +The tool application has a number of command line options. Here is > > +the > > sample command line: >=20 > Do we need to reword 'tool application' to either 'test application' or '= test > utility'? Will sync this. >=20 > > + > > +.. code-block:: console > > + > > + ./build/app/dpdk-test-compress-perf -l 4 -- --driver-name > > + compress_qat -- > > input-file test.txt --seg-sz 8192 > > + --compress-level 1:1:9 --num-iter 10 --extended-input-sz 1048576 > > + --max-num-sgl-segs 16 --huffman-enc fixed > > + > > diff --git a/doc/guides/tools/index.rst b/doc/guides/tools/index.rst > > index a6e2c4c..24235ba 100644 > > --- a/doc/guides/tools/index.rst > > +++ b/doc/guides/tools/index.rst > > @@ -42,3 +42,4 @@ DPDK Tools User Guides > > testbbdev > > cryptoperf > > testeventdev > > + comp_perf > > -- > > 2.7.4 Thx, Tomek