Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/104390/?format=api
https://patches.dpdk.org/api/patches/104390/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20211116094205.750359-12-joyce.kong@arm.com/", "project": { "id": 1, "url": "https://patches.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": "<20211116094205.750359-12-joyce.kong@arm.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20211116094205.750359-12-joyce.kong@arm.com", "date": "2021-11-16T09:42:04", "name": "[v2,11/12] app/bbdev: use compiler atomics for shared data sync", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "39a8ea122cacc28217effa548122b2ae676bf9f0", "submitter": { "id": 970, "url": "https://patches.dpdk.org/api/people/970/?format=api", "name": "Joyce Kong", "email": "joyce.kong@arm.com" }, "delegate": { "id": 24651, "url": "https://patches.dpdk.org/api/users/24651/?format=api", "username": "dmarchand", "first_name": "David", "last_name": "Marchand", "email": "david.marchand@redhat.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20211116094205.750359-12-joyce.kong@arm.com/mbox/", "series": [ { "id": 20583, "url": "https://patches.dpdk.org/api/series/20583/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=20583", "date": "2021-11-16T09:41:53", "name": "use compiler atomic builtins for app modules", "version": 2, "mbox": "https://patches.dpdk.org/series/20583/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/104390/comments/", "check": "success", "checks": "https://patches.dpdk.org/api/patches/104390/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 EC18CA0C43;\n\tTue, 16 Nov 2021 10:43:45 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 1089B411A4;\n\tTue, 16 Nov 2021 10:43:22 +0100 (CET)", "from foss.arm.com (foss.arm.com [217.140.110.172])\n by mails.dpdk.org (Postfix) with ESMTP id 8619A41238\n for <dev@dpdk.org>; Tue, 16 Nov 2021 10:43:20 +0100 (CET)", "from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14])\n by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 1739F1FB;\n Tue, 16 Nov 2021 01:43:20 -0800 (PST)", "from net-arm-n1amp-02.shanghai.arm.com\n (net-arm-n1amp-02.shanghai.arm.com [10.169.210.110])\n by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 049313F5A1;\n Tue, 16 Nov 2021 01:43:17 -0800 (PST)" ], "From": "Joyce Kong <joyce.kong@arm.com>", "To": "Nicolas Chautru <nicolas.chautru@intel.com>", "Cc": "dev@dpdk.org, honnappa.nagarahalli@arm.com, nd@arm.com,\n Joyce Kong <joyce.kong@arm.com>, Ruifeng Wang <ruifeng.wang@arm.com>", "Subject": "[PATCH v2 11/12] app/bbdev: use compiler atomics for shared data sync", "Date": "Tue, 16 Nov 2021 09:42:04 +0000", "Message-Id": "<20211116094205.750359-12-joyce.kong@arm.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20211116094205.750359-1-joyce.kong@arm.com>", "References": "<20211116094205.750359-1-joyce.kong@arm.com>", "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": "Convert rte_atomic usages to compiler atomic built-ins\nfor shared data sync in bbdev cases.\n\nSigned-off-by: Joyce Kong <joyce.kong@arm.com>\nReviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>\nReviewed-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>\n---\n app/test-bbdev/test_bbdev_perf.c | 135 ++++++++++++++-----------------\n 1 file changed, 59 insertions(+), 76 deletions(-)", "diff": "diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c\nindex 7b4529789b..0fa119a502 100644\n--- a/app/test-bbdev/test_bbdev_perf.c\n+++ b/app/test-bbdev/test_bbdev_perf.c\n@@ -133,7 +133,7 @@ struct test_op_params {\n \tuint16_t num_to_process;\n \tuint16_t num_lcores;\n \tint vector_mask;\n-\trte_atomic16_t sync;\n+\tuint16_t sync;\n \tstruct test_buffers q_bufs[RTE_MAX_NUMA_NODES][MAX_QUEUES];\n };\n \n@@ -148,9 +148,9 @@ struct thread_params {\n \tuint8_t iter_count;\n \tdouble iter_average;\n \tdouble bler;\n-\trte_atomic16_t nb_dequeued;\n-\trte_atomic16_t processing_status;\n-\trte_atomic16_t burst_sz;\n+\tuint16_t nb_dequeued;\n+\tint16_t processing_status;\n+\tuint16_t burst_sz;\n \tstruct test_op_params *op_params;\n \tstruct rte_bbdev_dec_op *dec_ops[MAX_BURST];\n \tstruct rte_bbdev_enc_op *enc_ops[MAX_BURST];\n@@ -2637,46 +2637,46 @@ dequeue_event_callback(uint16_t dev_id,\n \t}\n \n \tif (unlikely(event != RTE_BBDEV_EVENT_DEQUEUE)) {\n-\t\trte_atomic16_set(&tp->processing_status, TEST_FAILED);\n+\t\t__atomic_store_n(&tp->processing_status, TEST_FAILED, __ATOMIC_RELAXED);\n \t\tprintf(\n \t\t\t\"Dequeue interrupt handler called for incorrect event!\\n\");\n \t\treturn;\n \t}\n \n-\tburst_sz = rte_atomic16_read(&tp->burst_sz);\n+\tburst_sz = __atomic_load_n(&tp->burst_sz, __ATOMIC_RELAXED);\n \tnum_ops = tp->op_params->num_to_process;\n \n \tif (test_vector.op_type == RTE_BBDEV_OP_TURBO_DEC)\n \t\tdeq = rte_bbdev_dequeue_dec_ops(dev_id, queue_id,\n \t\t\t\t&tp->dec_ops[\n-\t\t\t\t\trte_atomic16_read(&tp->nb_dequeued)],\n+\t\t\t\t\t__atomic_load_n(&tp->nb_dequeued, __ATOMIC_RELAXED)],\n \t\t\t\tburst_sz);\n \telse if (test_vector.op_type == RTE_BBDEV_OP_LDPC_DEC)\n \t\tdeq = rte_bbdev_dequeue_ldpc_dec_ops(dev_id, queue_id,\n \t\t\t\t&tp->dec_ops[\n-\t\t\t\t\trte_atomic16_read(&tp->nb_dequeued)],\n+\t\t\t\t\t__atomic_load_n(&tp->nb_dequeued, __ATOMIC_RELAXED)],\n \t\t\t\tburst_sz);\n \telse if (test_vector.op_type == RTE_BBDEV_OP_LDPC_ENC)\n \t\tdeq = rte_bbdev_dequeue_ldpc_enc_ops(dev_id, queue_id,\n \t\t\t\t&tp->enc_ops[\n-\t\t\t\t\trte_atomic16_read(&tp->nb_dequeued)],\n+\t\t\t\t\t__atomic_load_n(&tp->nb_dequeued, __ATOMIC_RELAXED)],\n \t\t\t\tburst_sz);\n \telse /*RTE_BBDEV_OP_TURBO_ENC*/\n \t\tdeq = rte_bbdev_dequeue_enc_ops(dev_id, queue_id,\n \t\t\t\t&tp->enc_ops[\n-\t\t\t\t\trte_atomic16_read(&tp->nb_dequeued)],\n+\t\t\t\t\t__atomic_load_n(&tp->nb_dequeued, __ATOMIC_RELAXED)],\n \t\t\t\tburst_sz);\n \n \tif (deq < burst_sz) {\n \t\tprintf(\n \t\t\t\"After receiving the interrupt all operations should be dequeued. Expected: %u, got: %u\\n\",\n \t\t\tburst_sz, deq);\n-\t\trte_atomic16_set(&tp->processing_status, TEST_FAILED);\n+\t\t__atomic_store_n(&tp->processing_status, TEST_FAILED, __ATOMIC_RELAXED);\n \t\treturn;\n \t}\n \n-\tif (rte_atomic16_read(&tp->nb_dequeued) + deq < num_ops) {\n-\t\trte_atomic16_add(&tp->nb_dequeued, deq);\n+\tif (__atomic_load_n(&tp->nb_dequeued, __ATOMIC_RELAXED) + deq < num_ops) {\n+\t\t__atomic_fetch_add(&tp->nb_dequeued, deq, __ATOMIC_RELAXED);\n \t\treturn;\n \t}\n \n@@ -2713,7 +2713,7 @@ dequeue_event_callback(uint16_t dev_id,\n \n \tif (ret) {\n \t\tprintf(\"Buffers validation failed\\n\");\n-\t\trte_atomic16_set(&tp->processing_status, TEST_FAILED);\n+\t\t__atomic_store_n(&tp->processing_status, TEST_FAILED, __ATOMIC_RELAXED);\n \t}\n \n \tswitch (test_vector.op_type) {\n@@ -2734,7 +2734,7 @@ dequeue_event_callback(uint16_t dev_id,\n \t\tbreak;\n \tdefault:\n \t\tprintf(\"Unknown op type: %d\\n\", test_vector.op_type);\n-\t\trte_atomic16_set(&tp->processing_status, TEST_FAILED);\n+\t\t__atomic_store_n(&tp->processing_status, TEST_FAILED, __ATOMIC_RELAXED);\n \t\treturn;\n \t}\n \n@@ -2743,7 +2743,7 @@ dequeue_event_callback(uint16_t dev_id,\n \ttp->mbps += (((double)(num_ops * tb_len_bits)) / 1000000.0) /\n \t\t\t((double)total_time / (double)rte_get_tsc_hz());\n \n-\trte_atomic16_add(&tp->nb_dequeued, deq);\n+\t__atomic_fetch_add(&tp->nb_dequeued, deq, __ATOMIC_RELAXED);\n }\n \n static int\n@@ -2781,11 +2781,10 @@ throughput_intr_lcore_ldpc_dec(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\trte_atomic16_clear(&tp->processing_status);\n-\trte_atomic16_clear(&tp->nb_dequeued);\n+\t__atomic_store_n(&tp->processing_status, 0, __ATOMIC_RELAXED);\n+\t__atomic_store_n(&tp->nb_dequeued, 0, __ATOMIC_RELAXED);\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_dec_op_alloc_bulk(tp->op_params->mp, ops,\n \t\t\t\tnum_to_process);\n@@ -2833,17 +2832,15 @@ throughput_intr_lcore_ldpc_dec(void *arg)\n \t\t\t * the number of operations is not a multiple of\n \t\t\t * burst size.\n \t\t\t */\n-\t\t\trte_atomic16_set(&tp->burst_sz, num_to_enq);\n+\t\t\t__atomic_store_n(&tp->burst_sz, num_to_enq, __ATOMIC_RELAXED);\n \n \t\t\t/* Wait until processing of previous batch is\n \t\t\t * completed\n \t\t\t */\n-\t\t\twhile (rte_atomic16_read(&tp->nb_dequeued) !=\n-\t\t\t\t\t(int16_t) enqueued)\n-\t\t\t\trte_pause();\n+\t\t\trte_wait_until_equal_16(&tp->nb_dequeued, enqueued, __ATOMIC_RELAXED);\n \t\t}\n \t\tif (j != TEST_REPETITIONS - 1)\n-\t\t\trte_atomic16_clear(&tp->nb_dequeued);\n+\t\t\t__atomic_store_n(&tp->nb_dequeued, 0, __ATOMIC_RELAXED);\n \t}\n \n \treturn TEST_SUCCESS;\n@@ -2878,11 +2875,10 @@ throughput_intr_lcore_dec(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\trte_atomic16_clear(&tp->processing_status);\n-\trte_atomic16_clear(&tp->nb_dequeued);\n+\t__atomic_store_n(&tp->processing_status, 0, __ATOMIC_RELAXED);\n+\t__atomic_store_n(&tp->nb_dequeued, 0, __ATOMIC_RELAXED);\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_dec_op_alloc_bulk(tp->op_params->mp, ops,\n \t\t\t\tnum_to_process);\n@@ -2923,17 +2919,15 @@ throughput_intr_lcore_dec(void *arg)\n \t\t\t * the number of operations is not a multiple of\n \t\t\t * burst size.\n \t\t\t */\n-\t\t\trte_atomic16_set(&tp->burst_sz, num_to_enq);\n+\t\t\t__atomic_store_n(&tp->burst_sz, num_to_enq, __ATOMIC_RELAXED);\n \n \t\t\t/* Wait until processing of previous batch is\n \t\t\t * completed\n \t\t\t */\n-\t\t\twhile (rte_atomic16_read(&tp->nb_dequeued) !=\n-\t\t\t\t\t(int16_t) enqueued)\n-\t\t\t\trte_pause();\n+\t\t\trte_wait_until_equal_16(&tp->nb_dequeued, enqueued, __ATOMIC_RELAXED);\n \t\t}\n \t\tif (j != TEST_REPETITIONS - 1)\n-\t\t\trte_atomic16_clear(&tp->nb_dequeued);\n+\t\t\t__atomic_store_n(&tp->nb_dequeued, 0, __ATOMIC_RELAXED);\n \t}\n \n \treturn TEST_SUCCESS;\n@@ -2968,11 +2962,10 @@ throughput_intr_lcore_enc(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\trte_atomic16_clear(&tp->processing_status);\n-\trte_atomic16_clear(&tp->nb_dequeued);\n+\t__atomic_store_n(&tp->processing_status, 0, __ATOMIC_RELAXED);\n+\t__atomic_store_n(&tp->nb_dequeued, 0, __ATOMIC_RELAXED);\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_enc_op_alloc_bulk(tp->op_params->mp, ops,\n \t\t\tnum_to_process);\n@@ -3012,17 +3005,15 @@ throughput_intr_lcore_enc(void *arg)\n \t\t\t * the number of operations is not a multiple of\n \t\t\t * burst size.\n \t\t\t */\n-\t\t\trte_atomic16_set(&tp->burst_sz, num_to_enq);\n+\t\t\t__atomic_store_n(&tp->burst_sz, num_to_enq, __ATOMIC_RELAXED);\n \n \t\t\t/* Wait until processing of previous batch is\n \t\t\t * completed\n \t\t\t */\n-\t\t\twhile (rte_atomic16_read(&tp->nb_dequeued) !=\n-\t\t\t\t\t(int16_t) enqueued)\n-\t\t\t\trte_pause();\n+\t\t\trte_wait_until_equal_16(&tp->nb_dequeued, enqueued, __ATOMIC_RELAXED);\n \t\t}\n \t\tif (j != TEST_REPETITIONS - 1)\n-\t\t\trte_atomic16_clear(&tp->nb_dequeued);\n+\t\t\t__atomic_store_n(&tp->nb_dequeued, 0, __ATOMIC_RELAXED);\n \t}\n \n \treturn TEST_SUCCESS;\n@@ -3058,11 +3049,10 @@ throughput_intr_lcore_ldpc_enc(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\trte_atomic16_clear(&tp->processing_status);\n-\trte_atomic16_clear(&tp->nb_dequeued);\n+\t__atomic_store_n(&tp->processing_status, 0, __ATOMIC_RELAXED);\n+\t__atomic_store_n(&tp->nb_dequeued, 0, __ATOMIC_RELAXED);\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_enc_op_alloc_bulk(tp->op_params->mp, ops,\n \t\t\tnum_to_process);\n@@ -3104,17 +3094,15 @@ throughput_intr_lcore_ldpc_enc(void *arg)\n \t\t\t * the number of operations is not a multiple of\n \t\t\t * burst size.\n \t\t\t */\n-\t\t\trte_atomic16_set(&tp->burst_sz, num_to_enq);\n+\t\t\t__atomic_store_n(&tp->burst_sz, num_to_enq, __ATOMIC_RELAXED);\n \n \t\t\t/* Wait until processing of previous batch is\n \t\t\t * completed\n \t\t\t */\n-\t\t\twhile (rte_atomic16_read(&tp->nb_dequeued) !=\n-\t\t\t\t\t(int16_t) enqueued)\n-\t\t\t\trte_pause();\n+\t\t\trte_wait_until_equal_16(&tp->nb_dequeued, enqueued, __ATOMIC_RELAXED);\n \t\t}\n \t\tif (j != TEST_REPETITIONS - 1)\n-\t\t\trte_atomic16_clear(&tp->nb_dequeued);\n+\t\t\t__atomic_store_n(&tp->nb_dequeued, 0, __ATOMIC_RELAXED);\n \t}\n \n \treturn TEST_SUCCESS;\n@@ -3148,8 +3136,7 @@ throughput_pmd_lcore_dec(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_dec_op_alloc_bulk(tp->op_params->mp, ops_enq, num_ops);\n \tTEST_ASSERT_SUCCESS(ret, \"Allocation failed for %d ops\", num_ops);\n@@ -3252,8 +3239,7 @@ bler_pmd_lcore_ldpc_dec(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_dec_op_alloc_bulk(tp->op_params->mp, ops_enq, num_ops);\n \tTEST_ASSERT_SUCCESS(ret, \"Allocation failed for %d ops\", num_ops);\n@@ -3382,8 +3368,7 @@ throughput_pmd_lcore_ldpc_dec(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_dec_op_alloc_bulk(tp->op_params->mp, ops_enq, num_ops);\n \tTEST_ASSERT_SUCCESS(ret, \"Allocation failed for %d ops\", num_ops);\n@@ -3499,8 +3484,7 @@ throughput_pmd_lcore_enc(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_enc_op_alloc_bulk(tp->op_params->mp, ops_enq,\n \t\t\tnum_ops);\n@@ -3590,8 +3574,7 @@ throughput_pmd_lcore_ldpc_enc(void *arg)\n \n \tbufs = &tp->op_params->q_bufs[GET_SOCKET(info.socket_id)][queue_id];\n \n-\twhile (rte_atomic16_read(&tp->op_params->sync) == SYNC_WAIT)\n-\t\trte_pause();\n+\trte_wait_until_equal_16(&tp->op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \n \tret = rte_bbdev_enc_op_alloc_bulk(tp->op_params->mp, ops_enq,\n \t\t\tnum_ops);\n@@ -3774,7 +3757,7 @@ bler_test(struct active_device *ad,\n \telse\n \t\treturn TEST_SKIPPED;\n \n-\trte_atomic16_set(&op_params->sync, SYNC_WAIT);\n+\t__atomic_store_n(&op_params->sync, SYNC_WAIT, __ATOMIC_RELAXED);\n \n \t/* Main core is set at first entry */\n \tt_params[0].dev_id = ad->dev_id;\n@@ -3797,7 +3780,7 @@ bler_test(struct active_device *ad,\n \t\t\t\t&t_params[used_cores++], lcore_id);\n \t}\n \n-\trte_atomic16_set(&op_params->sync, SYNC_START);\n+\t__atomic_store_n(&op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \tret = bler_function(&t_params[0]);\n \n \t/* Main core is always used */\n@@ -3892,7 +3875,7 @@ throughput_test(struct active_device *ad,\n \t\t\tthroughput_function = throughput_pmd_lcore_enc;\n \t}\n \n-\trte_atomic16_set(&op_params->sync, SYNC_WAIT);\n+\t__atomic_store_n(&op_params->sync, SYNC_WAIT, __ATOMIC_RELAXED);\n \n \t/* Main core is set at first entry */\n \tt_params[0].dev_id = ad->dev_id;\n@@ -3915,7 +3898,7 @@ throughput_test(struct active_device *ad,\n \t\t\t\t&t_params[used_cores++], lcore_id);\n \t}\n \n-\trte_atomic16_set(&op_params->sync, SYNC_START);\n+\t__atomic_store_n(&op_params->sync, SYNC_START, __ATOMIC_RELAXED);\n \tret = throughput_function(&t_params[0]);\n \n \t/* Main core is always used */\n@@ -3945,29 +3928,29 @@ throughput_test(struct active_device *ad,\n \t * Wait for main lcore operations.\n \t */\n \ttp = &t_params[0];\n-\twhile ((rte_atomic16_read(&tp->nb_dequeued) <\n-\t\t\top_params->num_to_process) &&\n-\t\t\t(rte_atomic16_read(&tp->processing_status) !=\n-\t\t\tTEST_FAILED))\n+\twhile ((__atomic_load_n(&tp->nb_dequeued, __ATOMIC_RELAXED) <\n+\t\top_params->num_to_process) &&\n+\t\t(__atomic_load_n(&tp->processing_status, __ATOMIC_RELAXED) !=\n+\t\tTEST_FAILED))\n \t\trte_pause();\n \n \ttp->ops_per_sec /= TEST_REPETITIONS;\n \ttp->mbps /= TEST_REPETITIONS;\n-\tret |= (int)rte_atomic16_read(&tp->processing_status);\n+\tret |= (int)__atomic_load_n(&tp->processing_status, __ATOMIC_RELAXED);\n \n \t/* Wait for worker lcores operations */\n \tfor (used_cores = 1; used_cores < num_lcores; used_cores++) {\n \t\ttp = &t_params[used_cores];\n \n-\t\twhile ((rte_atomic16_read(&tp->nb_dequeued) <\n-\t\t\t\top_params->num_to_process) &&\n-\t\t\t\t(rte_atomic16_read(&tp->processing_status) !=\n-\t\t\t\tTEST_FAILED))\n+\t\twhile ((__atomic_load_n(&tp->nb_dequeued, __ATOMIC_RELAXED) <\n+\t\t\top_params->num_to_process) &&\n+\t\t\t(__atomic_load_n(&tp->processing_status, __ATOMIC_RELAXED) !=\n+\t\t\tTEST_FAILED))\n \t\t\trte_pause();\n \n \t\ttp->ops_per_sec /= TEST_REPETITIONS;\n \t\ttp->mbps /= TEST_REPETITIONS;\n-\t\tret |= (int)rte_atomic16_read(&tp->processing_status);\n+\t\tret |= (int)__atomic_load_n(&tp->processing_status, __ATOMIC_RELAXED);\n \t}\n \n \t/* Print throughput if test passed */\n", "prefixes": [ "v2", "11/12" ] }{ "id": 104390, "url": "