From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id 419A9FF5106 for ; Tue, 7 Apr 2026 15:19:21 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0305C40698; Tue, 7 Apr 2026 17:18:30 +0200 (CEST) Received: from mail-pg1-f178.google.com (mail-pg1-f178.google.com [209.85.215.178]) by mails.dpdk.org (Postfix) with ESMTP id 0584840698 for ; Tue, 7 Apr 2026 17:18:29 +0200 (CEST) Received: by mail-pg1-f178.google.com with SMTP id 41be03b00d2f7-c76864f4e58so2033807a12.1 for ; Tue, 07 Apr 2026 08:18:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20251104.gappssmtp.com; s=20251104; t=1775575108; x=1776179908; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=Dy/0Nu1RSKvgezVPrvrS8jOEpLCvzn7FNhwWPPP203g=; b=VgGOYI5CK1QrEQ69sOZ0S61YOwXM66+XjpsiVwieGhW8Rd9SgREBCZuZ0HOpy4Juxb tQKi71ubCzoVF+D+ShMKqpp8x40iTjUc/P7UbudajHzPzECvOcjJE62n7xzmCL5SuaSS gHazrU0QVsM07BeH2k/FnXjea8eK9/vDVElfR1SYuXsaeAvNdDFShBNCKRdGh6J2Z2NE /uPKYhyf1EHNHQC0R/8FDmEFcbABEZx2Ti141VrgWH4ax4eXMQrncVEHKU9N68OjC954 Tu5ej0GSvyKl6TcWGaDa6MU7SyiOQYTt87shryYHtEfbfphkT0EtJmNkrkI+WkmsJaXc xCOQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775575108; x=1776179908; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=Dy/0Nu1RSKvgezVPrvrS8jOEpLCvzn7FNhwWPPP203g=; b=ULDNt5vKzoIupa1fKEy+eAUIqs77XxVLkD5krc0zen7ajRdqPVGwCFtcQNPYrA/vGj hycIXeT/35o5MubM9LFCrbw1Zl0WYAnfL1Ck9WxZDZ4kSmaoSjispiyMle/U7lSTK8y5 ttlMDv9TPlHoJkOjN9fLx5L1C3Hho7gaGQYZyBnm1T6TolkedSvY1gSrKn5VEIz72d0c gxbd7zpL7alaau3a2P5HQMnDHL0NjWooADgH1EdziXZp14Vl6p+NzBUDiAWZTchC5PUl 3UYqF30AUlXmzGh8eFirrrMHvCcBSJH3hrsye7y1tTrnaMm98E1CP61y9vkheaYrMCmq 8oow== X-Gm-Message-State: AOJu0Yzr+WiQBj+S+GwcZ1L7yPsMzBRjr9ysC/2RHZ+DtGe/gIQbvR5d bS1Q1Zm8s1LvqYYm6SuVvaLQmthHUX2UUYV5YixQlS8YjONWKk1wXIS4enEbxX19gLYMh8B407q FF9fl X-Gm-Gg: AeBDieu8ROhXkBElFTQIHXPUP3pWPLd01vRVS052ngLEPZuRHUCZod/EmmQxpvgjJ+4 v7Vhke/mZHTfYgt5Ai/WwwSBrzu5o8D7XFXK0PkGLUvw5CmBbnYJn5PJtj2JpyX1ZGErtnHd+Nx flaVkz6cFtazh/gbhgiBlljt7Blttp60LMptIFKqxJRafy68Eeyfppy7rfr/6eCsQUlBWF1y9rL oNVp7fm/27UzXPec4dauyXrLTam3HGlsypYNUCU+34tfkn9RYdCSUNjR/rik5j4qi79dwosyhoM XLpfDtYrzOtDPIse54nWp/1kBsSxhzaILsAtSAt8GYVB0lSObIqMWwOezR4ZOxt9QyTwN37byok HCb8KhR+Ld9aOtRVHJLSjY9tN1/CV2iy3P8y3Dd6Lc3QODalFks3hnxarf9Jr09Mf8YvBj5wOcd qfGmrjwmQcvqeMcuhS/ZROgsLxko/vQjpu X-Received: by 2002:a05:6a20:6a11:b0:39b:be6f:3d4d with SMTP id adf61e73a8af0-39f2f0003d4mr17954066637.23.1775575107863; Tue, 07 Apr 2026 08:18:27 -0700 (PDT) Received: from phoenix.lan ([104.202.41.210]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c76c6477bbcsm15842933a12.3.2026.04.07.08.18.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Apr 2026 08:18:27 -0700 (PDT) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Bruce Richardson , Kirill Rybalchenko Subject: [PATCH v2 14/23] examples/ptpclient: resolve shadow variable warnings Date: Tue, 7 Apr 2026 08:16:10 -0700 Message-ID: <20260407151732.272195-15-stephen@networkplumber.org> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260407151732.272195-1-stephen@networkplumber.org> References: <20260407035209.650419-1-stephen@networkplumber.org> <20260407151732.272195-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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Rename function parameters that shadow global variables. In port_init() rename mbuf_pool to mp, and in print_clock_info(), delta_eval(), parse_sync(), parse_fup(), and parse_drsp() rename ptp_data to ptp. Signed-off-by: Stephen Hemminger Acked-by: Bruce Richardson --- examples/ptpclient/meson.build | 1 - examples/ptpclient/ptpclient.c | 138 ++++++++++++++++----------------- 2 files changed, 69 insertions(+), 70 deletions(-) diff --git a/examples/ptpclient/meson.build b/examples/ptpclient/meson.build index 2e9b7625fc..47d091ea21 100644 --- a/examples/ptpclient/meson.build +++ b/examples/ptpclient/meson.build @@ -10,4 +10,3 @@ allow_experimental_apis = true sources = files( 'ptpclient.c', ) -cflags += no_shadow_cflag diff --git a/examples/ptpclient/ptpclient.c b/examples/ptpclient/ptpclient.c index 174ca5dd70..8e1560a3ac 100644 --- a/examples/ptpclient/ptpclient.c +++ b/examples/ptpclient/ptpclient.c @@ -173,7 +173,7 @@ ns_to_timeval(int64_t nsec) * coming from the mbuf_pool passed as a parameter. */ static inline int -port_init(uint16_t port, struct rte_mempool *mbuf_pool) +port_init(uint16_t port, struct rte_mempool *mp) { struct rte_eth_dev_info dev_info; struct rte_eth_conf port_conf; @@ -223,7 +223,7 @@ port_init(uint16_t port, struct rte_mempool *mbuf_pool) rxconf->offloads = port_conf.rxmode.offloads; retval = rte_eth_rx_queue_setup(port, q, nb_rxd, - rte_eth_dev_socket_id(port), rxconf, mbuf_pool); + rte_eth_dev_socket_id(port), rxconf, mp); if (retval < 0) return retval; @@ -266,42 +266,42 @@ port_init(uint16_t port, struct rte_mempool *mbuf_pool) } static void -print_clock_info(struct ptpv2_time_receiver_ordinary *ptp_data) +print_clock_info(struct ptpv2_time_receiver_ordinary *ptp) { int64_t nsec; struct timespec net_time, sys_time; printf("time transmitter clock id: %02x:%02x:%02x:%02x:%02x:%02x:%02x:%02x", - ptp_data->transmitter_clock_id.id[0], - ptp_data->transmitter_clock_id.id[1], - ptp_data->transmitter_clock_id.id[2], - ptp_data->transmitter_clock_id.id[3], - ptp_data->transmitter_clock_id.id[4], - ptp_data->transmitter_clock_id.id[5], - ptp_data->transmitter_clock_id.id[6], - ptp_data->transmitter_clock_id.id[7]); + ptp->transmitter_clock_id.id[0], + ptp->transmitter_clock_id.id[1], + ptp->transmitter_clock_id.id[2], + ptp->transmitter_clock_id.id[3], + ptp->transmitter_clock_id.id[4], + ptp->transmitter_clock_id.id[5], + ptp->transmitter_clock_id.id[6], + ptp->transmitter_clock_id.id[7]); printf("\nT2 - time receiver clock. %lds %ldns", - (ptp_data->tstamp2.tv_sec), - (ptp_data->tstamp2.tv_nsec)); + (ptp->tstamp2.tv_sec), + (ptp->tstamp2.tv_nsec)); printf("\nT1 - time transmitter clock. %lds %ldns ", - ptp_data->tstamp1.tv_sec, - (ptp_data->tstamp1.tv_nsec)); + ptp->tstamp1.tv_sec, + (ptp->tstamp1.tv_nsec)); printf("\nT3 - time receiver clock. %lds %ldns", - ptp_data->tstamp3.tv_sec, - (ptp_data->tstamp3.tv_nsec)); + ptp->tstamp3.tv_sec, + (ptp->tstamp3.tv_nsec)); printf("\nT4 - time transmitter clock. %lds %ldns\n", - ptp_data->tstamp4.tv_sec, - (ptp_data->tstamp4.tv_nsec)); + ptp->tstamp4.tv_sec, + (ptp->tstamp4.tv_nsec)); printf("\nDelta between transmitter and receiver clocks:%"PRId64"ns\n", - ptp_data->delta); + ptp->delta); clock_gettime(CLOCK_REALTIME, &sys_time); - rte_eth_timesync_read_time(ptp_data->current_ptp_port, + rte_eth_timesync_read_time(ptp->current_ptp_port, &net_time); time_t ts = net_time.tv_sec; @@ -313,14 +313,14 @@ print_clock_info(struct ptpv2_time_receiver_ordinary *ptp_data) nsec = (int64_t)timespec64_to_ns(&net_time) - (int64_t)timespec64_to_ns(&sys_time); - ptp_data->new_adj = ns_to_timeval(nsec); + ptp->new_adj = ns_to_timeval(nsec); - gettimeofday(&ptp_data->new_adj, NULL); + gettimeofday(&ptp->new_adj, NULL); - time_t tp = ptp_data->new_adj.tv_sec; + time_t tp = ptp->new_adj.tv_sec; printf("\nCurrent SYS Time: %.24s %.6ld ns", - ctime(&tp), ptp_data->new_adj.tv_usec); + ctime(&tp), ptp->new_adj.tv_usec); printf("\nDelta between PTP and Linux Kernel time:%"PRId64"ns\n", nsec); @@ -332,7 +332,7 @@ print_clock_info(struct ptpv2_time_receiver_ordinary *ptp_data) } static int64_t -delta_eval(struct ptpv2_time_receiver_ordinary *ptp_data) +delta_eval(struct ptpv2_time_receiver_ordinary *ptp) { int64_t delta; uint64_t t1 = 0; @@ -340,10 +340,10 @@ delta_eval(struct ptpv2_time_receiver_ordinary *ptp_data) uint64_t t3 = 0; uint64_t t4 = 0; - t1 = timespec64_to_ns(&ptp_data->tstamp1); - t2 = timespec64_to_ns(&ptp_data->tstamp2); - t3 = timespec64_to_ns(&ptp_data->tstamp3); - t4 = timespec64_to_ns(&ptp_data->tstamp4); + t1 = timespec64_to_ns(&ptp->tstamp1); + t2 = timespec64_to_ns(&ptp->tstamp2); + t3 = timespec64_to_ns(&ptp->tstamp3); + t4 = timespec64_to_ns(&ptp->tstamp4); delta = -((int64_t)((t2 - t1) - (t4 - t3))) / 2; @@ -354,26 +354,26 @@ delta_eval(struct ptpv2_time_receiver_ordinary *ptp_data) * Parse the PTP SYNC message. */ static void -parse_sync(struct ptpv2_time_receiver_ordinary *ptp_data, uint16_t rx_tstamp_idx) +parse_sync(struct ptpv2_time_receiver_ordinary *ptp, uint16_t rx_tstamp_idx) { struct ptp_header *ptp_hdr; - ptp_hdr = rte_pktmbuf_mtod_offset(ptp_data->m, struct ptp_header *, + ptp_hdr = rte_pktmbuf_mtod_offset(ptp->m, struct ptp_header *, sizeof(struct rte_ether_hdr)); - ptp_data->seqID_SYNC = rte_be_to_cpu_16(ptp_hdr->seq_id); + ptp->seqID_SYNC = rte_be_to_cpu_16(ptp_hdr->seq_id); - if (ptp_data->ptpset == 0) { - ptp_data->transmitter_clock_id = ptp_hdr->source_port_id.clock_id; - ptp_data->ptpset = 1; + if (ptp->ptpset == 0) { + ptp->transmitter_clock_id = ptp_hdr->source_port_id.clock_id; + ptp->ptpset = 1; } - if (memcmp(&ptp_data->transmitter_clock_id, + if (memcmp(&ptp->transmitter_clock_id, &ptp_hdr->source_port_id.clock_id, sizeof(struct clock_id)) == 0) { - if (ptp_data->ptpset == 1) - rte_eth_timesync_read_rx_timestamp(ptp_data->portid, - &ptp_data->tstamp2, rx_tstamp_idx); + if (ptp->ptpset == 1) + rte_eth_timesync_read_rx_timestamp(ptp->portid, + &ptp->tstamp2, rx_tstamp_idx); } } @@ -382,7 +382,7 @@ parse_sync(struct ptpv2_time_receiver_ordinary *ptp_data, uint16_t rx_tstamp_idx * Parse the PTP FOLLOWUP message and send DELAY_REQ to the main clock. */ static void -parse_fup(struct ptpv2_time_receiver_ordinary *ptp_data) +parse_fup(struct ptpv2_time_receiver_ordinary *ptp) { struct rte_ether_hdr *eth_hdr; struct rte_ether_addr eth_addr; @@ -395,32 +395,32 @@ parse_fup(struct ptpv2_time_receiver_ordinary *ptp_data) struct rte_ether_addr eth_multicast = ether_multicast; size_t pkt_size; int wait_us; - struct rte_mbuf *m = ptp_data->m; + struct rte_mbuf *m = ptp->m; int ret; eth_hdr = rte_pktmbuf_mtod(m, struct rte_ether_hdr *); ptp_hdr = rte_pktmbuf_mtod_offset(m, struct ptp_header *, sizeof(struct rte_ether_hdr)); - if (memcmp(&ptp_data->transmitter_clock_id, + if (memcmp(&ptp->transmitter_clock_id, &ptp_hdr->source_port_id.clock_id, sizeof(struct clock_id)) != 0) return; - ptp_data->seqID_FOLLOWUP = rte_be_to_cpu_16(ptp_hdr->seq_id); + ptp->seqID_FOLLOWUP = rte_be_to_cpu_16(ptp_hdr->seq_id); ptp_msg = rte_pktmbuf_mtod_offset(m, struct ptp_message *, sizeof(struct rte_ether_hdr)); origin_tstamp = &ptp_msg->follow_up.precise_origin_tstamp; - ptp_data->tstamp1.tv_nsec = ntohl(origin_tstamp->ns); - ptp_data->tstamp1.tv_sec = + ptp->tstamp1.tv_nsec = ntohl(origin_tstamp->ns); + ptp->tstamp1.tv_sec = ((uint64_t)ntohl(origin_tstamp->sec_lsb)) | (((uint64_t)ntohs(origin_tstamp->sec_msb)) << 32); - if (ptp_data->seqID_FOLLOWUP == ptp_data->seqID_SYNC) { - ret = rte_eth_macaddr_get(ptp_data->portid, ð_addr); + if (ptp->seqID_FOLLOWUP == ptp->seqID_SYNC) { + ret = rte_eth_macaddr_get(ptp->portid, ð_addr); if (ret != 0) { printf("\nCore %u: port %u failed to get MAC address: %s\n", - rte_lcore_id(), ptp_data->portid, + rte_lcore_id(), ptp->portid, rte_strerror(-ret)); return; } @@ -446,7 +446,7 @@ parse_fup(struct ptpv2_time_receiver_ordinary *ptp_data) struct delay_req_msg *, sizeof(struct rte_ether_hdr)); - req_msg->hdr.seq_id = htons(ptp_data->seqID_SYNC); + req_msg->hdr.seq_id = htons(ptp->seqID_SYNC); req_msg->hdr.msg_type = DELAY_REQ; req_msg->hdr.ver = 2; req_msg->hdr.control = 1; @@ -468,25 +468,25 @@ parse_fup(struct ptpv2_time_receiver_ordinary *ptp_data) client_clkid->id[6] = eth_hdr->src_addr.addr_bytes[4]; client_clkid->id[7] = eth_hdr->src_addr.addr_bytes[5]; - ptp_data->client_clock_id = *client_clkid; + ptp->client_clock_id = *client_clkid; /* Enable flag for hardware timestamping. */ created_pkt->ol_flags |= RTE_MBUF_F_TX_IEEE1588_TMST; /*Read value from NIC to prevent latching with old value. */ - rte_eth_timesync_read_tx_timestamp(ptp_data->portid, - &ptp_data->tstamp3); + rte_eth_timesync_read_tx_timestamp(ptp->portid, + &ptp->tstamp3); /* Transmit the packet. */ - rte_eth_tx_burst(ptp_data->portid, 0, &created_pkt, 1); + rte_eth_tx_burst(ptp->portid, 0, &created_pkt, 1); wait_us = 0; - ptp_data->tstamp3.tv_nsec = 0; - ptp_data->tstamp3.tv_sec = 0; + ptp->tstamp3.tv_nsec = 0; + ptp->tstamp3.tv_sec = 0; /* Wait at least 1 us to read TX timestamp. */ - while ((rte_eth_timesync_read_tx_timestamp(ptp_data->portid, - &ptp_data->tstamp3) < 0) && (wait_us < 1000)) { + while ((rte_eth_timesync_read_tx_timestamp(ptp->portid, + &ptp->tstamp3) < 0) && (wait_us < 1000)) { rte_delay_us(1); wait_us++; } @@ -530,9 +530,9 @@ update_kernel_time(void) * Parse the DELAY_RESP message. */ static void -parse_drsp(struct ptpv2_time_receiver_ordinary *ptp_data) +parse_drsp(struct ptpv2_time_receiver_ordinary *ptp) { - struct rte_mbuf *m = ptp_data->m; + struct rte_mbuf *m = ptp->m; struct ptp_message *ptp_msg; struct tstamp *rx_tstamp; uint16_t seq_id; @@ -540,26 +540,26 @@ parse_drsp(struct ptpv2_time_receiver_ordinary *ptp_data) ptp_msg = rte_pktmbuf_mtod_offset(m, struct ptp_message *, sizeof(struct rte_ether_hdr)); seq_id = rte_be_to_cpu_16(ptp_msg->delay_resp.hdr.seq_id); - if (memcmp(&ptp_data->client_clock_id, + if (memcmp(&ptp->client_clock_id, &ptp_msg->delay_resp.req_port_id.clock_id, sizeof(struct clock_id)) == 0) { - if (seq_id == ptp_data->seqID_FOLLOWUP) { + if (seq_id == ptp->seqID_FOLLOWUP) { rx_tstamp = &ptp_msg->delay_resp.rx_tstamp; - ptp_data->tstamp4.tv_nsec = ntohl(rx_tstamp->ns); - ptp_data->tstamp4.tv_sec = + ptp->tstamp4.tv_nsec = ntohl(rx_tstamp->ns); + ptp->tstamp4.tv_sec = ((uint64_t)ntohl(rx_tstamp->sec_lsb)) | (((uint64_t)ntohs(rx_tstamp->sec_msb)) << 32); /* Evaluate the delta for adjustment. */ - ptp_data->delta = delta_eval(ptp_data); + ptp->delta = delta_eval(ptp); - rte_eth_timesync_adjust_time(ptp_data->portid, - ptp_data->delta); + rte_eth_timesync_adjust_time(ptp->portid, + ptp->delta); - ptp_data->current_ptp_port = ptp_data->portid; + ptp->current_ptp_port = ptp->portid; /* Update kernel time if enabled in app parameters. */ - if (ptp_data->kernel_time_set == 1) + if (ptp->kernel_time_set == 1) update_kernel_time(); -- 2.53.0