From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id CE100187 for ; Thu, 3 May 2018 09:25:33 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 03 May 2018 00:25:32 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,357,1520924400"; d="scan'208";a="55598385" Received: from irsmsx106.ger.corp.intel.com ([163.33.3.31]) by orsmga002.jf.intel.com with ESMTP; 03 May 2018 00:25:31 -0700 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.150]) by IRSMSX106.ger.corp.intel.com ([169.254.8.125]) with mapi id 14.03.0319.002; Thu, 3 May 2018 08:25:30 +0100 From: "De Lara Guarch, Pablo" To: "Kumar, Ravi1" , Thomas Monjalon CC: "dev@dpdk.org" , "hemant.agrawal@nxp.com" Thread-Topic: [dpdk-dev] [PATCH v5 17/19] crypto/ccp: support cpu based md5 and sha2 family authentication algo Thread-Index: AQHTv31K7SU2KUOvEkChaa79Uh9zsqQNW8GAgACwqQCAABecAIAAMQ1ggA9jrYCAACd/wA== Date: Thu, 3 May 2018 07:25:30 +0000 Message-ID: References: <1520584520-130522-1-git-send-email-Ravi1.kumar@amd.com> <14946114.ASIPbkMQcp@xps> <1698373.c9XDPgIzTm@xps> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYWY1M2M4ZTQtM2MyZS00NjRmLTk0ZDMtMWI3NDE1YjUyZDA5IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE2LjUuOS4zIiwiVHJ1c3RlZExhYmVsSGFzaCI6ImJzUjdyazE0RDlDN3pBM3RBblp0dEZNMmNCSzdXZUlpT3hjNHVtWThqSEk9In0= x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.200.100 dlp-reaction: no-action 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 v5 17/19] crypto/ccp: support cpu based md5 and sha2 family authentication algo 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: Thu, 03 May 2018 07:25:34 -0000 > -----Original Message----- > From: Kumar, Ravi1 [mailto:Ravi1.Kumar@amd.com] > Sent: Thursday, May 3, 2018 7:02 AM > To: De Lara Guarch, Pablo ; Thomas > Monjalon > Cc: dev@dpdk.org; hemant.agrawal@nxp.com > Subject: RE: [dpdk-dev] [PATCH v5 17/19] crypto/ccp: support cpu based md= 5 > and sha2 family authentication algo >=20 > >Hi Ravi, > > > >> -----Original Message----- > >> From: Thomas Monjalon [mailto:thomas@monjalon.net] > >> Sent: Monday, April 23, 2018 9:06 AM > >> To: Kumar, Ravi1 > >> Cc: De Lara Guarch, Pablo ; > >> dev@dpdk.org; hemant.agrawal@nxp.com > >> Subject: Re: [dpdk-dev] [PATCH v5 17/19] crypto/ccp: support cpu > >> based > >> md5 and sha2 family authentication algo > >> > >> 23/04/2018 08:41, Kumar, Ravi1: > >> > >> +CONFIG_RTE_LIBRTE_PMD_CCP_CPU_AUTH=3Dn > >> > > > >> > >Why introducing a compile-time option? > >> > >Can it be a run-time option of the device? > >> > >We must not add compile-time device option if not well justified. > >> [...] > >> > By default, all the crypto operations (cipher + auth) are offloaded > >> > to CCP > >> engines. When user enables CONFIG_RTE_LIBRTE_PMD_CCP_CPU_AUTH=3Dy, > the > >> auth operations are not offloaded to CCP and rather performed over > >> CPU. We kept this feature as a compile time option in order to let > >> user decide whether to run auth operations on CCP or CPU as some of > >> the auth operations performs faster on CPU as compared to their > performance on CCP. > >> > >> No, you do not let the user decide. > >> The compilation options are for the packager to decide. > >> The user can rely on pre-compiled packages and use only runtime option= s. > >> That's why we forbid compile-time options for such features. > >> > > > >Could you send a patch to remove this compile-time option and pass it as= an > option for the vdev? > >Look at how the crypto scheduler accepts parameters from "rte_vdev_init"= or - > -vdev, in scheduler_pmd.c. > > > >Thanks! > >Pablo > > > Hi Pablo, >=20 > I have uploaded 2 patches for this issue. >=20 > If needed, you could squash the patch-1 to our very initial patch (commit > 0054d84f6ec8: crypto/ccp: add AMD ccp skeleton PMD). >=20 > Please let me know if you need any other information from our side. This cannot be squashed into the first patch as it has already been merged, so they would be applied as they are. Pablo >=20 > Regards, > Ravi