From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============4850999014748384470==" MIME-Version: 1.0 From: kernel test robot To: kbuild-all@lists.01.org Subject: Re: [PATCH] random: do not take pool spinlock at boot Date: Sat, 12 Feb 2022 13:04:14 +0800 Message-ID: <202202121223.7CKs3pQ3-lkp@intel.com> In-Reply-To: <20220212002859.622233-1-Jason@zx2c4.com> List-Id: --===============4850999014748384470== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Hi "Jason, I love your patch! Perhaps something to improve: [auto build test WARNING on next-20220211] [cannot apply to char-misc/char-misc-testing linux/master linus/master v5.1= 7-rc3 v5.17-rc2 v5.17-rc1 v5.17-rc3] [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/0day-ci/linux/commits/Jason-A-Donenfeld/random-d= o-not-take-pool-spinlock-at-boot/20220212-083131 base: 6d9bd4ad4ca08b1114e814c2c42383b8b13be631 config: um-x86_64_defconfig (https://download.01.org/0day-ci/archive/202202= 12/202202121223.7CKs3pQ3-lkp(a)intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce (this is a W=3D1 build): # https://github.com/0day-ci/linux/commit/cac377f6ab22a8bfe966643eb= e19812e8187de3b git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Jason-A-Donenfeld/random-do-not-ta= ke-pool-spinlock-at-boot/20220212-083131 git checkout cac377f6ab22a8bfe966643ebe19812e8187de3b # save the config file to linux build tree mkdir build_dir make W=3D1 O=3Dbuild_dir ARCH=3Dum SUBARCH=3Dx86_64 SHELL=3D/bin/ba= sh drivers/char/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All warnings (new ones prefixed by >>): drivers/char/random.c: In function 'rand_initialize': drivers/char/random.c:162:3: error: implicit declaration of function '_m= ix_pool_bytes'; did you mean 'mix_pool_bytes'? [-Werror=3Dimplicit-function= -declaration] 162 | _mix_pool_bytes(&rv, sizeof(rv)); | ^~~~~~~~~~~~~~~ | mix_pool_bytes drivers/char/random.c: At top level: >> drivers/char/random.c:822:13: warning: conflicting types for '_mix_pool_= bytes' 822 | static void _mix_pool_bytes(const void *in, size_t nbytes) | ^~~~~~~~~~~~~~~ drivers/char/random.c:822:13: error: static declaration of '_mix_pool_by= tes' follows non-static declaration drivers/char/random.c:162:3: note: previous implicit declaration of '_mi= x_pool_bytes' was here 162 | _mix_pool_bytes(&rv, sizeof(rv)); | ^~~~~~~~~~~~~~~ drivers/char/random.c:1109:6: warning: no previous prototype for 'add_hw= generator_randomness' [-Wmissing-prototypes] 1109 | void add_hwgenerator_randomness(const void *buffer, size_t count, | ^~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +/_mix_pool_bytes +822 drivers/char/random.c b980955236922ae Theodore Ts'o 2013-03-04 821 = b67959013391e48 Jason A. Donenfeld 2022-02-10 @822 static void _mix_pool_b= ytes(const void *in, size_t nbytes) b67959013391e48 Jason A. Donenfeld 2022-02-10 823 { b67959013391e48 Jason A. Donenfeld 2022-02-10 824 blake2s_update(&input_= pool.hash, in, nbytes); ^1da177e4c3f415 Linus Torvalds 2005-04-16 825 } ^1da177e4c3f415 Linus Torvalds 2005-04-16 826 = --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org --===============4850999014748384470==--