From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kusztal, ArkadiuszX" Subject: Re: [PATCH] app/crypto-perf: add minimise-offload-cost flag Date: Thu, 11 May 2017 14:22:06 +0000 Message-ID: <80307F746F1522479831AB1253B7024E758DEE@IRSMSX102.ger.corp.intel.com> References: <1494346443-11130-1-git-send-email-fiona.trahe@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "Doherty, Declan" , "Griffin, John" , "De Lara Guarch, Pablo" , "Trahe, Fiona" To: "Trahe, Fiona" , "dev@dpdk.org" , "Mrozowicz, SlawomirX" Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 3EF5320F for ; Thu, 11 May 2017 16:22:09 +0200 (CEST) In-Reply-To: <1494346443-11130-1-git-send-email-fiona.trahe@intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Fiona Trahe > Sent: Tuesday, May 09, 2017 5:14 PM > To: dev@dpdk.org; Mrozowicz, SlawomirX > > Cc: Doherty, Declan ; Griffin, John > ; De Lara Guarch, Pablo > ; Trahe, Fiona > Subject: [dpdk-dev] [PATCH] app/crypto-perf: add minimise-offload-cost fl= ag >=20 > The throughput test enqueues and dequeues bursts of operations > to the device. For software devices the full burst size will > usually be successfully en/dequeued, on hardware devices > however the CPU can call the API more frequently than necessary, > as it has nothing else to do. > Minimum offload cost is achieved when the specified > burst_size is en/dequeued. So rather than > wasting CPU cycles continually retrying, with a > fraction of the burst being en/dequeued each time, > fewer CPU cycles are used by backing off until a full > burst can be enqueued. >=20 > This patch adds a --minimise-offload-cost flag. > When set the test backs off until full bursts are > en/dequeued and counts the cycles while waiting. > These cycles represent cycles saved by > offloading, which in a real application are > available for other work. Hence these cycles are > deducted from the total cycle-count to show the > minimum offload-cost which can be acheived. >=20 > Signed-off-by: Fiona Trahe > --- > app/test-crypto-perf/cperf_options.h | 2 + > app/test-crypto-perf/cperf_options_parsing.c | 12 +++++ > app/test-crypto-perf/cperf_test_throughput.c | 76 > ++++++++++++++++++++++------ > 3 files changed, 75 insertions(+), 15 deletions(-) >=20 > diff --git a/app/test-crypto-perf/cperf_options.h b/app/test-crypto- > perf/cperf_options.h > index b928c58..48ca1de 100644 > --- a/app/test-crypto-perf/cperf_options.h > +++ b/app/test-crypto-perf/cperf_options.h > @@ -31,6 +31,7 @@ > -- > 2.5.0 Acked-by: Arek Kusztal