Message ID | 1600244628-13062-1-git-send-email-phil.yang@arm.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 78B26A04C7; Wed, 16 Sep 2020 10:24:10 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C8C581C1D5; Wed, 16 Sep 2020 10:24:09 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id 9CDFC1C1CC for <dev@dpdk.org>; Wed, 16 Sep 2020 10:24:07 +0200 (CEST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 13E43101E; Wed, 16 Sep 2020 01:24:07 -0700 (PDT) Received: from phil-VirtualBox.shanghai.arm.com (phil-VirtualBox.shanghai.arm.com [10.169.182.49]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 1EE873F718; Wed, 16 Sep 2020 01:24:04 -0700 (PDT) From: Phil Yang <phil.yang@arm.com> To: david.marchand@redhat.com, dev@dpdk.org Cc: Honnappa.Nagarahalli@arm.com, Ruifeng.Wang@arm.com, nd@arm.com Date: Wed, 16 Sep 2020 16:23:44 +0800 Message-Id: <1600244628-13062-1-git-send-email-phil.yang@arm.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1599794967-17500-1-git-send-email-phil.yang@arm.com> References: <1599794967-17500-1-git-send-email-phil.yang@arm.com> Subject: [dpdk-dev] [PATCH v2 0/4] use C11 atomic builtins for libs 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 |
use C11 atomic builtins for libs
|
|
Message
Phil Yang
Sept. 16, 2020, 8:23 a.m. UTC
Since rte_atomicXX APIs are not allowed to be used[1][2], use C11 atomic builtins instead in eal, bbdev, power, and ethdev libs. [1] http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecation.rst#L87 [2] http://code.dpdk.org/dpdk/latest/source/devtools/checkpatches.sh#L80 v2: Fix Clang int-conversion warning. v1: Initial version. Phil Yang (4): eal: use C11 atomic builtins for already initialized check bbdev: use C11 atomic builtins for device processing counter power: use C11 atomic builtins for power in use state update ethdev: use C11 atomic builtins for link status update lib/librte_bbdev/rte_bbdev.c | 5 ++-- lib/librte_bbdev/rte_bbdev.h | 4 +-- lib/librte_eal/freebsd/eal.c | 18 +++++++------ lib/librte_eal/linux/eal.c | 20 ++++++++------- lib/librte_ethdev/rte_ethdev_driver.h | 19 ++++---------- lib/librte_power/power_acpi_cpufreq.c | 45 +++++++++++++++++++++++++-------- lib/librte_power/power_pstate_cpufreq.c | 45 +++++++++++++++++++++++++-------- 7 files changed, 100 insertions(+), 56 deletions(-)
Comments
On Wed, Sep 16, 2020 at 10:24 AM Phil Yang <phil.yang@arm.com> wrote: > > Since rte_atomicXX APIs are not allowed to be used[1][2], use C11 atomic > builtins instead in eal, bbdev, power, and ethdev libs. > > [1] http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecation.rst#L87 > [2] http://code.dpdk.org/dpdk/latest/source/devtools/checkpatches.sh#L80 > > v2: > Fix Clang int-conversion warning. > > v1: > Initial version. > > Phil Yang (4): > eal: use C11 atomic builtins for already initialized check > bbdev: use C11 atomic builtins for device processing counter > power: use C11 atomic builtins for power in use state update > ethdev: use C11 atomic builtins for link status update Thanks Phil. Just a small comment on the first patch, the rest lgtm. Nicolas, David H., could you have a look at (resp.) patch 2, 3?
David Marchand <david.marchand@redhat.com> writes: > Subject: Re: [PATCH v2 0/4] use C11 atomic builtins for libs > > On Wed, Sep 16, 2020 at 10:24 AM Phil Yang <phil.yang@arm.com> wrote: > > > > Since rte_atomicXX APIs are not allowed to be used[1][2], use C11 atomic > > builtins instead in eal, bbdev, power, and ethdev libs. > > > > [1] > http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecatio > n.rst#L87 > > [2] > http://code.dpdk.org/dpdk/latest/source/devtools/checkpatches.sh#L80 > > > > v2: > > Fix Clang int-conversion warning. > > > > v1: > > Initial version. > > > > Phil Yang (4): > > eal: use C11 atomic builtins for already initialized check > > bbdev: use C11 atomic builtins for device processing counter > > power: use C11 atomic builtins for power in use state update > > ethdev: use C11 atomic builtins for link status update > > Thanks Phil. > Just a small comment on the first patch, the rest lgtm. > > Nicolas, David H., could you have a look at (resp.) patch 2, 3? Thanks for your comments. I will address them in the next version. Thanks, Phil > > > -- > David Marchand