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/[email protected]/

Please squash this change into to devlink-health series, the link for the
last sent version is here:
https://lore.kernel.org/netdev/[email protected]/

Thanks in advance
Mateusz

CC: Simon Horman <[email protected]>
Signed-off-by: Mateusz Polchlopek <[email protected]>
---
 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

Reply via email to