Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/130527/?format=api
{ "id": 130527, "url": "http://patchwork.dpdk.org/api/patches/130527/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20230818174537.290222-3-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": "<20230818174537.290222-3-stephen@networkplumber.org>", "list_archive_url": "https://inbox.dpdk.org/dev/20230818174537.290222-3-stephen@networkplumber.org", "date": "2023-08-18T17:45:26", "name": "[v2,02/13] all: remove use of word abort", "commit_ref": null, "pull_url": null, "state": "rejected", "archived": true, "hash": "0abf683b4d540ea907425e73217973c9725872c8", "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/20230818174537.290222-3-stephen@networkplumber.org/mbox/", "series": [ { "id": 29282, "url": "http://patchwork.dpdk.org/api/series/29282/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=29282", "date": "2023-08-18T17:45:24", "name": "Replace us of term abort", "version": 2, "mbox": "http://patchwork.dpdk.org/series/29282/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/130527/comments/", "check": "warning", "checks": "http://patchwork.dpdk.org/api/patches/130527/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 823EF4309A;\n\tFri, 18 Aug 2023 19:47:00 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 01A8043254;\n\tFri, 18 Aug 2023 19:46:50 +0200 (CEST)", "from mail-pf1-f178.google.com (mail-pf1-f178.google.com\n [209.85.210.178])\n by mails.dpdk.org (Postfix) with ESMTP id 342BA42FB2\n for <dev@dpdk.org>; Fri, 18 Aug 2023 19:46:47 +0200 (CEST)", "by mail-pf1-f178.google.com with SMTP id\n d2e1a72fcca58-68879c4cadeso1002049b3a.2\n for <dev@dpdk.org>; Fri, 18 Aug 2023 10:46:47 -0700 (PDT)", "from hermes.local (204-195-127-207.wavecable.com. [204.195.127.207])\n by smtp.gmail.com with ESMTPSA id\n u20-20020aa78394000000b0066a2e8431a0sm1804610pfm.183.2023.08.18.10.46.44\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Fri, 18 Aug 2023 10:46:45 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=networkplumber-org.20221208.gappssmtp.com; s=20221208; t=1692380806;\n x=1692985606;\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=BQIz7tfV4tPIpLQXnpAgEPBXJAKBAGRqsPm/CDydxT8=;\n b=LvctaVG3+QPTrqgsxav900bYrKkYg2+kzNqyCPLVu6BICbh6rVvnJZ2k8wIxU8fvjl\n ZwOyv3sTaZAlMi/mTkj4pVynE/U9k2DeC0jKyUpuEbPu7UKKQignATyToOeCHmlIJYh/\n 4H//Y+8MLL7CLRGZy+EMSAH6HAXiMTCUftNBwHYVAzyiHAiXB25rrNyMX+0CZyEIvpGL\n 7oqiUmazxgE4bPeMOwzfZd4ggcSL686TP710OFqp3wtT/z8SMZJbje1qMo9IWsT9HQGl\n nMINUyDGVe3hc2b0LSGfrV0bkcG6KpCz/knPBAOYehu1ZdW/ZHSP8ETwRi7fyhA1toUY\n zP2Q==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20221208; t=1692380806; x=1692985606;\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=BQIz7tfV4tPIpLQXnpAgEPBXJAKBAGRqsPm/CDydxT8=;\n b=hzcpFYchw6kvle7r+MCy0dBUHY0mzLV7oIwJutf3UBwPGwIys0mgKP0AbY738oBvWu\n 2uIiQ+soiyVmpfuGbde5wwomOSSPJ2v9cgaIz/oAThqlQ+5SePWcbNf9/tQ29FEO85aG\n 7RUI4W5fD4Cm88Y4+48pT88GhNTdz7U9iTN+8faJDz7alnQ7yW2jJGAUISd1J/siW0AG\n OMxVyMEu5kjWuigB+Rzti6l9wW6PaFdO7NjEa1vi4wXwtYN/1yfFEPzHSB790bSKnstB\n dRlpfnF6fTUGSNlkfBVQThxcWuU+t5KXzfZU6BN3Zl7dRXEIDsEOAgvOSv5wh//kyyrT\n FY+w==", "X-Gm-Message-State": "AOJu0YxuqIAVNg9KsaaIu0r2kX1VXJU0vnDXAWuy/OfWuOnfDyHluD4U\n 0GrrIfEdRRv3LEvRE1VnaHUCS9HDGoSWuFOjk7/JKQ==", "X-Google-Smtp-Source": "\n AGHT+IEHBS1xBIg+vK9v19T/pJBNkwM3Fi0dir+nym3lc+iK2CMVqg/tQL42gb8pw2MWOR3C2Z6Y9g==", "X-Received": "by 2002:a05:6a00:24c6:b0:687:2f1e:156a with SMTP id\n d6-20020a056a0024c600b006872f1e156amr3522306pfv.5.1692380805746;\n Fri, 18 Aug 2023 10:46:45 -0700 (PDT)", "From": "Stephen Hemminger <stephen@networkplumber.org>", "To": "dev@dpdk.org", "Cc": "Stephen Hemminger <stephen@networkplumber.org>,\n Chengwen Feng <fengchengwen@huawei.com>, Kevin Laatz <kevin.laatz@intel.com>,\n Bruce Richardson <bruce.richardson@intel.com>,\n Konstantin Ananyev <konstantin.v.ananyev@yandex.ru>,\n Ciara Loftus <ciara.loftus@intel.com>, Qi Zhang <qi.z.zhang@intel.com>,\n Radu Nicolau <radu.nicolau@intel.com>, Akhil Goyal <gakhil@marvell.com>,\n\t=?utf-8?q?Mattias_R=C3=B6nnblom?= <mattias.ronnblom@ericsson.com>,\n Cristian Dumitrescu <cristian.dumitrescu@intel.com>,\n Thomas Monjalon <thomas@monjalon.net>, Ferruh Yigit <ferruh.yigit@amd.com>,\n Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>,\n Yipeng Wang <yipeng1.wang@intel.com>,\n Sameh Gobriel <sameh.gobriel@intel.com>,\n Vladimir Medvedkin <vladimir.medvedkin@intel.com>,\n Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>", "Subject": "[PATCH v2 02/13] all: remove use of word abort", "Date": "Fri, 18 Aug 2023 10:45:26 -0700", "Message-Id": "<20230818174537.290222-3-stephen@networkplumber.org>", "X-Mailer": "git-send-email 2.39.2", "In-Reply-To": "<20230818174537.290222-1-stephen@networkplumber.org>", "References": "<20230818025249.33994-1-stephen@networkplumber.org>\n <20230818174537.290222-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": "The term abort is on the Tier 1 \"replace immediately\" list.\n\nSigned-off-by: Stephen Hemminger <stephen@networkplumber.org>\n---\n .../comp_perf_test_cyclecount.c | 2 +-\n .../comp_perf_test_throughput.c | 2 +-\n app/test-compress-perf/comp_perf_test_verify.c | 2 +-\n app/test/test_dmadev.c | 2 +-\n doc/guides/prog_guide/bpf_lib.rst | 2 +-\n doc/guides/prog_guide/traffic_management.rst | 2 +-\n drivers/dma/hisilicon/hisi_dmadev.c | 4 ++--\n drivers/dma/hisilicon/hisi_dmadev.h | 2 +-\n drivers/net/af_xdp/rte_eth_af_xdp.c | 2 +-\n examples/ipsec-secgw/rt.c | 4 ++--\n lib/dmadev/rte_dmadev.h | 6 +++---\n lib/eal/include/generic/rte_power_intrinsics.h | 10 +++++-----\n lib/eal/include/generic/rte_rwlock.h | 4 ++--\n lib/eal/include/generic/rte_spinlock.h | 8 ++++----\n lib/eal/include/rte_seqlock.h | 2 +-\n lib/eal/x86/include/rte_rtm.h | 16 ++++++++--------\n lib/eal/x86/include/rte_spinlock.h | 12 ++++++------\n lib/eal/x86/rte_power_intrinsics.c | 8 ++++----\n lib/ethdev/rte_tm.h | 2 +-\n lib/hash/rte_cuckoo_hash.h | 2 +-\n lib/ring/rte_ring_peek_elem_pvt.h | 4 ++--\n 21 files changed, 49 insertions(+), 49 deletions(-)", "diff": "diff --git a/app/test-compress-perf/comp_perf_test_cyclecount.c b/app/test-compress-perf/comp_perf_test_cyclecount.c\nindex 4d336ec8d699..810db5704cc6 100644\n--- a/app/test-compress-perf/comp_perf_test_cyclecount.c\n+++ b/app/test-compress-perf/comp_perf_test_cyclecount.c\n@@ -461,7 +461,7 @@ main_loop(struct cperf_cyclecount_ctx *ctx, enum rte_comp_xform_type type)\n \n \tif (test_data->perf_comp_force_stop) {\n \t\tRTE_LOG(ERR, USER1,\n-\t\t \"lcore: %d Perf. test has been aborted by user\\n\",\n+\t\t \"lcore: %d Perf. test has been canceled by user\\n\",\n \t\t\tmem->lcore_id);\n \t\tres = -1;\n \t}\ndiff --git a/app/test-compress-perf/comp_perf_test_throughput.c b/app/test-compress-perf/comp_perf_test_throughput.c\nindex 1f7072d22383..6f27fdf826e2 100644\n--- a/app/test-compress-perf/comp_perf_test_throughput.c\n+++ b/app/test-compress-perf/comp_perf_test_throughput.c\n@@ -323,7 +323,7 @@ main_loop(struct cperf_benchmark_ctx *ctx, enum rte_comp_xform_type type)\n \n \tif (test_data->perf_comp_force_stop) {\n \t\tRTE_LOG(ERR, USER1,\n-\t\t \"lcore: %d Perf. test has been aborted by user\\n\",\n+\t\t \"lcore: %d Perf. test has been canceled by user\\n\",\n \t\t\tmem->lcore_id);\n \t\tres = -1;\n \t}\ndiff --git a/app/test-compress-perf/comp_perf_test_verify.c b/app/test-compress-perf/comp_perf_test_verify.c\nindex 7bd18073cff4..541c8d88c5ae 100644\n--- a/app/test-compress-perf/comp_perf_test_verify.c\n+++ b/app/test-compress-perf/comp_perf_test_verify.c\n@@ -382,7 +382,7 @@ main_loop(struct cperf_verify_ctx *ctx, enum rte_comp_xform_type type)\n \n \tif (test_data->perf_comp_force_stop) {\n \t\tRTE_LOG(ERR, USER1,\n-\t\t \"lcore: %d Perf. test has been aborted by user\\n\",\n+\t\t \"lcore: %d Perf. test has been canceled by user\\n\",\n \t\t\tmem->lcore_id);\n \t\tres = -1;\n \t}\ndiff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c\nindex 0736ff2a1867..e6ff3e5b4ec7 100644\n--- a/app/test/test_dmadev.c\n+++ b/app/test/test_dmadev.c\n@@ -57,7 +57,7 @@ runtest(const char *printable, int (*test_fn)(int16_t dev_id, uint16_t vchan), i\n \t\tif (stats.completed != stats.submitted)\n \t\t\tERR_RETURN(\"\\nError, not all submitted jobs are reported as completed\\n\");\n \t\tif (check_err_stats && stats.errors != 0)\n-\t\t\tERR_RETURN(\"\\nErrors reported during op processing, aborting tests\\n\");\n+\t\t\tERR_RETURN(\"\\nErrors reported during op processing, stopping tests\\n\");\n \t}\n \tprintf(\"\\n\");\n \treturn 0;\ndiff --git a/doc/guides/prog_guide/bpf_lib.rst b/doc/guides/prog_guide/bpf_lib.rst\nindex 1cf2d59429e5..17bb82313c29 100644\n--- a/doc/guides/prog_guide/bpf_lib.rst\n+++ b/doc/guides/prog_guide/bpf_lib.rst\n@@ -40,7 +40,7 @@ packet. Registers ``R1-R5`` are scratch registers\n and must not be used to store the data across these instructions.\n These instructions have implicit program exit condition as well. When\n eBPF program is trying to access the data beyond the packet boundary,\n-the interpreter will abort the execution of the program. JIT compilers\n+the interpreter will cancel the execution of the program. JIT compilers\n therefore must preserve this property. ``src_reg`` and ``imm32`` fields are\n explicit inputs to these instructions.\n For example, ``(BPF_IND | BPF_W | BPF_LD)`` means:\ndiff --git a/doc/guides/prog_guide/traffic_management.rst b/doc/guides/prog_guide/traffic_management.rst\nindex c356791a4543..c3673d3c74f2 100644\n--- a/doc/guides/prog_guide/traffic_management.rst\n+++ b/doc/guides/prog_guide/traffic_management.rst\n@@ -214,7 +214,7 @@ typically performs the following steps:\n with immediate effect once the port is started.\n \n This function fails when the currently configured hierarchy is not supported by\n-the Ethernet port, in which case the user can abort or try out another\n+the Ethernet port, in which case the user can retry with another\n hierarchy configuration (e.g. a hierarchy with less leaf nodes), which can be\n built from scratch or by modifying the existing hierarchy configuration. Note\n that this function can still fail due to other causes (e.g. not enough memory\ndiff --git a/drivers/dma/hisilicon/hisi_dmadev.c b/drivers/dma/hisilicon/hisi_dmadev.c\nindex 0e11ca14ccc3..f94e48f0f135 100644\n--- a/drivers/dma/hisilicon/hisi_dmadev.c\n+++ b/drivers/dma/hisilicon/hisi_dmadev.c\n@@ -753,8 +753,8 @@ hisi_dma_convert_status(uint16_t status)\n \t\treturn RTE_DMA_STATUS_INVALID_OPCODE;\n \tcase HISI_DMA_STATUS_INVALID_LENGTH:\n \t\treturn RTE_DMA_STATUS_INVALID_LENGTH;\n-\tcase HISI_DMA_STATUS_USER_ABORT:\n-\t\treturn RTE_DMA_STATUS_USER_ABORT;\n+\tcase HISI_DMA_STATUS_USER_CANCEL:\n+\t\treturn RTE_DMA_STATUS_USER_CANCEL;\n \tcase HISI_DMA_STATUS_REMOTE_READ_ERROR:\n \tcase HISI_DMA_STATUS_AXI_READ_ERROR:\n \t\treturn RTE_DMA_STATUS_BUS_READ_ERROR;\ndiff --git a/drivers/dma/hisilicon/hisi_dmadev.h b/drivers/dma/hisilicon/hisi_dmadev.h\nindex 5a17f9f69e25..ffb7ed3b1daf 100644\n--- a/drivers/dma/hisilicon/hisi_dmadev.h\n+++ b/drivers/dma/hisilicon/hisi_dmadev.h\n@@ -152,7 +152,7 @@ enum {\n #define HISI_DMA_STATUS_SUCCESS\t\t\t0x0\n #define HISI_DMA_STATUS_INVALID_OPCODE\t\t0x1\n #define HISI_DMA_STATUS_INVALID_LENGTH\t\t0x2\n-#define HISI_DMA_STATUS_USER_ABORT\t\t0x4\n+#define HISI_DMA_STATUS_USER_CANCEL\t\t0x4\n #define HISI_DMA_STATUS_REMOTE_READ_ERROR\t0x10\n #define HISI_DMA_STATUS_AXI_READ_ERROR\t\t0x20\n #define HISI_DMA_STATUS_AXI_WRITE_ERROR\t\t0x40\ndiff --git a/drivers/net/af_xdp/rte_eth_af_xdp.c b/drivers/net/af_xdp/rte_eth_af_xdp.c\nindex c7786cc53a5e..49ca937f3ef7 100644\n--- a/drivers/net/af_xdp/rte_eth_af_xdp.c\n+++ b/drivers/net/af_xdp/rte_eth_af_xdp.c\n@@ -776,7 +776,7 @@ eth_monitor_callback(const uint64_t value,\n \tconst uint64_t v = opaque[CLB_VAL_IDX];\n \tconst uint64_t m = (uint32_t)~0;\n \n-\t/* if the value has changed, abort entering power optimized state */\n+\t/* if the value has changed, cancel entering power optimized state */\n \treturn (value & m) == v ? 0 : -1;\n }\n \ndiff --git a/examples/ipsec-secgw/rt.c b/examples/ipsec-secgw/rt.c\nindex ce854ccb6018..eae3b086723f 100644\n--- a/examples/ipsec-secgw/rt.c\n+++ b/examples/ipsec-secgw/rt.c\n@@ -50,7 +50,7 @@ parse_rt_tokens(char **tokens, uint32_t n_tokens,\n \t\troute_ipv4 = &rt_ip4[*n_rts];\n \n \t\tAPP_CHECK(*n_rts <= RT_IPV4_MAX_RULES - 1, status,\n-\t\t\t\"too many rt rules, abort insertion\\n\");\n+\t\t\t\"too many rt rules, cancel insertion\\n\");\n \t\tif (status->status < 0)\n \t\t\treturn;\n \n@@ -59,7 +59,7 @@ parse_rt_tokens(char **tokens, uint32_t n_tokens,\n \t\troute_ipv6 = &rt_ip6[*n_rts];\n \n \t\tAPP_CHECK(*n_rts <= RT_IPV6_MAX_RULES - 1, status,\n-\t\t\t\"too many rt rules, abort insertion\\n\");\n+\t\t\t\"too many rt rules, cancel insertion\\n\");\n \t\tif (status->status < 0)\n \t\t\treturn;\n \t} else {\ndiff --git a/lib/dmadev/rte_dmadev.h b/lib/dmadev/rte_dmadev.h\nindex b157ab7600f2..f45aff1adc8b 100644\n--- a/lib/dmadev/rte_dmadev.h\n+++ b/lib/dmadev/rte_dmadev.h\n@@ -728,13 +728,13 @@ int rte_dma_dump(int16_t dev_id, FILE *f);\n enum rte_dma_status_code {\n \t/** The operation completed successfully. */\n \tRTE_DMA_STATUS_SUCCESSFUL,\n-\t/** The operation failed to complete due abort by user.\n+\t/** The operation failed to complete due being cancel by user.\n \t * This is mainly used when processing dev_stop, user could modify the\n-\t * descriptors (e.g. change one bit to tell hardware abort this job),\n+\t * descriptors (e.g. change one bit to tell hardware to cancel this job),\n \t * it allows outstanding requests to be complete as much as possible,\n \t * so reduce the time to stop the device.\n \t */\n-\tRTE_DMA_STATUS_USER_ABORT,\n+\tRTE_DMA_STATUS_USER_CANCEL,\n \t/** The operation failed to complete due to following scenarios:\n \t * The jobs in a particular batch are not attempted because they\n \t * appeared after a fence where a previous job failed. In some HW\ndiff --git a/lib/eal/include/generic/rte_power_intrinsics.h b/lib/eal/include/generic/rte_power_intrinsics.h\nindex f981df7d75a1..8de92b4cda64 100644\n--- a/lib/eal/include/generic/rte_power_intrinsics.h\n+++ b/lib/eal/include/generic/rte_power_intrinsics.h\n@@ -24,7 +24,7 @@\n /**\n * Callback definition for monitoring conditions. Callbacks with this signature\n * will be used by `rte_power_monitor()` to check if the entering of power\n- * optimized state should be aborted.\n+ * optimized state should not continue.\n *\n * @param val\n * The value read from memory.\n@@ -33,7 +33,7 @@\n *\n * @return\n * 0 if entering of power optimized state should proceed\n- * -1 if entering of power optimized state should be aborted\n+ * -1 if entering of power optimized state should not continue\n */\n typedef int (*rte_power_monitor_clb_t)(const uint64_t val,\n \t\tconst uint64_t opaque[RTE_POWER_MONITOR_OPAQUE_SZ]);\n@@ -47,7 +47,7 @@ struct rte_power_monitor_cond {\n \t */\n \trte_power_monitor_clb_t fn; /**< Callback to be used to check if\n \t * entering power optimized state should\n-\t * be aborted.\n+\t * not happen.\n \t */\n \tuint64_t opaque[RTE_POWER_MONITOR_OPAQUE_SZ];\n \t/**< Callback-specific data */\n@@ -66,7 +66,7 @@ struct rte_power_monitor_cond {\n * size (`pmc->size`) are provided in the `pmc` power monitoring condition. If\n * the mask is non-zero, the current value pointed to by the `pmc->addr` pointer\n * will be read and compared against the expected value, and if they match, the\n- * entering of optimized power state will be aborted. This is intended to\n+ * entering of optimized power state will be canceled. This is intended to\n * prevent the CPU from entering optimized power state and waiting on a write\n * that has already happened by the time this API is called.\n *\n@@ -141,7 +141,7 @@ int rte_power_pause(const uint64_t tsc_timestamp);\n * Additionally, `expected` 64-bit values and 64-bit masks are provided. If\n * mask is non-zero, the current value pointed to by the `p` pointer will be\n * checked against the expected value, and if they do not match, the entering of\n- * optimized power state may be aborted.\n+ * optimized power state may be canceled.\n *\n * @warning It is responsibility of the user to check if this function is\n * supported at runtime using `rte_cpu_get_intrinsics_support()` API call.\ndiff --git a/lib/eal/include/generic/rte_rwlock.h b/lib/eal/include/generic/rte_rwlock.h\nindex 9e083bbc61b1..c2c694ae1c77 100644\n--- a/lib/eal/include/generic/rte_rwlock.h\n+++ b/lib/eal/include/generic/rte_rwlock.h\n@@ -258,7 +258,7 @@ rte_rwlock_write_is_locked(rte_rwlock_t *rwl)\n * fails or not available take a read lock\n *\n * NOTE: An attempt to perform a HW I/O operation inside a hardware memory\n- * transaction always aborts the transaction since the CPU is not able to\n+ * transaction always cancels the transaction since the CPU is not able to\n * roll-back should the transaction fail. Therefore, hardware transactional\n * locks are not advised to be used around rte_eth_rx_burst() and\n * rte_eth_tx_burst() calls.\n@@ -285,7 +285,7 @@ rte_rwlock_read_unlock_tm(rte_rwlock_t *rwl)\n * fails or not available take a write lock\n *\n * NOTE: An attempt to perform a HW I/O operation inside a hardware memory\n- * transaction always aborts the transaction since the CPU is not able to\n+ * transaction always cancels the transaction since the CPU is not able to\n * roll-back should the transaction fail. Therefore, hardware transactional\n * locks are not advised to be used around rte_eth_rx_burst() and\n * rte_eth_tx_burst() calls.\ndiff --git a/lib/eal/include/generic/rte_spinlock.h b/lib/eal/include/generic/rte_spinlock.h\nindex c50ebaaa80fd..109c5a5adac2 100644\n--- a/lib/eal/include/generic/rte_spinlock.h\n+++ b/lib/eal/include/generic/rte_spinlock.h\n@@ -144,7 +144,7 @@ static inline int rte_tm_supported(void);\n * if it fails or not available take the spinlock.\n *\n * NOTE: An attempt to perform a HW I/O operation inside a hardware memory\n- * transaction always aborts the transaction since the CPU is not able to\n+ * transaction always cancels the transaction since the CPU is not able to\n * roll-back should the transaction fail. Therefore, hardware transactional\n * locks are not advised to be used around rte_eth_rx_burst() and\n * rte_eth_tx_burst() calls.\n@@ -172,7 +172,7 @@ rte_spinlock_unlock_tm(rte_spinlock_t *sl)\n * if it fails or not available try to take the lock.\n *\n * NOTE: An attempt to perform a HW I/O operation inside a hardware memory\n- * transaction always aborts the transaction since the CPU is not able to\n+ * transaction always cancels the transaction since the CPU is not able to\n * roll-back should the transaction fail. Therefore, hardware transactional\n * locks are not advised to be used around rte_eth_rx_burst() and\n * rte_eth_tx_burst() calls.\n@@ -277,7 +277,7 @@ static inline int rte_spinlock_recursive_trylock(rte_spinlock_recursive_t *slr)\n * if it fails or not available take the recursive spinlocks\n *\n * NOTE: An attempt to perform a HW I/O operation inside a hardware memory\n- * transaction always aborts the transaction since the CPU is not able to\n+ * transaction always cancels the transaction since the CPU is not able to\n * roll-back should the transaction fail. Therefore, hardware transactional\n * locks are not advised to be used around rte_eth_rx_burst() and\n * rte_eth_tx_burst() calls.\n@@ -303,7 +303,7 @@ static inline void rte_spinlock_recursive_unlock_tm(\n * if it fails or not available try to take the recursive lock\n *\n * NOTE: An attempt to perform a HW I/O operation inside a hardware memory\n- * transaction always aborts the transaction since the CPU is not able to\n+ * transaction always cancels the transaction since the CPU is not able to\n * roll-back should the transaction fail. Therefore, hardware transactional\n * locks are not advised to be used around rte_eth_rx_burst() and\n * rte_eth_tx_burst() calls.\ndiff --git a/lib/eal/include/rte_seqlock.h b/lib/eal/include/rte_seqlock.h\nindex fcbb9c586668..d2598fb42cb6 100644\n--- a/lib/eal/include/rte_seqlock.h\n+++ b/lib/eal/include/rte_seqlock.h\n@@ -60,7 +60,7 @@ extern \"C\" {\n * // Loads may be atomic or non-atomic, as in this example.\n * *param_x = config->param_x;\n * strcpy(param_y, config->param_y);\n- * // An alternative to an immediate retry is to abort and\n+ * // An alternative to an immediate retry is to quit and\n * // try again at some later time, assuming progress is\n * // possible without the data.\n * } while (rte_seqlock_read_retry(&config->lock, sn));\ndiff --git a/lib/eal/x86/include/rte_rtm.h b/lib/eal/x86/include/rte_rtm.h\nindex 36bf49846f08..4db9018543a9 100644\n--- a/lib/eal/x86/include/rte_rtm.h\n+++ b/lib/eal/x86/include/rte_rtm.h\n@@ -17,13 +17,13 @@ extern \"C\" {\n \n \n #define RTE_XBEGIN_STARTED\t\t(~0u)\n-#define RTE_XABORT_EXPLICIT\t\t(1 << 0)\n-#define RTE_XABORT_RETRY\t\t(1 << 1)\n-#define RTE_XABORT_CONFLICT\t\t(1 << 2)\n-#define RTE_XABORT_CAPACITY\t\t(1 << 3)\n-#define RTE_XABORT_DEBUG\t\t(1 << 4)\n-#define RTE_XABORT_NESTED\t\t(1 << 5)\n-#define RTE_XABORT_CODE(x)\t\t(((x) >> 24) & 0xff)\n+#define RTE_XCANCEL_EXPLICIT\t\t(1 << 0)\n+#define RTE_XCANCEL_RETRY\t\t(1 << 1)\n+#define RTE_XCANCEL_CONFLICT\t\t(1 << 2)\n+#define RTE_XCANCEL_CAPACITY\t\t(1 << 3)\n+#define RTE_XCANCEL_DEBUG\t\t(1 << 4)\n+#define RTE_XCANCEL_NESTED\t\t(1 << 5)\n+#define RTE_XCANCEL_CODE(x)\t\t(((x) >> 24) & 0xff)\n \n static __rte_always_inline\n unsigned int rte_xbegin(void)\n@@ -41,7 +41,7 @@ void rte_xend(void)\n }\n \n /* not an inline function to workaround a clang bug with -O0 */\n-#define rte_xabort(status) do { \\\n+#define rte_xcancel(status) do { \\\n \tasm volatile(\".byte 0xc6,0xf8,%P0\" :: \"i\" (status) : \"memory\"); \\\n } while (0)\n \ndiff --git a/lib/eal/x86/include/rte_spinlock.h b/lib/eal/x86/include/rte_spinlock.h\nindex 0b20ddfd73da..4e3a33f61ae8 100644\n--- a/lib/eal/x86/include/rte_spinlock.h\n+++ b/lib/eal/x86/include/rte_spinlock.h\n@@ -18,7 +18,7 @@ extern \"C\" {\n #include \"rte_cycles.h\"\n \n #define RTE_RTM_MAX_RETRIES (20)\n-#define RTE_XABORT_LOCK_BUSY (0xff)\n+#define RTE_XCANCEL_LOCK_BUSY (0xff)\n \n #ifndef RTE_FORCE_INTRINSICS\n static inline void\n@@ -93,16 +93,16 @@ rte_try_tm(volatile int *lock)\n \n \t\tif (likely(RTE_XBEGIN_STARTED == status)) {\n \t\t\tif (unlikely(*lock))\n-\t\t\t\trte_xabort(RTE_XABORT_LOCK_BUSY);\n+\t\t\t\trte_xcancel(RTE_XCANCEL_LOCK_BUSY);\n \t\t\telse\n \t\t\t\treturn 1;\n \t\t}\n \t\twhile (*lock)\n \t\t\trte_pause();\n \n-\t\tif ((status & RTE_XABORT_CONFLICT) ||\n-\t\t ((status & RTE_XABORT_EXPLICIT) &&\n-\t\t (RTE_XABORT_CODE(status) == RTE_XABORT_LOCK_BUSY))) {\n+\t\tif ((status & RTE_XCANCEL_CONFLICT) ||\n+\t\t ((status & RTE_XCANCEL_EXPLICIT) &&\n+\t\t (RTE_XCANCEL_CODE(status) == RTE_XCANCEL_LOCK_BUSY))) {\n \t\t\t/* add a small delay before retrying, basing the\n \t\t\t * delay on the number of times we've already tried,\n \t\t\t * to give a back-off type of behaviour. We\n@@ -116,7 +116,7 @@ rte_try_tm(volatile int *lock)\n \t\t\tcontinue;\n \t\t}\n \n-\t\tif ((status & RTE_XABORT_RETRY) == 0) /* do not retry */\n+\t\tif ((status & RTE_XCANCEL_RETRY) == 0) /* do not retry */\n \t\t\tbreak;\n \t}\n \treturn 0;\ndiff --git a/lib/eal/x86/rte_power_intrinsics.c b/lib/eal/x86/rte_power_intrinsics.c\nindex f749da9b851a..01101d4e3190 100644\n--- a/lib/eal/x86/rte_power_intrinsics.c\n+++ b/lib/eal/x86/rte_power_intrinsics.c\n@@ -118,7 +118,7 @@ rte_power_monitor(const struct rte_power_monitor_cond *pmc,\n \n \tcur_value = __get_umwait_val(pmc->addr, pmc->size);\n \n-\t/* check if callback indicates we should abort */\n+\t/* check if callback indicates we should not proceed */\n \tif (pmc->fn(cur_value, pmc->opaque) != 0)\n \t\tgoto end;\n \n@@ -242,7 +242,7 @@ rte_power_monitor_multi(const struct rte_power_monitor_cond pmc[],\n \t/* start new transaction region */\n \trc = rte_xbegin();\n \n-\t/* transaction abort, possible write to one of wait addresses */\n+\t/* transaction canceled, possible write to one of wait addresses */\n \tif (rc != RTE_XBEGIN_STARTED)\n \t\treturn 0;\n \n@@ -251,7 +251,7 @@ rte_power_monitor_multi(const struct rte_power_monitor_cond pmc[],\n \t * the read set. This means that when we trigger a wakeup from another\n \t * thread, even if we don't have a defined wakeup address and thus don't\n \t * actually cause any writes, the act of locking our lock will itself\n-\t * trigger the wakeup and abort the transaction.\n+\t * trigger the wakeup and cancel the transaction.\n \t */\n \trte_spinlock_is_locked(&s->lock);\n \n@@ -271,7 +271,7 @@ rte_power_monitor_multi(const struct rte_power_monitor_cond pmc[],\n \n \t\tconst uint64_t val = __get_umwait_val(c->addr, c->size);\n \n-\t\t/* abort if callback indicates that we need to stop */\n+\t\t/* cancel if callback indicates that we need to stop */\n \t\tif (c->fn(val, c->opaque) != 0)\n \t\t\tbreak;\n \t}\ndiff --git a/lib/ethdev/rte_tm.h b/lib/ethdev/rte_tm.h\nindex 08c5fafecdf3..8b5d477840e0 100644\n--- a/lib/ethdev/rte_tm.h\n+++ b/lib/ethdev/rte_tm.h\n@@ -1710,7 +1710,7 @@ rte_tm_node_resume(uint16_t port_id,\n * the current port, with immediate effect once the port is started.\n *\n * This function fails when the currently configured hierarchy is not supported\n- * by the Ethernet port, in which case the user can abort or try out another\n+ * by the Ethernet port, in which case the user can retry out another\n * hierarchy configuration (e.g. a hierarchy with less leaf nodes), which can be\n * build from scratch (when *clear_on_fail* is enabled) or by modifying the\n * existing hierarchy configuration (when *clear_on_fail* is disabled).\ndiff --git a/lib/hash/rte_cuckoo_hash.h b/lib/hash/rte_cuckoo_hash.h\nindex eb2644f74b1b..2bd01d133a9c 100644\n--- a/lib/hash/rte_cuckoo_hash.h\n+++ b/lib/hash/rte_cuckoo_hash.h\n@@ -124,7 +124,7 @@ const rte_hash_cmp_eq_t cmp_jump_table[NUM_KEY_CMP_CASES] = {\n \n #define RTE_HASH_BFS_QUEUE_MAX_LEN 1000\n \n-#define RTE_XABORT_CUCKOO_PATH_INVALIDED 0x4\n+#define RTE_CANCEL_CUCKOO_PATH_INVALIDED 0x4\n \n #define RTE_HASH_TSX_MAX_RETRY 10\n \ndiff --git a/lib/ring/rte_ring_peek_elem_pvt.h b/lib/ring/rte_ring_peek_elem_pvt.h\nindex bb0a7d567037..18764724be1e 100644\n--- a/lib/ring/rte_ring_peek_elem_pvt.h\n+++ b/lib/ring/rte_ring_peek_elem_pvt.h\n@@ -23,7 +23,7 @@\n * This function should be used only for single thread producer/consumer.\n * Check that user didn't request to move tail above the head.\n * In that situation:\n- * - return zero, that will cause abort any pending changes and\n+ * - return zero, that will result in canceling any pending changes and\n * return head to its previous position.\n * - throw an assert in debug mode.\n */\n@@ -67,7 +67,7 @@ __rte_ring_st_set_head_tail(struct rte_ring_headtail *ht, uint32_t tail,\n * This function should be used only for producer/consumer in MT_HTS mode.\n * Check that user didn't request to move tail above the head.\n * In that situation:\n- * - return zero, that will cause abort any pending changes and\n+ * - return zero, that will result in canceling any pending changes and\n * return head to its previous position.\n * - throw an assert in debug mode.\n */\n", "prefixes": [ "v2", "02/13" ] }