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 622E65B12 for ; Fri, 2 Mar 2018 10:53:37 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 02 Mar 2018 01:53:35 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.47,411,1515484800"; d="scan'208";a="34102855" Received: from irsmsx153.ger.corp.intel.com ([163.33.192.75]) by fmsmga004.fm.intel.com with ESMTP; 02 Mar 2018 01:53:33 -0800 Received: from irsmsx101.ger.corp.intel.com ([169.254.1.188]) by IRSMSX153.ger.corp.intel.com ([169.254.9.213]) with mapi id 14.03.0319.002; Fri, 2 Mar 2018 09:53:32 +0000 From: "Trahe, Fiona" To: Ahmed Mansour , "Verma, Shally" , "dev@dpdk.org" CC: "De Lara Guarch, Pablo" , "Athreya, Narayana Prasad" , "Gupta, Ashish" , "Sahu, Sunila" , "Challa, Mahipal" , "Jain, Deepak K" , Hemant Agrawal , "Roy Pledge" , Youri Querry , "Trahe, Fiona" Thread-Topic: [dpdk-dev] [PATCH] compressdev: implement API Thread-Index: AQHTnFNA34V0d0oQkkqZkaaruzarZKO83QBg Date: Fri, 2 Mar 2018 09:53:32 +0000 Message-ID: <348A99DA5F5B7549AA880327E580B4358932A333@IRSMSX101.ger.corp.intel.com> References: <1517595924-25963-1-git-send-email-fiona.trahe@intel.com> <12544144.czVLKRyaz4@xps> <348A99DA5F5B7549AA880327E580B43589325187@IRSMSX101.ger.corp.intel.com> <348A99DA5F5B7549AA880327E580B4358932983C@IRSMSX101.ger.corp.intel.com> <348A99DA5F5B7549AA880327E580B43589329F78@IRSMSX101.ger.corp.intel.com> In-Reply-To: Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMzg2YzI0NDYtODdlMC00YThlLWE1NWUtNmJjZjM4NWE3ZDdiIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE2LjUuOS4zIiwiVHJ1c3RlZExhYmVsSGFzaCI6InBWejVvTDVxeUFjelphUmQ3SG1wWVpTVkZJZWtVTHoyZUFYZ0tZRHV6TGc9In0= x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH] compressdev: implement API 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, 02 Mar 2018 09:53:37 -0000 > -----Original Message----- > From: Ahmed Mansour [mailto:ahmed.mansour@nxp.com] > Sent: Friday, March 2, 2018 12:56 AM > To: Trahe, Fiona ; Verma, Shally ; dev@dpdk.org > Cc: De Lara Guarch, Pablo ; Athreya, Nara= yana Prasad > ; Gupta, Ashish ; Sahu, Sunila > ; Challa, Mahipal ; Ja= in, Deepak K > ; Hemant Agrawal ; Roy P= ledge > ; Youri Querry > Subject: Re: [dpdk-dev] [PATCH] compressdev: implement API >=20 > On 3/1/2018 9:41 AM, Trahe, Fiona wrote: > > Hi Shally > > > > //snip// > >> [Shally] This looks better to me. So it mean app would always call xfo= rm_init() for stateless and attach > an > >> updated priv_xform to ops (depending upon if there's shareable or not)= . So it does not need to have > >> NULL pointer on priv_xform. right? > >> > > [Fiona] yes. The PMD must return a valid priv_xform pointer. >=20 > [Ahmed] What I understood is that the xform_init will be called once > initially. if the @flag returned is NONE_SHAREABLE then the application > must not attach two inflight ops to the same @priv_xform? Otherwise the > application can attach many ops in flight to the @priv_xform? [Fiona Yes. App calls the xform_init() once on a device where it plans to s= end stateless ops. If PMD returns shareable, then it doesn't need to call again and can attach= this to every stateless op going to that device. If PMD returns SINGLE_OP then it must call xform_init() before every other= =20 stateless op it wants to have inflight simultaneously. This does not mean i= t must be called before every op,=20 but probably will set up a batch of priv_xforms - it can reuse each priv_x= form once the op finishes with it.