From nobody Tue Apr 7 16:16:01 2026 Received: from mail-oi1-f170.google.com (mail-oi1-f170.google.com [209.85.167.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D08B83B95EB for ; Thu, 12 Mar 2026 15:05:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.170 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773327949; cv=none; b=HK0StdGbLzmCMIEy589keC1MBN18syzWg0ov8o4LF81SC6fvgs5V6XblwH7ysTI1XfsLGzRok01PITEaDdLJCgEuRVKp4H1nnXHIO/dVlS7/coP3NP3m5zeIdoRezeyfqad22sG3CZpxl7iKTAERc++WCkPm0D/+xusEvAUFTp4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773327949; c=relaxed/simple; bh=guxb5JuRwHt26SwsPKoWQpt9pRz3v9T6U6dLZoAc/AY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=dG39OVmsH1CK59oG7t9a4zXJtGOhSuf4tCmbDI2QBDWrqYmfnoS+6WuAcGGwrGmRFugrhEGrjXQKzjaC/bMrQs1snT28nvTPTUp2Ciac7gmDPx+hUCwDoRs+EUIAm2Kj0F9j8ZtIs7bVtIY69z2mbUnLNiZYCgznLrc+iKjq9yg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bitbyteword.org; spf=pass smtp.mailfrom=bitbyteword.org; dkim=pass (2048-bit key) header.d=bitbyteword.org header.i=@bitbyteword.org header.b=E7acLtoB; arc=none smtp.client-ip=209.85.167.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bitbyteword.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bitbyteword.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bitbyteword.org header.i=@bitbyteword.org header.b="E7acLtoB" Received: by mail-oi1-f170.google.com with SMTP id 5614622812f47-464bba3a9easo809850b6e.0 for ; Thu, 12 Mar 2026 08:05:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bitbyteword.org; s=google; t=1773327947; x=1773932747; darn=vger.kernel.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=0owfrX/RnsqtfciHeZROr5f5IRld381S+Ztj32xJcD4=; b=E7acLtoB8mTSc5rKgK+bcbcoNEW30IvGv1X5OTafcXa2xszBHSLO6ESNPg8lkFMH6E XwVwaqKbjjeXXjYHEj0zTphL8oUg0Ax0UdLbhlIAmFAUcl2bgBlcy0RBPxLhTWu8Hnyb 4jHazNJmreWykfp4yNYGTS28OHAiGmAk2YFsnb6es7ZIe5DHlrmTtTC22ulHeJ72rF45 yxjQ+vbXxJxnczKH0isIUCeXnrMJRRgsJmwTCHJUFXN+0n0G309wjzuaKJKxFW4fXFEM GNDQIlhKjv0NPLUM3CYDrBqAb+GyX7w13fVN3VpF+UKnrcHj9meKsF8oTEzq23X8QS14 wJFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1773327947; x=1773932747; 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=0owfrX/RnsqtfciHeZROr5f5IRld381S+Ztj32xJcD4=; b=P6szlQKeawJytjWwkJCWw3FJvetV5CYwatkHdffeNAgjiLoBw7k4rlfUxSMCZ7hmP0 isduMn0+1VOd+ichZ5W4OMPM7SVipxKBb0mDU7f07WDT4V0SvO8PZXClEDZ4wS8sEfuY mSVvAB2k19ohMFIL2tJB+GdxAn4avIkNeo9PAwMGsIgAkWpG5XGFPUhsKAqI5mu2mwMG 0NfjTuf0FeQ6X69sjDpeQzhfUcXd9I6+OUj3fzyvORauo7lkOdXAf6LFLwHHHXbRjKen VTrTujvMPeuX6ZYIPv4o4pFTAcP+CpxhSIKupWSO1BiOZ6P6azFbr4orHdvZzoW/S4k8 DdDA== X-Forwarded-Encrypted: i=1; AJvYcCXfaadxQ+MoN7Aj88kVxwOeV/mD/vL6l2dj2DTlzJO2LYlBVRTz38l6drYab4XGpQND0hKqw1bChQp0Rz8=@vger.kernel.org X-Gm-Message-State: AOJu0Yyx4ZH94WK3wjNuwmz+vJjqt0h5p8H3axwjjyIXTsmbjBQnnXg8 2TtOn0dTgluudRpWJrIBOgttnZfUrRFd3dkChG0mStWcvhyuAMIiu6aMj2LtECbOpeM= X-Gm-Gg: ATEYQzyyQlMWH1uzJkvnTa2w7QMLRZ8rOcV7VVMDwP6CLz06fOIBjdztMiharRzpwgP 3nbBIFmdsCbt5FKfl29bIOFgt0FH5HHEGsxenskjyKlbdIZmb7PeMdzqYehZsyyjGZ2++qbALeF pp01Qj0eSa8sOMBrqTTiZMxtYwzCVyxJ9pZsYhuK76kme2BNAYl/BIDKgkQvOybN4vjaiCNWwgP KnHrBri5UURnHNFVa7Uqkp/DGWivHYAPR4+HInEUrWv2WQmhiXJ1wI4TlaQawRTA9oSfBWur8wZ FSaTPGu2q8TxQzFENJGdykBee+yZjt0iTCaHx1rO51XDdlkFkNtPVSgNcFJa9K+hpiQyEWNcO+L ENgk0BJVWv0aFbQyeOqeVUswNpiDMQf4uAsaCmcy3sN7+zzjdE35hzXUVZwF3lUTfeLXpnGNgre OTExaAUCzPos//6kQeX9nShjT/IiT4Kla0VJ8QWRUV7ehxD5ls5/IXcz8fMygPJRRM6g== X-Received: by 2002:a05:6808:308c:b0:45e:69af:6bc6 with SMTP id 5614622812f47-467333c5d11mr5089686b6e.13.1773327946780; Thu, 12 Mar 2026 08:05:46 -0700 (PDT) Received: from vinmini.lan (c-73-143-21-186.hsd1.vt.comcast.net. [73.143.21.186]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7d76aedae57sm4321776a34.28.2026.03.12.08.05.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 12 Mar 2026 08:05:46 -0700 (PDT) From: "Vineeth Pillai (Google)" To: Cc: "Vineeth Pillai (Google)" , Steven Rostedt , Peter Zijlstra , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend , Stanislav Fomichev , Aaron Conole , Eelco Chaudron , Ilya Maximets , Marcelo Ricardo Leitner , Xin Long , Jon Maloy , Kuniyuki Iwashima , Samiullah Khawaja , Hangbin Liu , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org, dev@openvswitch.org, linux-sctp@vger.kernel.org, tipc-discussion@lists.sourceforge.net, linux-trace-kernel@vger.kernel.org Subject: [PATCH 04/15] net: Use trace_invoke_##name() at guarded tracepoint call sites Date: Thu, 12 Mar 2026 11:04:59 -0400 Message-ID: <20260312150523.2054552-5-vineeth@bitbyteword.org> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260312150523.2054552-1-vineeth@bitbyteword.org> References: <20260312150523.2054552-1-vineeth@bitbyteword.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Replace trace_foo() with the new trace_invoke_foo() at sites already guarded by trace_foo_enabled(), avoiding a redundant static_branch_unlikely() re-evaluation inside the tracepoint. trace_invoke_foo() calls the tracepoint callbacks directly without utilizing the static branch again. Suggested-by: Steven Rostedt Suggested-by: Peter Zijlstra Signed-off-by: Vineeth Pillai (Google) Assisted-by: Claude:claude-sonnet-4-6 --- net/core/dev.c | 2 +- net/core/xdp.c | 2 +- net/openvswitch/actions.c | 2 +- net/openvswitch/datapath.c | 2 +- net/sctp/outqueue.c | 2 +- net/tipc/node.c | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index 14a83f2035b93..a48fae2bbf57e 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -6444,7 +6444,7 @@ void netif_receive_skb_list(struct list_head *head) return; if (trace_netif_receive_skb_list_entry_enabled()) { list_for_each_entry(skb, head, list) - trace_netif_receive_skb_list_entry(skb); + trace_invoke_netif_receive_skb_list_entry(skb); } netif_receive_skb_list_internal(head); trace_netif_receive_skb_list_exit(0); diff --git a/net/core/xdp.c b/net/core/xdp.c index 9890a30584ba7..53acc887c3434 100644 --- a/net/core/xdp.c +++ b/net/core/xdp.c @@ -362,7 +362,7 @@ int xdp_rxq_info_reg_mem_model(struct xdp_rxq_info *xdp= _rxq, xsk_pool_set_rxq_info(allocator, xdp_rxq); =20 if (trace_mem_connect_enabled() && xdp_alloc) - trace_mem_connect(xdp_alloc, xdp_rxq); + trace_invoke_mem_connect(xdp_alloc, xdp_rxq); return 0; } =20 diff --git a/net/openvswitch/actions.c b/net/openvswitch/actions.c index 792ca44a461da..420eb19322e85 100644 --- a/net/openvswitch/actions.c +++ b/net/openvswitch/actions.c @@ -1259,7 +1259,7 @@ static int do_execute_actions(struct datapath *dp, st= ruct sk_buff *skb, int err =3D 0; =20 if (trace_ovs_do_execute_action_enabled()) - trace_ovs_do_execute_action(dp, skb, key, a, rem); + trace_invoke_ovs_do_execute_action(dp, skb, key, a, rem); =20 /* Actions that rightfully have to consume the skb should do it * and return directly. diff --git a/net/openvswitch/datapath.c b/net/openvswitch/datapath.c index e209099218b41..02451629e888e 100644 --- a/net/openvswitch/datapath.c +++ b/net/openvswitch/datapath.c @@ -335,7 +335,7 @@ int ovs_dp_upcall(struct datapath *dp, struct sk_buff *= skb, int err; =20 if (trace_ovs_dp_upcall_enabled()) - trace_ovs_dp_upcall(dp, skb, key, upcall_info); + trace_invoke_ovs_dp_upcall(dp, skb, key, upcall_info); =20 if (upcall_info->portid =3D=3D 0) { err =3D -ENOTCONN; diff --git a/net/sctp/outqueue.c b/net/sctp/outqueue.c index f6b8c13dafa4a..9831afbff070f 100644 --- a/net/sctp/outqueue.c +++ b/net/sctp/outqueue.c @@ -1267,7 +1267,7 @@ int sctp_outq_sack(struct sctp_outq *q, struct sctp_c= hunk *chunk) /* SCTP path tracepoint for congestion control debugging. */ if (trace_sctp_probe_path_enabled()) { list_for_each_entry(transport, transport_list, transports) - trace_sctp_probe_path(transport, asoc); + trace_invoke_sctp_probe_path(transport, asoc); } =20 sack_ctsn =3D ntohl(sack->cum_tsn_ack); diff --git a/net/tipc/node.c b/net/tipc/node.c index af442a5ef8f3d..01e07ec18c56c 100644 --- a/net/tipc/node.c +++ b/net/tipc/node.c @@ -1943,7 +1943,7 @@ static bool tipc_node_check_state(struct tipc_node *n= , struct sk_buff *skb, =20 if (trace_tipc_node_check_state_enabled()) { trace_tipc_skb_dump(skb, false, "skb for node state check"); - trace_tipc_node_check_state(n, true, " "); + trace_invoke_tipc_node_check_state(n, true, " "); } l =3D n->links[bearer_id].link; if (!l) --=20 2.53.0