From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 6419A48BF9; Mon, 1 Dec 2025 12:45:47 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8C2B04065E; Mon, 1 Dec 2025 12:45:12 +0100 (CET) Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.12]) by mails.dpdk.org (Postfix) with ESMTP id 2F26D40650; Mon, 1 Dec 2025 12:45:10 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1764589510; x=1796125510; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=vTeKnZ6MWQuwaWKEHlamXAilNAVP4treVH4cMNGFb4M=; b=g+8sYXwc5PzHOTj5lmgoUMET6J4OJ8vHvbU1m4AW1fvC3nXXeW36sdwz dePOQJ9jOeXU4lI15do/XpTiAx+UDkHpmFzBPDwRPyruiLjiWdCnYluJv I0cr32RuJYrfS8U+Y3E1ibdCrQiW0OgVn6bioI8OUgHiQJDZPAfY9cxfL 304ntFjv5M5aKqXBHJ09zTLFNOoebrgKTc8kv0VqWMWgGSJIlLMVgQ2tW jweTzh5n1PIsiIoAmm35sJe5w/dPKOM/1XcPa2y2VvoAwsMuxCOPmKIGV fLU9KA7RpTo4flb6fOxK2jT16a+Uz0Cc6zmT4g8TaJhn8uk/dh2IjxX36 A==; X-CSE-ConnectionGUID: Y2aKtzxyRYWqHr9gpeNpNg== X-CSE-MsgGUID: OecNRm3OQnuc0Crkht2lHg== X-IronPort-AV: E=McAfee;i="6800,10657,11629"; a="77991705" X-IronPort-AV: E=Sophos;i="6.20,240,1758610800"; d="scan'208";a="77991705" Received: from fmviesa003.fm.intel.com ([10.60.135.143]) by orvoesa104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Dec 2025 03:45:09 -0800 X-CSE-ConnectionGUID: o6YE6XdDSyqtRheplePBNw== X-CSE-MsgGUID: qkWPPNzETMG8Ui86S5+4oA== X-ExtLoop1: 1 Received: from silpixa00401385.ir.intel.com ([10.20.224.226]) by fmviesa003.fm.intel.com with ESMTP; 01 Dec 2025 03:45:08 -0800 From: Bruce Richardson To: dev@dpdk.org Cc: Bruce Richardson , stable@dpdk.org, Stephen Hemminger Subject: [PATCH v3 06/31] pipeline: fix variable shadowing Date: Mon, 1 Dec 2025 11:44:23 +0000 Message-ID: <20251201114448.1441377-7-bruce.richardson@intel.com> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20251201114448.1441377-1-bruce.richardson@intel.com> References: <20251106140948.2894678-1-bruce.richardson@intel.com> <20251201114448.1441377-1-bruce.richardson@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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Remove unnecessary inner variables and rename others in order to remove variable shadowing warnings from the code. Fixes: ea5ab65f5743 ("pipeline: relax table match field requirements") Fixes: cdaa937d3eaa ("pipeline: support selector table") Fixes: 5f3e6104227c ("pipeline: prepare for variable size headers") Fixes: 724f3ef422e9 ("pipeline: generate custom instruction functions") Fixes: 68b95704a6a3 ("pipeline: add API for shared library-based pipeline build") Cc: stable@dpdk.org Signed-off-by: Bruce Richardson Acked-by: Stephen Hemminger --- lib/pipeline/rte_swx_ctl.c | 11 +++++------ lib/pipeline/rte_swx_pipeline.c | 21 ++++++++++----------- lib/pipeline/rte_swx_pipeline_internal.h | 10 +++++----- 3 files changed, 20 insertions(+), 22 deletions(-) diff --git a/lib/pipeline/rte_swx_ctl.c b/lib/pipeline/rte_swx_ctl.c index 4e9bb842a1..d24764cbbe 100644 --- a/lib/pipeline/rte_swx_ctl.c +++ b/lib/pipeline/rte_swx_ctl.c @@ -210,7 +210,7 @@ table_params_get(struct rte_swx_ctl_pipeline *ctl, uint32_t table_id) uint32_t key_size = 0, key_offset = 0, action_data_size = 0, i; if (table->info.n_match_fields) { - uint32_t n_match_fields_em = 0, i; + uint32_t n_match_fields_em = 0; /* Find first (smallest offset) and last (biggest offset) match fields. */ first = &table->mf[0]; @@ -855,22 +855,21 @@ selector_free(struct rte_swx_ctl_pipeline *ctl) for (i = 0; i < ctl->info.n_selectors; i++) { struct selector *s = &ctl->selectors[i]; - uint32_t i; /* selector_fields. */ free(s->selector_fields); /* groups. */ if (s->groups) - for (i = 0; i < s->info.n_groups_max; i++) - selector_group_members_free(s, i); + for (uint32_t j = 0; j < s->info.n_groups_max; j++) + selector_group_members_free(s, j); free(s->groups); /* pending_groups. */ if (s->pending_groups) - for (i = 0; i < s->info.n_groups_max; i++) - selector_pending_group_members_free(s, i); + for (uint32_t j = 0; j < s->info.n_groups_max; j++) + selector_pending_group_members_free(s, j); free(s->pending_groups); diff --git a/lib/pipeline/rte_swx_pipeline.c b/lib/pipeline/rte_swx_pipeline.c index 2193bc4ebf..a9157815e4 100644 --- a/lib/pipeline/rte_swx_pipeline.c +++ b/lib/pipeline/rte_swx_pipeline.c @@ -1515,10 +1515,10 @@ static int header_build(struct rte_swx_pipeline *p) { struct header *h; - uint32_t n_bytes = 0, i; + uint32_t total_bytes = 0, i; TAILQ_FOREACH(h, &p->headers, node) { - n_bytes += h->st->n_bits / 8; + total_bytes += h->st->n_bits / 8; } for (i = 0; i < RTE_SWX_PIPELINE_THREADS_MAX; i++) { @@ -1533,10 +1533,10 @@ header_build(struct rte_swx_pipeline *p) sizeof(struct header_out_runtime)); CHECK(t->headers_out, ENOMEM); - t->header_storage = calloc(1, n_bytes); + t->header_storage = calloc(1, total_bytes); CHECK(t->header_storage, ENOMEM); - t->header_out_storage = calloc(1, n_bytes); + t->header_out_storage = calloc(1, total_bytes); CHECK(t->header_out_storage, ENOMEM); TAILQ_FOREACH(h, &p->headers, node) { @@ -14031,7 +14031,6 @@ instruction_group_list_create(struct rte_swx_pipeline *p) for (i = 0; i < p->n_instructions; i++) { struct instruction_data *data = &p->instruction_data[i]; - struct instruction_group *g; uint32_t j; /* Continue when the current instruction is not a jump destination. */ @@ -14746,11 +14745,11 @@ rte_swx_pipeline_build_from_lib(struct rte_swx_pipeline **pipeline, /* Action instructions. */ TAILQ_FOREACH(a, &p->actions, node) { - char name[RTE_SWX_NAME_SIZE * 2]; + char action_name[RTE_SWX_NAME_SIZE * 2]; - snprintf(name, sizeof(name), "action_%s_run", a->name); + snprintf(action_name, sizeof(action_name), "action_%s_run", a->name); - p->action_funcs[a->id] = dlsym(lib, name); + p->action_funcs[a->id] = dlsym(lib, action_name); if (!p->action_funcs[a->id]) { status = -EINVAL; goto free; @@ -14765,14 +14764,14 @@ rte_swx_pipeline_build_from_lib(struct rte_swx_pipeline **pipeline, } TAILQ_FOREACH(g, igl, node) { - char name[RTE_SWX_NAME_SIZE * 2]; + char pipeline_name[RTE_SWX_NAME_SIZE * 2]; if (g->first_instr_id == g->last_instr_id) continue; - snprintf(name, sizeof(name), "pipeline_func_%u", g->group_id); + snprintf(pipeline_name, sizeof(pipeline_name), "pipeline_func_%u", g->group_id); - g->func = dlsym(lib, name); + g->func = dlsym(lib, pipeline_name); if (!g->func) { status = -EINVAL; goto free; diff --git a/lib/pipeline/rte_swx_pipeline_internal.h b/lib/pipeline/rte_swx_pipeline_internal.h index df864ea066..f527857861 100644 --- a/lib/pipeline/rte_swx_pipeline_internal.h +++ b/lib/pipeline/rte_swx_pipeline_internal.h @@ -1772,14 +1772,14 @@ emit_handler(struct thread *t) if ((t->n_headers_out == 2) && (h1->ptr + h1->n_bytes == t->ptr) && (h0->ptr == h0->ptr0)) { - uint32_t offset; + uint32_t ofs; TRACE("Emit handler: header encapsulation.\n"); - offset = h0->n_bytes + h1->n_bytes; - memcpy(t->ptr - offset, h0->ptr, h0->n_bytes); - t->pkt.offset -= offset; - t->pkt.length += offset; + ofs = h0->n_bytes + h1->n_bytes; + memcpy(t->ptr - ofs, h0->ptr, h0->n_bytes); + t->pkt.offset -= ofs; + t->pkt.length += ofs; return; } -- 2.51.0