Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/132241/?format=api
http://patchwork.dpdk.org/api/patches/132241/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20230929181328.104311-6-hernan.vargas@intel.com/", "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": "<20230929181328.104311-6-hernan.vargas@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230929181328.104311-6-hernan.vargas@intel.com", "date": "2023-09-29T18:13:22", "name": "[v1,05/11] test/bbdev: improve test log messages", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "c107933bd11c2ef04649a7a11f55457a6ae0dc2d", "submitter": { "id": 2659, "url": "http://patchwork.dpdk.org/api/people/2659/?format=api", "name": "Hernan Vargas", "email": "hernan.vargas@intel.com" }, "delegate": { "id": 2642, "url": "http://patchwork.dpdk.org/api/users/2642/?format=api", "username": "mcoquelin", "first_name": "Maxime", "last_name": "Coquelin", "email": "maxime.coquelin@redhat.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20230929181328.104311-6-hernan.vargas@intel.com/mbox/", "series": [ { "id": 29705, "url": "http://patchwork.dpdk.org/api/series/29705/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=29705", "date": "2023-09-29T18:13:17", "name": "test-bbdev changes for 23.11", "version": 1, "mbox": "http://patchwork.dpdk.org/series/29705/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/132241/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/132241/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 9658142677;\n\tFri, 29 Sep 2023 22:16:19 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id A291C40693;\n\tFri, 29 Sep 2023 22:15:47 +0200 (CEST)", "from mgamail.intel.com (mgamail.intel.com [192.198.163.7])\n by mails.dpdk.org (Postfix) with ESMTP id 632DA402AC\n for <dev@dpdk.org>; Fri, 29 Sep 2023 22:15:41 +0200 (CEST)", "from fmsmga002.fm.intel.com ([10.253.24.26])\n by fmvoesa101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 29 Sep 2023 11:14:02 -0700", "from unknown (HELO csl-npg-qt0.la.intel.com) ([10.233.181.103])\n by fmsmga002.fm.intel.com with ESMTP; 29 Sep 2023 11:14:01 -0700" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1696018541; x=1727554541;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=TYpA89+DUjPFLdRWyVohzbO4ydKxFLO4PDlvbSZuSDY=;\n b=cfetM6qpZVjx3SVwo9WTKWd0sTSGahPoBxZtuCGgkY3gtumX3KP8O6bK\n N4CNWaAfwzfnPzOus73+QT/1+ryHHN+BkLbaA0ZQk5IkZ6zxSaC4myQam\n 99+XXSnd9y37KfeHfX9uuIVzun9Em4D1Ahme7TzEBaWb8l7kr/A6c+9oa\n nW7Ov3CnD8uaVaZMCOBwfcKxUqYEtck2l/wRRxOuCwaBMHrGRT0wLH2/k\n 4itO1ZgkplRaI5jH1RUeDNrvaSiJSlNNetpAodOmxu7d86ZsH+bRbcE57\n xiSftcwTKNOUIHI4vimvLv5Wkn8Wm+X/dtdvs5Guzov+0XBnHDg3v03DD Q==;", "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,10848\"; a=\"3925497\"", "E=Sophos;i=\"6.03,188,1694761200\";\n d=\"scan'208\";a=\"3925497\"", "E=McAfee;i=\"6600,9927,10848\"; a=\"865777284\"", "E=Sophos;i=\"6.03,188,1694761200\"; d=\"scan'208\";a=\"865777284\"" ], "X-ExtLoop1": "1", "From": "Hernan Vargas <hernan.vargas@intel.com>", "To": "dev@dpdk.org, gakhil@marvell.com, trix@redhat.com,\n maxime.coquelin@redhat.com", "Cc": "nicolas.chautru@intel.com, qi.z.zhang@intel.com,\n Hernan Vargas <hernan.vargas@intel.com>", "Subject": "[PATCH v1 05/11] test/bbdev: improve test log messages", "Date": "Fri, 29 Sep 2023 11:13:22 -0700", "Message-Id": "<20230929181328.104311-6-hernan.vargas@intel.com>", "X-Mailer": "git-send-email 2.37.1", "In-Reply-To": "<20230929181328.104311-1-hernan.vargas@intel.com>", "References": "<20230929181328.104311-1-hernan.vargas@intel.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": "Add a print message for failure to retrieve stats on bbdev.\nAdd vector name in logs.\nRemove unnecessary prints.\nUpdate code comments and cosmetic changes.\nNo functional impact.\n\nSigned-off-by: Hernan Vargas <hernan.vargas@intel.com>\n---\n app/test-bbdev/main.c | 3 ++-\n app/test-bbdev/test_bbdev_perf.c | 26 ++++++++++++++------------\n 2 files changed, 16 insertions(+), 13 deletions(-)", "diff": "diff --git a/app/test-bbdev/main.c b/app/test-bbdev/main.c\nindex ec830eb32bc8..8f6852e2efdb 100644\n--- a/app/test-bbdev/main.c\n+++ b/app/test-bbdev/main.c\n@@ -107,7 +107,8 @@ unit_test_suite_runner(struct unit_test_suite *suite)\n \tend = rte_rdtsc_precise();\n \n \tprintf(\" + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +\\n\");\n-\tprintf(\" + Test Suite Summary : %s\\n\", suite->suite_name);\n+\tprintf(\" + Test Suite Summary : %s - %s\\n\",\n+\t\t\tsuite->suite_name, get_vector_filename());\n \tprintf(\" + Tests Total : %2d\\n\", total);\n \tprintf(\" + Tests Skipped : %2d\\n\", skipped);\n \tprintf(\" + Tests Passed : %2d\\n\", succeeded);\ndiff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c\nindex 54cb2090f988..4f8e226e587b 100644\n--- a/app/test-bbdev/test_bbdev_perf.c\n+++ b/app/test-bbdev/test_bbdev_perf.c\n@@ -721,9 +721,9 @@ add_bbdev_dev(uint8_t dev_id, struct rte_bbdev_info *info,\n \t\t\tconf.vf_dl_queues_number[i] = VF_DL_5G_QUEUE_VALUE;\n \t\t}\n \n-\t\t/* UL bandwidth. Needed for schedule algorithm */\n+\t\t/* UL bandwidth. Needed only for Vista Creek 5GNR schedule algorithm */\n \t\tconf.ul_bandwidth = UL_5G_BANDWIDTH;\n-\t\t/* DL bandwidth */\n+\t\t/* DL bandwidth. Needed only for Vista Creek 5GNR schedule algorithm */\n \t\tconf.dl_bandwidth = DL_5G_BANDWIDTH;\n \n \t\t/* UL & DL load Balance Factor to 64 */\n@@ -743,7 +743,7 @@ add_bbdev_dev(uint8_t dev_id, struct rte_bbdev_info *info,\n \t\tstruct rte_acc_conf conf;\n \t\tunsigned int i;\n \n-\t\tprintf(\"Configure ACC100/ACC101 FEC Driver %s with default values\\n\",\n+\t\tprintf(\"Configure ACC100 FEC device %s with default values\\n\",\n \t\t\t\tinfo->drv.driver_name);\n \n \t\t/* clear default configuration before initialization */\n@@ -1047,13 +1047,15 @@ ut_setup(void)\n static void\n ut_teardown(void)\n {\n-\tuint8_t i, dev_id;\n+\tuint8_t i, dev_id, ret;\n \tstruct rte_bbdev_stats stats;\n \n \tfor (i = 0; i < nb_active_devs; i++) {\n \t\tdev_id = active_devs[i].dev_id;\n \t\t/* read stats and print */\n-\t\trte_bbdev_stats_get(dev_id, &stats);\n+\t\tret = rte_bbdev_stats_get(dev_id, &stats);\n+\t\tif (ret != 0)\n+\t\t\tprintf(\"Failed to get stats on bbdev %u\\n\", dev_id);\n \t\t/* Stop the device */\n \t\trte_bbdev_stop(dev_id);\n \t}\n@@ -2227,9 +2229,11 @@ validate_op_harq_chain(struct rte_bbdev_op_data *op,\n \t\t\t\tif ((error > 8 && (abs_harq_origin <\n \t\t\t\t\t\t(llr_max - 16))) ||\n \t\t\t\t\t\t(error > 16)) {\n+\t\t\t\t\t/*\n \t\t\t\t\tprintf(\"HARQ mismatch %d: exp %d act %d => %d\\n\",\n \t\t\t\t\t\t\tj, harq_orig[j],\n \t\t\t\t\t\t\tharq_out[jj], error);\n+\t\t\t\t\t*/\n \t\t\t\t\tbyte_error++;\n \t\t\t\t\tcum_error += error;\n \t\t\t\t}\n@@ -5270,7 +5274,7 @@ offload_latency_test_fft(struct rte_mempool *mempool, struct test_buffers *bufs,\n \t\t\tburst_sz = num_to_process - dequeued;\n \n \t\tret = rte_bbdev_fft_op_alloc_bulk(mempool, ops_enq, burst_sz);\n-\t\tTEST_ASSERT_SUCCESS(ret, \"Allocation failed for %d ops\", burst_sz);\n+\t\tTEST_ASSERT_SUCCESS(ret, \"rte_bbdev_fft_op_alloc_bulk() failed\");\n \t\tif (test_vector.op_type != RTE_BBDEV_OP_NONE)\n \t\t\tcopy_reference_fft_op(ops_enq, burst_sz, dequeued,\n \t\t\t\t\tbufs->inputs,\n@@ -5352,7 +5356,7 @@ offload_latency_test_dec(struct rte_mempool *mempool, struct test_buffers *bufs,\n \t\t\tburst_sz = num_to_process - dequeued;\n \n \t\tret = rte_bbdev_dec_op_alloc_bulk(mempool, ops_enq, burst_sz);\n-\t\tTEST_ASSERT_SUCCESS(ret, \"Allocation failed for %d ops\", burst_sz);\n+\t\tTEST_ASSERT_SUCCESS(ret, \"rte_bbdev_dec_op_alloc_bulk() failed\");\n \t\tref_op->turbo_dec.iter_max = get_iter_max();\n \t\tif (test_vector.op_type != RTE_BBDEV_OP_NONE)\n \t\t\tcopy_reference_dec_op(ops_enq, burst_sz, dequeued,\n@@ -5439,7 +5443,7 @@ offload_latency_test_ldpc_dec(struct rte_mempool *mempool,\n \t\t\tburst_sz = num_to_process - dequeued;\n \n \t\tret = rte_bbdev_dec_op_alloc_bulk(mempool, ops_enq, burst_sz);\n-\t\tTEST_ASSERT_SUCCESS(ret, \"Allocation failed for %d ops\", burst_sz);\n+\t\tTEST_ASSERT_SUCCESS(ret, \"rte_bbdev_dec_op_alloc_bulk() failed\");\n \t\tref_op->ldpc_dec.iter_max = get_iter_max();\n \t\tif (test_vector.op_type != RTE_BBDEV_OP_NONE)\n \t\t\tcopy_reference_ldpc_dec_op(ops_enq, burst_sz, dequeued,\n@@ -5534,8 +5538,7 @@ offload_latency_test_enc(struct rte_mempool *mempool, struct test_buffers *bufs,\n \t\t\tburst_sz = num_to_process - dequeued;\n \n \t\tret = rte_bbdev_enc_op_alloc_bulk(mempool, ops_enq, burst_sz);\n-\t\tTEST_ASSERT_SUCCESS(ret,\n-\t\t\t\t\"rte_bbdev_enc_op_alloc_bulk() failed\");\n+\t\tTEST_ASSERT_SUCCESS(ret, \"rte_bbdev_enc_op_alloc_bulk() failed\");\n \t\tif (test_vector.op_type != RTE_BBDEV_OP_NONE)\n \t\t\tcopy_reference_enc_op(ops_enq, burst_sz, dequeued,\n \t\t\t\t\tbufs->inputs,\n@@ -5617,8 +5620,7 @@ offload_latency_test_ldpc_enc(struct rte_mempool *mempool,\n \t\t\tburst_sz = num_to_process - dequeued;\n \n \t\tret = rte_bbdev_enc_op_alloc_bulk(mempool, ops_enq, burst_sz);\n-\t\tTEST_ASSERT_SUCCESS(ret,\n-\t\t\t\t\"rte_bbdev_enc_op_alloc_bulk() failed\");\n+\t\tTEST_ASSERT_SUCCESS(ret, \"rte_bbdev_enc_op_alloc_bulk() failed\");\n \t\tif (test_vector.op_type != RTE_BBDEV_OP_NONE)\n \t\t\tcopy_reference_ldpc_enc_op(ops_enq, burst_sz, dequeued,\n \t\t\t\t\tbufs->inputs,\n", "prefixes": [ "v1", "05/11" ] }{ "id": 132241, "url": "