Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/129979/?format=api
{ "id": 129979, "url": "http://patchwork.dpdk.org/api/patches/129979/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20230809001017.293037-15-stephen@networkplumber.org/", "project": { "id": 1, "url": "http://patchwork.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20230809001017.293037-15-stephen@networkplumber.org>", "list_archive_url": "https://inbox.dpdk.org/dev/20230809001017.293037-15-stephen@networkplumber.org", "date": "2023-08-09T00:10:02", "name": "[v2,14/29] dmadev: mark API's as not experimental", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "27204fdec111af77533c42ebdaae930e4828f25d", "submitter": { "id": 27, "url": "http://patchwork.dpdk.org/api/people/27/?format=api", "name": "Stephen Hemminger", "email": "stephen@networkplumber.org" }, "delegate": { "id": 1, "url": "http://patchwork.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20230809001017.293037-15-stephen@networkplumber.org/mbox/", "series": [ { "id": 29134, "url": "http://patchwork.dpdk.org/api/series/29134/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=29134", "date": "2023-08-09T00:09:48", "name": "promote many API's to stable", "version": 2, "mbox": "http://patchwork.dpdk.org/series/29134/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/129979/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/129979/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "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])\n\tby inbox.dpdk.org (Postfix) with ESMTP id C140143011;\n\tWed, 9 Aug 2023 02:12:07 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 47C64432A4;\n\tWed, 9 Aug 2023 02:10:48 +0200 (CEST)", "from mail-pl1-f182.google.com (mail-pl1-f182.google.com\n [209.85.214.182])\n by mails.dpdk.org (Postfix) with ESMTP id 43C8743289\n for <dev@dpdk.org>; Wed, 9 Aug 2023 02:10:43 +0200 (CEST)", "by mail-pl1-f182.google.com with SMTP id\n d9443c01a7336-1bba04b9df3so54679395ad.0\n for <dev@dpdk.org>; Tue, 08 Aug 2023 17:10:43 -0700 (PDT)", "from hermes.local (204-195-127-207.wavecable.com. [204.195.127.207])\n by smtp.gmail.com with ESMTPSA id\n ix11-20020a170902f80b00b001bc676df6a9sm5745629plb.132.2023.08.08.17.10.41\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Tue, 08 Aug 2023 17:10:41 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=networkplumber-org.20221208.gappssmtp.com; s=20221208; t=1691539842;\n x=1692144642;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=gs3UWff/60uqgyNk8yw83kWn/jT1eKYtU8H4VdScDAg=;\n b=kAGWXDQcZ/JQPs7WNMOSpFaIHvHFSo+85w4ngFcJlA8jKAz5CG948TgqZMmy8yL1dS\n J6uOxMiEHVf+y63+VfIVIuWq0IuORVBMkMuJ9Jic13t3Aayl2pfADRRSCwbweXQX35ie\n cogvuHQeAluCO7EXA2R84ouYgDZh+j1v9y3nYAGZOYEuwj88CdNBdlSWsbXB5A1Mo1Az\n ANGRvkCvvjv45c5enIsrBQXP7lNaAM25OWO2uvV1AvBEa3uCErobnuwNvjC/CO6CqrBk\n Fnf6CAuQYEPJcMEIZiM9HEh2g83RmCvVY+KJKtnl5fhVqkoteBo5uoc7mha2dzipr9GM\n 3/hA==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20221208; t=1691539842; x=1692144642;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=gs3UWff/60uqgyNk8yw83kWn/jT1eKYtU8H4VdScDAg=;\n b=l37Mts3KByQOrmVvcWJhV78m8Wg8AeU21g4vels+nqhNUCSzSvgUtpoqK9n5LcMCsg\n xR2GpBr7eIpn2xEzDcm2RiRwfG8cqS2iQ/Fzl4Y+g0In1VvUX9bLx4HkOOGCfrwsgQK/\n rF/eA4sdeBew6PEp/lgA31CjGLnfQPE4+e46KI3ZRU/wK3vxqKPqAKLrm32h8q+KGMPV\n X3UocNdfGtPl9np0Ol6I2Bwa2S8qAAgl34pf/svBjl4qoFOhmkCdo2mqfu7XGSdh1qIL\n j0VwbmS3KrWvf2Jo9JkkZYilNPSON/xrb3MHlhHEpxUHJhRvnsfp2nQiF1loOYFBniyp\n TNzg==", "X-Gm-Message-State": "AOJu0Yz5rg/E+P0UEMB5Ghp1Jnd+2yFp9cuEL2a1cT08aNJNFtekDaC7\n HkaQQHUnrU6MD7FGRV5PxlJnGWvQYbtmDkLM1YwSKA==", "X-Google-Smtp-Source": "\n AGHT+IHICkK33c2KgUOpXx59DqWn0SGe3JQpapM3gtlpBt+aSgPxO7rhajNraX7eAoxEVia81VuOJg==", "X-Received": "by 2002:a17:902:d492:b0:1bc:2f17:c628 with SMTP id\n c18-20020a170902d49200b001bc2f17c628mr1275189plg.56.1691539842096;\n Tue, 08 Aug 2023 17:10:42 -0700 (PDT)", "From": "Stephen Hemminger <stephen@networkplumber.org>", "To": "dev@dpdk.org", "Cc": "Stephen Hemminger <stephen@networkplumber.org>,\n Thomas Monjalon <thomas@monjalon.net>,\n Chengwen Feng <fengchengwen@huawei.com>,\n Kevin Laatz <kevin.laatz@intel.com>,\n Bruce Richardson <bruce.richardson@intel.com>", "Subject": "[PATCH v2 14/29] dmadev: mark API's as not experimental", "Date": "Tue, 8 Aug 2023 17:10:02 -0700", "Message-Id": "<20230809001017.293037-15-stephen@networkplumber.org>", "X-Mailer": "git-send-email 2.39.2", "In-Reply-To": "<20230809001017.293037-1-stephen@networkplumber.org>", "References": "<20230808173527.186042-1-stephen@networkplumber.org>\n <20230809001017.293037-1-stephen@networkplumber.org>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "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>,\n <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>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "These were added in 20.11 time now to remove experimental flag.\n\nSigned-off-by: Stephen Hemminger <stephen@networkplumber.org>\n---\n MAINTAINERS | 2 +-\n lib/dmadev/rte_dmadev.h | 85 -----------------------------------------\n lib/dmadev/version.map | 2 +-\n 3 files changed, 2 insertions(+), 87 deletions(-)", "diff": "diff --git a/MAINTAINERS b/MAINTAINERS\nindex 462223dd65d0..f020972b609b 100644\n--- a/MAINTAINERS\n+++ b/MAINTAINERS\n@@ -482,7 +482,7 @@ F: doc/guides/prog_guide/mldev.rst\n F: app/test-mldev/\n F: doc/guides/tools/testmldev.rst\n \n-DMA device API - EXPERIMENTAL\n+DMA device API\n M: Chengwen Feng <fengchengwen@huawei.com>\n F: lib/dmadev/\n F: drivers/dma/skeleton/\ndiff --git a/lib/dmadev/rte_dmadev.h b/lib/dmadev/rte_dmadev.h\nindex e61d71959e82..ecc98517e852 100644\n--- a/lib/dmadev/rte_dmadev.h\n+++ b/lib/dmadev/rte_dmadev.h\n@@ -158,9 +158,6 @@ extern \"C\" {\n #define RTE_DMADEV_DEFAULT_MAX 64\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Configure the maximum number of dmadevs.\n * @note This function can be invoked before the primary process rte_eal_init()\n * to change the maximum number of dmadevs. If not invoked, the maximum number\n@@ -172,13 +169,9 @@ extern \"C\" {\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_dev_max(size_t dev_max);\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Get the device identifier for the named DMA device.\n *\n * @param name\n@@ -188,13 +181,9 @@ int rte_dma_dev_max(size_t dev_max);\n * Returns DMA device identifier on success.\n * - <0: Failure to find named DMA device.\n */\n-__rte_experimental\n int rte_dma_get_dev_id_by_name(const char *name);\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Check whether the dev_id is valid.\n *\n * @param dev_id\n@@ -203,20 +192,15 @@ int rte_dma_get_dev_id_by_name(const char *name);\n * @return\n * - If the device index is valid (true) or not (false).\n */\n-__rte_experimental\n bool rte_dma_is_valid(int16_t dev_id);\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Get the total number of DMA devices that have been successfully\n * initialised.\n *\n * @return\n * The total number of usable DMA devices.\n */\n-__rte_experimental\n uint16_t rte_dma_count_avail(void);\n \n /**\n@@ -227,7 +211,6 @@ uint16_t rte_dma_count_avail(void);\n * @return\n * Next valid dmadev, UINT16_MAX if there is none.\n */\n-__rte_experimental\n int16_t rte_dma_next_dev(int16_t start_dev_id);\n \n /** Utility macro to iterate over all available dmadevs */\n@@ -310,9 +293,6 @@ struct rte_dma_info {\n };\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Retrieve information of a DMA device.\n *\n * @param dev_id\n@@ -324,7 +304,6 @@ struct rte_dma_info {\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_info_get(int16_t dev_id, struct rte_dma_info *dev_info);\n \n /**\n@@ -349,9 +328,6 @@ struct rte_dma_conf {\n };\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Configure a DMA device.\n *\n * This function must be invoked first before any other function in the\n@@ -367,13 +343,9 @@ struct rte_dma_conf {\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_configure(int16_t dev_id, const struct rte_dma_conf *dev_conf);\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Start a DMA device.\n *\n * The device start step is the last one and consists of setting the DMA\n@@ -385,13 +357,9 @@ int rte_dma_configure(int16_t dev_id, const struct rte_dma_conf *dev_conf);\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_start(int16_t dev_id);\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Stop a DMA device.\n *\n * The device can be restarted with a call to rte_dma_start().\n@@ -402,13 +370,9 @@ int rte_dma_start(int16_t dev_id);\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_stop(int16_t dev_id);\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Close a DMA device.\n *\n * The device cannot be restarted after this call.\n@@ -419,7 +383,6 @@ int rte_dma_stop(int16_t dev_id);\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_close(int16_t dev_id);\n \n /**\n@@ -585,9 +548,6 @@ struct rte_dma_vchan_conf {\n };\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Allocate and set up a virtual DMA channel.\n *\n * @param dev_id\n@@ -602,7 +562,6 @@ struct rte_dma_vchan_conf {\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_vchan_setup(int16_t dev_id, uint16_t vchan,\n \t\t\tconst struct rte_dma_vchan_conf *conf);\n \n@@ -631,9 +590,6 @@ struct rte_dma_stats {\n #define RTE_DMA_ALL_VCHAN\t0xFFFFu\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Retrieve basic statistics of a or all virtual DMA channel(s).\n *\n * @param dev_id\n@@ -648,14 +604,10 @@ struct rte_dma_stats {\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_stats_get(int16_t dev_id, uint16_t vchan,\n \t\t struct rte_dma_stats *stats);\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Reset basic statistics of a or all virtual DMA channel(s).\n *\n * @param dev_id\n@@ -667,7 +619,6 @@ int rte_dma_stats_get(int16_t dev_id, uint16_t vchan,\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_stats_reset(int16_t dev_id, uint16_t vchan);\n \n /**\n@@ -683,9 +634,6 @@ enum rte_dma_vchan_status {\n };\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Determine if all jobs have completed on a device channel.\n * This function is primarily designed for testing use, as it allows a process to check if\n * all jobs are completed, without actually gathering completions from those jobs.\n@@ -700,14 +648,10 @@ enum rte_dma_vchan_status {\n * 0 - call completed successfully\n * < 0 - error code indicating there was a problem calling the API\n */\n-__rte_experimental\n int\n rte_dma_vchan_status(int16_t dev_id, uint16_t vchan, enum rte_dma_vchan_status *status);\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Dump DMA device info.\n *\n * @param dev_id\n@@ -718,7 +662,6 @@ rte_dma_vchan_status(int16_t dev_id, uint16_t vchan, enum rte_dma_vchan_status *\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n int rte_dma_dump(int16_t dev_id, FILE *f);\n \n /**\n@@ -822,9 +765,6 @@ struct rte_dma_sge {\n /**@}*/\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Enqueue a copy operation onto the virtual DMA channel.\n *\n * This queues up a copy operation to be performed by hardware, if the 'flags'\n@@ -850,7 +790,6 @@ struct rte_dma_sge {\n * - -ENOSPC: if no space left to enqueue.\n * - other values < 0 on failure.\n */\n-__rte_experimental\n static inline int\n rte_dma_copy(int16_t dev_id, uint16_t vchan, rte_iova_t src, rte_iova_t dst,\n \t uint32_t length, uint64_t flags)\n@@ -868,9 +807,6 @@ rte_dma_copy(int16_t dev_id, uint16_t vchan, rte_iova_t src, rte_iova_t dst,\n }\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Enqueue a scatter-gather list copy operation onto the virtual DMA channel.\n *\n * This queues up a scatter-gather list copy operation to be performed by\n@@ -900,7 +836,6 @@ rte_dma_copy(int16_t dev_id, uint16_t vchan, rte_iova_t src, rte_iova_t dst,\n * - -ENOSPC: if no space left to enqueue.\n * - other values < 0 on failure.\n */\n-__rte_experimental\n static inline int\n rte_dma_copy_sg(int16_t dev_id, uint16_t vchan, struct rte_dma_sge *src,\n \t\tstruct rte_dma_sge *dst, uint16_t nb_src, uint16_t nb_dst,\n@@ -921,9 +856,6 @@ rte_dma_copy_sg(int16_t dev_id, uint16_t vchan, struct rte_dma_sge *src,\n }\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Enqueue a fill operation onto the virtual DMA channel.\n *\n * This queues up a fill operation to be performed by hardware, if the 'flags'\n@@ -949,7 +881,6 @@ rte_dma_copy_sg(int16_t dev_id, uint16_t vchan, struct rte_dma_sge *src,\n * - -ENOSPC: if no space left to enqueue.\n * - other values < 0 on failure.\n */\n-__rte_experimental\n static inline int\n rte_dma_fill(int16_t dev_id, uint16_t vchan, uint64_t pattern,\n \t rte_iova_t dst, uint32_t length, uint64_t flags)\n@@ -968,9 +899,6 @@ rte_dma_fill(int16_t dev_id, uint16_t vchan, uint64_t pattern,\n }\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Trigger hardware to begin performing enqueued operations.\n *\n * Writes the \"doorbell\" to the hardware to trigger it\n@@ -984,7 +912,6 @@ rte_dma_fill(int16_t dev_id, uint16_t vchan, uint64_t pattern,\n * @return\n * 0 on success. Otherwise negative value is returned.\n */\n-__rte_experimental\n static inline int\n rte_dma_submit(int16_t dev_id, uint16_t vchan)\n {\n@@ -1001,9 +928,6 @@ rte_dma_submit(int16_t dev_id, uint16_t vchan)\n }\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Return the number of operations that have been successfully completed.\n * Once an operation has been reported as completed, the results of that\n * operation will be visible to all cores on the system.\n@@ -1025,7 +949,6 @@ rte_dma_submit(int16_t dev_id, uint16_t vchan)\n * The number of operations that successfully completed. This return value\n * must be less than or equal to the value of nb_cpls.\n */\n-__rte_experimental\n static inline uint16_t\n rte_dma_completed(int16_t dev_id, uint16_t vchan, const uint16_t nb_cpls,\n \t\t uint16_t *last_idx, bool *has_error)\n@@ -1060,9 +983,6 @@ rte_dma_completed(int16_t dev_id, uint16_t vchan, const uint16_t nb_cpls,\n }\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Return the number of operations that have been completed, and the operations\n * result may succeed or fail.\n * Once an operation has been reported as completed successfully, the results of that\n@@ -1088,7 +1008,6 @@ rte_dma_completed(int16_t dev_id, uint16_t vchan, const uint16_t nb_cpls,\n * If this number is greater than zero (assuming n), then n values in the\n * status array are also set.\n */\n-__rte_experimental\n static inline uint16_t\n rte_dma_completed_status(int16_t dev_id, uint16_t vchan,\n \t\t\t const uint16_t nb_cpls, uint16_t *last_idx,\n@@ -1112,9 +1031,6 @@ rte_dma_completed_status(int16_t dev_id, uint16_t vchan,\n }\n \n /**\n- * @warning\n- * @b EXPERIMENTAL: this API may change without prior notice.\n- *\n * Check remaining capacity in descriptor ring for the current burst.\n *\n * @param dev_id\n@@ -1126,7 +1042,6 @@ rte_dma_completed_status(int16_t dev_id, uint16_t vchan,\n * - Remaining space in the descriptor ring for the current burst.\n * - 0 on error\n */\n-__rte_experimental\n static inline uint16_t\n rte_dma_burst_capacity(int16_t dev_id, uint16_t vchan)\n {\ndiff --git a/lib/dmadev/version.map b/lib/dmadev/version.map\nindex 7031d6b335e9..2a3736514c8f 100644\n--- a/lib/dmadev/version.map\n+++ b/lib/dmadev/version.map\n@@ -1,4 +1,4 @@\n-EXPERIMENTAL {\n+DPDK_24 {\n \tglobal:\n \n \trte_dma_close;\n", "prefixes": [ "v2", "14/29" ] }