From patchwork Tue Oct 24 02:54:19 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 133256 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 2E245431EB; Tue, 24 Oct 2023 04:56:53 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 84E5642E4F; Tue, 24 Oct 2023 04:55:05 +0200 (CEST) Received: from mail-pf1-f180.google.com (mail-pf1-f180.google.com [209.85.210.180]) by mails.dpdk.org (Postfix) with ESMTP id 975C242E24 for ; Tue, 24 Oct 2023 04:54:54 +0200 (CEST) Received: by mail-pf1-f180.google.com with SMTP id d2e1a72fcca58-6b77ab73c6fso2903078b3a.1 for ; Mon, 23 Oct 2023 19:54:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1698116093; x=1698720893; 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=2JBJD6+tpGubJY4wOvnGIPe0s+cCrgMYtTGRA+MdC3I=; b=k9pNvrfZuDbQ3t2rY2qnKS6eCwn3yyjGtDn19Ey0+IkWtcGAnE3lJAMUK1H3ls0gYc oqxx9hGs7PEs82hruPHonv17EyEoN+wWd+nM5vTCfiJuqqXjwaRpf6LnHi5A4vs9aB3X 9YO7BtjbUZoGtJCNkVJWLQ0Tf7VMUP5HMGvQZ95xPxqFwOJPzfMP2Hx+jA2o9J/f75KZ Ue891nj66v70baGfluVAXyMf0HOie4NJ4WJneA0CzkFCsdNtC7NJ24BiMG82QDOTX/PG yub5g9uOtZbuBqkiD9lC6mHjSlZkkgaY2ffufkBSSiCzKiMmWjSVL416iabX1pLT5MgI 6UqA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1698116093; x=1698720893; 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=2JBJD6+tpGubJY4wOvnGIPe0s+cCrgMYtTGRA+MdC3I=; b=MFAqfOa1/gL6xYNfXZ0yPgMFIUJZOb+X0e8Avucnr0VJbZ/lgoyaMvAYK5ZNp89cqp MUI69hGv1/RoIQDQViEoKf+QyHHn3cCJ2BmcCZJgRseWfsklUxDY6TcXyA1BeeudkzCN R/q7rKekxbvFRGaosoRfbsCBipA91Csdeiaiy44CV0ZXUDG5+iiC6cec+58c2wpZkil7 j9pTrPw5P6HW6GdKrb5jYntOOvLrzE9RvowI+KO3d9BomEARtw+3DtgNqgWGxsSqz5bM QBsZXJDdDRSnBDhhJ7I7uPBmFIrfAzzldFafIp6DebV4xkn8Yl6NkKutjZUAlLcVbh5w NROA== X-Gm-Message-State: AOJu0YyYznZ0C5LiIwiHac/VhQOqZ5eqBLfQGFMsAkhCEFTaSm+IRDLI 7sFQJD9XTP9XcJgUw2VwkcrIIQ0ULuOYmkHXcpDYRA== X-Google-Smtp-Source: AGHT+IEl4xtq0i65t+yExcCBWNac3br+zhp9zyUKGPo2j5Uiid5SJMFDt9Db7vsC4sLU7/0lWJ9PTQ== X-Received: by 2002:a05:6a20:9187:b0:17a:eff5:fbbe with SMTP id v7-20020a056a20918700b0017aeff5fbbemr1873686pzd.8.1698116093574; Mon, 23 Oct 2023 19:54:53 -0700 (PDT) Received: from hermes.local (204-195-126-68.wavecable.com. [204.195.126.68]) by smtp.gmail.com with ESMTPSA id b22-20020aa79516000000b0069b772c4325sm6802348pfp.87.2023.10.23.19.54.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 23 Oct 2023 19:54:52 -0700 (PDT) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Bruce Richardson , Thomas Monjalon , Fan Zhang , Ashish Gupta Subject: [PATCH v7 22/25] compressdev: remove experimental flag Date: Mon, 23 Oct 2023 19:54:19 -0700 Message-Id: <20231024025422.126021-23-stephen@networkplumber.org> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20231024025422.126021-1-stephen@networkplumber.org> References: <20230808173527.186042-1-stephen@networkplumber.org> <20231024025422.126021-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 The compressdev was added back in 2018. Remove the experimental flag and make ABI stable. Signed-off-by: Stephen Hemminger Acked-by: Bruce Richardson --- MAINTAINERS | 2 +- lib/compressdev/rte_comp.h | 6 ------ lib/compressdev/rte_compressdev.h | 26 -------------------------- lib/compressdev/rte_compressdev_pmd.h | 6 ------ lib/compressdev/version.map | 2 +- 5 files changed, 2 insertions(+), 40 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 2f1ab5a0f328..789507ddccb7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -459,7 +459,7 @@ F: lib/security/ F: doc/guides/prog_guide/rte_security.rst F: app/test/test_security* -Compression API - EXPERIMENTAL +Compression API M: Fan Zhang M: Ashish Gupta T: git://dpdk.org/next/dpdk-next-crypto diff --git a/lib/compressdev/rte_comp.h b/lib/compressdev/rte_comp.h index bf896d07223c..232564cf5e9a 100644 --- a/lib/compressdev/rte_comp.h +++ b/lib/compressdev/rte_comp.h @@ -499,7 +499,6 @@ struct rte_comp_op { * - On success pointer to mempool * - On failure NULL */ -__rte_experimental struct rte_mempool * rte_comp_op_pool_create(const char *name, unsigned int nb_elts, unsigned int cache_size, @@ -515,7 +514,6 @@ rte_comp_op_pool_create(const char *name, * - On success returns a valid rte_comp_op structure * - On failure returns NULL */ -__rte_experimental struct rte_comp_op * rte_comp_op_alloc(struct rte_mempool *mempool); @@ -532,7 +530,6 @@ rte_comp_op_alloc(struct rte_mempool *mempool); * - nb_ops: Success, the nb_ops requested was allocated * - 0: Not enough entries in the mempool; no ops are retrieved. */ -__rte_experimental int rte_comp_op_bulk_alloc(struct rte_mempool *mempool, struct rte_comp_op **ops, uint16_t nb_ops); @@ -546,7 +543,6 @@ rte_comp_op_bulk_alloc(struct rte_mempool *mempool, * Compress operation pointer allocated from rte_comp_op_alloc() * If op is NULL, no operation is performed. */ -__rte_experimental void rte_comp_op_free(struct rte_comp_op *op); @@ -561,7 +557,6 @@ rte_comp_op_free(struct rte_comp_op *op); * @param nb_ops * Number of operations to free */ -__rte_experimental void rte_comp_op_bulk_free(struct rte_comp_op **ops, uint16_t nb_ops); @@ -574,7 +569,6 @@ rte_comp_op_bulk_free(struct rte_comp_op **ops, uint16_t nb_ops); * @return * The name of this flag, or NULL if it's not a valid feature flag. */ -__rte_experimental const char * rte_comp_get_feature_name(uint64_t flag); diff --git a/lib/compressdev/rte_compressdev.h b/lib/compressdev/rte_compressdev.h index 13a418631893..8cb5db0e3f7d 100644 --- a/lib/compressdev/rte_compressdev.h +++ b/lib/compressdev/rte_compressdev.h @@ -10,10 +10,6 @@ * * RTE Compression Device APIs. * - * @warning - * @b EXPERIMENTAL: - * All functions in this file may be changed or removed without prior notice. - * * Defines comp device APIs for the provisioning of compression operations. */ @@ -54,7 +50,6 @@ struct rte_compressdev_capabilities { #define RTE_COMP_END_OF_CAPABILITIES_LIST() \ { RTE_COMP_ALGO_UNSPECIFIED } -__rte_experimental const struct rte_compressdev_capabilities * rte_compressdev_capability_get(uint8_t dev_id, enum rte_comp_algorithm algo); @@ -94,7 +89,6 @@ rte_compressdev_capability_get(uint8_t dev_id, * @return * The name of this flag, or NULL if it's not a valid feature flag. */ -__rte_experimental const char * rte_compressdev_get_feature_name(uint64_t flag); @@ -133,7 +127,6 @@ struct rte_compressdev_stats { * - Returns compress device identifier on success. * - Return -1 on failure to find named compress device. */ -__rte_experimental int rte_compressdev_get_dev_id(const char *name); @@ -146,7 +139,6 @@ rte_compressdev_get_dev_id(const char *name); * - Returns compress device name. * - Returns NULL if compress device is not present. */ -__rte_experimental const char * rte_compressdev_name_get(uint8_t dev_id); @@ -157,7 +149,6 @@ rte_compressdev_name_get(uint8_t dev_id); * @return * - The total number of usable compress devices. */ -__rte_experimental uint8_t rte_compressdev_count(void); @@ -175,7 +166,6 @@ rte_compressdev_count(void); * @return * Returns number of attached compress devices. */ -__rte_experimental uint8_t rte_compressdev_devices_get(const char *driver_name, uint8_t *devices, uint8_t nb_devices); @@ -190,7 +180,6 @@ rte_compressdev_devices_get(const char *driver_name, uint8_t *devices, * a default of zero if the socket could not be determined. * -1 if returned is the dev_id value is out of range. */ -__rte_experimental int rte_compressdev_socket_id(uint8_t dev_id); @@ -221,7 +210,6 @@ struct rte_compressdev_config { * - 0: Success, device configured. * - <0: Error code returned by the driver configuration function. */ -__rte_experimental int rte_compressdev_configure(uint8_t dev_id, struct rte_compressdev_config *config); @@ -240,7 +228,6 @@ rte_compressdev_configure(uint8_t dev_id, * - 0: Success, device started. * - <0: Error code of the driver device start function. */ -__rte_experimental int rte_compressdev_start(uint8_t dev_id); @@ -251,7 +238,6 @@ rte_compressdev_start(uint8_t dev_id); * @param dev_id * Compress device identifier */ -__rte_experimental void rte_compressdev_stop(uint8_t dev_id); @@ -269,7 +255,6 @@ rte_compressdev_stop(uint8_t dev_id); * - 0 on successfully closing device * - <0 on failure to close device */ -__rte_experimental int rte_compressdev_close(uint8_t dev_id); @@ -296,7 +281,6 @@ rte_compressdev_close(uint8_t dev_id); * - 0: Success, queue pair correctly set up. * - <0: Queue pair configuration failed */ -__rte_experimental int rte_compressdev_queue_pair_setup(uint8_t dev_id, uint16_t queue_pair_id, uint32_t max_inflight_ops, int socket_id); @@ -309,7 +293,6 @@ rte_compressdev_queue_pair_setup(uint8_t dev_id, uint16_t queue_pair_id, * @return * - The number of configured queue pairs. */ -__rte_experimental uint16_t rte_compressdev_queue_pair_count(uint8_t dev_id); @@ -327,7 +310,6 @@ rte_compressdev_queue_pair_count(uint8_t dev_id); * - Zero if successful. * - Non-zero otherwise. */ -__rte_experimental int rte_compressdev_stats_get(uint8_t dev_id, struct rte_compressdev_stats *stats); @@ -337,7 +319,6 @@ rte_compressdev_stats_get(uint8_t dev_id, struct rte_compressdev_stats *stats); * @param dev_id * The identifier of the device. */ -__rte_experimental void rte_compressdev_stats_reset(uint8_t dev_id); @@ -355,7 +336,6 @@ rte_compressdev_stats_reset(uint8_t dev_id); * The element after the last valid element has it's op field set to * RTE_COMP_ALGO_UNSPECIFIED. */ -__rte_experimental void rte_compressdev_info_get(uint8_t dev_id, struct rte_compressdev_info *dev_info); @@ -413,7 +393,6 @@ rte_compressdev_info_get(uint8_t dev_id, struct rte_compressdev_info *dev_info); * of pointers to *rte_comp_op* structures effectively supplied to the * *ops* array. */ -__rte_experimental uint16_t rte_compressdev_dequeue_burst(uint8_t dev_id, uint16_t qp_id, struct rte_comp_op **ops, uint16_t nb_ops); @@ -468,7 +447,6 @@ rte_compressdev_dequeue_burst(uint8_t dev_id, uint16_t qp_id, * comp devices queue is full or if invalid parameters are specified in * a *rte_comp_op*. */ -__rte_experimental uint16_t rte_compressdev_enqueue_burst(uint8_t dev_id, uint16_t qp_id, struct rte_comp_op **ops, uint16_t nb_ops); @@ -496,7 +474,6 @@ rte_compressdev_enqueue_burst(uint8_t dev_id, uint16_t qp_id, * - Returns -ENOTSUP if comp device does not support the comp transform. * - Returns -ENOMEM if the private stream could not be allocated. */ -__rte_experimental int rte_compressdev_stream_create(uint8_t dev_id, const struct rte_comp_xform *xform, @@ -518,7 +495,6 @@ rte_compressdev_stream_create(uint8_t dev_id, * - Returns -ENOTSUP if comp device does not support STATEFUL operations. * - Returns -EBUSY if can't free stream as there are inflight operations */ -__rte_experimental int rte_compressdev_stream_free(uint8_t dev_id, void *stream); @@ -545,7 +521,6 @@ rte_compressdev_stream_free(uint8_t dev_id, void *stream); * - Returns -ENOTSUP if comp device does not support the comp transform. * - Returns -ENOMEM if the private_xform could not be allocated. */ -__rte_experimental int rte_compressdev_private_xform_create(uint8_t dev_id, const struct rte_comp_xform *xform, @@ -567,7 +542,6 @@ rte_compressdev_private_xform_create(uint8_t dev_id, * - <0 in error cases * - Returns -EINVAL if input parameters are invalid. */ -__rte_experimental int rte_compressdev_private_xform_free(uint8_t dev_id, void *private_xform); diff --git a/lib/compressdev/rte_compressdev_pmd.h b/lib/compressdev/rte_compressdev_pmd.h index ea012908b783..fa233492fe1f 100644 --- a/lib/compressdev/rte_compressdev_pmd.h +++ b/lib/compressdev/rte_compressdev_pmd.h @@ -59,7 +59,6 @@ struct rte_compressdev_global { * @return * - The rte_compressdev structure pointer for the given device identifier. */ -__rte_experimental struct rte_compressdev * rte_compressdev_pmd_get_named_dev(const char *name); @@ -292,7 +291,6 @@ struct rte_compressdev_ops { * @return * - Slot in the rte_dev_devices array for a new device; */ -__rte_experimental struct rte_compressdev * rte_compressdev_pmd_allocate(const char *name, int socket_id); @@ -308,7 +306,6 @@ rte_compressdev_pmd_allocate(const char *name, int socket_id); * @return * - 0 on success, negative on error */ -__rte_experimental int rte_compressdev_pmd_release_device(struct rte_compressdev *dev); @@ -331,7 +328,6 @@ rte_compressdev_pmd_release_device(struct rte_compressdev *dev); * - 0 on success * - errno on failure */ -__rte_experimental int rte_compressdev_pmd_parse_input_args( struct rte_compressdev_pmd_init_params *params, @@ -353,7 +349,6 @@ rte_compressdev_pmd_parse_input_args( * - comp device instance on success * - NULL on creation failure */ -__rte_experimental struct rte_compressdev * rte_compressdev_pmd_create(const char *name, struct rte_device *device, @@ -372,7 +367,6 @@ rte_compressdev_pmd_create(const char *name, * - 0 on success * - errno on failure */ -__rte_experimental int rte_compressdev_pmd_destroy(struct rte_compressdev *dev); diff --git a/lib/compressdev/version.map b/lib/compressdev/version.map index e2a108b6509f..fa891f669b5d 100644 --- a/lib/compressdev/version.map +++ b/lib/compressdev/version.map @@ -1,4 +1,4 @@ -EXPERIMENTAL { +DPDK_24 { global: rte_compressdev_capability_get;