| 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: <202209220623.YTuu3co4-lkp@intel.com>
Date: Thu, 22 Sep 2022 06:59:25 +0800
From: kernel test robot <lkp@...el.com>
To: Christophe JAILLET <christophe.jaillet@...adoo.fr>,
Kumaravel Thiagarajan <kumaravel.thiagarajan@...rochip.com>,
Arnd Bergmann <arnd@...db.de>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: llvm@...ts.linux.dev, kbuild-all@...ts.01.org,
linux-kernel@...r.kernel.org, kernel-janitors@...r.kernel.org,
Christophe JAILLET <christophe.jaillet@...adoo.fr>,
linux-gpio@...r.kernel.org
Subject: Re: [PATCH 3/3] misc: microchip: pci1xxxx: Fix the error handling
paths of gp_aux_bus_probe()
Hi Christophe,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on char-misc/char-misc-testing]
[also build test WARNING on next-20220921]
[cannot apply to linus/master v6.0-rc6]
[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/Christophe-JAILLET/misc-microchip-pci1xxxx-Fix-the-error-handling-path-of-gp_aux_bus_probe/20220918-143022
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git ceecbbddbf549fe0b7ffa3804a6e255b3360030f
config: riscv-randconfig-r032-20220921
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 791a7ae1ba3efd6bca96338e10ffde557ba83920)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://github.com/intel-lab-lkp/linux/commit/089c1639fdebdad9be8de56c1546308eac15747d
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Christophe-JAILLET/misc-microchip-pci1xxxx-Fix-the-error-handling-path-of-gp_aux_bus_probe/20220918-143022
git checkout 089c1639fdebdad9be8de56c1546308eac15747d
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/clk/ drivers/misc/mchp_pci1xxxx/ drivers/pinctrl/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@...el.com>
All warnings (new ones prefixed by >>):
>> drivers/misc/mchp_pci1xxxx/mchp_pci1xxxx_gp.c:35:6: warning: variable 'irq' set but not used [-Wunused-but-set-variable]
int irq, retval;
^
1 warning generated.
vim +/irq +35 drivers/misc/mchp_pci1xxxx/mchp_pci1xxxx_gp.c
31
32 static int gp_aux_bus_probe(struct pci_dev *pdev, const struct pci_device_id *id)
33 {
34 struct aux_bus_device *aux_bus;
> 35 int irq, retval;
36
37 retval = pcim_enable_device(pdev);
38 if (retval)
39 return retval;
40
41 aux_bus = devm_kzalloc(&pdev->dev, sizeof(*aux_bus), GFP_KERNEL);
42 if (!aux_bus)
43 return -ENOMEM;
44
45 aux_bus->aux_device_wrapper[0] = kzalloc(sizeof(*aux_bus->aux_device_wrapper[0]),
46 GFP_KERNEL);
47 if (!aux_bus->aux_device_wrapper[0])
48 return -ENOMEM;
49
50 retval = ida_alloc(&gp_client_ida, GFP_KERNEL);
51 if (retval < 0) {
52 kfree(aux_bus->aux_device_wrapper[0]);
53 return retval;
54 }
55
56 aux_bus->aux_device_wrapper[0]->aux_dev.name = aux_dev_otp_e2p_name;
57 aux_bus->aux_device_wrapper[0]->aux_dev.dev.parent = &pdev->dev;
58 aux_bus->aux_device_wrapper[0]->aux_dev.dev.release = gp_auxiliary_device_release;
59 aux_bus->aux_device_wrapper[0]->aux_dev.id = retval;
60
61 aux_bus->aux_device_wrapper[0]->gp_aux_data.region_start = pci_resource_start(pdev, 0);
62 aux_bus->aux_device_wrapper[0]->gp_aux_data.region_length = pci_resource_end(pdev, 0);
63
64 retval = auxiliary_device_init(&aux_bus->aux_device_wrapper[0]->aux_dev);
65 if (retval < 0) {
66 ida_free(&gp_client_ida, aux_bus->aux_device_wrapper[0]->aux_dev.id);
67 kfree(aux_bus->aux_device_wrapper[0]);
68 goto err_aux_dev_uninit_0;
69 }
70
71 retval = auxiliary_device_add(&aux_bus->aux_device_wrapper[0]->aux_dev);
72 if (retval)
73 goto err_aux_dev_uninit_0;
74
75
76 retval = pci_alloc_irq_vectors(pdev, 1, 1, PCI_IRQ_ALL_TYPES);
77 if (retval < 0)
78 goto err_aux_dev_del_0;
79
80 retval = pci_irq_vector(pdev, 0);
81 if (retval < 0)
82 goto err_aux_dev_del_0;
83 irq = retval;
84
85 aux_bus->aux_device_wrapper[1] = kzalloc(sizeof(*aux_bus->aux_device_wrapper[1]),
86 GFP_KERNEL);
87 if (!aux_bus->aux_device_wrapper[1]) {
88 retval = -ENOMEM;
89 goto err_aux_dev_del_0;
90 }
91
92 retval = ida_alloc(&gp_client_ida, GFP_KERNEL);
93 if (retval < 0) {
94 kfree(aux_bus->aux_device_wrapper[1]);
95 goto err_aux_dev_del_0;
96 }
97
98 aux_bus->aux_device_wrapper[1]->aux_dev.name = aux_dev_gpio_name;
99 aux_bus->aux_device_wrapper[1]->aux_dev.dev.parent = &pdev->dev;
100 aux_bus->aux_device_wrapper[1]->aux_dev.dev.release = gp_auxiliary_device_release;
101 aux_bus->aux_device_wrapper[1]->aux_dev.id = retval;
102
103 aux_bus->aux_device_wrapper[1]->gp_aux_data.region_start = pci_resource_start(pdev, 0);
104 aux_bus->aux_device_wrapper[1]->gp_aux_data.region_length = pci_resource_end(pdev, 0);
105
106 pdev->irq = retval;
107 aux_bus->aux_device_wrapper[1]->gp_aux_data.irq_num = pdev->irq;
108
109 retval = auxiliary_device_init(&aux_bus->aux_device_wrapper[1]->aux_dev);
110 if (retval < 0) {
111 ida_free(&gp_client_ida, aux_bus->aux_device_wrapper[1]->aux_dev.id);
112 kfree(aux_bus->aux_device_wrapper[1]);
113 goto err_aux_dev_del_0;
114 }
115
116 retval = auxiliary_device_add(&aux_bus->aux_device_wrapper[1]->aux_dev);
117 if (retval)
118 goto err_aux_dev_uninit_1;
119
120 pci_set_drvdata(pdev, aux_bus);
121 pci_set_master(pdev);
122
123 return 0;
124
125 err_aux_dev_uninit_1:
126 auxiliary_device_uninit(&aux_bus->aux_device_wrapper[1]->aux_dev);
127
128 err_aux_dev_del_0:
129 auxiliary_device_delete(&aux_bus->aux_device_wrapper[0]->aux_dev);
130
131 err_aux_dev_uninit_0:
132 auxiliary_device_uninit(&aux_bus->aux_device_wrapper[0]->aux_dev);
133
134 return retval;
135 }
136
--
0-DAY CI Kernel Test Service
https://01.org/lkp
View attachment "config" of type "text/plain" (205282 bytes)
Powered by blists - more mailing lists