DORMANTno replies

[PATCH iwl-next] fixup! devlink: add devlink_fmsg_dump_skb() function

From: Mateusz Polchlopek <hidden>
Date: 2024-12-10 11:57:01
Also in: intel-wired-lan, lkml
Subsystem: devlink, networking [general], the rest · Maintainers: Jiri Pirko, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds

Hello Tony!

In the commit related to "devlink-health dump" the sparse reports new issue.
It has been also reported some time ago by Simon:
https://lore.kernel.org/netdev/20240822104007.GL2164@kernel.org/ (local)

Please squash this change into to devlink-health series, the link for the
last sent version is here:
https://lore.kernel.org/netdev/20240821133714.61417-1-przemyslaw.kitszel@intel.com/ (local)

Thanks in advance
Mateusz

CC: Simon Horman <horms@kernel.org>
Signed-off-by: Mateusz Polchlopek <redacted>
---
 net/devlink/health.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/devlink/health.c b/net/devlink/health.c
index 3f6241d51007..57db6799722a 100644
--- a/net/devlink/health.c
+++ b/net/devlink/health.c
@@ -1270,7 +1270,7 @@ void devlink_fmsg_dump_skb(struct devlink_fmsg *fmsg, const struct sk_buff *skb)
 			 has_trans ? skb_network_header_len(skb) : -1);
 	devlink_fmsg_put(fmsg, "transport hdr",
 			 has_trans ? skb->transport_header : -1);
-	devlink_fmsg_put(fmsg, "csum", skb->csum);
+	devlink_fmsg_put(fmsg, "csum", (__force u32)skb->csum);
 	devlink_fmsg_put(fmsg, "csum_ip_summed", (u8)skb->ip_summed);
 	devlink_fmsg_put(fmsg, "csum_complete_sw", !!skb->csum_complete_sw);
 	devlink_fmsg_put(fmsg, "csum_valid", !!skb->csum_valid);
-- 
2.38.1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help