| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <202402190552.9D7I1cA3-lkp@intel.com>
Date: Mon, 19 Feb 2024 06:01:08 +0800
From: kernel test robot <lkp@...el.com>
To: Alexander Shishkin <alexander.shishkin@...ux.intel.com>
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: drivers/hwtracing/intel_th/core.c:812: warning: Function parameter
or member 'ndevres' not described in 'intel_th_alloc'
Hi Alexander,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 6c160f16be5df1f66f6afe186c961ad446d7f94b
commit: db73a059de00eed721f13051c0d6ff3e7de90fe8 intel_th: Rework resource passing between glue layers and core
date: 4 years, 10 months ago
config: x86_64-buildonly-randconfig-001-20231012 (https://download.01.org/0day-ci/archive/20240219/202402190552.9D7I1cA3-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240219/202402190552.9D7I1cA3-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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202402190552.9D7I1cA3-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/hwtracing/intel_th/core.c:812: warning: Function parameter or member 'drvdata' not described in 'intel_th_alloc'
>> drivers/hwtracing/intel_th/core.c:812: warning: Function parameter or member 'ndevres' not described in 'intel_th_alloc'
vim +812 drivers/hwtracing/intel_th/core.c
39f4034693b7c7 Alexander Shishkin 2015-09-22 802
39f4034693b7c7 Alexander Shishkin 2015-09-22 803 /**
39f4034693b7c7 Alexander Shishkin 2015-09-22 804 * intel_th_alloc() - allocate a new Intel TH device and its subdevices
39f4034693b7c7 Alexander Shishkin 2015-09-22 805 * @dev: parent device
db73a059de00ee Alexander Shishkin 2019-05-03 806 * @devres: resources indexed by th_mmio_idx
39f4034693b7c7 Alexander Shishkin 2015-09-22 807 * @irq: irq number
39f4034693b7c7 Alexander Shishkin 2015-09-22 808 */
39f4034693b7c7 Alexander Shishkin 2015-09-22 809 struct intel_th *
3321371b5d6484 Alexander Shishkin 2017-08-18 810 intel_th_alloc(struct device *dev, struct intel_th_drvdata *drvdata,
3321371b5d6484 Alexander Shishkin 2017-08-18 811 struct resource *devres, unsigned int ndevres, int irq)
39f4034693b7c7 Alexander Shishkin 2015-09-22 @812 {
39f4034693b7c7 Alexander Shishkin 2015-09-22 813 struct intel_th *th;
661b0df8489a35 Alexander Shishkin 2017-08-23 814 int err, r;
661b0df8489a35 Alexander Shishkin 2017-08-23 815
db73a059de00ee Alexander Shishkin 2019-05-03 816 if (ndevres < TH_MMIO_END)
db73a059de00ee Alexander Shishkin 2019-05-03 817 return ERR_PTR(-EINVAL);
39f4034693b7c7 Alexander Shishkin 2015-09-22 818
39f4034693b7c7 Alexander Shishkin 2015-09-22 819 th = kzalloc(sizeof(*th), GFP_KERNEL);
39f4034693b7c7 Alexander Shishkin 2015-09-22 820 if (!th)
39f4034693b7c7 Alexander Shishkin 2015-09-22 821 return ERR_PTR(-ENOMEM);
39f4034693b7c7 Alexander Shishkin 2015-09-22 822
39f4034693b7c7 Alexander Shishkin 2015-09-22 823 th->id = ida_simple_get(&intel_th_ida, 0, 0, GFP_KERNEL);
39f4034693b7c7 Alexander Shishkin 2015-09-22 824 if (th->id < 0) {
39f4034693b7c7 Alexander Shishkin 2015-09-22 825 err = th->id;
39f4034693b7c7 Alexander Shishkin 2015-09-22 826 goto err_alloc;
39f4034693b7c7 Alexander Shishkin 2015-09-22 827 }
39f4034693b7c7 Alexander Shishkin 2015-09-22 828
39f4034693b7c7 Alexander Shishkin 2015-09-22 829 th->major = __register_chrdev(0, 0, TH_POSSIBLE_OUTPUTS,
39f4034693b7c7 Alexander Shishkin 2015-09-22 830 "intel_th/output", &intel_th_output_fops);
39f4034693b7c7 Alexander Shishkin 2015-09-22 831 if (th->major < 0) {
39f4034693b7c7 Alexander Shishkin 2015-09-22 832 err = th->major;
39f4034693b7c7 Alexander Shishkin 2015-09-22 833 goto err_ida;
39f4034693b7c7 Alexander Shishkin 2015-09-22 834 }
39f4034693b7c7 Alexander Shishkin 2015-09-22 835 th->dev = dev;
3321371b5d6484 Alexander Shishkin 2017-08-18 836 th->drvdata = drvdata;
39f4034693b7c7 Alexander Shishkin 2015-09-22 837
db73a059de00ee Alexander Shishkin 2019-05-03 838 for (r = 0; r < ndevres; r++)
db73a059de00ee Alexander Shishkin 2019-05-03 839 th->resource[r] = devres[r];
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 840 th->num_resources = ndevres;
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 841 th->irq = irq;
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 842
d7b1787161b78a Alexander Shishkin 2016-02-15 843 dev_set_drvdata(dev, th);
d7b1787161b78a Alexander Shishkin 2016-02-15 844
142dfeb2020960 Alexander Shishkin 2016-06-22 845 pm_runtime_no_callbacks(dev);
142dfeb2020960 Alexander Shishkin 2016-06-22 846 pm_runtime_put(dev);
142dfeb2020960 Alexander Shishkin 2016-06-22 847 pm_runtime_allow(dev);
142dfeb2020960 Alexander Shishkin 2016-06-22 848
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 849 err = intel_th_populate(th);
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 850 if (err) {
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 851 /* free the subdevices and undo everything */
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 852 intel_th_free(th);
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 853 return ERR_PTR(err);
a753bfcfdb1f31 Alexander Shishkin 2017-08-10 854 }
39f4034693b7c7 Alexander Shishkin 2015-09-22 855
39f4034693b7c7 Alexander Shishkin 2015-09-22 856 return th;
39f4034693b7c7 Alexander Shishkin 2015-09-22 857
39f4034693b7c7 Alexander Shishkin 2015-09-22 858 err_ida:
39f4034693b7c7 Alexander Shishkin 2015-09-22 859 ida_simple_remove(&intel_th_ida, th->id);
39f4034693b7c7 Alexander Shishkin 2015-09-22 860
39f4034693b7c7 Alexander Shishkin 2015-09-22 861 err_alloc:
39f4034693b7c7 Alexander Shishkin 2015-09-22 862 kfree(th);
39f4034693b7c7 Alexander Shishkin 2015-09-22 863
39f4034693b7c7 Alexander Shishkin 2015-09-22 864 return ERR_PTR(err);
39f4034693b7c7 Alexander Shishkin 2015-09-22 865 }
39f4034693b7c7 Alexander Shishkin 2015-09-22 866 EXPORT_SYMBOL_GPL(intel_th_alloc);
39f4034693b7c7 Alexander Shishkin 2015-09-22 867
:::::: The code at line 812 was first introduced by commit
:::::: 39f4034693b7c7bd1fe4cb58c93259d600f55561 intel_th: Add driver infrastructure for Intel(R) Trace Hub devices
:::::: TO: Alexander Shishkin <alexander.shishkin@...ux.intel.com>
:::::: CC: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists