From: kernel test robot <lkp@intel.com> To: Thomas Gleixner <tglx@linutronix.de> Cc: kbuild-all@lists.01.org, linux-kernel@vger.kernel.org Subject: [tglx-devel:hrtimer 5/12] net/can/bcm.c:628:60: error: expected ')' before ';' token Date: Mon, 20 Sep 2021 07:33:32 +0800 [thread overview] Message-ID: <202109200723.1K3ut3o0-lkp@intel.com> (raw) [-- Attachment #1: Type: text/plain, Size: 4795 bytes --] tree: https://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git hrtimer head: 16c372aeaca31d5d9d998466e75e845a2926685f commit: df0b0ce19903f84991732242cb7c058dd229d587 [5/12] can: bcm: Use hrtimer_forward_now() config: nds32-randconfig-r026-20210919 (attached as .config) compiler: nds32le-linux-gcc (GCC) 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/tglx/devel.git/commit/?id=df0b0ce19903f84991732242cb7c058dd229d587 git remote add tglx-devel https://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git git fetch --no-tags tglx-devel hrtimer git checkout df0b0ce19903f84991732242cb7c058dd229d587 # save the attached .config to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=nds32 SHELL=/bin/bash net/can/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All error/warnings (new ones prefixed by >>): net/can/bcm.c: In function 'bcm_rx_thr_handler': net/can/bcm.c:628:45: warning: left-hand operand of comma expression has no effect [-Wunused-value] 628 | hrtimer_forward_now((hrtimer, op->kt_ival2); | ^ >> net/can/bcm.c:628:60: error: expected ')' before ';' token 628 | hrtimer_forward_now((hrtimer, op->kt_ival2); | ~ ^ | ) >> net/can/bcm.c:628:45: warning: passing argument 1 of 'hrtimer_forward_now' makes pointer from integer without a cast [-Wint-conversion] 628 | hrtimer_forward_now((hrtimer, op->kt_ival2); | ~~~~~~~~^~~~~~~~~~~~~~~ | | | ktime_t {aka long long int} In file included from include/linux/sched.h:19, from include/linux/sched/task_stack.h:9, from arch/nds32/include/asm/fpu.h:9, from arch/nds32/include/asm/elf.h:12, from include/linux/elf.h:6, from include/linux/module.h:19, from net/can/bcm.c:43: include/linux/hrtimer.h:503:55: note: expected 'struct hrtimer *' but argument is of type 'ktime_t' {aka 'long long int'} 503 | static inline u64 hrtimer_forward_now(struct hrtimer *timer, | ~~~~~~~~~~~~~~~~^~~~~ >> net/can/bcm.c:628:17: error: too few arguments to function 'hrtimer_forward_now' 628 | hrtimer_forward_now((hrtimer, op->kt_ival2); | ^~~~~~~~~~~~~~~~~~~ In file included from include/linux/sched.h:19, from include/linux/sched/task_stack.h:9, from arch/nds32/include/asm/fpu.h:9, from arch/nds32/include/asm/elf.h:12, from include/linux/elf.h:6, from include/linux/module.h:19, from net/can/bcm.c:43: include/linux/hrtimer.h:503:19: note: declared here 503 | static inline u64 hrtimer_forward_now(struct hrtimer *timer, | ^~~~~~~~~~~~~~~~~~~ >> net/can/bcm.c:629:40: error: expected ';' before '}' token 629 | return HRTIMER_RESTART; | ^ | ; 630 | } else { | ~ net/can/bcm.c:635:1: error: control reaches end of non-void function [-Werror=return-type] 635 | } | ^ cc1: some warnings being treated as errors vim +628 net/can/bcm.c 618 619 /* 620 * bcm_rx_thr_handler - the time for blocked content updates is over now: 621 * Check for throttled data and send it to the userspace 622 */ 623 static enum hrtimer_restart bcm_rx_thr_handler(struct hrtimer *hrtimer) 624 { 625 struct bcm_op *op = container_of(hrtimer, struct bcm_op, thrtimer); 626 627 if (bcm_rx_thr_flush(op)) { > 628 hrtimer_forward_now((hrtimer, op->kt_ival2); > 629 return HRTIMER_RESTART; 630 } else { 631 /* rearm throttle handling */ 632 op->kt_lastmsg = 0; 633 return HRTIMER_NORESTART; 634 } 635 } 636 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 33606 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: [tglx-devel:hrtimer 5/12] net/can/bcm.c:628:60: error: expected ')' before '; ' token Date: Mon, 20 Sep 2021 07:33:32 +0800 [thread overview] Message-ID: <202109200723.1K3ut3o0-lkp@intel.com> (raw) [-- Attachment #1: Type: text/plain, Size: 4891 bytes --] tree: https://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git hrtimer head: 16c372aeaca31d5d9d998466e75e845a2926685f commit: df0b0ce19903f84991732242cb7c058dd229d587 [5/12] can: bcm: Use hrtimer_forward_now() config: nds32-randconfig-r026-20210919 (attached as .config) compiler: nds32le-linux-gcc (GCC) 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/tglx/devel.git/commit/?id=df0b0ce19903f84991732242cb7c058dd229d587 git remote add tglx-devel https://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git git fetch --no-tags tglx-devel hrtimer git checkout df0b0ce19903f84991732242cb7c058dd229d587 # save the attached .config to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=nds32 SHELL=/bin/bash net/can/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All error/warnings (new ones prefixed by >>): net/can/bcm.c: In function 'bcm_rx_thr_handler': net/can/bcm.c:628:45: warning: left-hand operand of comma expression has no effect [-Wunused-value] 628 | hrtimer_forward_now((hrtimer, op->kt_ival2); | ^ >> net/can/bcm.c:628:60: error: expected ')' before ';' token 628 | hrtimer_forward_now((hrtimer, op->kt_ival2); | ~ ^ | ) >> net/can/bcm.c:628:45: warning: passing argument 1 of 'hrtimer_forward_now' makes pointer from integer without a cast [-Wint-conversion] 628 | hrtimer_forward_now((hrtimer, op->kt_ival2); | ~~~~~~~~^~~~~~~~~~~~~~~ | | | ktime_t {aka long long int} In file included from include/linux/sched.h:19, from include/linux/sched/task_stack.h:9, from arch/nds32/include/asm/fpu.h:9, from arch/nds32/include/asm/elf.h:12, from include/linux/elf.h:6, from include/linux/module.h:19, from net/can/bcm.c:43: include/linux/hrtimer.h:503:55: note: expected 'struct hrtimer *' but argument is of type 'ktime_t' {aka 'long long int'} 503 | static inline u64 hrtimer_forward_now(struct hrtimer *timer, | ~~~~~~~~~~~~~~~~^~~~~ >> net/can/bcm.c:628:17: error: too few arguments to function 'hrtimer_forward_now' 628 | hrtimer_forward_now((hrtimer, op->kt_ival2); | ^~~~~~~~~~~~~~~~~~~ In file included from include/linux/sched.h:19, from include/linux/sched/task_stack.h:9, from arch/nds32/include/asm/fpu.h:9, from arch/nds32/include/asm/elf.h:12, from include/linux/elf.h:6, from include/linux/module.h:19, from net/can/bcm.c:43: include/linux/hrtimer.h:503:19: note: declared here 503 | static inline u64 hrtimer_forward_now(struct hrtimer *timer, | ^~~~~~~~~~~~~~~~~~~ >> net/can/bcm.c:629:40: error: expected ';' before '}' token 629 | return HRTIMER_RESTART; | ^ | ; 630 | } else { | ~ net/can/bcm.c:635:1: error: control reaches end of non-void function [-Werror=return-type] 635 | } | ^ cc1: some warnings being treated as errors vim +628 net/can/bcm.c 618 619 /* 620 * bcm_rx_thr_handler - the time for blocked content updates is over now: 621 * Check for throttled data and send it to the userspace 622 */ 623 static enum hrtimer_restart bcm_rx_thr_handler(struct hrtimer *hrtimer) 624 { 625 struct bcm_op *op = container_of(hrtimer, struct bcm_op, thrtimer); 626 627 if (bcm_rx_thr_flush(op)) { > 628 hrtimer_forward_now((hrtimer, op->kt_ival2); > 629 return HRTIMER_RESTART; 630 } else { 631 /* rearm throttle handling */ 632 op->kt_lastmsg = 0; 633 return HRTIMER_NORESTART; 634 } 635 } 636 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org [-- Attachment #2: config.gz --] [-- Type: application/gzip, Size: 33606 bytes --]
next reply other threads:[~2021-09-19 23:33 UTC|newest] Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-09-19 23:33 kernel test robot [this message] 2021-09-19 23:33 ` [tglx-devel:hrtimer 5/12] net/can/bcm.c:628:60: error: expected ')' before '; ' token kernel test robot
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=202109200723.1K3ut3o0-lkp@intel.com \ --to=lkp@intel.com \ --cc=kbuild-all@lists.01.org \ --cc=linux-kernel@vger.kernel.org \ --cc=tglx@linutronix.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.