From patchwork Fri Oct 20 21:41:13 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 133112 X-Patchwork-Delegate: david.marchand@redhat.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 395C9431BE; Fri, 20 Oct 2023 23:42:33 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1C35A42D7D; Fri, 20 Oct 2023 23:41:45 +0200 (CEST) Received: from mail-pf1-f175.google.com (mail-pf1-f175.google.com [209.85.210.175]) by mails.dpdk.org (Postfix) with ESMTP id 13B6540EF0 for ; Fri, 20 Oct 2023 23:41:39 +0200 (CEST) Received: by mail-pf1-f175.google.com with SMTP id d2e1a72fcca58-6be0277c05bso1136368b3a.0 for ; Fri, 20 Oct 2023 14:41:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1697838098; x=1698442898; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=XChDISV/dXS65avk97hsI2n1WXEiMyYioiE/QjgOQzc=; b=taaELtamsUzjPBo9k/pQKeqycdsh5dpLeg/thoxSDKnDrFR1ebvgBE9iSQ+5bROUsh mj6pCc8vOvDxh73JzHOP7OfzeSPUpI/JPM7LLs6SlNLMSDFsmhOozNfIR/RiDvOhFO4a oQPk0nK/OIyduJ/lDL0jJCnk9sxsjs0r9KctPEiSXLGAubtksmlDGfULw0AYl70gaY5B Ls8ljcjHTPEoNmfStR/u2ORizIe67740SZEycYwVvvRcFTFa5szGVWov2lmJZ36kdS3u RzJ0yf40+4TEtosmpNORfoMj8/QZ3YI4GCTngUWgybkcH8mlyYXSUjlpqVbC7dmAhJr4 wVcw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697838098; x=1698442898; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=XChDISV/dXS65avk97hsI2n1WXEiMyYioiE/QjgOQzc=; b=lgOLArt5tC5zjWs4IR6i/UYPLQxER3Xlu0+cp3EB6UBgQLLMnARzc6Pcvki44BKet1 kQiwUjCJVRpy1QVQjroU+lfcMhtUNT0omWZcCj7rt5hc1pB3pcvCwYlRp7oVzeyeCK01 P1lpYl3XLvhrgWU6PioDRdcGn1b9z4ljEf7XgBVgFwEAtZCmbou3QxRhlhvOlNOfeh/+ 2SW2ip5G4dW+tExcCDG9irJSBV+dmEdjfhRHF8gXEQD/AnYTE8X46ewtGZd8Nt58zApq 7Hzt2F2cdFwOEO9yR79kDDS/WZC/r27pblI9QbpsOH8rOP4QkVXvb2M7uMRQ4bfrBzrX RWrg== X-Gm-Message-State: AOJu0YzwJ7jsJrv+2OSaoBOyCMsqatK6kXeMB0+7eB5ptEYXCN1SSr/Z Y+Ixfs3T6iSALVjU48V5knbHkbMZznNLu6xR1hI45Q== X-Google-Smtp-Source: AGHT+IGwaKdtSHhl5Dpx4y5fiaMtxHBIEMe8Cpo+gPycl8H2PStnuZL3fMNYLca7XRX5UdDaIMMfOg== X-Received: by 2002:a05:6a21:a5a3:b0:16b:f3b1:c040 with SMTP id gd35-20020a056a21a5a300b0016bf3b1c040mr3952658pzc.38.1697838098000; Fri, 20 Oct 2023 14:41:38 -0700 (PDT) Received: from hermes.local (204-195-126-68.wavecable.com. [204.195.126.68]) by smtp.gmail.com with ESMTPSA id 19-20020a631753000000b005894450b404sm1886543pgx.63.2023.10.20.14.41.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 20 Oct 2023 14:41:37 -0700 (PDT) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , =?utf-8?q?Mattias_R?= =?utf-8?q?=C3=B6nnblom?= Subject: [PATCH v2 08/14] eal: make seqcount and seqlock stable Date: Fri, 20 Oct 2023 14:41:13 -0700 Message-Id: <20231020214119.255491-9-stephen@networkplumber.org> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20231020214119.255491-1-stephen@networkplumber.org> References: <20230809164312.308093-1-stephen@networkplumber.org> <20231020214119.255491-1-stephen@networkplumber.org> MIME-Version: 1.0 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 These were add back in 22.07 release. Signed-off-by: Stephen Hemminger Acked-by: Mattias Rönnblom Acked-by: Mattias Rönnblom --- lib/eal/include/rte_seqcount.h | 23 ----------------------- lib/eal/include/rte_seqlock.h | 21 --------------------- 2 files changed, 44 deletions(-) diff --git a/lib/eal/include/rte_seqcount.h b/lib/eal/include/rte_seqcount.h index 4f9cefb33862..ff8985310377 100644 --- a/lib/eal/include/rte_seqcount.h +++ b/lib/eal/include/rte_seqcount.h @@ -41,15 +41,11 @@ typedef struct { #define RTE_SEQCOUNT_INITIALIZER { .sn = 0 } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * Initialize the sequence counter. * * @param seqcount * A pointer to the sequence counter. */ -__rte_experimental static inline void rte_seqcount_init(rte_seqcount_t *seqcount) { @@ -57,9 +53,6 @@ rte_seqcount_init(rte_seqcount_t *seqcount) } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * Begin a read-side critical section. * * A call to this function marks the beginning of a read-side critical @@ -101,8 +94,6 @@ rte_seqcount_init(rte_seqcount_t *seqcount) * * @see rte_seqcount_read_retry() */ - -__rte_experimental static inline uint32_t rte_seqcount_read_begin(const rte_seqcount_t *seqcount) { @@ -114,9 +105,6 @@ rte_seqcount_read_begin(const rte_seqcount_t *seqcount) } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * End a read-side critical section. * * A call to this function marks the end of a read-side critical @@ -146,8 +134,6 @@ rte_seqcount_read_begin(const rte_seqcount_t *seqcount) * * @see rte_seqcount_read_begin() */ - -__rte_experimental static inline bool rte_seqcount_read_retry(const rte_seqcount_t *seqcount, uint32_t begin_sn) { @@ -172,9 +158,6 @@ rte_seqcount_read_retry(const rte_seqcount_t *seqcount, uint32_t begin_sn) } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * Begin a write-side critical section. * * A call to this function marks the beginning of a write-side @@ -196,8 +179,6 @@ rte_seqcount_read_retry(const rte_seqcount_t *seqcount, uint32_t begin_sn) * * @see rte_seqcount_write_end() */ - -__rte_experimental static inline void rte_seqcount_write_begin(rte_seqcount_t *seqcount) { @@ -214,9 +195,6 @@ rte_seqcount_write_begin(rte_seqcount_t *seqcount) } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * End a write-side critical section. * * A call to this function marks the end of the write-side critical @@ -228,7 +206,6 @@ rte_seqcount_write_begin(rte_seqcount_t *seqcount) * * @see rte_seqcount_write_begin() */ -__rte_experimental static inline void rte_seqcount_write_end(rte_seqcount_t *seqcount) { diff --git a/lib/eal/include/rte_seqlock.h b/lib/eal/include/rte_seqlock.h index fcbb9c586668..589c98188529 100644 --- a/lib/eal/include/rte_seqlock.h +++ b/lib/eal/include/rte_seqlock.h @@ -114,9 +114,6 @@ typedef struct { } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * Initialize the seqlock. * * This function initializes the seqlock, and leaves the writer-side @@ -125,7 +122,6 @@ typedef struct { * @param seqlock * A pointer to the seqlock. */ -__rte_experimental static inline void rte_seqlock_init(rte_seqlock_t *seqlock) { @@ -134,9 +130,6 @@ rte_seqlock_init(rte_seqlock_t *seqlock) } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * Begin a read-side critical section. * * See rte_seqcount_read_retry() for details. @@ -150,8 +143,6 @@ rte_seqlock_init(rte_seqlock_t *seqlock) * @see rte_seqlock_read_retry() * @see rte_seqcount_read_retry() */ - -__rte_experimental static inline uint32_t rte_seqlock_read_begin(const rte_seqlock_t *seqlock) { @@ -159,9 +150,6 @@ rte_seqlock_read_begin(const rte_seqlock_t *seqlock) } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * End a read-side critical section. * * See rte_seqcount_read_retry() for details. @@ -177,7 +165,6 @@ rte_seqlock_read_begin(const rte_seqlock_t *seqlock) * * @see rte_seqlock_read_begin() */ -__rte_experimental static inline bool rte_seqlock_read_retry(const rte_seqlock_t *seqlock, uint32_t begin_sn) { @@ -185,9 +172,6 @@ rte_seqlock_read_retry(const rte_seqlock_t *seqlock, uint32_t begin_sn) } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * Begin a write-side critical section. * * A call to this function acquires the write lock associated @p @@ -212,7 +196,6 @@ rte_seqlock_read_retry(const rte_seqlock_t *seqlock, uint32_t begin_sn) * * @see rte_seqlock_write_unlock() */ -__rte_experimental static inline void rte_seqlock_write_lock(rte_seqlock_t *seqlock) __rte_exclusive_lock_function(&seqlock->lock) @@ -224,9 +207,6 @@ rte_seqlock_write_lock(rte_seqlock_t *seqlock) } /** - * @warning - * @b EXPERIMENTAL: this API may change without prior notice. - * * End a write-side critical section. * * A call to this function marks the end of the write-side critical @@ -238,7 +218,6 @@ rte_seqlock_write_lock(rte_seqlock_t *seqlock) * * @see rte_seqlock_write_lock() */ -__rte_experimental static inline void rte_seqlock_write_unlock(rte_seqlock_t *seqlock) __rte_unlock_function(&seqlock->lock)