From: kernel test robot <lkp@intel.com> To: David Gow <davidgow@google.com>, Brendan Higgins <brendanhiggins@google.com>, Luis Chamberlain <mcgrof@kernel.org>, Jeremy Kerr <jk@codeconstruct.com.au>, Daniel Latypov <dlatypov@google.com>, Shuah Khan <skhan@linuxfoundation.org>, Andrew Jeffery <andrew@aj.id.au>, Mika Westerberg <mika.westerberg@linux.intel.com>, Andra Paraschiv <andraprs@amazon.com>, Longpeng <longpeng2@huawei.com> Cc: kbuild-all@lists.01.org, "David Gow" <davidgow@google.com>, linux-aspeed@lists.ozlabs.org, "Maíra Canal" <maira.canal@usp.br>, openbmc@lists.ozlabs.org, linux-usb@vger.kernel.org, linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, "Matt Johnston" <matt@codeconstruct.com.au>, linux-modules@vger.kernel.org, kunit-dev@googlegroups.com Subject: Re: [PATCH 2/5] kunit: flatten kunit_suite*** to kunit_suite** in .kunit_test_suites Date: Sat, 18 Jun 2022 21:58:28 +0800 [thread overview] Message-ID: <202206182117.58z5vWxq-lkp@intel.com> (raw) In-Reply-To: <20220618090310.1174932-3-davidgow@google.com> Hi David, Thank you for the patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.19-rc2 next-20220617] [cannot apply to mcgrof/modules-next] [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/David-Gow/Rework-KUnit-test-execution-in-modules/20220618-170653 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 4b35035bcf80ddb47c0112c4fbd84a63a2836a18 config: parisc-allyesconfig (https://download.01.org/0day-ci/archive/20220618/202206182117.58z5vWxq-lkp@intel.com/config) compiler: hppa-linux-gcc (GCC) 11.3.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/intel-lab-lkp/linux/commit/c2386c54cc9fd471e5353f375ff71734214ed3c6 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review David-Gow/Rework-KUnit-test-execution-in-modules/20220618-170653 git checkout c2386c54cc9fd471e5353f375ff71734214ed3c6 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=parisc SHELL=/bin/bash If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@intel.com> Note: the linux-review/David-Gow/Rework-KUnit-test-execution-in-modules/20220618-170653 HEAD fddb3ea0ed5627098eabc542fdba5a8b4b769066 builds fine. It only hurts bisectability. All errors (new ones prefixed by >>): drivers/mmc/host/sdhci-of-aspeed.c: In function 'aspeed_sdc_tests_init': >> drivers/mmc/host/sdhci-of-aspeed.c:612:16: error: too few arguments to function '__kunit_test_suites_init' 612 | return __kunit_test_suites_init(aspeed_sdc_test_suites); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from drivers/mmc/host/sdhci-of-aspeed-test.c:4, from drivers/mmc/host/sdhci-of-aspeed.c:608: include/kunit/test.h:240:5: note: declared here 240 | int __kunit_test_suites_init(struct kunit_suite * const * const suites, int num_suites); | ^~~~~~~~~~~~~~~~~~~~~~~~ drivers/mmc/host/sdhci-of-aspeed.c: In function 'aspeed_sdc_tests_exit': >> drivers/mmc/host/sdhci-of-aspeed.c:617:9: error: too few arguments to function '__kunit_test_suites_exit' 617 | __kunit_test_suites_exit(aspeed_sdc_test_suites); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from drivers/mmc/host/sdhci-of-aspeed-test.c:4, from drivers/mmc/host/sdhci-of-aspeed.c:608: include/kunit/test.h:242:6: note: declared here 242 | void __kunit_test_suites_exit(struct kunit_suite **suites, int num_suites); | ^~~~~~~~~~~~~~~~~~~~~~~~ drivers/mmc/host/sdhci-of-aspeed.c: In function 'aspeed_sdc_tests_init': drivers/mmc/host/sdhci-of-aspeed.c:613:1: error: control reaches end of non-void function [-Werror=return-type] 613 | } | ^ cc1: some warnings being treated as errors vim +/__kunit_test_suites_init +612 drivers/mmc/host/sdhci-of-aspeed.c 4af307f574260c Andrew Jeffery 2021-01-22 609 4af307f574260c Andrew Jeffery 2021-01-22 610 static inline int aspeed_sdc_tests_init(void) 4af307f574260c Andrew Jeffery 2021-01-22 611 { 4af307f574260c Andrew Jeffery 2021-01-22 @612 return __kunit_test_suites_init(aspeed_sdc_test_suites); 4af307f574260c Andrew Jeffery 2021-01-22 613 } 4af307f574260c Andrew Jeffery 2021-01-22 614 4af307f574260c Andrew Jeffery 2021-01-22 615 static inline void aspeed_sdc_tests_exit(void) 4af307f574260c Andrew Jeffery 2021-01-22 616 { 4af307f574260c Andrew Jeffery 2021-01-22 @617 __kunit_test_suites_exit(aspeed_sdc_test_suites); 4af307f574260c Andrew Jeffery 2021-01-22 618 } 4af307f574260c Andrew Jeffery 2021-01-22 619 #else 4af307f574260c Andrew Jeffery 2021-01-22 620 static inline int aspeed_sdc_tests_init(void) 4af307f574260c Andrew Jeffery 2021-01-22 621 { 4af307f574260c Andrew Jeffery 2021-01-22 622 return 0; 4af307f574260c Andrew Jeffery 2021-01-22 623 } 4af307f574260c Andrew Jeffery 2021-01-22 624 -- 0-DAY CI Kernel Test Service https://01.org/lkp
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: David Gow <davidgow@google.com>, Brendan Higgins <brendanhiggins@google.com>, Luis Chamberlain <mcgrof@kernel.org>, Jeremy Kerr <jk@codeconstruct.com.au>, Daniel Latypov <dlatypov@google.com>, Shuah Khan <skhan@linuxfoundation.org>, Andrew Jeffery <andrew@aj.id.au>, Mika Westerberg <mika.westerberg@linux.intel.com>, Andra Paraschiv <andraprs@amazon.com>, Longpeng <longpeng2@huawei.com> Cc: kbuild-all@lists.01.org, kunit-dev@googlegroups.com, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org, "Maíra Canal" <maira.canal@usp.br>, linux-mmc@vger.kernel.org, linux-aspeed@lists.ozlabs.org, openbmc@lists.ozlabs.org, linux-usb@vger.kernel.org, linux-modules@vger.kernel.org, "Matt Johnston" <matt@codeconstruct.com.au>, "David Gow" <davidgow@google.com> Subject: Re: [PATCH 2/5] kunit: flatten kunit_suite*** to kunit_suite** in .kunit_test_suites Date: Sat, 18 Jun 2022 21:58:28 +0800 [thread overview] Message-ID: <202206182117.58z5vWxq-lkp@intel.com> (raw) In-Reply-To: <20220618090310.1174932-3-davidgow@google.com> Hi David, Thank you for the patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.19-rc2 next-20220617] [cannot apply to mcgrof/modules-next] [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/David-Gow/Rework-KUnit-test-execution-in-modules/20220618-170653 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 4b35035bcf80ddb47c0112c4fbd84a63a2836a18 config: parisc-allyesconfig (https://download.01.org/0day-ci/archive/20220618/202206182117.58z5vWxq-lkp@intel.com/config) compiler: hppa-linux-gcc (GCC) 11.3.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/intel-lab-lkp/linux/commit/c2386c54cc9fd471e5353f375ff71734214ed3c6 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review David-Gow/Rework-KUnit-test-execution-in-modules/20220618-170653 git checkout c2386c54cc9fd471e5353f375ff71734214ed3c6 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=parisc SHELL=/bin/bash If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@intel.com> Note: the linux-review/David-Gow/Rework-KUnit-test-execution-in-modules/20220618-170653 HEAD fddb3ea0ed5627098eabc542fdba5a8b4b769066 builds fine. It only hurts bisectability. All errors (new ones prefixed by >>): drivers/mmc/host/sdhci-of-aspeed.c: In function 'aspeed_sdc_tests_init': >> drivers/mmc/host/sdhci-of-aspeed.c:612:16: error: too few arguments to function '__kunit_test_suites_init' 612 | return __kunit_test_suites_init(aspeed_sdc_test_suites); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from drivers/mmc/host/sdhci-of-aspeed-test.c:4, from drivers/mmc/host/sdhci-of-aspeed.c:608: include/kunit/test.h:240:5: note: declared here 240 | int __kunit_test_suites_init(struct kunit_suite * const * const suites, int num_suites); | ^~~~~~~~~~~~~~~~~~~~~~~~ drivers/mmc/host/sdhci-of-aspeed.c: In function 'aspeed_sdc_tests_exit': >> drivers/mmc/host/sdhci-of-aspeed.c:617:9: error: too few arguments to function '__kunit_test_suites_exit' 617 | __kunit_test_suites_exit(aspeed_sdc_test_suites); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from drivers/mmc/host/sdhci-of-aspeed-test.c:4, from drivers/mmc/host/sdhci-of-aspeed.c:608: include/kunit/test.h:242:6: note: declared here 242 | void __kunit_test_suites_exit(struct kunit_suite **suites, int num_suites); | ^~~~~~~~~~~~~~~~~~~~~~~~ drivers/mmc/host/sdhci-of-aspeed.c: In function 'aspeed_sdc_tests_init': drivers/mmc/host/sdhci-of-aspeed.c:613:1: error: control reaches end of non-void function [-Werror=return-type] 613 | } | ^ cc1: some warnings being treated as errors vim +/__kunit_test_suites_init +612 drivers/mmc/host/sdhci-of-aspeed.c 4af307f574260c Andrew Jeffery 2021-01-22 609 4af307f574260c Andrew Jeffery 2021-01-22 610 static inline int aspeed_sdc_tests_init(void) 4af307f574260c Andrew Jeffery 2021-01-22 611 { 4af307f574260c Andrew Jeffery 2021-01-22 @612 return __kunit_test_suites_init(aspeed_sdc_test_suites); 4af307f574260c Andrew Jeffery 2021-01-22 613 } 4af307f574260c Andrew Jeffery 2021-01-22 614 4af307f574260c Andrew Jeffery 2021-01-22 615 static inline void aspeed_sdc_tests_exit(void) 4af307f574260c Andrew Jeffery 2021-01-22 616 { 4af307f574260c Andrew Jeffery 2021-01-22 @617 __kunit_test_suites_exit(aspeed_sdc_test_suites); 4af307f574260c Andrew Jeffery 2021-01-22 618 } 4af307f574260c Andrew Jeffery 2021-01-22 619 #else 4af307f574260c Andrew Jeffery 2021-01-22 620 static inline int aspeed_sdc_tests_init(void) 4af307f574260c Andrew Jeffery 2021-01-22 621 { 4af307f574260c Andrew Jeffery 2021-01-22 622 return 0; 4af307f574260c Andrew Jeffery 2021-01-22 623 } 4af307f574260c Andrew Jeffery 2021-01-22 624 -- 0-DAY CI Kernel Test Service https://01.org/lkp
next prev parent reply other threads:[~2022-06-18 14:00 UTC|newest] Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-06-18 9:03 [PATCH 0/5] Rework KUnit test execution in modules David Gow 2022-06-18 9:03 ` David Gow 2022-06-18 9:03 ` [PATCH 1/5] kunit: unify module and builtin suite definitions David Gow 2022-06-18 9:03 ` David Gow 2022-06-18 11:36 ` kernel test robot 2022-06-18 11:36 ` kernel test robot 2022-06-18 12:47 ` kernel test robot 2022-06-18 12:47 ` kernel test robot 2022-06-18 9:03 ` [PATCH 2/5] kunit: flatten kunit_suite*** to kunit_suite** in .kunit_test_suites David Gow 2022-06-18 9:03 ` David Gow 2022-06-18 13:58 ` kernel test robot [this message] 2022-06-18 13:58 ` kernel test robot 2022-06-18 14:49 ` kernel test robot 2022-06-18 14:49 ` kernel test robot 2022-06-18 9:03 ` [PATCH 3/5] thunderbolt: test: Use kunit_test_suite() macro David Gow 2022-06-18 9:03 ` David Gow 2022-06-20 5:05 ` Mika Westerberg 2022-06-18 9:03 ` [PATCH 4/5] nitro_enclaves: " David Gow 2022-06-18 9:03 ` David Gow 2022-06-20 9:26 ` Paraschiv, Andra-Irina 2022-06-20 9:26 ` Paraschiv, Andra-Irina 2022-06-18 9:03 ` [PATCH 5/5] mmc: sdhci-of-aspeed: " David Gow 2022-06-18 9:03 ` David Gow 2022-06-18 17:11 ` [PATCH 0/5] Rework KUnit test execution in modules Maíra Canal 2022-06-18 17:11 ` Maíra Canal 2022-06-18 17:41 ` Christophe Leroy 2022-06-18 17:41 ` Christophe Leroy
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=202206182117.58z5vWxq-lkp@intel.com \ --to=lkp@intel.com \ --cc=andraprs@amazon.com \ --cc=andrew@aj.id.au \ --cc=brendanhiggins@google.com \ --cc=davidgow@google.com \ --cc=dlatypov@google.com \ --cc=jk@codeconstruct.com.au \ --cc=kbuild-all@lists.01.org \ --cc=kunit-dev@googlegroups.com \ --cc=linux-aspeed@lists.ozlabs.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-kselftest@vger.kernel.org \ --cc=linux-mmc@vger.kernel.org \ --cc=linux-modules@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=longpeng2@huawei.com \ --cc=maira.canal@usp.br \ --cc=matt@codeconstruct.com.au \ --cc=mcgrof@kernel.org \ --cc=mika.westerberg@linux.intel.com \ --cc=openbmc@lists.ozlabs.org \ --cc=skhan@linuxfoundation.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: 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.