From: Dan Carpenter <dan.carpenter@linaro.org>
To: oe-kbuild@lists.linux.dev, Mina Almasry <almasrymina@google.com>,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: lkp@intel.com, oe-kbuild-all@lists.linux.dev,
Mina Almasry <almasrymina@google.com>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Simon Horman <horms@kernel.org>, Joe Damato <jdamato@fastly.com>,
Stanislav Fomichev <sdf@fomichev.me>
Subject: Re: [PATCH net-next v1] net: devmem: NULL check netdev_nl_get_dma_dev return value
Date: Mon, 1 Sep 2025 14:35:23 +0300 [thread overview]
Message-ID: <202508310205.YQmsa9gY-lkp@intel.com> (raw)
In-Reply-To: <20250829220003.3310242-1-almasrymina@google.com>
Hi Mina,
kernel test robot noticed the following build warnings:
url: https://github.com/intel-lab-lkp/linux/commits/Mina-Almasry/net-devmem-NULL-check-netdev_nl_get_dma_dev-return-value/20250830-060251
base: 4f54dff818d7b5b1d84becd5d90bc46e6233c0d7
patch link: https://lore.kernel.org/r/20250829220003.3310242-1-almasrymina%40google.com
patch subject: [PATCH net-next v1] net: devmem: NULL check netdev_nl_get_dma_dev return value
config: x86_64-randconfig-161-20250830 (https://download.01.org/0day-ci/archive/20250831/202508310205.YQmsa9gY-lkp@intel.com/config)
compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
| Closes: https://lore.kernel.org/r/202508310205.YQmsa9gY-lkp@intel.com/
New smatch warnings:
net/core/netdev-genl.c:1102 netdev_nl_bind_tx_doit() warn: passing zero to 'PTR_ERR'
vim +/PTR_ERR +1102 net/core/netdev-genl.c
8802087d20c0e1 Stanislav Fomichev 2025-05-08 1046 int netdev_nl_bind_tx_doit(struct sk_buff *skb, struct genl_info *info)
8802087d20c0e1 Stanislav Fomichev 2025-05-08 1047 {
bd61848900bff5 Mina Almasry 2025-05-08 1048 struct net_devmem_dmabuf_binding *binding;
bd61848900bff5 Mina Almasry 2025-05-08 1049 struct netdev_nl_sock *priv;
bd61848900bff5 Mina Almasry 2025-05-08 1050 struct net_device *netdev;
512c88fb0e884c Dragos Tatulea 2025-08-27 1051 struct device *dma_dev;
bd61848900bff5 Mina Almasry 2025-05-08 1052 u32 ifindex, dmabuf_fd;
bd61848900bff5 Mina Almasry 2025-05-08 1053 struct sk_buff *rsp;
bd61848900bff5 Mina Almasry 2025-05-08 1054 int err = 0;
bd61848900bff5 Mina Almasry 2025-05-08 1055 void *hdr;
bd61848900bff5 Mina Almasry 2025-05-08 1056
bd61848900bff5 Mina Almasry 2025-05-08 1057 if (GENL_REQ_ATTR_CHECK(info, NETDEV_A_DEV_IFINDEX) ||
bd61848900bff5 Mina Almasry 2025-05-08 1058 GENL_REQ_ATTR_CHECK(info, NETDEV_A_DMABUF_FD))
bd61848900bff5 Mina Almasry 2025-05-08 1059 return -EINVAL;
bd61848900bff5 Mina Almasry 2025-05-08 1060
bd61848900bff5 Mina Almasry 2025-05-08 1061 ifindex = nla_get_u32(info->attrs[NETDEV_A_DEV_IFINDEX]);
bd61848900bff5 Mina Almasry 2025-05-08 1062 dmabuf_fd = nla_get_u32(info->attrs[NETDEV_A_DMABUF_FD]);
bd61848900bff5 Mina Almasry 2025-05-08 1063
bd61848900bff5 Mina Almasry 2025-05-08 1064 priv = genl_sk_priv_get(&netdev_nl_family, NETLINK_CB(skb).sk);
bd61848900bff5 Mina Almasry 2025-05-08 1065 if (IS_ERR(priv))
bd61848900bff5 Mina Almasry 2025-05-08 1066 return PTR_ERR(priv);
bd61848900bff5 Mina Almasry 2025-05-08 1067
bd61848900bff5 Mina Almasry 2025-05-08 1068 rsp = genlmsg_new(GENLMSG_DEFAULT_SIZE, GFP_KERNEL);
bd61848900bff5 Mina Almasry 2025-05-08 1069 if (!rsp)
bd61848900bff5 Mina Almasry 2025-05-08 1070 return -ENOMEM;
bd61848900bff5 Mina Almasry 2025-05-08 1071
bd61848900bff5 Mina Almasry 2025-05-08 1072 hdr = genlmsg_iput(rsp, info);
bd61848900bff5 Mina Almasry 2025-05-08 1073 if (!hdr) {
bd61848900bff5 Mina Almasry 2025-05-08 1074 err = -EMSGSIZE;
bd61848900bff5 Mina Almasry 2025-05-08 1075 goto err_genlmsg_free;
bd61848900bff5 Mina Almasry 2025-05-08 1076 }
bd61848900bff5 Mina Almasry 2025-05-08 1077
bd61848900bff5 Mina Almasry 2025-05-08 1078 mutex_lock(&priv->lock);
bd61848900bff5 Mina Almasry 2025-05-08 1079
bd61848900bff5 Mina Almasry 2025-05-08 1080 netdev = netdev_get_by_index_lock(genl_info_net(info), ifindex);
bd61848900bff5 Mina Almasry 2025-05-08 1081 if (!netdev) {
bd61848900bff5 Mina Almasry 2025-05-08 1082 err = -ENODEV;
bd61848900bff5 Mina Almasry 2025-05-08 1083 goto err_unlock_sock;
bd61848900bff5 Mina Almasry 2025-05-08 1084 }
bd61848900bff5 Mina Almasry 2025-05-08 1085
bd61848900bff5 Mina Almasry 2025-05-08 1086 if (!netif_device_present(netdev)) {
bd61848900bff5 Mina Almasry 2025-05-08 1087 err = -ENODEV;
bd61848900bff5 Mina Almasry 2025-05-08 1088 goto err_unlock_netdev;
bd61848900bff5 Mina Almasry 2025-05-08 1089 }
bd61848900bff5 Mina Almasry 2025-05-08 1090
ae28cb114727dd Mina Almasry 2025-05-08 1091 if (!netdev->netmem_tx) {
ae28cb114727dd Mina Almasry 2025-05-08 1092 err = -EOPNOTSUPP;
ae28cb114727dd Mina Almasry 2025-05-08 1093 NL_SET_ERR_MSG(info->extack,
ae28cb114727dd Mina Almasry 2025-05-08 1094 "Driver does not support netmem TX");
ae28cb114727dd Mina Almasry 2025-05-08 1095 goto err_unlock_netdev;
ae28cb114727dd Mina Almasry 2025-05-08 1096 }
ae28cb114727dd Mina Almasry 2025-05-08 1097
512c88fb0e884c Dragos Tatulea 2025-08-27 1098 dma_dev = netdev_queue_get_dma_dev(netdev, 0);
512c88fb0e884c Dragos Tatulea 2025-08-27 1099 binding = net_devmem_bind_dmabuf(netdev, dma_dev, DMA_TO_DEVICE,
512c88fb0e884c Dragos Tatulea 2025-08-27 1100 dmabuf_fd, priv, info->extack);
991dbef67e2c35 Mina Almasry 2025-08-29 1101 if (IS_ERR_OR_NULL(binding)) {
bd61848900bff5 Mina Almasry 2025-05-08 @1102 err = PTR_ERR(binding);
bd61848900bff5 Mina Almasry 2025-05-08 1103 goto err_unlock_netdev;
net_devmem_bind_dmabuf() can't return NULL. See my blog for more
details:
https://staticthinking.wordpress.com/2022/08/01/mixing-error-pointers-and-null/
bd61848900bff5 Mina Almasry 2025-05-08 1104 }
bd61848900bff5 Mina Almasry 2025-05-08 1105
bd61848900bff5 Mina Almasry 2025-05-08 1106 nla_put_u32(rsp, NETDEV_A_DMABUF_ID, binding->id);
bd61848900bff5 Mina Almasry 2025-05-08 1107 genlmsg_end(rsp, hdr);
bd61848900bff5 Mina Almasry 2025-05-08 1108
bd61848900bff5 Mina Almasry 2025-05-08 1109 netdev_unlock(netdev);
bd61848900bff5 Mina Almasry 2025-05-08 1110 mutex_unlock(&priv->lock);
bd61848900bff5 Mina Almasry 2025-05-08 1111
bd61848900bff5 Mina Almasry 2025-05-08 1112 return genlmsg_reply(rsp, info);
bd61848900bff5 Mina Almasry 2025-05-08 1113
bd61848900bff5 Mina Almasry 2025-05-08 1114 err_unlock_netdev:
bd61848900bff5 Mina Almasry 2025-05-08 1115 netdev_unlock(netdev);
bd61848900bff5 Mina Almasry 2025-05-08 1116 err_unlock_sock:
bd61848900bff5 Mina Almasry 2025-05-08 1117 mutex_unlock(&priv->lock);
bd61848900bff5 Mina Almasry 2025-05-08 1118 err_genlmsg_free:
bd61848900bff5 Mina Almasry 2025-05-08 1119 nlmsg_free(rsp);
bd61848900bff5 Mina Almasry 2025-05-08 1120 return err;
8802087d20c0e1 Stanislav Fomichev 2025-05-08 1121 }
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
prev parent reply other threads:[~2025-09-01 11:35 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-29 21:59 Mina Almasry
2025-08-29 22:02 ` Mina Almasry
2025-08-30 0:52 ` Jakub Kicinski
2025-09-02 15:54 ` Stanislav Fomichev
2025-09-02 16:14 ` Mina Almasry
2025-09-01 11:35 ` Dan Carpenter [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202508310205.YQmsa9gY-lkp@intel.com \
--to=dan.carpenter@linaro.org \
--cc=almasrymina@google.com \
--cc=edumazet@google.com \
--cc=horms@kernel.org \
--cc=jdamato@fastly.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkp@intel.com \
--cc=netdev@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=oe-kbuild@lists.linux.dev \
--cc=pabeni@redhat.com \
--cc=sdf@fomichev.me \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®