[kbuild] drivers/char/tpm/tpm_atmel.h:96 tpm_read_index() warn: ignoring unreachable code.
CC: kbuild-...@lists.01.org BCC: l...@intel.com CC: linux-ker...@vger.kernel.org TO: Bart Van Assche CC: Helge Deller tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: e729dbe8ea1c6145ae7b9efd6a00a5613746d3b0 commit: 6968e707d371ef80511448c6771daf445b4a5cf5 parisc: io: Improve the outb(), outw() and outl() macros date: 3 months ago :: branch date: 4 hours ago :: commit date: 3 months ago config: parisc-randconfig-m031-20220330 (https://download.01.org/0day-ci/archive/20220401/202204010708.094loetn-...@intel.com/config) compiler: hppa-linux-gcc (GCC) 11.2.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot Reported-by: Dan Carpenter New smatch warnings: drivers/char/tpm/tpm_atmel.h:96 tpm_read_index() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:894 das16_do_insn_bits() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:140 das6402_set_mode() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:153 das6402_clear_all_interrupts() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:160 das6402_ai_clear_eoc() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:355 das6402_ai_cancel() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:362 das6402_ai_soft_trig() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:508 das6402_do_insn_bits() warn: ignoring unreachable code. drivers/comedi/drivers/dt2811.c:209 dt2811_interrupt() warn: ignoring unreachable code. drivers/comedi/drivers/dt2811.c:237 dt2811_ai_cancel() warn: ignoring unreachable code. drivers/comedi/drivers/dt2811.c:248 dt2811_ai_set_chanspec() warn: ignoring unreachable code. drivers/comedi/drivers/dt2811.c:531 dt2811_do_insn_bits() warn: ignoring unreachable code. drivers/comedi/drivers/dt2814.c:214 dt2814_ai_cmd() warn: ignoring unreachable code. drivers/comedi/drivers/dt2814.c:235 dt2814_ai_cancel() warn: ignoring unreachable code. drivers/comedi/drivers/dt2814.c:295 dt2814_interrupt() warn: ignoring unreachable code. drivers/comedi/drivers/dt2801.c:328 dt2801_writecmd() warn: ignoring unreachable code. drivers/comedi/drivers/ni_at_a2150.c:217 a2150_interrupt() warn: ignoring unreachable code. drivers/comedi/drivers/ni_at_a2150.c:767 a2150_detach() warn: ignoring unreachable code. drivers/comedi/drivers/ni_at_ao.c:123 atao_select_reg_group() warn: ignoring unreachable code. drivers/comedi/drivers/ni_at_ao.c:159 atao_dio_insn_bits() warn: ignoring unreachable code. drivers/comedi/drivers/ni_at_ao.c:194 atao_dio_insn_config() warn: ignoring unreachable code. drivers/comedi/drivers/ni_atmio16d.c:534 atmio16d_dio_insn_bits() warn: ignoring unreachable code. drivers/comedi/drivers/ni_atmio16d.c:565 atmio16d_dio_insn_config() warn: ignoring unreachable code. drivers/comedi/drivers/s526.c:507 s526_dio_insn_bits() warn: ignoring unreachable code. drivers/comedi/drivers/s526.c:545 s526_dio_insn_config() warn: ignoring unreachable code. Old smatch warnings: drivers/comedi/drivers/das16.c:542 das16_ai_set_mux_range() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:714 das16_cmd_exec() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:726 das16_cmd_exec() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:731 das16_cmd_exec() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:734 das16_cmd_exec() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:773 das16_cancel() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:838 das16_ai_insn_read() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:871 das16_ao_insn_write() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:920 das16_reset() warn: ignoring unreachable code. drivers/comedi/drivers/das16.c:1145 das16_attach() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:146 das6402_set_extended() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:234 das6402_ai_cmd() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:392 das6402_ai_insn_read() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:433 das6402_ao_insn_write() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:460 das6402_ao_insn_write() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:466 das6402_ao_insn_write() warn: ignoring unreachable code. drivers/comedi/drivers/das6402.c:520 das6402_reset() warn: ignoring unreachable code. drivers/comedi/drivers/dt2811.c:296 dt2811_ai_cmd() warn: ignoring unreachable code. drivers/comedi/drivers/dt2811.c:506 dt2811_ao_insn_write() warn: ignoring unreachable code. drivers/comedi/drivers/dt2811.c:541 dt2811_reset() warn: ignoring unreachable code. drivers/comedi/drivers/dt2814.c:112 dt2814_ai_insn_read() warn: ignoring unreachable code.
[kbuild] drivers/gpio/gpio-sim.c:839:2: warning: There is an unknown macro here somewhere. Configuration is required. If fwnode_for_each_child_node is a macro then please configure it. [unknownMacro]
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Bartosz Golaszewski
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: cb8c474e79be458f58e9df073f51ca159f3a2aa0 gpio: sim: new testing module
date: 3 months ago
:: branch date: 23 hours ago
:: commit date: 3 months ago
compiler: ia64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
>> drivers/gpio/gpiolib-acpi.c:252:2: warning: There is an unknown macro here
>> somewhere. Configuration is required. If list_for_each_entry is a macro then
>> please configure it. [unknownMacro]
list_for_each_entry(event, _gpio->events, node)
^
--
>> drivers/gpio/gpio-sim.c:839:2: warning: There is an unknown macro here
>> somewhere. Configuration is required. If fwnode_for_each_child_node is a
>> macro then please configure it. [unknownMacro]
fwnode_for_each_child_node(swnode, child)
^
vim +839 drivers/gpio/gpio-sim.c
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 834
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 835 static void
gpio_sim_remove_swnode_recursive(struct fwnode_handle *swnode)
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 836 {
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 837 struct fwnode_handle
*child;
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 838
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 @839
fwnode_for_each_child_node(swnode, child)
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 840
fwnode_remove_software_node(child);
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 841
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 842
fwnode_remove_software_node(swnode);
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 843 }
cb8c474e79be45 Bartosz Golaszewski 2021-12-07 844
--
0-DAY CI Kernel Test Service
https://01.org/lkp
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] drivers/firmware/cirrus/cs_dsp.c:1211:2: warning: Value stored to 'adsp1_sizes' is never read [clang-analyzer-deadcode.DeadStores]
CC: l...@lists.linux.dev
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Simon Trimmer
CC: Mark Brown
CC: Charles Keepax
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: e729dbe8ea1c6145ae7b9efd6a00a5613746d3b0
commit: f6bc909e7673c30abcbdb329e7d0aa2e83c103d7 firmware: cs_dsp: add driver
to support firmware loading on Cirrus Logic DSPs
date: 6 months ago
:: branch date: 2 hours ago
:: commit date: 6 months ago
config: mips-randconfig-c004-20220331
(https://download.01.org/0day-ci/archive/20220401/202204010650.0w4oeiqy-...@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project
0f6d9501cf49ce02937099350d08f20c4af86f3d)
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 mips cross compiling tool for clang build
# apt-get install binutils-mips-linux-gnu
#
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f6bc909e7673c30abcbdb329e7d0aa2e83c103d7
git remote add linus
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout f6bc909e7673c30abcbdb329e7d0aa2e83c103d7
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
clang-analyzer warnings: (new ones prefixed by >>)
^
include/linux/uaccess.h:193:2: note: Returning zero (loaded from 'n'), which
participates in a condition later
return n;
^~~~
drivers/hwtracing/stm/core.c:736:6: note: Returning from 'copy_from_user'
if (copy_from_user(, arg, sizeof(size)))
^~~~
drivers/hwtracing/stm/core.c:736:2: note: Taking false branch
if (copy_from_user(, arg, sizeof(size)))
^
drivers/hwtracing/stm/core.c:739:11: note: The left operand of '<' is a
garbage value
if (size < sizeof(*id) || size >= PATH_MAX + sizeof(*id))
^
drivers/hwtracing/stm/core.c:821:10: warning: 5th function call argument is
an uninitialized value [clang-analyzer-core.CallAndMessage]
err = stm_data->set_options(stm_data,
^
drivers/hwtracing/stm/core.c:803:2: note: 'options' declared without an
initial value
u64 options;
^~~
drivers/hwtracing/stm/core.c:805:2: note: Control jumps to 'case
2148017410:' at line 816
switch (cmd) {
^
drivers/hwtracing/stm/core.c:817:7: note: Calling 'copy_from_user'
if (copy_from_user(, (u64 __user *)arg, sizeof(u64)))
^~~~
include/linux/uaccess.h:191:2: note: Taking true branch
if (likely(check_copy_size(to, n, false)))
^
include/linux/uaccess.h:192:7: note: Calling '_copy_from_user'
n = _copy_from_user(to, from, n);
^~~~
include/linux/uaccess.h:157:7: note: Calling 'should_fail_usercopy'
if (!should_fail_usercopy() && likely(access_ok(from, n))) {
^~
include/linux/fault-inject-usercopy.h:18:49: note: Returning zero, which
participates in a condition later
static inline bool should_fail_usercopy(void) { return false; }
^~~~
include/linux/uaccess.h:157:7: note: Returning from 'should_fail_usercopy'
if (!should_fail_usercopy() && likely(access_ok(from, n))) {
^~
include/linux/uaccess.h:157:6: note: Left side of '&&' is true
if (!should_fail_usercopy() && likely(access_ok(from, n))) {
^
include/linux/uaccess.h:157:2: note: Taking true branch
if (!should_fail_usercopy() && likely(access_ok(from, n))) {
^
include/linux/uaccess.h:159:9: note: Calling 'raw_copy_from_user'
res = raw_copy_from_user(to, from, n);
^~~
arch/mips/include/asm/uaccess.h:455:2: note: Returning without writing to
'*to'
return __cu_len_r;
^
include/linux/uaccess.h:159:9: note: Returning from 'raw_copy_from_user'
res = raw_copy_from_user(to, from, n);
^~~
include/linux/uaccess.h:161:6: note: Assuming 'res' is 0, which participates
in a condition later
if (unlikely(res))
^
include/l
[kbuild] kernel/bpf/btf.c:6357:1: warning: There is an unknown macro here somewhere. Configuration is required. If BTF_ID_LIST_GLOBAL is a macro then please configure it. [unknownMacro]
CC: kbuild-...@lists.01.org BCC: l...@intel.com CC: linux-ker...@vger.kernel.org TO: Song Liu CC: Alexei Starovoitov tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: e729dbe8ea1c6145ae7b9efd6a00a5613746d3b0 commit: d19ddb476a539fd78ad1028ae13bb38506286931 bpf: Introduce btf_tracing_ids date: 5 months ago :: branch date: 2 hours ago :: commit date: 5 months ago compiler: or1k-linux-gcc (GCC) 11.2.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot cppcheck warnings: (new ones prefixed by >>) >> net/core/rtnetlink.c:5220:9: warning: Local variable 'err' shadows outer >> variable [shadowVariable] int err; ^ net/core/rtnetlink.c:5120:6: note: Shadowed declaration int err; ^ net/core/rtnetlink.c:5220:9: note: Shadow variable int err; ^ cppcheck possible warnings: (new ones prefixed by >>, may not real problems) >> net/core/rtnetlink.c:4846:6: warning: Redundant initialization for 'err'. >> The initialized value is overwritten before it is read. >> [redundantInitialization] err = dev->netdev_ops->ndo_bridge_getlink(skb, 0, 0, dev, 0, 0); ^ net/core/rtnetlink.c:4835:10: note: err is initialized int err = -EOPNOTSUPP; ^ net/core/rtnetlink.c:4846:6: note: err is overwritten err = dev->netdev_ops->ndo_bridge_getlink(skb, 0, 0, dev, 0, 0); ^ >> net/core/rtnetlink.c:2238:52: warning: Parameter 'tb' can be declared with >> const [constParameter] static int rtnl_ensure_unique_netns(struct nlattr *tb[], ^ -- >> kernel/bpf/btf.c:6357:1: warning: There is an unknown macro here somewhere. >> Configuration is required. If BTF_ID_LIST_GLOBAL is a macro then please >> configure it. [unknownMacro] BTF_ID_LIST_GLOBAL(btf_tracing_ids, MAX_BTF_TRACING_TYPE) ^ -- >> net/core/dev.c:1651:2: warning: There is an unknown macro here somewhere. >> Configuration is required. If netdev_for_each_lower_dev is a macro then >> please configure it. [unknownMacro] netdev_for_each_lower_dev(dev, lower_dev, iter) ^ vim +6357 kernel/bpf/btf.c eb529c5b10b940 Daniel Xu 2021-08-25 6356 d19ddb476a539f Song Liu2021-11-12 @6357 BTF_ID_LIST_GLOBAL(btf_tracing_ids, MAX_BTF_TRACING_TYPE) d19ddb476a539f Song Liu2021-11-12 6358 #define BTF_TRACING_TYPE(name, type) BTF_ID(struct, type) d19ddb476a539f Song Liu2021-11-12 6359 BTF_TRACING_TYPE_xxx d19ddb476a539f Song Liu2021-11-12 6360 #undef BTF_TRACING_TYPE 14f267d95fe4b0 Kumar Kartikeya Dwivedi 2021-10-02 6361 -- 0-DAY CI Kernel Test Service https://01.org/lkp ___ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] fs/erofs/zdata.c:767 z_erofs_do_read_page() error: we previously assumed 'fe->cl' could be null (see line 666)
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Gao Xiang
CC: Yue Hu
CC: Chao Yu , Chao Yu
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: 5c6dcc57e2e50553405f2cf8b949f99b8820a685 erofs: get rid of `struct
z_erofs_collector'
date: 2 weeks ago
:: branch date: 22 hours ago
:: commit date: 2 weeks ago
config: riscv-randconfig-m031-20220327
(https://download.01.org/0day-ci/archive/20220401/202204010444.7wz3lnx4-...@intel.com/config)
compiler: riscv32-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
New smatch warnings:
fs/erofs/zdata.c:767 z_erofs_do_read_page() error: we previously assumed
'fe->cl' could be null (see line 666)
Old smatch warnings:
fs/erofs/zdata.c:471 z_erofs_register_collection() warn: passing a valid
pointer to 'PTR_ERR'
fs/erofs/zdata.c:1244 jobqueueset_init() warn: maybe use && instead of &
vim +767 fs/erofs/zdata.c
92e6efd566c4a1 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-12-08
639
97e86a858bc360 drivers/staging/erofs/zdata.c Gao Xiang 2019-07-31
640 static int z_erofs_do_read_page(struct z_erofs_decompress_frontend *fe,
eaa9172ad988b3 fs/erofs/zdata.c Gao Xiang 2021-10-22
641 struct page *page, struct page **pagepool)
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
642 {
97e86a858bc360 drivers/staging/erofs/zdata.c Gao Xiang 2019-07-31
643 struct inode *const inode = fe->inode;
bda17a4577da72 fs/erofs/zdata.c Gao Xiang 2019-10-08
644 struct erofs_sb_info *const sbi = EROFS_I_SB(inode);
3b423417d0d1d5 drivers/staging/erofs/unzip_vle.c Chao Yu 2019-01-15
645 struct erofs_map_blocks *const map = >map;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
646 const loff_t offset = page_offset(page);
dc76ea8c1087b5 fs/erofs/zdata.c Gao Xiang 2019-09-22
647 bool tight = true;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
648
92e6efd566c4a1 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-12-08
649 enum z_erofs_cache_alloctype cache_strategy;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
650 enum z_erofs_page_type page_type;
7dd68b147d60e5 drivers/staging/erofs/unzip_vle.c Thomas Weißschuh 2018-09-10
651 unsigned int cur, end, spiltted, index;
1e05ff36e6921c drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-09-18
652 int err = 0;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
653
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
654 /* register locked file pages as online pages in pack */
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
655 z_erofs_onlinepage_init(page);
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
656
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
657 spiltted = 0;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
658 end = PAGE_SIZE;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
659 repeat:
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
660 cur = end - 1;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
661
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
662 /* lucky, within the range of the current map_blocks */
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
663 if (offset + cur >= map->m_la &&
1e5ceeab692958 drivers/staging/erofs/unzip_vle.c Gao Xiang 2019-02-27
664 offset + cur < map->m_la + map->m_llen) {
97e86a858bc360 drivers/staging/erofs/zdata.c Gao Xiang 2019-07-31
665 /* didn't get a valid collection previously (very rare) */
5c6dcc57e2e505 fs/erofs/zdata.c Gao Xiang 2022-03-02
@666 if (!fe->cl)
1e5ceeab692958 drivers/staging/erofs/unzip_vle.c Gao Xiang 2019-02-27
667 goto restart_now;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
668 goto hitted;
1e5ceeab692958 drivers/staging/erofs/unzip_vle.c Gao Xiang 2019-02-27
669 }
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
670
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26
671 /* go ahead the next map_blocks */
4f761fa253b49f fs/erofs/zdata.c Gao Xiang
[kbuild] fs/btrfs/disk-io.c:2263:2: warning: There is an unknown macro here somewhere. Configuration is required. If rbtree_postorder_for_each_entry_safe is a macro then please configure it. [unknownM
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Josef Bacik
CC: David Sterba
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: abed4aaae4f71a7bcdbe90a65319b6e772a2689d btrfs: track the csum, extent,
and free space trees in a rb tree
date: 3 months ago
:: branch date: 22 hours ago
:: commit date: 3 months ago
compiler: ia64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
>> fs/btrfs/disk-io.c:2263:2: warning: There is an unknown macro here
>> somewhere. Configuration is required. If
>> rbtree_postorder_for_each_entry_safe is a macro then please configure it.
>> [unknownMacro]
rbtree_postorder_for_each_entry_safe(root, tmp,
^
>> fs/btrfs/volumes.c:1181:2: warning: There is an unknown macro here
>> somewhere. Configuration is required. If list_for_each_entry_safe is a macro
>> then please configure it. [unknownMacro]
list_for_each_entry_safe(device, tmp, _devices->devices, dev_list)
^
vim +2263 fs/btrfs/disk-io.c
2e9f5954978cd5 Rashika 2013-10-31 2258
abed4aaae4f71a Josef Bacik 2021-11-05 2259 static void
free_global_root_pointers(struct btrfs_fs_info *fs_info)
abed4aaae4f71a Josef Bacik 2021-11-05 2260 {
abed4aaae4f71a Josef Bacik 2021-11-05 2261 struct btrfs_root *root, *tmp;
abed4aaae4f71a Josef Bacik 2021-11-05 2262
abed4aaae4f71a Josef Bacik 2021-11-05 @2263
rbtree_postorder_for_each_entry_safe(root, tmp,
abed4aaae4f71a Josef Bacik 2021-11-05 2264
_info->global_root_tree,
abed4aaae4f71a Josef Bacik 2021-11-05 2265
rb_node)
abed4aaae4f71a Josef Bacik 2021-11-05 2266
free_root_extent_buffers(root);
abed4aaae4f71a Josef Bacik 2021-11-05 2267 }
abed4aaae4f71a Josef Bacik 2021-11-05 2268
--
0-DAY CI Kernel Test Service
https://01.org/lkp
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] arch/x86/boot/compressed/../string.h:15:23: warning: '__builtin_memcpy' reading 52 bytes from a region of size 0
CC: kbuild-...@lists.01.org BCC: l...@intel.com CC: linux-ker...@vger.kernel.org TO: Arvind Sankar tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 787af64d05cd528aac9ad16752d11bb1c6061bb9 commit: 394b19d6cb58ae292c0e1ad6b893fed8ece477ce x86/boot/compressed: Use builtin mem functions for decompressor date: 1 year, 7 months ago :: branch date: 21 hours ago :: commit date: 1 year, 7 months ago config: i386-randconfig-c001 (https://download.01.org/0day-ci/archive/20220401/202204010318.cvt0hpzm-...@intel.com/config) compiler: gcc-11 (Debian 11.2.0-19) 11.2.0 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 # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=394b19d6cb58ae292c0e1ad6b893fed8ece477ce git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout 394b19d6cb58ae292c0e1ad6b893fed8ece477ce # save the config file to linux build tree If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All warnings (new ones prefixed by >>): In file included from arch/x86/boot/compressed/misc.c:18: In function 'parse_elf', inlined from 'extract_kernel' at arch/x86/boot/compressed/misc.c:442:2: >> arch/x86/boot/compressed/../string.h:15:23: warning: '__builtin_memcpy' >> reading 52 bytes from a region of size 0 [-Wstringop-overread] 15 | #define memcpy(d,s,l) __builtin_memcpy(d,s,l) | ^~~ arch/x86/boot/compressed/misc.c:283:9: note: in expansion of macro 'memcpy' 283 | memcpy(, output, sizeof(ehdr)); | ^~ vim +/__builtin_memcpy +15 arch/x86/boot/compressed/../string.h c041b5ad8640dd Vivek Goyal 2014-03-18 13 394b19d6cb58ae Arvind Sankar 2020-08-04 14 /* Access builtin version by default. */ c041b5ad8640dd Vivek Goyal 2014-03-18 @15 #define memcpy(d,s,l) __builtin_memcpy(d,s,l) c041b5ad8640dd Vivek Goyal 2014-03-18 16 #define memset(d,c,l) __builtin_memset(d,c,l) fb4cac573ef6dc Vivek Goyal 2014-03-18 17 #define memcmp __builtin_memcmp c041b5ad8640dd Vivek Goyal 2014-03-18 18 :: The code at line 15 was first introduced by commit :: c041b5ad8640dd89ccf1411cd2636ef7c1cfee92 x86, boot: Create a separate string.h file to provide standard string functions :: TO: Vivek Goyal :: CC: H. Peter Anvin -- 0-DAY CI Kernel Test Service https://01.org/lkp ___ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] mm/slab_common.c:443:24: warning: Uninitialized variables: s.object_size, s.size, s.align, s.flags, s.useroffset, s.usersize, s.name, s.refcount, s.ctor, s.list [uninitvar]
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Vlastimil Babka
CC: Roman Gushchin
CC: Hyeonggon Yoo <42.hye...@gmail.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: 401fb12c68c257b9c9116b1475c0ac26b646fcc0 mm/sl*b: Differentiate struct
slab fields by sl*b implementations
date: 3 months ago
:: branch date: 19 hours ago
:: commit date: 3 months ago
compiler: or1k-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
mm/slab_common.c:187:48: warning: Parameter 'ctor' can be declared with
const [constParameter]
slab_flags_t flags, const char *name, void (*ctor)(void *))
^
>> mm/slab_common.c:443:24: warning: Uninitialized variables: s.object_size,
>> s.size, s.align, s.flags, s.useroffset, s.usersize, s.name, s.refcount,
>> s.ctor, s.list [uninitvar]
debugfs_slab_release(s);
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:437:16: note: Assuming condition is false
if (list_empty(_destroy))
^
mm/slab_common.c:443:24: note: Uninitialized variables: s.object_size,
s.size, s.align, s.flags, s.useroffset, s.usersize, s.name, s.refcount, s.ctor,
s.list
debugfs_slab_release(s);
^
vim +443 mm/slab_common.c
3965fc36522446 Vladimir Davydov2014-01-23 418
657dc2f9722092 Tejun Heo 2017-02-22 419 static void
slab_caches_to_rcu_destroy_workfn(struct work_struct *work)
d5b3cf7139b877 Vladimir Davydov2015-02-10 420 {
657dc2f9722092 Tejun Heo 2017-02-22 421 LIST_HEAD(to_destroy);
657dc2f9722092 Tejun Heo 2017-02-22 422 struct kmem_cache *s,
*s2;
d5b3cf7139b877 Vladimir Davydov2015-02-10 423
657dc2f9722092 Tejun Heo 2017-02-22 424 /*
5f0d5a3ae7cff0 Paul E. McKenney2017-01-18 425 * On destruction,
SLAB_TYPESAFE_BY_RCU kmem_caches are put on the
657dc2f9722092 Tejun Heo 2017-02-22 426 *
@slab_caches_to_rcu_destroy list. The slab pages are freed
081a06fa299066 Randy Dunlap2020-08-11 427 * through RCU and the
associated kmem_cache are dereferenced
657dc2f9722092 Tejun Heo 2017-02-22 428 * while freeing the
pages, so the kmem_caches should be freed only
657dc2f9722092 Tejun Heo 2017-02-22 429 * after the pending
RCU operations are finished. As rcu_barrier()
657dc2f9722092 Tejun Heo 2017-02-22 430 * is a pretty slow
operation, we batch all pending destructions
657dc2f9722092 Tejun Heo 2017-02-22 431 * asynchronously.
657dc2f9722092 Tejun Heo 2017-02-22 432 */
657dc2f9722092 Tejun Heo 2017-02-22 433 mutex_lock(_mutex);
657dc2f9722092 Tejun Heo 2017-02-22 434
list_splice_init(_caches_to_rcu_destroy, _destroy);
657dc2f9722092 Tejun Heo 2017-02-22 435
mutex_unlock(_mutex);
d5b3cf7139b877 Vladimir Davydov2015-02-10 436
657dc2f9722092 Tejun Heo 2017-02-22 437 if
(list_empty(_destroy))
657dc2f9722092 Tejun Heo 2017-02-22 438 return;
657dc2f9722092 Tejun Heo 2017-02-22 439
657dc2f9722092 Tejun Heo 2017-02-22 440 rcu_barrier();
657dc2f9722092 Tejun Heo 2017-02-22 441
657dc2f9722092 Tejun Heo 2017-02-22 442
list_for_each_entry_safe(s, s2, _destroy, list) {
64dd68497be76a Faiyaz Mohammed 2021-06-28 @443
debugfs_slab_release(s);
d3fb45f370d927 Alexander Potapenko 2021-02-25 444
kfence_shutdown_cache(s);
657dc2f9722092 Tejun Heo 2017-02-22 445 #ifdef SLAB_SUPPORTS_SYSFS
657dc2f9722092 Tejun Heo 2017-02-22 446
[kbuild] drivers/hid/hid-core.c:1767:2: warning: There is an unknown macro here somewhere. Configuration is required. If list_for_each_entry is a macro then please configure it. [unknownMacro]
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Benjamin Tissoires
CC: Jiri Kosina
CC: Ping Cheng
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: 22f4b026c3ddd4b26c5baa202bd3ee38feaa2e9a HID: compute an ordered list
of input fields to process
date: 4 weeks ago
:: branch date: 16 hours ago
:: commit date: 4 weeks ago
compiler: or1k-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
>> drivers/hid/hid-core.c:1767:2: warning: There is an unknown macro here
>> somewhere. Configuration is required. If list_for_each_entry is a macro then
>> please configure it. [unknownMacro]
list_for_each_entry(report, _enum->report_list, list)
^
vim +1767 drivers/hid/hid-core.c
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1761
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1762 static void
hid_process_ordering(struct hid_device *hid)
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1763 {
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1764 struct hid_report
*report;
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1765 struct hid_report_enum
*report_enum = >report_enum[HID_INPUT_REPORT];
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1766
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 @1767
list_for_each_entry(report, _enum->report_list, list)
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1768
hid_report_process_ordering(hid, report);
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1769 }
22f4b026c3ddd4 Benjamin Tissoires 2022-02-03 1770
--
0-DAY CI Kernel Test Service
https://01.org/lkp
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] Re: [RESEND][PATCH v3 2/2] serial: 8250: Add proper clock handling for OxSemi PCIe devices
CC: kbuild-...@lists.01.org BCC: l...@intel.com In-Reply-To: References: TO: "Maciej W. Rozycki" TO: "Greg Kroah-Hartman" TO: Jiri Slaby CC: Andy Shevchenko CC: linux-ser...@vger.kernel.org CC: linux-ker...@vger.kernel.org Hi "Maciej, I love your patch! Perhaps something to improve: [auto build test WARNING on tty/tty-testing] [also build test WARNING on usb/usb-testing helgaas-pci/next v5.17 next-20220331] [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/intel-lab-lkp/linux/commits/Maciej-W-Rozycki/serial-8250-Fixes-for-Oxford-Semiconductor-950-UARTs/20220331-151612 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git tty-testing :: branch date: 7 hours ago :: commit date: 7 hours ago config: x86_64-randconfig-m001 (https://download.01.org/0day-ci/archive/20220331/202203312211.zjqjarhw-...@intel.com/config) compiler: gcc-11 (Debian 11.2.0-19) 11.2.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot Reported-by: Dan Carpenter New smatch warnings: drivers/tty/serial/8250/8250_pci.c:1171 pci_oxsemi_tornado_get_divisor() error: uninitialized symbol 'tcr'. drivers/tty/serial/8250/8250_pci.c:1172 pci_oxsemi_tornado_get_divisor() error: uninitialized symbol 'quot'. drivers/tty/serial/8250/8250_pci.c:1180 pci_oxsemi_tornado_get_divisor() error: uninitialized symbol 'cpr'. Old smatch warnings: drivers/tty/serial/8250/8250_pci.c:1176 pci_oxsemi_tornado_get_divisor() error: uninitialized symbol 'quot'. drivers/tty/serial/8250/8250_pci.c:1190 pci_oxsemi_tornado_get_divisor() error: uninitialized symbol 'cpr'. vim +/tcr +1171 drivers/tty/serial/8250/8250_pci.c b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1049 b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1050 /* b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1051 * Determine the oversampling rate, the clock prescaler, and the clock b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1052 * divisor for the requested baud rate. The clock rate is 62.5 MHz, b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1053 * which is four times the baud base, and the prescaler increments in b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1054 * steps of 1/8. Therefore to make calculations on integers we need b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1055 * to use a scaled clock rate, which is the baud base multiplied by 32 b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1056 * (or our assumed UART clock rate multiplied by 2). b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1057 * b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1058 * The allowed oversampling rates are from 4 up to 16 inclusive (values b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1059 * from 0 to 3 inclusive map to 16). Likewise the clock prescaler allows b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1060 * values between 1.000 and 63.875 inclusive (operation for values from b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1061 * 0.000 to 0.875 has not been specified). The clock divisor is the usual b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1062 * unsigned 16-bit integer. b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1063 * b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1064 * For the most accurate baud rate we use a table of predetermined b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1065 * oversampling rates and clock prescalers that records all possible b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1066 * products of the two parameters in the range from 4 up to 255 inclusive, b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1067 * and additionally 335 for the 150bps rate, with the prescaler scaled b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1068 * by 8. The table is sorted by the decreasing value of the oversampling b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1069 * rate and ties are resolved by sorting by the decreasing value of the b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1070 * product. This way preference is given to higher oversampling rates. b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1071 * b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1072 * We iterate over the table and choose the product of an oversampling b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1073 * rate and a clock prescaler that gives the lowest integer division b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1074 * result deviation, or if an exact integer divider is found we stop b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1075 * looking for it right away. We do some fixup if the resulting clock b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1076 * divisor required would be out of its unsigned 16-bit integer range. b3a554d50d82ad Maciej W. Rozycki 2022-03-31 1077 * b3a554d50d82ad Maciej W. Rozycki 2022-0
[kbuild] [sailus-media-tree:master 28/56] drivers/media/v4l2-core/v4l2-subdev.c:875 __v4l2_subdev_state_alloc() warn: Please consider using kvcalloc instead
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Tomi Valkeinen
CC: Sakari Ailus
CC: Laurent Pinchart
CC: Hans Verkuil
CC: Jacopo Mondi
tree: git://linuxtv.org/sailus/media_tree.git master
head: 2052fcce377325e5662418ec65a1fabe59e4cfbf
commit: 38a25fa468f080196b04a18edf9fd173ceaf6466 [28/56] media: subdev: rename
subdev-state alloc & free
:: branch date: 27 hours ago
:: commit date: 8 days ago
config: powerpc64-randconfig-m031-20220330
(https://download.01.org/0day-ci/archive/20220331/202203312125.s4p9lsqs-...@intel.com/config)
compiler: powerpc64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
smatch warnings:
drivers/media/v4l2-core/v4l2-subdev.c:875 __v4l2_subdev_state_alloc() warn:
Please consider using kvcalloc instead
vim +875 drivers/media/v4l2-core/v4l2-subdev.c
9b02cbb3ede89b Laurent Pinchart 2015-04-24 864
38a25fa468f080 Tomi Valkeinen 2022-03-01 865 struct v4l2_subdev_state
*__v4l2_subdev_state_alloc(struct v4l2_subdev *sd)
9b02cbb3ede89b Laurent Pinchart 2015-04-24 866 {
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 867 struct
v4l2_subdev_state *state;
9b02cbb3ede89b Laurent Pinchart 2015-04-24 868 int ret;
9b02cbb3ede89b Laurent Pinchart 2015-04-24 869
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 870 state =
kzalloc(sizeof(*state), GFP_KERNEL);
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 871 if (!state)
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 872 return
ERR_PTR(-ENOMEM);
9b02cbb3ede89b Laurent Pinchart 2015-04-24 873
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 874 if
(sd->entity.num_pads) {
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 @875 state->pads =
kvmalloc_array(sd->entity.num_pads,
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 876
sizeof(*state->pads),
758d90e161382c Tomasz Figa 2017-06-19 877
GFP_KERNEL | __GFP_ZERO);
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 878 if
(!state->pads) {
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 879 ret =
-ENOMEM;
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 880 goto
err;
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 881 }
9b02cbb3ede89b Laurent Pinchart 2015-04-24 882 }
9b02cbb3ede89b Laurent Pinchart 2015-04-24 883
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 884 ret =
v4l2_subdev_call(sd, pad, init_cfg, state);
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 885 if (ret < 0 && ret !=
-ENOIOCTLCMD)
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 886 goto err;
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 887
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 888 return state;
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 889
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 890 err:
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 891 if (state &&
state->pads)
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 892
kvfree(state->pads);
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 893
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 894 kfree(state);
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 895
0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 896 return ERR_PTR(ret);
9b02cbb3ede89b Laurent Pinchart 2015-04-24 897 }
38a25fa468f080 Tomi Valkeinen 2022-03-01 898
EXPORT_SYMBOL_GPL(__v4l2_subdev_state_alloc);
9b02cbb3ede89b Laurent Pinchart 2015-04-24 899
:: The code at line 875 was first introduced by commit
:: 0d346d2a6f54f06f36b224fd27cd6eafe8c83be9 media: v4l2-subdev: add
subdev-wide state struct
:: TO: Tomi Valkeinen
:: CC: Mauro Carvalho Chehab
--
0-DAY CI Kernel Test Service
https://01.org/lkp
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] drivers/net/ethernet/xilinx/ll_temac_main.c:1257:10: warning: Call to function 'sprintf' is insecure as it does not provide bounding of the memory buffer or security checks introduced in the
CC: l...@lists.linux.dev CC: kbuild-...@lists.01.org BCC: l...@intel.com CC: linux-ker...@vger.kernel.org TO: Arnd Bergmann CC: Masahiro Yamada CC: Alex Shi CC: Nick Desaulniers CC: Miguel Ojeda CC: Nathan Chancellor tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 787af64d05cd528aac9ad16752d11bb1c6061bb9 commit: e8c07082a810fbb9db303a2b66b66b8d7e588b53 Kbuild: move to -std=gnu11 date: 3 weeks ago :: branch date: 15 hours ago :: commit date: 3 weeks ago config: arm-randconfig-c002-20220330 (https://download.01.org/0day-ci/archive/20220331/202203312148.3vraxwar-...@intel.com/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 0f6d9501cf49ce02937099350d08f20c4af86f3d) 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 arm cross compiling tool for clang build # apt-get install binutils-arm-linux-gnueabi # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e8c07082a810fbb9db303a2b66b66b8d7e588b53 git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout e8c07082a810fbb9db303a2b66b66b8d7e588b53 # save the config file to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm clang-analyzer If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot clang-analyzer warnings: (new ones prefixed by >>) ^~ lib/memcpy_kunit.c:270:2: warning: Call to function 'memset' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memset_s' in case of C11 [clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling] memset_startat(, 0x79, four); ^ include/linux/string.h:290:2: note: expanded from macro 'memset_startat' memset(__ptr + offsetof(typeof(*(obj)), member), __val, \ ^~ lib/memcpy_kunit.c:270:2: note: Call to function 'memset' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memset_s' in case of C11 memset_startat(, 0x79, four); ^ include/linux/string.h:290:2: note: expanded from macro 'memset_startat' memset(__ptr + offsetof(typeof(*(obj)), member), __val, \ ^~ Suppressed 30 warnings (30 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 32 warnings generated. drivers/char/xillybus/xillybus_class.c:66:3: warning: Call to function 'snprintf' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'snprintf_s' in case of C11 [clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling] snprintf(unit->name, UNITNAMELEN, "%s", prefix); ^~~~ drivers/char/xillybus/xillybus_class.c:66:3: note: Call to function 'snprintf' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'snprintf_s' in case of C11 snprintf(unit->name, UNITNAMELEN, "%s", prefix); ^~~~ drivers/char/xillybus/xillybus_class.c:69:3: warning: Call to function 'snprintf' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'snprintf_s' in case of C11 [clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling] snprintf(unit->name, UNITNAMELEN, "%s_%02d", ^~~~ drivers/char/xillybus/xillybus_class.c:69:3: note: Call to function 'snprintf' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'snprintf_s' in case of C11 snprintf(unit->name, UNITNAMELEN, "%s_%02d", ^~~~ drivers/char/xillybus/xillybus_class.c:118:3: warning: Call to function 'snprintf' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary c
[kbuild] drivers/isdn/mISDN/dsp_pipeline.c:45:8: warning: Call to function 'sprintf' is insecure as it does not provide bounding of the memory buffer or security checks introduced in the C11 standard.
CC: l...@lists.linux.dev CC: kbuild-...@lists.01.org BCC: l...@intel.com CC: linux-ker...@vger.kernel.org TO: Arnd Bergmann CC: Masahiro Yamada CC: Alex Shi CC: Nick Desaulniers CC: Miguel Ojeda CC: Nathan Chancellor tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 787af64d05cd528aac9ad16752d11bb1c6061bb9 commit: e8c07082a810fbb9db303a2b66b66b8d7e588b53 Kbuild: move to -std=gnu11 date: 3 weeks ago :: branch date: 15 hours ago :: commit date: 3 weeks ago config: riscv-randconfig-c006-20220331 (https://download.01.org/0day-ci/archive/20220331/202203312123.hkfjxzxb-...@intel.com/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 22a95dd3e6c911c2854dd75f42ba32d34840a61a) 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://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e8c07082a810fbb9db303a2b66b66b8d7e588b53 git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout e8c07082a810fbb9db303a2b66b66b8d7e588b53 # save the config file to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv clang-analyzer If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot clang-analyzer warnings: (new ones prefixed by >>) ^~ drivers/spi/spi-zynqmp-gqspi.c:347:2: note: Call to function 'memcpy' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memcpy_s' in case of C11 memcpy(xqspi->rxbuf, , size); ^~ drivers/spi/spi-zynqmp-gqspi.c:513:4: warning: Call to function 'memcpy' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memcpy_s' in case of C11 [clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling] memcpy(, xqspi->txbuf, 4); ^~ drivers/spi/spi-zynqmp-gqspi.c:513:4: note: Call to function 'memcpy' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memcpy_s' in case of C11 memcpy(, xqspi->txbuf, 4); ^~ drivers/spi/spi-zynqmp-gqspi.c:518:4: warning: Call to function 'memcpy' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memcpy_s' in case of C11 [clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling] memcpy(, xqspi->txbuf, ^~ drivers/spi/spi-zynqmp-gqspi.c:518:4: note: Call to function 'memcpy' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memcpy_s' in case of C11 memcpy(, xqspi->txbuf, ^~ Suppressed 43 warnings (36 in non-user code, 7 with check filters). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 95 warnings generated. net/bridge/br_input.c:294:2: warning: Call to function 'memset' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memset_s' in case of C11 [clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling] memset(skb->cb, 0, sizeof(struct br_input_skb_cb)); ^~ net/bridge/br_input.c:294:2: note: Call to function 'memset' is insecure as it does not provide security checks introduced in the C11 standard. Replace with analogous functions that support length arguments or provides boundary checks such as 'memset_s' in case of C11 memset(skb->cb, 0, sizeof(struct br_input_skb_cb)); ^~ Suppressed 94 warnings (87 in non-user code, 7 with check filters). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 100 warnings generated. net/bridge/br_ioctl.c:166:3: warnin
[kbuild] mm/damon/dbgfs.c:525 dbgfs_init_regions_write() warn: passing a valid pointer to 'PTR_ERR'
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: SeongJae Park
CC: Andrew Morton
CC: Linux Memory Management List
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: 90bebce9fcd6488ba6b010af3a16a0a0d7e44cb6 mm/damon/dbgfs: allow users to
set initial monitoring target regions
date: 5 months ago
:: branch date: 15 hours ago
:: commit date: 5 months ago
config: arc-randconfig-m031-20220331
(https://download.01.org/0day-ci/archive/20220331/202203312110.1i6nofqu-...@intel.com/config)
compiler: arceb-elf-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
smatch warnings:
mm/damon/dbgfs.c:525 dbgfs_init_regions_write() warn: passing a valid pointer
to 'PTR_ERR'
vim +/PTR_ERR +525 mm/damon/dbgfs.c
90bebce9fcd648 SeongJae Park 2021-11-05 513
90bebce9fcd648 SeongJae Park 2021-11-05 514 static ssize_t
dbgfs_init_regions_write(struct file *file,
90bebce9fcd648 SeongJae Park 2021-11-05 515
const char __user *buf, size_t count,
90bebce9fcd648 SeongJae Park 2021-11-05 516
loff_t *ppos)
90bebce9fcd648 SeongJae Park 2021-11-05 517 {
90bebce9fcd648 SeongJae Park 2021-11-05 518struct damon_ctx *ctx =
file->private_data;
90bebce9fcd648 SeongJae Park 2021-11-05 519char *kbuf;
90bebce9fcd648 SeongJae Park 2021-11-05 520ssize_t ret = count;
90bebce9fcd648 SeongJae Park 2021-11-05 521int err;
90bebce9fcd648 SeongJae Park 2021-11-05 522
90bebce9fcd648 SeongJae Park 2021-11-05 523kbuf = user_input_str(buf,
count, ppos);
90bebce9fcd648 SeongJae Park 2021-11-05 524if (IS_ERR(kbuf))
90bebce9fcd648 SeongJae Park 2021-11-05 @525return PTR_ERR(kbuf);
90bebce9fcd648 SeongJae Park 2021-11-05 526
90bebce9fcd648 SeongJae Park 2021-11-05 527mutex_lock(>kdamond_lock);
90bebce9fcd648 SeongJae Park 2021-11-05 528if (ctx->kdamond) {
90bebce9fcd648 SeongJae Park 2021-11-05 529ret = -EBUSY;
90bebce9fcd648 SeongJae Park 2021-11-05 530goto unlock_out;
90bebce9fcd648 SeongJae Park 2021-11-05 531}
90bebce9fcd648 SeongJae Park 2021-11-05 532
90bebce9fcd648 SeongJae Park 2021-11-05 533err = set_init_regions(ctx,
kbuf, ret);
90bebce9fcd648 SeongJae Park 2021-11-05 534if (err)
90bebce9fcd648 SeongJae Park 2021-11-05 535ret = err;
90bebce9fcd648 SeongJae Park 2021-11-05 536
90bebce9fcd648 SeongJae Park 2021-11-05 537 unlock_out:
90bebce9fcd648 SeongJae Park 2021-11-05 538
mutex_unlock(>kdamond_lock);
90bebce9fcd648 SeongJae Park 2021-11-05 539kfree(kbuf);
90bebce9fcd648 SeongJae Park 2021-11-05 540return ret;
90bebce9fcd648 SeongJae Park 2021-11-05 541 }
90bebce9fcd648 SeongJae Park 2021-11-05 542
--
0-DAY CI Kernel Test Service
https://01.org/lkp
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] drivers/clk/clk-lmk04832.c:357:15: warning: Variable 'pll2_p' can be declared with const [constVariable]
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Liam Beguin
CC: Stephen Boyd
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: 3bc61cfd6f4a57de32132075b15b0ac8987ced1d clk: add support for the
lmk04832
date: 9 months ago
:: branch date: 14 hours ago
:: commit date: 9 months ago
compiler: ia64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
>> drivers/clk/clk-lmk04832.c:357:15: warning: Variable 'pll2_p' can be
>> declared with const [constVariable]
unsigned int pll2_p[] = {8, 2, 2, 3, 4, 5, 6, 7};
^
vim +/pll2_p +357 drivers/clk/clk-lmk04832.c
3bc61cfd6f4a57d Liam Beguin 2021-04-22 352
3bc61cfd6f4a57d Liam Beguin 2021-04-22 353 static unsigned long
lmk04832_vco_recalc_rate(struct clk_hw *hw,
3bc61cfd6f4a57d Liam Beguin 2021-04-22 354
unsigned long prate)
3bc61cfd6f4a57d Liam Beguin 2021-04-22 355 {
3bc61cfd6f4a57d Liam Beguin 2021-04-22 356 struct lmk04832 *lmk =
container_of(hw, struct lmk04832, vco);
3bc61cfd6f4a57d Liam Beguin 2021-04-22 @357 unsigned int pll2_p[] = {8, 2,
2, 3, 4, 5, 6, 7};
3bc61cfd6f4a57d Liam Beguin 2021-04-22 358 unsigned int pll2_n, p, pll2_r;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 359 unsigned int pll2_misc;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 360 unsigned long vco_rate;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 361 u8 tmp[3];
3bc61cfd6f4a57d Liam Beguin 2021-04-22 362 int ret;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 363
3bc61cfd6f4a57d Liam Beguin 2021-04-22 364 ret = regmap_read(lmk->regmap,
LMK04832_REG_PLL2_MISC, _misc);
3bc61cfd6f4a57d Liam Beguin 2021-04-22 365 if (ret)
3bc61cfd6f4a57d Liam Beguin 2021-04-22 366 return ret;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 367
3bc61cfd6f4a57d Liam Beguin 2021-04-22 368 p =
FIELD_GET(LMK04832_BIT_PLL2_MISC_P, pll2_misc);
3bc61cfd6f4a57d Liam Beguin 2021-04-22 369
3bc61cfd6f4a57d Liam Beguin 2021-04-22 370 ret =
regmap_bulk_read(lmk->regmap, LMK04832_REG_PLL2_N_0, , 3);
3bc61cfd6f4a57d Liam Beguin 2021-04-22 371 if (ret)
3bc61cfd6f4a57d Liam Beguin 2021-04-22 372 return ret;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 373
3bc61cfd6f4a57d Liam Beguin 2021-04-22 374 pll2_n = FIELD_PREP(0x03,
tmp[0]) |
3bc61cfd6f4a57d Liam Beguin 2021-04-22 375 FIELD_PREP(0x00ff00,
tmp[1]) |
3bc61cfd6f4a57d Liam Beguin 2021-04-22 376 FIELD_PREP(0xff,
tmp[2]);
3bc61cfd6f4a57d Liam Beguin 2021-04-22 377
3bc61cfd6f4a57d Liam Beguin 2021-04-22 378 ret =
regmap_bulk_read(lmk->regmap, LMK04832_REG_PLL2_R_MSB, , 2);
3bc61cfd6f4a57d Liam Beguin 2021-04-22 379 if (ret)
3bc61cfd6f4a57d Liam Beguin 2021-04-22 380 return ret;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 381
3bc61cfd6f4a57d Liam Beguin 2021-04-22 382 pll2_r = FIELD_PREP(0x0f00,
tmp[0]) |
3bc61cfd6f4a57d Liam Beguin 2021-04-22 383 FIELD_PREP(0x00ff,
tmp[1]);
3bc61cfd6f4a57d Liam Beguin 2021-04-22 384
3bc61cfd6f4a57d Liam Beguin 2021-04-22 385 vco_rate = (prate <<
FIELD_GET(LMK04832_BIT_PLL2_MISC_REF_2X_EN,
3bc61cfd6f4a57d Liam Beguin 2021-04-22 386
pll2_misc)) * pll2_n * pll2_p[p] / pll2_r;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 387
3bc61cfd6f4a57d Liam Beguin 2021-04-22 388 return vco_rate;
3bc61cfd6f4a57d Liam Beguin 2021-04-22 389 };
3bc61cfd6f4a57d Liam Beguin 2021-04-22 390
--
0-DAY CI Kernel Test Service
https://01.org/lkp
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] mm/damon/dbgfs.c:227 dbgfs_schemes_write() warn: passing a valid pointer to 'PTR_ERR'
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: SeongJae Park
CC: Andrew Morton
CC: Linux Memory Management List
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: af122dd8f3c0099349bc98ff69f0d90efd8b149f mm/damon/dbgfs: support
DAMON-based Operation Schemes
date: 5 months ago
:: branch date: 13 hours ago
:: commit date: 5 months ago
config: arc-randconfig-m031-20220331
(https://download.01.org/0day-ci/archive/20220331/202203311938.cvfj0j5j-...@intel.com/config)
compiler: arceb-elf-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
smatch warnings:
mm/damon/dbgfs.c:227 dbgfs_schemes_write() warn: passing a valid pointer to
'PTR_ERR'
vim +/PTR_ERR +227 mm/damon/dbgfs.c
af122dd8f3c009 SeongJae Park 2021-11-05 215
af122dd8f3c009 SeongJae Park 2021-11-05 216 static ssize_t
dbgfs_schemes_write(struct file *file, const char __user *buf,
af122dd8f3c009 SeongJae Park 2021-11-05 217size_t count, loff_t
*ppos)
af122dd8f3c009 SeongJae Park 2021-11-05 218 {
af122dd8f3c009 SeongJae Park 2021-11-05 219struct damon_ctx *ctx =
file->private_data;
af122dd8f3c009 SeongJae Park 2021-11-05 220char *kbuf;
af122dd8f3c009 SeongJae Park 2021-11-05 221struct damos **schemes;
af122dd8f3c009 SeongJae Park 2021-11-05 222ssize_t nr_schemes = 0, ret =
count;
af122dd8f3c009 SeongJae Park 2021-11-05 223int err;
af122dd8f3c009 SeongJae Park 2021-11-05 224
af122dd8f3c009 SeongJae Park 2021-11-05 225kbuf = user_input_str(buf,
count, ppos);
af122dd8f3c009 SeongJae Park 2021-11-05 226if (IS_ERR(kbuf))
af122dd8f3c009 SeongJae Park 2021-11-05 @227return PTR_ERR(kbuf);
af122dd8f3c009 SeongJae Park 2021-11-05 228
af122dd8f3c009 SeongJae Park 2021-11-05 229schemes = str_to_schemes(kbuf,
ret, _schemes);
af122dd8f3c009 SeongJae Park 2021-11-05 230if (!schemes) {
af122dd8f3c009 SeongJae Park 2021-11-05 231ret = -EINVAL;
af122dd8f3c009 SeongJae Park 2021-11-05 232goto out;
af122dd8f3c009 SeongJae Park 2021-11-05 233}
af122dd8f3c009 SeongJae Park 2021-11-05 234
af122dd8f3c009 SeongJae Park 2021-11-05 235mutex_lock(>kdamond_lock);
af122dd8f3c009 SeongJae Park 2021-11-05 236if (ctx->kdamond) {
af122dd8f3c009 SeongJae Park 2021-11-05 237ret = -EBUSY;
af122dd8f3c009 SeongJae Park 2021-11-05 238goto unlock_out;
af122dd8f3c009 SeongJae Park 2021-11-05 239}
af122dd8f3c009 SeongJae Park 2021-11-05 240
af122dd8f3c009 SeongJae Park 2021-11-05 241err = damon_set_schemes(ctx,
schemes, nr_schemes);
af122dd8f3c009 SeongJae Park 2021-11-05 242if (err)
af122dd8f3c009 SeongJae Park 2021-11-05 243ret = err;
af122dd8f3c009 SeongJae Park 2021-11-05 244else
af122dd8f3c009 SeongJae Park 2021-11-05 245nr_schemes = 0;
af122dd8f3c009 SeongJae Park 2021-11-05 246 unlock_out:
af122dd8f3c009 SeongJae Park 2021-11-05 247
mutex_unlock(>kdamond_lock);
af122dd8f3c009 SeongJae Park 2021-11-05 248free_schemes_arr(schemes,
nr_schemes);
af122dd8f3c009 SeongJae Park 2021-11-05 249 out:
af122dd8f3c009 SeongJae Park 2021-11-05 250kfree(kbuf);
af122dd8f3c009 SeongJae Park 2021-11-05 251return ret;
af122dd8f3c009 SeongJae Park 2021-11-05 252 }
af122dd8f3c009 SeongJae Park 2021-11-05 253
--
0-DAY CI Kernel Test Service
https://01.org/lkp
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] drivers/input/joystick/adi.c:395:13-14: WARNING opportunity for min()
CC: kbuild-...@lists.01.org BCC: l...@intel.com CC: linux-ker...@vger.kernel.org TO: Denis Efremov CC: Julia Lawall tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 787af64d05cd528aac9ad16752d11bb1c6061bb9 commit: 5f66f73b9ff4dcabd4e2405ba9c32e80e02f9408 coccinelle: misc: add minmax script date: 11 months ago :: branch date: 10 hours ago :: commit date: 11 months ago config: i386-randconfig-c001 (https://download.01.org/0day-ci/archive/20220331/202203311613.ig9rkbjl-...@intel.com/config) compiler: gcc-11 (Debian 11.2.0-19) 11.2.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot Reported-by: Julia Lawall cocci warnings: (new ones prefixed by >>) >> drivers/input/joystick/adi.c:395:13-14: WARNING opportunity for min() Please review and possibly fold the followup patch. -- 0-DAY CI Kernel Test Service https://01.org/lkp ___ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] [PATCH] coccinelle: misc: fix minmax.cocci warnings
CC: kbuild-...@lists.01.org BCC: l...@intel.com CC: linux-ker...@vger.kernel.org TO: Denis Efremov CC: Julia Lawall CC: Dmitry Torokhov CC: linux-in...@vger.kernel.org CC: linux-ker...@vger.kernel.org From: kernel test robot drivers/input/joystick/adi.c:395:13-14: WARNING opportunity for min() Check for opencoded min(), max() implementations. Generated patches sometimes require adding a cast to fix compile warning. Warnings/patches scope intentionally limited to a function body. Generated by: scripts/coccinelle/misc/minmax.cocci CC: Denis Efremov Reported-by: kernel test robot Signed-off-by: kernel test robot --- tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 787af64d05cd528aac9ad16752d11bb1c6061bb9 commit: 5f66f73b9ff4dcabd4e2405ba9c32e80e02f9408 coccinelle: misc: add minmax script :: branch date: 10 hours ago :: commit date: 11 months ago Please take the patch only if it's a positive warning. Thanks! drivers/input/joystick/adi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/input/joystick/adi.c +++ b/drivers/input/joystick/adi.c @@ -392,7 +392,7 @@ static int adi_init_input(struct adi *ad if (!input_dev) return -ENOMEM; - t = adi->id < ADI_ID_MAX ? adi->id : ADI_ID_MAX; + t = min(adi->id, ADI_ID_MAX); snprintf(buf, ADI_MAX_PHYS_LENGTH, adi_names[t], adi->id); snprintf(adi->name, ADI_MAX_NAME_LENGTH, "Logitech %s [%s]", buf, adi->cname); ___ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] [esmil:visionfive 28/62] drivers/char/hw_random/starfive-vic-rng.c:209:2-9: line 209 is redundant because platform_get_irq() already prints an error
CC: kbuild-...@lists.01.org BCC: l...@intel.com CC: linux-ker...@vger.kernel.org TO: Huan Feng CC: Emil Renner Berthing tree: https://github.com/esmil/linux visionfive head: 6c7cd35771680c033c40b4af48cbe7e3093eab3d commit: 1d3f42fdd786e76c5138aff9663180a7f17c68bd [28/62] drivers/hw_random: Add StarFive JH7100 Random Number Generator driver :: branch date: 14 hours ago :: commit date: 9 days ago config: nios2-randconfig-c003-20220331 (https://download.01.org/0day-ci/archive/20220331/202203311416.frqojzon-...@intel.com/config) compiler: nios2-linux-gcc (GCC) 11.2.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot Reported-by: Julia Lawall cocci warnings: (new ones prefixed by >>) >> drivers/char/hw_random/starfive-vic-rng.c:209:2-9: line 209 is redundant >> because platform_get_irq() already prints an error Please review and possibly fold the followup patch. -- 0-DAY CI Kernel Test Service https://01.org/lkp ___ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] [PATCH] drivers/hw_random: fix platform_get_irq.cocci warnings
CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Huan Feng
CC: Emil Renner Berthing
From: kernel test robot
drivers/char/hw_random/starfive-vic-rng.c:209:2-9: line 209 is redundant
because platform_get_irq() already prints an error
Remove dev_err() messages after platform_get_irq*() failures
Generated by: scripts/coccinelle/api/platform_get_irq.cocci
CC: Huan Feng
Reported-by: kernel test robot
Signed-off-by: kernel test robot
---
tree: https://github.com/esmil/linux visionfive
head: 6c7cd35771680c033c40b4af48cbe7e3093eab3d
commit: 1d3f42fdd786e76c5138aff9663180a7f17c68bd [28/62] drivers/hw_random: Add
StarFive JH7100 Random Number Generator driver
:: branch date: 14 hours ago
:: commit date: 9 days ago
Please take the patch only if it's a positive warning. Thanks!
drivers/char/hw_random/starfive-vic-rng.c |1 -
1 file changed, 1 deletion(-)
--- a/drivers/char/hw_random/starfive-vic-rng.c
+++ b/drivers/char/hw_random/starfive-vic-rng.c
@@ -206,7 +206,6 @@ static int vic_rng_probe(struct platform
irq = platform_get_irq(pdev, 0);
if (irq <= 0) {
- dev_err(>dev, "Couldn't get irq %d\n", irq);
return irq;
}
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
