From: kernel test robot <lkp@intel.com> To: Saurav Girepunje <saurav.girepunje@gmail.com>, Larry.Finger@lwfinger.net, phil@philpotter.co.uk, gregkh@linuxfoundation.org, fabioaiuto83@gmail.com, straube.linux@gmail.com, ross.schm.dev@gmail.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Cc: kbuild-all@lists.01.org, saurav.girepunje@hotmail.com Subject: Re: [PATCH] staging: r8188eu: core: remove null check before vfree Date: Sun, 29 Aug 2021 03:22:31 +0800 [thread overview] Message-ID: <202108290317.UhD2FXl6-lkp@intel.com> (raw) In-Reply-To: <YSp4UP1+HrhmDA3C@user> [-- Attachment #1: Type: text/plain, Size: 3592 bytes --] Hi Saurav, Thank you for the patch! Yet something to improve: [auto build test ERROR on staging/staging-testing] url: https://github.com/0day-ci/linux/commits/Saurav-Girepunje/staging-r8188eu-core-remove-null-check-before-vfree/20210829-015616 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 4adb389e08c95fdf91995271932c59250ff0d561 config: sh-allmodconfig (attached as .config) compiler: sh4-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://github.com/0day-ci/linux/commit/9c7f84adad9db1009055b063408e70c89fc51170 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Saurav-Girepunje/staging-r8188eu-core-remove-null-check-before-vfree/20210829-015616 git checkout 9c7f84adad9db1009055b063408e70c89fc51170 # 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=sh SHELL=/bin/bash drivers/staging/ 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 >>): drivers/staging/r8188eu/core/rtw_sta_mgt.c: In function '_rtw_free_sta_priv': >> drivers/staging/r8188eu/core/rtw_sta_mgt.c:163:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] 163 | struct sta_info *rtw_alloc_stainfo(struct sta_priv *pstapriv, u8 *hwaddr) | ^~~~~~ >> drivers/staging/r8188eu/core/rtw_sta_mgt.c:506:1: error: expected declaration or statement at end of input 506 | } | ^ drivers/staging/r8188eu/core/rtw_sta_mgt.c:507: error: control reaches end of non-void function [-Werror=return-type] At top level: drivers/staging/r8188eu/core/rtw_sta_mgt.c:469:4: warning: 'rtw_access_ctrl' defined but not used [-Wunused-function] 469 | u8 rtw_access_ctrl(struct adapter *padapter, u8 *mac_addr) | ^~~~~~~~~~~~~~~ drivers/staging/r8188eu/core/rtw_sta_mgt.c:458:18: warning: 'rtw_get_bcmc_stainfo' defined but not used [-Wunused-function] 458 | struct sta_info *rtw_get_bcmc_stainfo(struct adapter *padapter) | ^~~~~~~~~~~~~~~~~~~~ drivers/staging/r8188eu/core/rtw_sta_mgt.c:436:5: warning: 'rtw_init_bcmc_stainfo' defined but not used [-Wunused-function] 436 | u32 rtw_init_bcmc_stainfo(struct adapter *padapter) | ^~~~~~~~~~~~~~~~~~~~~ drivers/staging/r8188eu/core/rtw_sta_mgt.c:367:6: warning: 'rtw_free_all_stainfo' defined but not used [-Wunused-function] 367 | void rtw_free_all_stainfo(struct adapter *padapter) | ^~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors Kconfig warnings: (for reference only) WARNING: unmet direct dependencies detected for SND_ATMEL_SOC_PDC Depends on SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC && HAS_DMA Selected by - SND_ATMEL_SOC_SSC && SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC - SND_ATMEL_SOC_SSC_PDC && SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC && ATMEL_SSC vim +506 drivers/staging/r8188eu/core/rtw_sta_mgt.c 15865124feed88 Phillip Potter 2021-07-28 504 15865124feed88 Phillip Potter 2021-07-28 505 return res; 15865124feed88 Phillip Potter 2021-07-28 @506 } --- 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: 55027 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [PATCH] staging: r8188eu: core: remove null check before vfree Date: Sun, 29 Aug 2021 03:22:31 +0800 [thread overview] Message-ID: <202108290317.UhD2FXl6-lkp@intel.com> (raw) In-Reply-To: <YSp4UP1+HrhmDA3C@user> [-- Attachment #1: Type: text/plain, Size: 3660 bytes --] Hi Saurav, Thank you for the patch! Yet something to improve: [auto build test ERROR on staging/staging-testing] url: https://github.com/0day-ci/linux/commits/Saurav-Girepunje/staging-r8188eu-core-remove-null-check-before-vfree/20210829-015616 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 4adb389e08c95fdf91995271932c59250ff0d561 config: sh-allmodconfig (attached as .config) compiler: sh4-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://github.com/0day-ci/linux/commit/9c7f84adad9db1009055b063408e70c89fc51170 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Saurav-Girepunje/staging-r8188eu-core-remove-null-check-before-vfree/20210829-015616 git checkout 9c7f84adad9db1009055b063408e70c89fc51170 # 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=sh SHELL=/bin/bash drivers/staging/ 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 >>): drivers/staging/r8188eu/core/rtw_sta_mgt.c: In function '_rtw_free_sta_priv': >> drivers/staging/r8188eu/core/rtw_sta_mgt.c:163:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] 163 | struct sta_info *rtw_alloc_stainfo(struct sta_priv *pstapriv, u8 *hwaddr) | ^~~~~~ >> drivers/staging/r8188eu/core/rtw_sta_mgt.c:506:1: error: expected declaration or statement at end of input 506 | } | ^ drivers/staging/r8188eu/core/rtw_sta_mgt.c:507: error: control reaches end of non-void function [-Werror=return-type] At top level: drivers/staging/r8188eu/core/rtw_sta_mgt.c:469:4: warning: 'rtw_access_ctrl' defined but not used [-Wunused-function] 469 | u8 rtw_access_ctrl(struct adapter *padapter, u8 *mac_addr) | ^~~~~~~~~~~~~~~ drivers/staging/r8188eu/core/rtw_sta_mgt.c:458:18: warning: 'rtw_get_bcmc_stainfo' defined but not used [-Wunused-function] 458 | struct sta_info *rtw_get_bcmc_stainfo(struct adapter *padapter) | ^~~~~~~~~~~~~~~~~~~~ drivers/staging/r8188eu/core/rtw_sta_mgt.c:436:5: warning: 'rtw_init_bcmc_stainfo' defined but not used [-Wunused-function] 436 | u32 rtw_init_bcmc_stainfo(struct adapter *padapter) | ^~~~~~~~~~~~~~~~~~~~~ drivers/staging/r8188eu/core/rtw_sta_mgt.c:367:6: warning: 'rtw_free_all_stainfo' defined but not used [-Wunused-function] 367 | void rtw_free_all_stainfo(struct adapter *padapter) | ^~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors Kconfig warnings: (for reference only) WARNING: unmet direct dependencies detected for SND_ATMEL_SOC_PDC Depends on SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC && HAS_DMA Selected by - SND_ATMEL_SOC_SSC && SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC - SND_ATMEL_SOC_SSC_PDC && SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC && ATMEL_SSC vim +506 drivers/staging/r8188eu/core/rtw_sta_mgt.c 15865124feed88 Phillip Potter 2021-07-28 504 15865124feed88 Phillip Potter 2021-07-28 505 return res; 15865124feed88 Phillip Potter 2021-07-28 @506 } --- 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: 55027 bytes --]
next prev parent reply other threads:[~2021-08-28 19:22 UTC|newest] Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-08-28 17:54 [PATCH] staging: r8188eu: core: remove null check before vfree Saurav Girepunje 2021-08-28 19:20 ` Michael Straube 2021-08-28 19:30 ` Michael Straube 2021-08-28 19:47 ` Kari Argillander 2021-08-28 19:56 ` Michael Straube 2021-08-28 19:22 ` kernel test robot [this message] 2021-08-28 19:22 ` 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=202108290317.UhD2FXl6-lkp@intel.com \ --to=lkp@intel.com \ --cc=Larry.Finger@lwfinger.net \ --cc=fabioaiuto83@gmail.com \ --cc=gregkh@linuxfoundation.org \ --cc=kbuild-all@lists.01.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-staging@lists.linux.dev \ --cc=phil@philpotter.co.uk \ --cc=ross.schm.dev@gmail.com \ --cc=saurav.girepunje@gmail.com \ --cc=saurav.girepunje@hotmail.com \ --cc=straube.linux@gmail.com \ /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.