On Fri, Jun 16, 2023 at 08:43:56AM +0800, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master > head: 925294c9aa184801cc0a451b69a18dd0fe7d847d > commit: 3a41db531e5124adaa3a9ab9ca0c724aee85b10c [6153/10326] pktcdvd: Get rid of custom printing macros > config: riscv-buildonly-randconfig-r001-20230615 (https://download.01.org/0day-ci/archive/20230616/202306160811.nV1bMsK4-lkp@xxxxxxxxx/config) > compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project.git f28c006a5895fc0e329fe15fead81e37457cb1d1) > reproduce (this is a W=1 build): > mkdir -p ~/bin > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # install riscv cross compiling tool for clang build > # apt-get install binutils-riscv64-linux-gnu > # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=3a41db531e5124adaa3a9ab9ca0c724aee85b10c > git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git > git fetch --no-tags linux-next master > git checkout 3a41db531e5124adaa3a9ab9ca0c724aee85b10c > # save the config file > mkdir build_dir && cp config build_dir/.config > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang ~/bin/make.cross W=1 O=build_dir ARCH=riscv olddefconfig > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang ~/bin/make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/ mm// > > If you fix the issue in a separate patch/commit (i.e. not just a new version of > the same patch/commit), kindly add following tags > | Reported-by: kernel test robot <lkp@xxxxxxxxx> > | Closes: https://lore.kernel.org/oe-kbuild-all/202306160811.nV1bMsK4-lkp@xxxxxxxxx/ > > All warnings (new ones prefixed by >>): > > drivers/block/pktcdvd.c:1364:12: warning: stack frame size (3648) exceeds limit (2048) in 'kcdrwd' [-Wframe-larger-than] > static int kcdrwd(void *foobar) > ^ > >> drivers/block/pktcdvd.c:1315:13: warning: stack frame size (2496) exceeds limit (2048) in 'pkt_handle_packets' [-Wframe-larger-than] > static void pkt_handle_packets(struct pktcdvd_device *pd) > ^ > 2 warnings generated. Ha, the added line > 4b83e99ee7092d Jens Axboe 2023-01-04 @1315 static void pkt_handle_packets(struct pktcdvd_device *pd) > 4b83e99ee7092d Jens Axboe 2023-01-04 1316 { > 3a41db531e5124 Andy Shevchenko 2023-03-10 1317 struct device *ddev = disk_to_dev(pd->disk); is a simple macro #define disk_to_dev(disk) \ (&((disk)->part0->bd_device)) and looking into implementation it might be that clang mistakenly allocates stack based on bd_device size. And since it's a randconfig, quite likely something like GCOV, KASAN, etc. is involved. Also note, that struct block_device { ... /* * keep this out-of-line as it's both big and not needed in the fast * path */ struct device bd_device; } __randomize_layout; > 4b83e99ee7092d Jens Axboe 2023-01-04 1318 struct packet_data *pkt, *next; -- With Best Regards, Andy Shevchenko
