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 E5100A2F6B for ; Tue, 8 Oct 2019 12:53:48 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1A7FF1C036; Tue, 8 Oct 2019 12:53:48 +0200 (CEST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id B5B991C034 for ; Tue, 8 Oct 2019 12:53:46 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Oct 2019 03:53:45 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.67,270,1566889200"; d="scan'208";a="197661341" Received: from irsmsx151.ger.corp.intel.com ([163.33.192.59]) by orsmga006.jf.intel.com with ESMTP; 08 Oct 2019 03:53:43 -0700 Received: from irsmsx156.ger.corp.intel.com (10.108.20.68) by IRSMSX151.ger.corp.intel.com (163.33.192.59) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 8 Oct 2019 11:53:43 +0100 Received: from irsmsx101.ger.corp.intel.com ([169.254.1.129]) by IRSMSX156.ger.corp.intel.com ([169.254.3.246]) with mapi id 14.03.0439.000; Tue, 8 Oct 2019 11:53:42 +0100 From: "Trahe, Fiona" To: Akhil Goyal , "dev@dpdk.org" CC: "Kusztal, ArkadiuszX" , "Trahe, Fiona" Thread-Topic: [PATCH v2 0/3] Add dual threading in QAT PMD Thread-Index: AQHVZM3qhzEgx9AsEUS9FLtPuux0XKdKaBGAgAZaz1A= Date: Tue, 8 Oct 2019 10:53:42 +0000 Message-ID: <348A99DA5F5B7549AA880327E580B4358982F455@IRSMSX101.ger.corp.intel.com> References: <1567784675-1401-1-git-send-email-fiona.trahe@intel.com> <1567786349-25353-1-git-send-email-fiona.trahe@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: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYWNlZWI4YWUtOTBlYS00NDhlLWFlMWItYjA2MzQ5ZDhmOWQwIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiT05TT2JIaXJLMHFnaFgycVFXQndxMzJoUmpIbXkzZytCRTFjSDVKbTF3dmhDUFZ6Rk1uVUk5aVR3TlZISG9oSiJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.2.0.6 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 v2 0/3] Add dual threading in QAT PMD 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Akhil, We're going to defer this patchset this to 20.02.=20 We want to add another patch to it to mitigate some performance impacts it = causes.# I'm not sure of the process for this - do I just mark it in patchwork as de= ferred? Or is there a tag I can put in an email, like Deferred-by? Fiona > -----Original Message----- > From: Akhil Goyal [mailto:akhil.goyal@nxp.com] > Sent: Friday, October 4, 2019 11:48 AM > To: Trahe, Fiona ; dev@dpdk.org > Cc: Kusztal, ArkadiuszX > Subject: RE: [PATCH v2 0/3] Add dual threading in QAT PMD >=20 > Hi Fiona, >=20 > This patchset need a rebase. Could you please send v3. >=20 > Thanks, > Akhil >=20 > > -----Original Message----- > > From: Fiona Trahe > > Sent: Friday, September 6, 2019 9:42 PM > > To: dev@dpdk.org > > Cc: Akhil Goyal ; arkadiuszx.kusztal@intel.com; > > fiona.trahe@intel.com > > Subject: [PATCH v2 0/3] Add dual threading in QAT PMD > > > > Remove the limitation whereby enqueue and dequeue must be > > done in same thread. > > The inflight calculation is reworked to be thread-safe for 2 > > threads - note this is not general multi-thread support, i.e > > all enqueues to a qp must still be done in one thread and > > all dequeues must be done in one thread, but enqueues and > > dequeues may be in separate threads. > > As the tail-coalescing feature is not > > threadsafe it is removed first. > > > > v2 changes: > > - clarified wording in docs > > > > Fiona Trahe (3): > > common/qat: remove tail write coalescing feature > > common/qat: move max inflights param into qp > > common/qat: add dual thread support > > > > doc/guides/compressdevs/qat_comp.rst | 5 ++- > > doc/guides/cryptodevs/qat.rst | 11 +++++- > > drivers/common/qat/qat_qp.c | 77 ++++++++++++++++++----------= -------- > > drivers/common/qat/qat_qp.h | 11 ++---- > > 4 files changed, 54 insertions(+), 50 deletions(-) > > > > -- > > 2.13.6