[kees:for-next/pstore 3/3] fs//pstore/inode.c:105:3: warning: 'return' with no value, in function returning non-void
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git for-next/pstore
head: 6099dcfe31fb9eee10c9f272244603dd890472ce
commit: 6099dcfe31fb9eee10c9f272244603dd890472ce [3/3] pstore: pstore_ftrace_seq_next should increase position index
config: x86_64-randconfig-s2-20200227 (attached as .config)
compiler: gcc-7 (Debian 7.5.0-5) 7.5.0
reproduce:
git checkout 6099dcfe31fb9eee10c9f272244603dd890472ce
# save the attached .config to linux build tree
make ARCH=x86_64
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
fs//pstore/inode.c: In function 'pstore_ftrace_seq_show':
>> fs//pstore/inode.c:105:3: warning: 'return' with no value, in function returning non-void [-Wreturn-type]
return;
^~~~~~
fs//pstore/inode.c:98:12: note: declared here
static int pstore_ftrace_seq_show(struct seq_file *s, void *v)
^~~~~~~~~~~~~~~~~~~~~~
Cyclomatic Complexity 5 include/linux/compiler.h:__read_once_size
Cyclomatic Complexity 5 include/linux/compiler.h:__write_once_size
Cyclomatic Complexity 1 arch/x86/include/asm/bitops.h:fls64
Cyclomatic Complexity 1 include/linux/log2.h:__ilog2_u64
Cyclomatic Complexity 1 include/linux/list.h:__list_del
Cyclomatic Complexity 1 arch/x86/include/asm/atomic64_64.h:arch_atomic64_read
Cyclomatic Complexity 1 include/linux/spinlock.h:spinlock_check
Cyclomatic Complexity 1 include/linux/spinlock.h:spin_unlock_irqrestore
Cyclomatic Complexity 1 include/linux/dcache.h:d_inode
Cyclomatic Complexity 3 include/linux/slab.h:kmalloc_type
Cyclomatic Complexity 28 include/linux/slab.h:kmalloc_index
Cyclomatic Complexity 1 include/linux/slab.h:kmalloc_large
Cyclomatic Complexity 4 include/linux/slab.h:kmalloc
Cyclomatic Complexity 1 include/linux/slab.h:kzalloc
Cyclomatic Complexity 1 include/linux/pstore.h:pstore_ftrace_decode_cpu
Cyclomatic Complexity 1 include/linux/pstore.h:pstore_ftrace_read_timestamp
Cyclomatic Complexity 2 fs//pstore/inode.c:pstore_ftrace_seq_next
Cyclomatic Complexity 1 fs//pstore/inode.c:pstore_kill_sb
Cyclomatic Complexity 1 fs//pstore/inode.c:pstore_mount
Cyclomatic Complexity 2 fs//pstore/inode.c:pstore_unlink
Cyclomatic Complexity 2 fs//pstore/inode.c:pstore_show_options
Cyclomatic Complexity 2 fs//pstore/inode.c:pstore_ftrace_seq_show
Cyclomatic Complexity 2 include/linux/list.h:__list_del_entry
Cyclomatic Complexity 1 include/linux/list.h:list_del
Cyclomatic Complexity 3 fs//pstore/inode.c:free_pstore_private
Cyclomatic Complexity 2 fs//pstore/inode.c:pstore_evict_inode
Cyclomatic Complexity 1 fs//pstore/inode.c:pstore_ftrace_seq_stop
Cyclomatic Complexity 3 fs//pstore/inode.c:pstore_ftrace_seq_start
Cyclomatic Complexity 6 fs//pstore/inode.c:parse_options
Cyclomatic Complexity 1 fs//pstore/inode.c:pstore_remount
Cyclomatic Complexity 2 fs//pstore/inode.c:pstore_get_inode
Cyclomatic Complexity 3 fs//pstore/inode.c:pstore_file_open
Cyclomatic Complexity 2 fs//pstore/inode.c:pstore_file_read
Cyclomatic Complexity 2 fs//pstore/inode.c:pstore_file_llseek
Cyclomatic Complexity 1 include/asm-generic/atomic-instrumented.h:atomic64_read
Cyclomatic Complexity 1 include/asm-generic/atomic-long.h:atomic_long_read
Cyclomatic Complexity 1 include/linux/rwsem.h:rwsem_is_locked
Cyclomatic Complexity 1 include/linux/fs.h:inode_is_locked
Cyclomatic Complexity 68 include/asm-generic/getorder.h:get_order
Cyclomatic Complexity 2 include/linux/list.h:__list_add
Cyclomatic Complexity 1 include/linux/list.h:list_add
Cyclomatic Complexity 1 include/linux/fs.h:inode_lock
Cyclomatic Complexity 1 include/linux/fs.h:inode_unlock
Cyclomatic Complexity 3 fs//pstore/inode.c:pstore_init_fs
Cyclomatic Complexity 1 fs//pstore/inode.c:pstore_exit_fs
Cyclomatic Complexity 1 fs//pstore/inode.c:pstore_is_mounted
Cyclomatic Complexity 12 fs//pstore/inode.c:pstore_mkfile
Cyclomatic Complexity 3 fs//pstore/inode.c:pstore_get_records
Cyclomatic Complexity 3 fs//pstore/inode.c:pstore_fill_super
Cyclomatic Complexity 1 fs//pstore/inode.c:_GLOBAL__sub_D_65535_0_pstore_is_mounted
Cyclomatic Complexity 1 fs//pstore/inode.c:_GLOBAL__sub_I_65535_1_pstore_is_mounted
vim +/return +105 fs//pstore/inode.c
97
98 static int pstore_ftrace_seq_show(struct seq_file *s, void *v)
99 {
100 struct pstore_private *ps = s->private;
101 struct pstore_ftrace_seq_data *data = v;
102 struct pstore_ftrace_record *rec;
103
104 if (!data)
> 105 return;
106
107 rec = (struct pstore_ftrace_record *)(ps->record->buf + data->off);
108
109 seq_printf(s, "CPU:%d ts:%llu %08lx %08lx %ps <- %pS\n",
110 pstore_ftrace_decode_cpu(rec),
111 pstore_ftrace_read_timestamp(rec),
112 rec->ip, rec->parent_ip, (void *)rec->ip,
113 (void *)rec->parent_ip);
114
115 return 0;
116 }
117
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
[broonie-misc:arm64-bti 8/11] arch/arm64/kernel/traps.c:731:14: error: 'sys64_hooks' undeclared; did you mean 'sys64_hook'?
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/misc.git arm64-bti
head: d6897bb309fc4ef374e1de8242eb94d1fb97c13b
commit: f224ac4523ed07315be3422867829cbc868c10ae [8/11] arm64: traps: Shuffle code to eliminate forward declarations
config: arm64-alldefconfig (attached as .config)
compiler: aarch64-linux-gcc (GCC) 7.5.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout f224ac4523ed07315be3422867829cbc868c10ae
# save the attached .config to linux build tree
GCC_VERSION=7.5.0 make.cross ARCH=arm64
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
arch/arm64/kernel/traps.c: In function 'do_sysinstr':
>> arch/arm64/kernel/traps.c:731:14: error: 'sys64_hooks' undeclared (first use in this function); did you mean 'sys64_hook'?
for (hook = sys64_hooks; hook->handler; hook++)
^~~~~~~~~~~
sys64_hook
arch/arm64/kernel/traps.c:731:14: note: each undeclared identifier is reported only once for each function it appears in
>> arch/arm64/kernel/traps.c:731:31: error: dereferencing pointer to incomplete type 'const struct sys64_hook'
for (hook = sys64_hooks; hook->handler; hook++)
^~
>> arch/arm64/kernel/traps.c:731:46: error: increment of pointer to an incomplete type 'const struct sys64_hook'
for (hook = sys64_hooks; hook->handler; hook++)
^~
vim +731 arch/arm64/kernel/traps.c
70c63cdfd6ee61 Marc Zyngier 2018-09-27 726
afa7c0e5b965cd James Morse 2019-10-25 727 void do_sysinstr(unsigned int esr, struct pt_regs *regs)
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 728 {
37143dcc44f8f3 Mark Rutland 2019-08-13 729 const struct sys64_hook *hook;
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 730
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 @731 for (hook = sys64_hooks; hook->handler; hook++)
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 732 if ((hook->esr_mask & esr) == hook->esr_val) {
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 733 hook->handler(esr, regs);
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 734 return;
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 735 }
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 736
49f6cba617fef4 Mark Rutland 2017-01-27 737 /*
49f6cba617fef4 Mark Rutland 2017-01-27 738 * New SYS instructions may previously have been undefined at EL0. Fall
49f6cba617fef4 Mark Rutland 2017-01-27 739 * back to our usual undefined instruction handler so that we handle
49f6cba617fef4 Mark Rutland 2017-01-27 740 * these consistently.
49f6cba617fef4 Mark Rutland 2017-01-27 741 */
49f6cba617fef4 Mark Rutland 2017-01-27 742 do_undefinstr(regs);
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 743 }
b6e43c0e3129ff James Morse 2019-10-25 744 NOKPROBE_SYMBOL(do_sysinstr);
9dbd5bb25c56e3 Suzuki K Poulose 2016-09-09 745
:::::: The code at line 731 was first introduced by commit
:::::: 9dbd5bb25c56e35e6b4c34d968689a1ded850924 arm64: Refactor sysinstr exception handling
:::::: TO: Suzuki K Poulose <suzuki.poulose(a)arm.com>
:::::: CC: Will Deacon <will.deacon(a)arm.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
Re: [PATCH v2 09/19] arm64: mte: Add specific SIGSEGV codes
by kbuild test robot
Hi Catalin,
I love your patch! Yet something to improve:
[auto build test ERROR on arm-soc/for-next]
[also build test ERROR on arm/for-next linus/master v5.6-rc3 next-20200226]
[cannot apply to arm64/for-next/core kvmarm/next xlnx/master arm-perf/for-next/perf]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Catalin-Marinas/arm64-Memory-Tag...
base: https://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc.git for-next
config: x86_64-defconfig (attached as .config)
compiler: gcc-7 (Debian 7.5.0-5) 7.5.0
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All error/warnings (new ones prefixed by >>):
In file included from include/linux/export.h:43:0,
from include/linux/linkage.h:7,
from arch/x86/include/asm/cache.h:5,
from include/linux/cache.h:6,
from include/linux/time.h:5,
from include/linux/compat.h:10,
from arch/x86/kernel/signal_compat.c:2:
In function 'signal_compat_build_tests',
inlined from 'sigaction_compat_abi' at arch/x86/kernel/signal_compat.c:166:2:
>> include/linux/compiler.h:350:38: error: call to '__compiletime_assert_30' declared with attribute error: BUILD_BUG_ON failed: NSIGSEGV != 7
_compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
^
include/linux/compiler.h:331:4: note: in definition of macro '__compiletime_assert'
prefix ## suffix(); \
^~~~~~
include/linux/compiler.h:350:2: note: in expansion of macro '_compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
^~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
#define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
^~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:50:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
^~~~~~~~~~~~~~~~
>> arch/x86/kernel/signal_compat.c:30:2: note: in expansion of macro 'BUILD_BUG_ON'
BUILD_BUG_ON(NSIGSEGV != 7);
^~~~~~~~~~~~
--
In file included from include/linux/export.h:43:0,
from include/linux/linkage.h:7,
from arch/x86/include/asm/cache.h:5,
from include/linux/cache.h:6,
from include/linux/time.h:5,
from include/linux/compat.h:10,
from arch/x86//kernel/signal_compat.c:2:
In function 'signal_compat_build_tests',
inlined from 'sigaction_compat_abi' at arch/x86//kernel/signal_compat.c:166:2:
>> include/linux/compiler.h:350:38: error: call to '__compiletime_assert_30' declared with attribute error: BUILD_BUG_ON failed: NSIGSEGV != 7
_compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
^
include/linux/compiler.h:331:4: note: in definition of macro '__compiletime_assert'
prefix ## suffix(); \
^~~~~~
include/linux/compiler.h:350:2: note: in expansion of macro '_compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
^~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
#define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
^~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:50:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
^~~~~~~~~~~~~~~~
arch/x86//kernel/signal_compat.c:30:2: note: in expansion of macro 'BUILD_BUG_ON'
BUILD_BUG_ON(NSIGSEGV != 7);
^~~~~~~~~~~~
vim +/__compiletime_assert_30 +350 include/linux/compiler.h
9a8ab1c39970a4 Daniel Santos 2013-02-21 336
9a8ab1c39970a4 Daniel Santos 2013-02-21 337 #define _compiletime_assert(condition, msg, prefix, suffix) \
9a8ab1c39970a4 Daniel Santos 2013-02-21 338 __compiletime_assert(condition, msg, prefix, suffix)
9a8ab1c39970a4 Daniel Santos 2013-02-21 339
9a8ab1c39970a4 Daniel Santos 2013-02-21 340 /**
9a8ab1c39970a4 Daniel Santos 2013-02-21 341 * compiletime_assert - break build and emit msg if condition is false
9a8ab1c39970a4 Daniel Santos 2013-02-21 342 * @condition: a compile-time constant condition to check
9a8ab1c39970a4 Daniel Santos 2013-02-21 343 * @msg: a message to emit if condition is false
9a8ab1c39970a4 Daniel Santos 2013-02-21 344 *
9a8ab1c39970a4 Daniel Santos 2013-02-21 345 * In tradition of POSIX assert, this macro will break the build if the
9a8ab1c39970a4 Daniel Santos 2013-02-21 346 * supplied condition is *false*, emitting the supplied error message if the
9a8ab1c39970a4 Daniel Santos 2013-02-21 347 * compiler has support to do so.
9a8ab1c39970a4 Daniel Santos 2013-02-21 348 */
9a8ab1c39970a4 Daniel Santos 2013-02-21 349 #define compiletime_assert(condition, msg) \
9a8ab1c39970a4 Daniel Santos 2013-02-21 @350 _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
9a8ab1c39970a4 Daniel Santos 2013-02-21 351
:::::: The code at line 350 was first introduced by commit
:::::: 9a8ab1c39970a4938a72d94e6fd13be88a797590 bug.h, compiler.h: introduce compiletime_assert & BUILD_BUG_ON_MSG
:::::: TO: Daniel Santos <daniel.santos(a)pobox.com>
:::::: CC: Linus Torvalds <torvalds(a)linux-foundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
Re: [PATCH] i2c:i2c-core-of:remove redundant dev_err message
by kbuild test robot
Hi tangbin,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on wsa/i2c/for-next]
[also build test WARNING on v5.6-rc3 next-20200226]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/tangbin/i2c-i2c-core-of-remove-r...
base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-next
config: i386-allyesconfig (attached as .config)
compiler: gcc-7 (Debian 7.5.0-5) 7.5.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/i2c/i2c-core-of.c: In function 'of_i2c_register_devices':
>> drivers/i2c/i2c-core-of.c:106:11: warning: 'return' with a value, in function returning void
return PTR_ERR(client);
^~~~~~~~~~~~~~~
drivers/i2c/i2c-core-of.c:85:6: note: declared here
void of_i2c_register_devices(struct i2c_adapter *adap)
^~~~~~~~~~~~~~~~~~~~~~~
>> drivers/i2c/i2c-core-of.c:106:11: warning: ignoring return value of 'PTR_ERR', declared with attribute warn_unused_result [-Wunused-result]
return PTR_ERR(client);
^~~~~~~~~~~~~~~
vim +/return +106 drivers/i2c/i2c-core-of.c
84
85 void of_i2c_register_devices(struct i2c_adapter *adap)
86 {
87 struct device_node *bus, *node;
88 struct i2c_client *client;
89
90 /* Only register child devices if the adapter has a node pointer set */
91 if (!adap->dev.of_node)
92 return;
93
94 dev_dbg(&adap->dev, "of_i2c: walking child nodes\n");
95
96 bus = of_get_child_by_name(adap->dev.of_node, "i2c-bus");
97 if (!bus)
98 bus = of_node_get(adap->dev.of_node);
99
100 for_each_available_child_of_node(bus, node) {
101 if (of_node_test_and_set_flag(node, OF_POPULATED))
102 continue;
103
104 client = of_i2c_register_device(adap, node);
105 if (IS_ERR(client)) {
> 106 return PTR_ERR(client);
107 of_node_clear_flag(node, OF_POPULATED);
108 }
109 }
110
111 of_node_put(bus);
112 }
113
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
Re: [PATCH] i2c:i2c-core-of:remove redundant dev_err message
by 唐彬
Hi tangbin,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on wsa/i2c/for-next]
[also build test WARNING on v5.6-rc3 next-20200226]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/tangbin/i2c-i2c-core-of-remove-r...
base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-next
config: i386-allyesconfig (attached as .config)
compiler: gcc-7 (Debian 7.5.0-5) 7.5.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/i2c/i2c-core-of.c: In function 'of_i2c_register_devices':
>> drivers/i2c/i2c-core-of.c:106:11: warning: 'return' with a value, in function returning void
return PTR_ERR(client);
^~~~~~~~~~~~~~~
drivers/i2c/i2c-core-of.c:85:6: note: declared here
void of_i2c_register_devices(struct i2c_adapter *adap)
^~~~~~~~~~~~~~~~~~~~~~~
>> drivers/i2c/i2c-core-of.c:106:11: warning: ignoring return value of 'PTR_ERR', declared with attribute warn_unused_result [-Wunused-result]
return PTR_ERR(client);
^~~~~~~~~~~~~~~
vim +/return +106 drivers/i2c/i2c-core-of.c
84
85 void of_i2c_register_devices(struct i2c_adapter *adap)
86 {
87 struct device_node *bus, *node;
88 struct i2c_client *client;
89
90 /* Only register child devices if the adapter has a node pointer set */
91 if (!adap->dev.of_node)
92 return;
93
94 dev_dbg(&adap->dev, "of_i2c: walking child nodes\n");
95
96 bus = of_get_child_by_name(adap->dev.of_node, "i2c-bus");
97 if (!bus)
98 bus = of_node_get(adap->dev.of_node);
99
100 for_each_available_child_of_node(bus, node) {
101 if (of_node_test_and_set_flag(node, OF_POPULATED))
102 continue;
103
104 client = of_i2c_register_device(adap, node);
105 if (IS_ERR(client)) {
> 106 return PTR_ERR(client);
107 of_node_clear_flag(node, OF_POPULATED);
108 }
109 }
110
111 of_node_put(bus);
112 }
113
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
Re: [RFC PATCH] USB: hub: fix port suspend/resume
by kbuild test robot
Hi Marco,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on usb/usb-testing]
[also build test WARNING on v5.6-rc3 next-20200227]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Marco-Felsch/USB-hub-fix-port-su...
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: x86_64-defconfig (attached as .config)
compiler: gcc-7 (Debian 7.5.0-5) 7.5.0
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings
All warnings (new ones prefixed by >>):
drivers/usb/core/port.c: In function '_usb_port_suspend':
>> drivers/usb/core/port.c:296:23: warning: 'retval' may be used uninitialized in this function [-Wmaybe-uninitialized]
if (retval && retval != -EAGAIN)
smatch warnings:
drivers/usb/core/port.c:296 _usb_port_suspend() error: uninitialized symbol 'retval'.
vim +/retval +296 drivers/usb/core/port.c
285
286 static int __maybe_unused _usb_port_suspend(struct device *dev)
287 {
288 struct usb_port *port_dev = to_usb_port(dev);
289 struct usb_device *udev = port_dev->child;
290 int retval;
291
292 if (!udev->do_remote_wakeup && udev->persist_enabled)
293 retval = usb_port_runtime_suspend(dev);
294
295 /* Do not force the user to enable the power-off feature */
> 296 if (retval && retval != -EAGAIN)
297 return retval;
298
299 return 0;
300 }
301
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
Re: [RFC PATCH] USB: hub: fix port suspend/resume
by kbuild test robot
Hi Marco,
[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on usb/usb-testing]
[also build test ERROR on v5.6-rc3 next-20200227]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Marco-Felsch/USB-hub-fix-port-su...
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: openrisc-randconfig-a001-20200227 (attached as .config)
compiler: or1k-linux-gcc (GCC) 9.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=9.2.0 make.cross ARCH=openrisc
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/usb/core/port.c: In function '_usb_port_suspend':
>> drivers/usb/core/port.c:292:11: error: 'struct usb_device' has no member named 'do_remote_wakeup'
292 | if (!udev->do_remote_wakeup && udev->persist_enabled)
| ^~
drivers/usb/core/port.c: In function '_usb_port_resume':
drivers/usb/core/port.c:307:11: error: 'struct usb_device' has no member named 'do_remote_wakeup'
307 | if (!udev->do_remote_wakeup && udev->persist_enabled)
| ^~
vim +292 drivers/usb/core/port.c
285
286 static int __maybe_unused _usb_port_suspend(struct device *dev)
287 {
288 struct usb_port *port_dev = to_usb_port(dev);
289 struct usb_device *udev = port_dev->child;
290 int retval;
291
> 292 if (!udev->do_remote_wakeup && udev->persist_enabled)
293 retval = usb_port_runtime_suspend(dev);
294
295 /* Do not force the user to enable the power-off feature */
296 if (retval && retval != -EAGAIN)
297 return retval;
298
299 return 0;
300 }
301
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
[PATCH] dmaengine: tegra-apb: fix platform_get_irq.cocci warnings
by Julia Lawall
From: kbuild test robot <lkp(a)intel.com>
Remove dev_err() messages after platform_get_irq*() failures.
platform_get_irq() already prints an error.
Generated by: scripts/coccinelle/api/platform_get_irq.cocci
Fixes: 6c41ac96ad92 ("dmaengine: tegra-apb: Support COMPILE_TEST")
CC: Dmitry Osipenko <digetx(a)gmail.com>
Signed-off-by: kbuild test robot <lkp(a)intel.com>
Signed-off-by: Julia Lawall <julia.lawall(a)inria.fr>
---
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: f3fef7e990dae5fcd7cd8ccbd9b2a98bdb481da8
commit: 6c41ac96ad9217fe2a6f31c9dcc31b97365b21f6 [4196/4617] dmaengine: tegra-apb: Support COMPILE_TEST
:::::: branch date: 17 hours ago
:::::: commit date: 2 days ago
Please take the patch only if it's a positive warning. Thanks!
tegra20-apb-dma.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/dma/tegra20-apb-dma.c
+++ b/drivers/dma/tegra20-apb-dma.c
@@ -1477,7 +1477,6 @@ static int tegra_dma_probe(struct platfo
irq = platform_get_irq(pdev, i);
if (irq < 0) {
ret = irq;
- dev_err(&pdev->dev, "No irq resource for chan %d\n", i);
goto err_pm_disable;
}
2 years, 3 months
Re: [RFC PATCH 2/2] KUnit: KASAN Integration
by kbuild test robot
Hi Patricia,
[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on tip/sched/core]
[also build test ERROR on linus/master v5.6-rc3]
[cannot apply to linux/master next-20200227]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Patricia-Alfonso/Port-KASAN-Test...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git a0f03b617c3b2644d3d47bf7d9e60aed01bd5b10
config: x86_64-defconfig (attached as .config)
compiler: clang version 11.0.0 (git://gitmirror/llvm_project 949134e2fefd34a38ed71de90dffe2300e2e1139)
reproduce:
# FIXME the reproduce steps for clang is not ready yet
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from arch/x86/entry/vdso/vdso32/vclock_gettime.c:31:
In file included from arch/x86/entry/vdso/vdso32/../vclock_gettime.c:15:
In file included from arch/x86/entry/vdso/../../../../lib/vdso/gettimeofday.c:26:
In file included from arch/x86/include/asm/vdso/gettimeofday.h:21:
In file included from include/clocksource/hyperv_timer.h:18:
In file included from arch/x86/include/asm/mshyperv.h:6:
In file included from include/linux/nmi.h:8:
In file included from include/linux/sched.h:35:
In file included from include/kunit/test.h:15:
In file included from include/linux/module.h:30:
>> arch/x86/include/asm/module.h:69:2: error: unknown processor family
#error unknown processor family
^
1 error generated.
vim +69 arch/x86/include/asm/module.h
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 15
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 16 #ifdef CONFIG_X86_64
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 17 /* X86_64 does not define MODULE_PROC_FAMILY */
87d6021b814353 arch/x86/include/asm/module.h Arnd Bergmann 2019-10-01 18 #elif defined CONFIG_M486SX
87d6021b814353 arch/x86/include/asm/module.h Arnd Bergmann 2019-10-01 19 #define MODULE_PROC_FAMILY "486SX "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 20 #elif defined CONFIG_M486
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 21 #define MODULE_PROC_FAMILY "486 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 22 #elif defined CONFIG_M586
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 23 #define MODULE_PROC_FAMILY "586 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 24 #elif defined CONFIG_M586TSC
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 25 #define MODULE_PROC_FAMILY "586TSC "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 26 #elif defined CONFIG_M586MMX
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 27 #define MODULE_PROC_FAMILY "586MMX "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 28 #elif defined CONFIG_MCORE2
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 29 #define MODULE_PROC_FAMILY "CORE2 "
366d19e181be87 arch/x86/include/asm/module.h Tobias Doerffel 2009-08-21 30 #elif defined CONFIG_MATOM
366d19e181be87 arch/x86/include/asm/module.h Tobias Doerffel 2009-08-21 31 #define MODULE_PROC_FAMILY "ATOM "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 32 #elif defined CONFIG_M686
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 33 #define MODULE_PROC_FAMILY "686 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 34 #elif defined CONFIG_MPENTIUMII
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 35 #define MODULE_PROC_FAMILY "PENTIUMII "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 36 #elif defined CONFIG_MPENTIUMIII
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 37 #define MODULE_PROC_FAMILY "PENTIUMIII "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 38 #elif defined CONFIG_MPENTIUMM
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 39 #define MODULE_PROC_FAMILY "PENTIUMM "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 40 #elif defined CONFIG_MPENTIUM4
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 41 #define MODULE_PROC_FAMILY "PENTIUM4 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 42 #elif defined CONFIG_MK6
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 43 #define MODULE_PROC_FAMILY "K6 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 44 #elif defined CONFIG_MK7
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 45 #define MODULE_PROC_FAMILY "K7 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 46 #elif defined CONFIG_MK8
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 47 #define MODULE_PROC_FAMILY "K8 "
ce9c99af8d4b3b arch/x86/include/asm/module.h Ian Campbell 2011-04-08 48 #elif defined CONFIG_MELAN
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 49 #define MODULE_PROC_FAMILY "ELAN "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 50 #elif defined CONFIG_MCRUSOE
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 51 #define MODULE_PROC_FAMILY "CRUSOE "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 52 #elif defined CONFIG_MEFFICEON
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 53 #define MODULE_PROC_FAMILY "EFFICEON "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 54 #elif defined CONFIG_MWINCHIPC6
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 55 #define MODULE_PROC_FAMILY "WINCHIPC6 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 56 #elif defined CONFIG_MWINCHIP3D
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 57 #define MODULE_PROC_FAMILY "WINCHIP3D "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 58 #elif defined CONFIG_MCYRIXIII
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 59 #define MODULE_PROC_FAMILY "CYRIXIII "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 60 #elif defined CONFIG_MVIAC3_2
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 61 #define MODULE_PROC_FAMILY "VIAC3-2 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 62 #elif defined CONFIG_MVIAC7
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 63 #define MODULE_PROC_FAMILY "VIAC7 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 64 #elif defined CONFIG_MGEODEGX1
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 65 #define MODULE_PROC_FAMILY "GEODEGX1 "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 66 #elif defined CONFIG_MGEODE_LX
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 67 #define MODULE_PROC_FAMILY "GEODE "
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 68 #else
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 @69 #error unknown processor family
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 70 #endif
0ddc9cc8fdfe3d include/asm-x86/module.h Harvey Harrison 2007-12-18 71
:::::: The code at line 69 was first introduced by commit
:::::: 0ddc9cc8fdfe3df7a90557e66069e3da2c584725 x86: unify module_{32|64}.h
:::::: TO: Harvey Harrison <harvey.harrison(a)gmail.com>
:::::: CC: Ingo Molnar <mingo(a)elte.hu>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months