From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3326742F9C; Tue, 1 Aug 2023 09:33:08 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0DAC340A7D; Tue, 1 Aug 2023 09:33:08 +0200 (CEST) Received: from dkmailrelay1.smartsharesystems.com (smartserver.smartsharesystems.com [77.243.40.215]) by mails.dpdk.org (Postfix) with ESMTP id CE2BA400D5 for ; Tue, 1 Aug 2023 09:33:06 +0200 (CEST) Received: from smartserver.smartsharesystems.com (smartserver.smartsharesys.local [192.168.4.10]) by dkmailrelay1.smartsharesystems.com (Postfix) with ESMTP id 0C4A52085A; Tue, 1 Aug 2023 09:33:05 +0200 (CEST) Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Subject: RE: [PATCH v2 0/4] eal: update public API to use stdatomic atomics Date: Tue, 1 Aug 2023 09:33:02 +0200 X-MimeOLE: Produced By Microsoft Exchange V6.5 Message-ID: <98CBD80474FA8B44BF855DF32C47DC35D87AB3@smartserver.smartshare.dk> In-Reply-To: <1690866234-28365-1-git-send-email-roretzla@linux.microsoft.com> X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH v2 0/4] eal: update public API to use stdatomic atomics Thread-Index: AdnENZSOwxqSqgK7Rdau9A5cI6pUPAAFK4gA References: <1690837661-27573-1-git-send-email-roretzla@linux.microsoft.com> <1690866234-28365-1-git-send-email-roretzla@linux.microsoft.com> From: =?iso-8859-1?Q?Morten_Br=F8rup?= To: "Tyler Retzlaff" , Cc: "Gaetan Rivet" , "Bruce Richardson" , "Thomas Monjalon" , "Nicolas Chautru" , "Yipeng Wang" , "Sameh Gobriel" , "Vladimir Medvedkin" , "Honnappa Nagarahalli" , "Konstantin Ananyev" , "Anatoly Burakov" , "Olivier Matz" , "Andrew Rybchenko" , "Joyce Kong" , "Erik Gabriel Carrillo" , "Liang Ma" , "Peter Mccarthy" , "Jerin Jacob" , "Maciej Czekaj" , "David Hunt" , "Ruifeng Wang" , "Min Zhou" , "David Christensen" , "Stanislaw Kardach" , , X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org > From: Tyler Retzlaff [mailto:roretzla@linux.microsoft.com] > Sent: Tuesday, 1 August 2023 07.04 >=20 > Update EAL public API to use standard C11 atomics. In accordance with > technical board discussion February 2023. > http://mails.dpdk.org/archives/dev/2023-February/263516.html >=20 > This initial series makes no attempt to adapt every use of builtin = atomics > from GCC it intends only to adapt the public API of the EAL presented = as > a compatibility surface to applications. >=20 > Further series will be submitted to adapt the rest of DPDK to use = standard > atomics that are 'internal' and not do not straddle the EAL API. >=20 > The choice has been made to break these two changes apart to allow = more > precise analysis of any potential performance impact this change or > subsequent changes may introduce. >=20 > Additionally, it is not possible to break the commits up per = driver/lib as > both the provider and consumer of the APIs need to be changed in the = same > commit to maintain non-broken build and function. >=20 > Finally, the first patch in this series is a duplication of Bruce = Richardson > series https://patchwork.dpdk.org/project/dpdk/list/?series=3D29048 = and is > only being provided to enable the CI run on this series. It is = expected that > Bruce's series when committed will allow the build: require minimum = c11 > compiler > here to be dropped from this series. >=20 > v2: > * remove volatile qualification from variables, structure fields and > parameters that have been qualified _Atomic > * restore removal of #include in rte_ring_core.h the = ring > library still uses other rte_atomic APIs and types that are not = replaced > by the standard or compiler intrinsics > * add missing adaption to stdatomic in ring/rte_ring_c11_pvt.h Series-acked-by: Morten Br=F8rup