From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 4080AA0547;
	Wed, 27 Oct 2021 12:58:15 +0200 (CEST)
Received: from [217.70.189.124] (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 31F89410FC;
	Wed, 27 Oct 2021 12:58:15 +0200 (CEST)
Received: from mail-io1-f43.google.com (mail-io1-f43.google.com
 [209.85.166.43]) by mails.dpdk.org (Postfix) with ESMTP id B9B9840DDA
 for <dev@dpdk.org>; Wed, 27 Oct 2021 12:58:14 +0200 (CEST)
Received: by mail-io1-f43.google.com with SMTP id h196so3127197iof.2
 for <dev@dpdk.org>; Wed, 27 Oct 2021 03:58:14 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112;
 h=mime-version:references:in-reply-to:from:date:message-id:subject:to
 :cc; bh=c1y12lDsoWqtCsAFdYPkjM8DcJKmOKPhI2a2o2vs6ic=;
 b=dg2xgg3jM7HAK7t8jW0mCXCM7I79iNkMtQG2uT7CZU5WLoLRxkzSpfr73UujpzsnVe
 oedJuIIr25TogwqSZMwk6pUWGOTfSWNYt+G29iVPnpF7jMwdut321ri4tLQA52Ye+PLh
 n3ZtaWHGiCqtNd3tbxBdOsjT/MIxqcVn+SqwrIEtMMV3hDJMnkEWuSQTAFCuRV4vqGsJ
 ddsOSKTwiDbKTLjWfwVAtiZowGTPKD6IsIKWWYdHQfnPNTEJIM+3I6+Tjs1kFsIfFhL2
 sfPvRhvwyYauEB1EpJxF8JIm+A4dbwSbbfKHu+nu3p/vv33XhHSDq0NWN/kpBEBNHp1k
 nn7A==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20210112;
 h=x-gm-message-state:mime-version:references:in-reply-to:from:date
 :message-id:subject:to:cc;
 bh=c1y12lDsoWqtCsAFdYPkjM8DcJKmOKPhI2a2o2vs6ic=;
 b=fG/+bBG7O9AQW0Lb4pRiFTuP0RYQlr67tcDKgLjiozzrqvExR/waOW3vF37kXRI09t
 fk70TqmC4hS9a5VaUkEoHmNldNXjNsKhON7eQCkfMpB5Q4jry9VuAyBlS/KHBbsB1Efs
 Lf1Qw8zd8OCkCzdo8rYHmo8RstC3CFDOciGQtRQRU4AlOHybrYhyZNIqwnBQBTFNIQrA
 J415MzWDk8BWxzMWFSw+ndrywWLA0Hqfuc10rHJleFpT5MMUOE5OFqwJFUlU1NN1a0GC
 4rN+9crorTTaRvR+TnzVs/zApUrNSgxotql9U+Lo+njCzHG28zJbOuxz0906xspEUiYw
 wm6A==
X-Gm-Message-State: AOAM530rBWRBgujyPMYCWrs69ErlCy/IbhKhe+IiBmdd3/RBLsO058Gx
 NiHSlXcmlq9Q+H+RVfjv0nUJeqs/aLyYn9+KGtM=
X-Google-Smtp-Source: ABdhPJy78mxzf+TewmUhErM/3j5lxNQz2vHU54iqTXTBpj7eDwMR6nIEvfNK1/QGnWnTjqsoOjLU9+dhh9Z70frKcBk=
X-Received: by 2002:a05:6602:2d13:: with SMTP id
 c19mr19464661iow.199.1635332294088; 
 Wed, 27 Oct 2021 03:58:14 -0700 (PDT)
MIME-Version: 1.0
References: <20210902053253.3017858-1-feifei.wang2@arm.com>
 <20211027081018.653746-1-feifei.wang2@arm.com>
In-Reply-To: <20211027081018.653746-1-feifei.wang2@arm.com>
From: Jerin Jacob <jerinjacobk@gmail.com>
Date: Wed, 27 Oct 2021 16:27:48 +0530
Message-ID: <CALBAE1MbP3fA-5Fo4Vhn3F8By-gx_1PQcaHse69yDLtHJisO_Q@mail.gmail.com>
To: Feifei Wang <feifei.wang2@arm.com>, 
 "Ananyev, Konstantin" <konstantin.ananyev@intel.com>, 
 Stephen Hemminger <stephen@networkplumber.org>,
 David Marchand <david.marchand@redhat.com>, 
 Thomas Monjalon <thomas@monjalon.net>
Cc: dpdk-dev <dev@dpdk.org>, nd <nd@arm.com>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [dpdk-dev] [PATCH v6 0/4] add new definitions for wait scheme
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.29
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>

On Wed, Oct 27, 2021 at 1:40 PM Feifei Wang <feifei.wang2@arm.com> wrote:
>
> Add new definitions for wait scheme, and apply this new definitions into
> lib to replace rte_pause.
>
> v2:
> 1. use macro to create new wait scheme (Stephen)
>
> v3:
> 1. delete unnecessary bug fix in bpf (Konstantin)
>
> v4:
> 1. put size into the macro body (Konstantin)
> 2. replace assert with BUILD_BUG_ON (Stephen)
> 3. delete unnecessary compiler barrier for bpf (Konstantin)
>
> v5:
> 1. 'size' is not the parameter (Konstantin)
> 2. put () around macro parameters (Konstantin)
> 3. fix some original typo issue (Jerin)
> 4. swap 'rte_wait_event' parameter location (Jerin)
> 5. add new macro '__LOAD_EXC'
> 6. delete 'undef' to prevent compilation warning

+ David, Konstantin, Stephen,

Please make a practice to add exiting reviewers.

undef the local marco may result in conflict with other libraries.
Please add __RTE_ARM_ for existing macros (mark as internal) to fix
the namespace if we are taking that path

>
> v6:
> 1. fix patch style check warning
> 2. delete 'bpf' patch due to 'const' limit
>
> Feifei Wang (4):
>   eal: add new definitions for wait scheme
>   eal: use wait event for read pflock
>   eal: use wait event scheme for mcslock
>   lib/distributor: use wait event scheme
>
>  lib/distributor/rte_distributor_single.c |  10 +-
>  lib/eal/arm/include/rte_pause_64.h       | 136 +++++++++++++----------
>  lib/eal/include/generic/rte_mcslock.h    |   9 +-
>  lib/eal/include/generic/rte_pause.h      |  28 +++++
>  lib/eal/include/generic/rte_pflock.h     |   4 +-
>  5 files changed, 119 insertions(+), 68 deletions(-)
>
> --
> 2.25.1
>