* Re: [PATCH] crypto: caam: fix netdev memory leak in dpaa2_caam_probe
[not found] <20260116014455.2575351-1-jianpeng.chang.cn@windriver.com>
@ 2026-01-16 8:19 ` kernel test robot
2026-01-16 9:05 ` Chang, Jianpeng (CN)
0 siblings, 1 reply; 2+ messages in thread
From: kernel test robot @ 2026-01-16 8:19 UTC (permalink / raw)
To: Jianpeng Chang, horia.geanta, pankaj.gupta, gaurav.jain, herbert,
davem, leitao, kuba
Cc: llvm, oe-kbuild-all, linux-crypto, linux-kernel, Jianpeng Chang
Hi Jianpeng,
kernel test robot noticed the following build errors:
[auto build test ERROR on herbert-cryptodev-2.6/master]
[also build test ERROR on herbert-crypto-2.6/master linus/master v6.19-rc5 next-20260115]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Jianpeng-Chang/crypto-caam-fix-netdev-memory-leak-in-dpaa2_caam_probe/20260116-094800
base: https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git master
patch link: https://lore.kernel.org/r/20260116014455.2575351-1-jianpeng.chang.cn%40windriver.com
patch subject: [PATCH] crypto: caam: fix netdev memory leak in dpaa2_caam_probe
config: x86_64-buildonly-randconfig-004-20260116 (https://download.01.org/0day-ci/archive/20260116/202601161604.ynMta3vK-lkp@intel.com/config)
compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260116/202601161604.ynMta3vK-lkp@intel.com/reproduce)
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>
| Closes: https://lore.kernel.org/oe-kbuild-all/202601161604.ynMta3vK-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/crypto/caam/caamalg_qi2.c:5132:19: error: array type 'cpumask_var_t' (aka 'struct cpumask[1]') is not assignable
5132 | priv->clean_mask = clean_mask;
| ~~~~~~~~~~~~~~~~ ^
1 error generated.
vim +5132 drivers/crypto/caam/caamalg_qi2.c
5007
5008 static int __cold dpaa2_dpseci_setup(struct fsl_mc_device *ls_dev)
5009 {
5010 struct device *dev = &ls_dev->dev;
5011 struct dpaa2_caam_priv *priv;
5012 struct dpaa2_caam_priv_per_cpu *ppriv;
5013 cpumask_var_t clean_mask;
5014 int err, cpu;
5015 u8 i;
5016
5017 err = -ENOMEM;
5018 if (!zalloc_cpumask_var(&clean_mask, GFP_KERNEL))
5019 goto err_cpumask;
5020
5021 priv = dev_get_drvdata(dev);
5022
5023 priv->dev = dev;
5024 priv->dpsec_id = ls_dev->obj_desc.id;
5025
5026 /* Get a handle for the DPSECI this interface is associate with */
5027 err = dpseci_open(priv->mc_io, 0, priv->dpsec_id, &ls_dev->mc_handle);
5028 if (err) {
5029 dev_err(dev, "dpseci_open() failed: %d\n", err);
5030 goto err_open;
5031 }
5032
5033 err = dpseci_get_api_version(priv->mc_io, 0, &priv->major_ver,
5034 &priv->minor_ver);
5035 if (err) {
5036 dev_err(dev, "dpseci_get_api_version() failed\n");
5037 goto err_get_vers;
5038 }
5039
5040 dev_info(dev, "dpseci v%d.%d\n", priv->major_ver, priv->minor_ver);
5041
5042 if (DPSECI_VER(priv->major_ver, priv->minor_ver) > DPSECI_VER(5, 3)) {
5043 err = dpseci_reset(priv->mc_io, 0, ls_dev->mc_handle);
5044 if (err) {
5045 dev_err(dev, "dpseci_reset() failed\n");
5046 goto err_get_vers;
5047 }
5048 }
5049
5050 err = dpseci_get_attributes(priv->mc_io, 0, ls_dev->mc_handle,
5051 &priv->dpseci_attr);
5052 if (err) {
5053 dev_err(dev, "dpseci_get_attributes() failed\n");
5054 goto err_get_vers;
5055 }
5056
5057 err = dpseci_get_sec_attr(priv->mc_io, 0, ls_dev->mc_handle,
5058 &priv->sec_attr);
5059 if (err) {
5060 dev_err(dev, "dpseci_get_sec_attr() failed\n");
5061 goto err_get_vers;
5062 }
5063
5064 err = dpaa2_dpseci_congestion_setup(priv, ls_dev->mc_handle);
5065 if (err) {
5066 dev_err(dev, "setup_congestion() failed\n");
5067 goto err_get_vers;
5068 }
5069
5070 priv->num_pairs = min(priv->dpseci_attr.num_rx_queues,
5071 priv->dpseci_attr.num_tx_queues);
5072 if (priv->num_pairs > num_online_cpus()) {
5073 dev_warn(dev, "%d queues won't be used\n",
5074 priv->num_pairs - num_online_cpus());
5075 priv->num_pairs = num_online_cpus();
5076 }
5077
5078 for (i = 0; i < priv->dpseci_attr.num_rx_queues; i++) {
5079 err = dpseci_get_rx_queue(priv->mc_io, 0, ls_dev->mc_handle, i,
5080 &priv->rx_queue_attr[i]);
5081 if (err) {
5082 dev_err(dev, "dpseci_get_rx_queue() failed\n");
5083 goto err_get_rx_queue;
5084 }
5085 }
5086
5087 for (i = 0; i < priv->dpseci_attr.num_tx_queues; i++) {
5088 err = dpseci_get_tx_queue(priv->mc_io, 0, ls_dev->mc_handle, i,
5089 &priv->tx_queue_attr[i]);
5090 if (err) {
5091 dev_err(dev, "dpseci_get_tx_queue() failed\n");
5092 goto err_get_rx_queue;
5093 }
5094 }
5095
5096 i = 0;
5097 for_each_online_cpu(cpu) {
5098 u8 j;
5099
5100 j = i % priv->num_pairs;
5101
5102 ppriv = per_cpu_ptr(priv->ppriv, cpu);
5103 ppriv->req_fqid = priv->tx_queue_attr[j].fqid;
5104
5105 /*
5106 * Allow all cores to enqueue, while only some of them
5107 * will take part in dequeuing.
5108 */
5109 if (++i > priv->num_pairs)
5110 continue;
5111
5112 ppriv->rsp_fqid = priv->rx_queue_attr[j].fqid;
5113 ppriv->prio = j;
5114
5115 dev_dbg(dev, "pair %d: rx queue %d, tx queue %d\n", j,
5116 priv->rx_queue_attr[j].fqid,
5117 priv->tx_queue_attr[j].fqid);
5118
5119 ppriv->net_dev = alloc_netdev_dummy(0);
5120 if (!ppriv->net_dev) {
5121 err = -ENOMEM;
5122 goto err_alloc_netdev;
5123 }
5124 cpumask_set_cpu(cpu, clean_mask);
5125 ppriv->net_dev->dev = *dev;
5126
5127 netif_napi_add_tx_weight(ppriv->net_dev, &ppriv->napi,
5128 dpaa2_dpseci_poll,
5129 DPAA2_CAAM_NAPI_WEIGHT);
5130 }
5131
> 5132 priv->clean_mask = clean_mask;
5133 return 0;
5134
5135 err_alloc_netdev:
5136 free_dpaa2_pcpu_netdev(priv, clean_mask);
5137 err_get_rx_queue:
5138 dpaa2_dpseci_congestion_free(priv);
5139 err_get_vers:
5140 dpseci_close(priv->mc_io, 0, ls_dev->mc_handle);
5141 err_open:
5142 free_cpumask_var(clean_mask);
5143 err_cpumask:
5144 return err;
5145 }
5146
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] crypto: caam: fix netdev memory leak in dpaa2_caam_probe
2026-01-16 8:19 ` [PATCH] crypto: caam: fix netdev memory leak in dpaa2_caam_probe kernel test robot
@ 2026-01-16 9:05 ` Chang, Jianpeng (CN)
0 siblings, 0 replies; 2+ messages in thread
From: Chang, Jianpeng (CN) @ 2026-01-16 9:05 UTC (permalink / raw)
To: kernel test robot, horia.geanta, pankaj.gupta, gaurav.jain,
herbert, davem, leitao, kuba
Cc: llvm, oe-kbuild-all, linux-crypto, linux-kernel
On 1/16/2026 4:19 PM, kernel test robot wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> Hi Jianpeng,
>
> kernel test robot noticed the following build errors:
>
> [auto build test ERROR on herbert-cryptodev-2.6/master]
> [also build test ERROR on herbert-crypto-2.6/master linus/master v6.19-rc5 next-20260115]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Jianpeng-Chang/crypto-caam-fix-netdev-memory-leak-in-dpaa2_caam_probe/20260116-094800
> base: https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git master
> patch link: https://lore.kernel.org/r/20260116014455.2575351-1-jianpeng.chang.cn%40windriver.com
> patch subject: [PATCH] crypto: caam: fix netdev memory leak in dpaa2_caam_probe
> config: x86_64-buildonly-randconfig-004-20260116 (https://download.01.org/0day-ci/archive/20260116/202601161604.ynMta3vK-lkp@intel.com/config)
> compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260116/202601161604.ynMta3vK-lkp@intel.com/reproduce)
>
> 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>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202601161604.ynMta3vK-lkp@intel.com/
>
> All errors (new ones prefixed by >>):
>
>>> drivers/crypto/caam/caamalg_qi2.c:5132:19: error: array type 'cpumask_var_t' (aka 'struct cpumask[1]') is not assignable
> 5132 | priv->clean_mask = clean_mask;
> | ~~~~~~~~~~~~~~~~ ^
> 1 error generated.
Tested only with CONFIG_CPUMASK_OFFSTACK enabled.
Please ignore this patch. I will resend the patch with the error fixed.
Sorry for the inconvenience.
Regards,
Jianpeng
>
>
> vim +5132 drivers/crypto/caam/caamalg_qi2.c
>
> 5007
> 5008 static int __cold dpaa2_dpseci_setup(struct fsl_mc_device *ls_dev)
> 5009 {
> 5010 struct device *dev = &ls_dev->dev;
> 5011 struct dpaa2_caam_priv *priv;
> 5012 struct dpaa2_caam_priv_per_cpu *ppriv;
> 5013 cpumask_var_t clean_mask;
> 5014 int err, cpu;
> 5015 u8 i;
> 5016
> 5017 err = -ENOMEM;
> 5018 if (!zalloc_cpumask_var(&clean_mask, GFP_KERNEL))
> 5019 goto err_cpumask;
> 5020
> 5021 priv = dev_get_drvdata(dev);
> 5022
> 5023 priv->dev = dev;
> 5024 priv->dpsec_id = ls_dev->obj_desc.id;
> 5025
> 5026 /* Get a handle for the DPSECI this interface is associate with */
> 5027 err = dpseci_open(priv->mc_io, 0, priv->dpsec_id, &ls_dev->mc_handle);
> 5028 if (err) {
> 5029 dev_err(dev, "dpseci_open() failed: %d\n", err);
> 5030 goto err_open;
> 5031 }
> 5032
> 5033 err = dpseci_get_api_version(priv->mc_io, 0, &priv->major_ver,
> 5034 &priv->minor_ver);
> 5035 if (err) {
> 5036 dev_err(dev, "dpseci_get_api_version() failed\n");
> 5037 goto err_get_vers;
> 5038 }
> 5039
> 5040 dev_info(dev, "dpseci v%d.%d\n", priv->major_ver, priv->minor_ver);
> 5041
> 5042 if (DPSECI_VER(priv->major_ver, priv->minor_ver) > DPSECI_VER(5, 3)) {
> 5043 err = dpseci_reset(priv->mc_io, 0, ls_dev->mc_handle);
> 5044 if (err) {
> 5045 dev_err(dev, "dpseci_reset() failed\n");
> 5046 goto err_get_vers;
> 5047 }
> 5048 }
> 5049
> 5050 err = dpseci_get_attributes(priv->mc_io, 0, ls_dev->mc_handle,
> 5051 &priv->dpseci_attr);
> 5052 if (err) {
> 5053 dev_err(dev, "dpseci_get_attributes() failed\n");
> 5054 goto err_get_vers;
> 5055 }
> 5056
> 5057 err = dpseci_get_sec_attr(priv->mc_io, 0, ls_dev->mc_handle,
> 5058 &priv->sec_attr);
> 5059 if (err) {
> 5060 dev_err(dev, "dpseci_get_sec_attr() failed\n");
> 5061 goto err_get_vers;
> 5062 }
> 5063
> 5064 err = dpaa2_dpseci_congestion_setup(priv, ls_dev->mc_handle);
> 5065 if (err) {
> 5066 dev_err(dev, "setup_congestion() failed\n");
> 5067 goto err_get_vers;
> 5068 }
> 5069
> 5070 priv->num_pairs = min(priv->dpseci_attr.num_rx_queues,
> 5071 priv->dpseci_attr.num_tx_queues);
> 5072 if (priv->num_pairs > num_online_cpus()) {
> 5073 dev_warn(dev, "%d queues won't be used\n",
> 5074 priv->num_pairs - num_online_cpus());
> 5075 priv->num_pairs = num_online_cpus();
> 5076 }
> 5077
> 5078 for (i = 0; i < priv->dpseci_attr.num_rx_queues; i++) {
> 5079 err = dpseci_get_rx_queue(priv->mc_io, 0, ls_dev->mc_handle, i,
> 5080 &priv->rx_queue_attr[i]);
> 5081 if (err) {
> 5082 dev_err(dev, "dpseci_get_rx_queue() failed\n");
> 5083 goto err_get_rx_queue;
> 5084 }
> 5085 }
> 5086
> 5087 for (i = 0; i < priv->dpseci_attr.num_tx_queues; i++) {
> 5088 err = dpseci_get_tx_queue(priv->mc_io, 0, ls_dev->mc_handle, i,
> 5089 &priv->tx_queue_attr[i]);
> 5090 if (err) {
> 5091 dev_err(dev, "dpseci_get_tx_queue() failed\n");
> 5092 goto err_get_rx_queue;
> 5093 }
> 5094 }
> 5095
> 5096 i = 0;
> 5097 for_each_online_cpu(cpu) {
> 5098 u8 j;
> 5099
> 5100 j = i % priv->num_pairs;
> 5101
> 5102 ppriv = per_cpu_ptr(priv->ppriv, cpu);
> 5103 ppriv->req_fqid = priv->tx_queue_attr[j].fqid;
> 5104
> 5105 /*
> 5106 * Allow all cores to enqueue, while only some of them
> 5107 * will take part in dequeuing.
> 5108 */
> 5109 if (++i > priv->num_pairs)
> 5110 continue;
> 5111
> 5112 ppriv->rsp_fqid = priv->rx_queue_attr[j].fqid;
> 5113 ppriv->prio = j;
> 5114
> 5115 dev_dbg(dev, "pair %d: rx queue %d, tx queue %d\n", j,
> 5116 priv->rx_queue_attr[j].fqid,
> 5117 priv->tx_queue_attr[j].fqid);
> 5118
> 5119 ppriv->net_dev = alloc_netdev_dummy(0);
> 5120 if (!ppriv->net_dev) {
> 5121 err = -ENOMEM;
> 5122 goto err_alloc_netdev;
> 5123 }
> 5124 cpumask_set_cpu(cpu, clean_mask);
> 5125 ppriv->net_dev->dev = *dev;
> 5126
> 5127 netif_napi_add_tx_weight(ppriv->net_dev, &ppriv->napi,
> 5128 dpaa2_dpseci_poll,
> 5129 DPAA2_CAAM_NAPI_WEIGHT);
> 5130 }
> 5131
>> 5132 priv->clean_mask = clean_mask;
> 5133 return 0;
> 5134
> 5135 err_alloc_netdev:
> 5136 free_dpaa2_pcpu_netdev(priv, clean_mask);
> 5137 err_get_rx_queue:
> 5138 dpaa2_dpseci_congestion_free(priv);
> 5139 err_get_vers:
> 5140 dpseci_close(priv->mc_io, 0, ls_dev->mc_handle);
> 5141 err_open:
> 5142 free_cpumask_var(clean_mask);
> 5143 err_cpumask:
> 5144 return err;
> 5145 }
> 5146
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-01-16 9:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20260116014455.2575351-1-jianpeng.chang.cn@windriver.com>
2026-01-16 8:19 ` [PATCH] crypto: caam: fix netdev memory leak in dpaa2_caam_probe kernel test robot
2026-01-16 9:05 ` Chang, Jianpeng (CN)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox