Message ID | 1567786349-25353-1-git-send-email-fiona.trahe@intel.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B4F161F414; Fri, 6 Sep 2019 18:12:42 +0200 (CEST) Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id B36DB1F40E for <dev@dpdk.org>; Fri, 6 Sep 2019 18:12:40 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Sep 2019 09:12:39 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,473,1559545200"; d="scan'208";a="358825665" Received: from sivswdev09.ir.intel.com (HELO localhost.localdomain) ([10.237.217.48]) by orsmga005.jf.intel.com with ESMTP; 06 Sep 2019 09:12:37 -0700 From: Fiona Trahe <fiona.trahe@intel.com> To: dev@dpdk.org Cc: akhil.goyal@nxp.com, arkadiuszx.kusztal@intel.com, fiona.trahe@intel.com Date: Fri, 6 Sep 2019 17:12:26 +0100 Message-Id: <1567786349-25353-1-git-send-email-fiona.trahe@intel.com> X-Mailer: git-send-email 1.7.0.7 In-Reply-To: <1567784675-1401-1-git-send-email-fiona.trahe@intel.com> References: <1567784675-1401-1-git-send-email-fiona.trahe@intel.com> Subject: [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 <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Series |
Add dual threading in QAT PMD
|
|
Message
Fiona Trahe
Sept. 6, 2019, 4:12 p.m. UTC
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(-)
Comments
Hi Fiona, This patchset need a rebase. Could you please send v3. Thanks, Akhil > -----Original Message----- > From: Fiona Trahe <fiona.trahe@intel.com> > Sent: Friday, September 6, 2019 9:42 PM > To: dev@dpdk.org > Cc: Akhil Goyal <akhil.goyal@nxp.com>; 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
Hi Akhil, We're going to defer this patchset this to 20.02. 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 deferred? 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 <fiona.trahe@intel.com>; dev@dpdk.org > Cc: Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com> > Subject: RE: [PATCH v2 0/3] Add dual threading in QAT PMD > > Hi Fiona, > > This patchset need a rebase. Could you please send v3. > > Thanks, > Akhil > > > -----Original Message----- > > From: Fiona Trahe <fiona.trahe@intel.com> > > Sent: Friday, September 6, 2019 9:42 PM > > To: dev@dpdk.org > > Cc: Akhil Goyal <akhil.goyal@nxp.com>; 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