From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: Re: [RFC PATCH] devres: better type safety with devm_*_action()
Date: Wed, 10 Mar 2021 17:19:07 +0800 [thread overview]
Message-ID: <202103101748.Fmex2Wnv-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 10496 bytes --]
CC: kbuild-all(a)lists.01.org
In-Reply-To: <20210309235917.2134565-1-linux@rasmusvillemoes.dk>
References: <20210309235917.2134565-1-linux@rasmusvillemoes.dk>
TO: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Hi Rasmus,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on driver-core/driver-core-testing]
[also build test WARNING on linux/master linus/master v5.12-rc2 next-20210310]
[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]
url: https://github.com/0day-ci/linux/commits/Rasmus-Villemoes/devres-better-type-safety-with-devm_-_action/20210310-080019
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git a38fd8748464831584a19438cbb3082b5a2dab15
:::::: branch date: 9 hours ago
:::::: commit date: 9 hours ago
config: i386-randconfig-s001-20210309 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.3-262-g5e674421-dirty
# https://github.com/0day-ci/linux/commit/663d41317b8fa291fc1c82100b3e2526627ab4d4
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Rasmus-Villemoes/devres-better-type-safety-with-devm_-_action/20210310-080019
git checkout 663d41317b8fa291fc1c82100b3e2526627ab4d4
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
"sparse warnings: (new ones prefixed by >>)"
block/keyslot-manager.c: note: in included file (through include/linux/blk_types.h, include/linux/bio.h, include/linux/keyslot-manager.h):
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: static assertion failed: "__same_type(action + 0, void (*)(void *)) || __same_type(action + 0, void (*)(typeof(data)))"
>> block/keyslot-manager.c:160:16: sparse: sparse: arithmetics on pointers to functions
>> block/keyslot-manager.c:160:16: sparse: sparse: arithmetics on pointers to functions
>> block/keyslot-manager.c:160:16: sparse: sparse: static assertion failed: "__same_type(blk_ksm_destroy_callback + 0, void (*)(void *)) || __same_type(blk_ksm_destroy_callback + 0, void (*)(typeof(ksm)))"
--
drivers/clk/clk-cdce925.c: note: in included file (through include/linux/acpi.h, include/linux/i2c.h):
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: static assertion failed: "__same_type(action + 0, void (*)(void *)) || __same_type(action + 0, void (*)(typeof(data)))"
>> drivers/clk/clk-cdce925.c:626:16: sparse: sparse: arithmetics on pointers to functions
>> drivers/clk/clk-cdce925.c:626:16: sparse: sparse: arithmetics on pointers to functions
>> drivers/clk/clk-cdce925.c:626:16: sparse: sparse: static assertion failed: "__same_type(cdce925_regulator_disable + 0, void (*)(void *)) || __same_type(cdce925_regulator_disable + 0, void (*)(typeof(regulator)))"
--
drivers/counter/ftm-quaddec.c: note: in included file (through include/linux/platform_device.h):
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: static assertion failed: "__same_type(action + 0, void (*)(void *)) || __same_type(action + 0, void (*)(typeof(data)))"
>> drivers/counter/ftm-quaddec.c:316:15: sparse: sparse: arithmetics on pointers to functions
>> drivers/counter/ftm-quaddec.c:316:15: sparse: sparse: arithmetics on pointers to functions
>> drivers/counter/ftm-quaddec.c:316:15: sparse: sparse: static assertion failed: "__same_type(ftm_quaddec_disable + 0, void (*)(void *)) || __same_type(ftm_quaddec_disable + 0, void (*)(typeof(ftm)))"
--
drivers/counter/microchip-tcb-capture.c: note: in included file (through include/linux/counter.h):
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: static assertion failed: "__same_type(action + 0, void (*)(void *)) || __same_type(action + 0, void (*)(typeof(data)))"
>> drivers/counter/microchip-tcb-capture.c:359:23: sparse: sparse: arithmetics on pointers to functions
>> drivers/counter/microchip-tcb-capture.c:359:23: sparse: sparse: arithmetics on pointers to functions
>> drivers/counter/microchip-tcb-capture.c:359:23: sparse: sparse: static assertion failed: "__same_type(mchp_tc_clk_remove + 0, void (*)(void *)) || __same_type(mchp_tc_clk_remove + 0, void (*)(typeof(clk[i])))"
--
drivers/gpio/gpio-exar.c: note: in included file:
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: static assertion failed: "__same_type(action + 0, void (*)(void *)) || __same_type(action + 0, void (*)(typeof(data)))"
>> drivers/gpio/gpio-exar.c:168:15: sparse: sparse: arithmetics on pointers to functions
>> drivers/gpio/gpio-exar.c:168:15: sparse: sparse: arithmetics on pointers to functions
>> drivers/gpio/gpio-exar.c:168:15: sparse: sparse: static assertion failed: "__same_type(exar_devm_ida_free + 0, void (*)(void *)) || __same_type(exar_devm_ida_free + 0, void (*)(typeof(exar_gpio)))"
--
drivers/char/xillybus/xillybus_of.c: note: in included file:
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: static assertion failed: "__same_type(action + 0, void (*)(void *)) || __same_type(action + 0, void (*)(typeof(data)))"
>> drivers/char/xillybus/xillybus_of.c:96:16: sparse: sparse: arithmetics on pointers to functions
>> drivers/char/xillybus/xillybus_of.c:96:16: sparse: sparse: arithmetics on pointers to functions
>> drivers/char/xillybus/xillybus_of.c:96:16: sparse: sparse: static assertion failed: "__same_type(xilly_of_unmap + 0, void (*)(void *)) || __same_type(xilly_of_unmap + 0, void (*)(typeof(this)))"
--
drivers/input/touchscreen/edt-ft5x06.c: note: in included file (through include/linux/acpi.h, include/linux/i2c.h):
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: static assertion failed: "__same_type(action + 0, void (*)(void *)) || __same_type(action + 0, void (*)(typeof(data)))"
>> drivers/input/touchscreen/edt-ft5x06.c:1116:17: sparse: sparse: arithmetics on pointers to functions
>> drivers/input/touchscreen/edt-ft5x06.c:1116:17: sparse: sparse: arithmetics on pointers to functions
>> drivers/input/touchscreen/edt-ft5x06.c:1116:17: sparse: sparse: static assertion failed: "__same_type(edt_ft5x06_disable_regulator + 0, void (*)(void *)) || __same_type(edt_ft5x06_disable_regulator + 0, void (*)(typeof(tsdata)))"
--
sound/soc/codecs/da7213.c: note: in included file (through include/linux/acpi.h):
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: arithmetics on pointers to functions
include/linux/device.h:290:15: sparse: sparse: static assertion failed: "__same_type(action + 0, void (*)(void *)) || __same_type(action + 0, void (*)(typeof(data)))"
>> sound/soc/codecs/da7213.c:1976:15: sparse: sparse: arithmetics on pointers to functions
>> sound/soc/codecs/da7213.c:1976:15: sparse: sparse: arithmetics on pointers to functions
>> sound/soc/codecs/da7213.c:1976:15: sparse: sparse: static assertion failed: "__same_type(da7213_power_off + 0, void (*)(void *)) || __same_type(da7213_power_off + 0, void (*)(typeof(da7213)))"
vim +160 block/keyslot-manager.c
5851d3b042b694 Eric Biggers 2021-01-21 140
5851d3b042b694 Eric Biggers 2021-01-21 141 /**
5851d3b042b694 Eric Biggers 2021-01-21 142 * devm_blk_ksm_init() - Resource-managed blk_ksm_init()
5851d3b042b694 Eric Biggers 2021-01-21 143 * @dev: The device which owns the blk_keyslot_manager.
5851d3b042b694 Eric Biggers 2021-01-21 144 * @ksm: The blk_keyslot_manager to initialize.
5851d3b042b694 Eric Biggers 2021-01-21 145 * @num_slots: The number of key slots to manage.
5851d3b042b694 Eric Biggers 2021-01-21 146 *
5851d3b042b694 Eric Biggers 2021-01-21 147 * Like blk_ksm_init(), but causes blk_ksm_destroy() to be called automatically
5851d3b042b694 Eric Biggers 2021-01-21 148 * on driver detach.
5851d3b042b694 Eric Biggers 2021-01-21 149 *
5851d3b042b694 Eric Biggers 2021-01-21 150 * Return: 0 on success, or else a negative error code.
5851d3b042b694 Eric Biggers 2021-01-21 151 */
5851d3b042b694 Eric Biggers 2021-01-21 152 int devm_blk_ksm_init(struct device *dev, struct blk_keyslot_manager *ksm,
5851d3b042b694 Eric Biggers 2021-01-21 153 unsigned int num_slots)
5851d3b042b694 Eric Biggers 2021-01-21 154 {
5851d3b042b694 Eric Biggers 2021-01-21 155 int err = blk_ksm_init(ksm, num_slots);
5851d3b042b694 Eric Biggers 2021-01-21 156
5851d3b042b694 Eric Biggers 2021-01-21 157 if (err)
5851d3b042b694 Eric Biggers 2021-01-21 158 return err;
5851d3b042b694 Eric Biggers 2021-01-21 159
5851d3b042b694 Eric Biggers 2021-01-21 @160 return devm_add_action_or_reset(dev, blk_ksm_destroy_callback, ksm);
5851d3b042b694 Eric Biggers 2021-01-21 161 }
5851d3b042b694 Eric Biggers 2021-01-21 162 EXPORT_SYMBOL_GPL(devm_blk_ksm_init);
5851d3b042b694 Eric Biggers 2021-01-21 163
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 38542 bytes --]
next reply other threads:[~2021-03-10 9:19 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-10 9:19 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2021-03-10 15:38 [RFC PATCH] devres: better type safety with devm_*_action() kernel test robot
2021-03-10 14:08 kernel test robot
2021-03-10 11:17 kernel test robot
2021-03-10 10:46 kernel test robot
2021-03-10 8:58 kernel test robot
2021-03-10 7:52 kernel test robot
2021-03-10 7:37 kernel test robot
2021-03-10 7:00 kernel test robot
2021-03-10 3:53 kernel test robot
2021-03-09 23:59 Rasmus Villemoes
2021-03-18 20:30 ` Rasmus Villemoes
2021-03-18 21:58 ` Kees Cook
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=202103101748.Fmex2Wnv-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild@lists.01.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.