Messages in this thread | | | Date | Mon, 15 Aug 2022 07:59:53 +0800 | From | kernel test robot <> | Subject | [stable:linux-5.15.y 972/9027] drivers/usb/host/max3421-hcd.c:1378:1: warning: stack frame size (11288) exceeds limit (8192) in 'max3421_spi_thread' |
| |
tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-5.15.y head: 7217df81279835a7aee62a07aabb7b8fb8c766f2 commit: 8d0956438eecfafbefb7cd15b7773087549b38d4 [972/9027] fortify: Explicitly disable Clang support config: x86_64-randconfig-a013-20220815 (https://download.01.org/0day-ci/archive/20220815/202208150728.vFqeXmnu-lkp@intel.com/config) compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 3329cec2f79185bafd678f310fafadba2a8c76d2) 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/stable/linux-stable.git/commit/?id=8d0956438eecfafbefb7cd15b7773087549b38d4 git remote add stable https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git git fetch --no-tags stable linux-5.15.y git checkout 8d0956438eecfafbefb7cd15b7773087549b38d4 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/media/dvb-frontends/ drivers/usb/host/
If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/usb/host/max3421-hcd.c:1968:34: warning: unused variable 'max3421_of_match_table' [-Wunused-const-variable] static const struct of_device_id max3421_of_match_table[] = { ^ >> drivers/usb/host/max3421-hcd.c:1378:1: warning: stack frame size (11288) exceeds limit (8192) in 'max3421_spi_thread' [-Wframe-larger-than] max3421_spi_thread(void *dev_id) ^ 2 warnings generated. -- >> drivers/media/dvb-frontends/nxt200x.c:1087:12: warning: stack frame size (15160) exceeds limit (8192) in 'nxt200x_init' [-Wframe-larger-than] static int nxt200x_init(struct dvb_frontend* fe) ^ >> drivers/media/dvb-frontends/nxt200x.c:519:12: warning: stack frame size (9624) exceeds limit (8192) in 'nxt200x_setup_frontend_parameters' [-Wframe-larger-than] static int nxt200x_setup_frontend_parameters(struct dvb_frontend *fe) ^ 2 warnings generated. -- >> drivers/media/dvb-frontends/stv0367.c:995:12: warning: stack frame size (9464) exceeds limit (8192) in 'stv0367ter_algo' [-Wframe-larger-than] static int stv0367ter_algo(struct dvb_frontend *fe) ^ >> drivers/media/dvb-frontends/stv0367.c:2540:12: warning: stack frame size (9368) exceeds limit (8192) in 'stv0367cab_set_frontend' [-Wframe-larger-than] static int stv0367cab_set_frontend(struct dvb_frontend *fe) ^ 2 warnings generated.
vim +/max3421_spi_thread +1378 drivers/usb/host/max3421-hcd.c
2d53139f31626b David Mosberger 2014-04-28 1376 2d53139f31626b David Mosberger 2014-04-28 1377 static int 2d53139f31626b David Mosberger 2014-04-28 @1378 max3421_spi_thread(void *dev_id) 2d53139f31626b David Mosberger 2014-04-28 1379 { 2d53139f31626b David Mosberger 2014-04-28 1380 struct usb_hcd *hcd = dev_id; 2d53139f31626b David Mosberger 2014-04-28 1381 struct spi_device *spi = to_spi_device(hcd->self.controller); 2d53139f31626b David Mosberger 2014-04-28 1382 struct max3421_hcd *max3421_hcd = hcd_to_max3421(hcd); 2d53139f31626b David Mosberger 2014-04-28 1383 int i, i_worked = 1; 2d53139f31626b David Mosberger 2014-04-28 1384 2d53139f31626b David Mosberger 2014-04-28 1385 /* set full-duplex SPI mode, low-active interrupt pin: */ 2d53139f31626b David Mosberger 2014-04-28 1386 spi_wr8(hcd, MAX3421_REG_PINCTL, 2d53139f31626b David Mosberger 2014-04-28 1387 (BIT(MAX3421_PINCTL_FDUPSPI_BIT) | /* full-duplex */ 2d53139f31626b David Mosberger 2014-04-28 1388 BIT(MAX3421_PINCTL_INTLEVEL_BIT))); /* low-active irq */ 2d53139f31626b David Mosberger 2014-04-28 1389 2d53139f31626b David Mosberger 2014-04-28 1390 while (!kthread_should_stop()) { 2d53139f31626b David Mosberger 2014-04-28 1391 max3421_hcd->rev = spi_rd8(hcd, MAX3421_REG_REVISION); 2d53139f31626b David Mosberger 2014-04-28 1392 if (max3421_hcd->rev == 0x12 || max3421_hcd->rev == 0x13) 2d53139f31626b David Mosberger 2014-04-28 1393 break; 2d53139f31626b David Mosberger 2014-04-28 1394 dev_err(&spi->dev, "bad rev 0x%02x", max3421_hcd->rev); 2d53139f31626b David Mosberger 2014-04-28 1395 msleep(10000); 2d53139f31626b David Mosberger 2014-04-28 1396 } 2d53139f31626b David Mosberger 2014-04-28 1397 dev_info(&spi->dev, "rev 0x%x, SPI clk %dHz, bpw %u, irq %d\n", 2d53139f31626b David Mosberger 2014-04-28 1398 max3421_hcd->rev, spi->max_speed_hz, spi->bits_per_word, 2d53139f31626b David Mosberger 2014-04-28 1399 spi->irq); 2d53139f31626b David Mosberger 2014-04-28 1400 2d53139f31626b David Mosberger 2014-04-28 1401 while (!kthread_should_stop()) { 2d53139f31626b David Mosberger 2014-04-28 1402 if (!i_worked) { 2d53139f31626b David Mosberger 2014-04-28 1403 /* 2d53139f31626b David Mosberger 2014-04-28 1404 * We'll be waiting for wakeups from the hard 2d53139f31626b David Mosberger 2014-04-28 1405 * interrupt handler, so now is a good time to 2d53139f31626b David Mosberger 2014-04-28 1406 * sync our hien with the chip: 2d53139f31626b David Mosberger 2014-04-28 1407 */ 2d53139f31626b David Mosberger 2014-04-28 1408 spi_wr8(hcd, MAX3421_REG_HIEN, max3421_hcd->hien); 2d53139f31626b David Mosberger 2014-04-28 1409 2d53139f31626b David Mosberger 2014-04-28 1410 set_current_state(TASK_INTERRUPTIBLE); 2eb5dbdd1263a4 David Mosberger-Tang 2014-06-19 1411 if (test_and_clear_bit(ENABLE_IRQ, &max3421_hcd->todo)) 2d53139f31626b David Mosberger 2014-04-28 1412 enable_irq(spi->irq); 2d53139f31626b David Mosberger 2014-04-28 1413 schedule(); 2d53139f31626b David Mosberger 2014-04-28 1414 __set_current_state(TASK_RUNNING); 2d53139f31626b David Mosberger 2014-04-28 1415 } 2d53139f31626b David Mosberger 2014-04-28 1416 2d53139f31626b David Mosberger 2014-04-28 1417 i_worked = 0; 2d53139f31626b David Mosberger 2014-04-28 1418 2d53139f31626b David Mosberger 2014-04-28 1419 if (max3421_hcd->urb_done) 2d53139f31626b David Mosberger 2014-04-28 1420 i_worked |= max3421_urb_done(hcd); 2d53139f31626b David Mosberger 2014-04-28 1421 else if (max3421_handle_irqs(hcd)) 2d53139f31626b David Mosberger 2014-04-28 1422 i_worked = 1; 2d53139f31626b David Mosberger 2014-04-28 1423 else if (!max3421_hcd->curr_urb) 2d53139f31626b David Mosberger 2014-04-28 1424 i_worked |= max3421_select_and_start_urb(hcd); 2d53139f31626b David Mosberger 2014-04-28 1425 2eb5dbdd1263a4 David Mosberger-Tang 2014-06-19 1426 if (test_and_clear_bit(RESET_HCD, &max3421_hcd->todo)) 2d53139f31626b David Mosberger 2014-04-28 1427 /* reset the HCD: */ 2d53139f31626b David Mosberger 2014-04-28 1428 i_worked |= max3421_reset_hcd(hcd); 2eb5dbdd1263a4 David Mosberger-Tang 2014-06-19 1429 if (test_and_clear_bit(RESET_PORT, &max3421_hcd->todo)) { 2d53139f31626b David Mosberger 2014-04-28 1430 /* perform a USB bus reset: */ 2d53139f31626b David Mosberger 2014-04-28 1431 spi_wr8(hcd, MAX3421_REG_HCTL, 2d53139f31626b David Mosberger 2014-04-28 1432 BIT(MAX3421_HCTL_BUSRST_BIT)); 2d53139f31626b David Mosberger 2014-04-28 1433 i_worked = 1; 2d53139f31626b David Mosberger 2014-04-28 1434 } 2eb5dbdd1263a4 David Mosberger-Tang 2014-06-19 1435 if (test_and_clear_bit(CHECK_UNLINK, &max3421_hcd->todo)) 2d53139f31626b David Mosberger 2014-04-28 1436 i_worked |= max3421_check_unlink(hcd); 2eb5dbdd1263a4 David Mosberger-Tang 2014-06-19 1437 if (test_and_clear_bit(IOPIN_UPDATE, &max3421_hcd->todo)) { 2d53139f31626b David Mosberger 2014-04-28 1438 /* 2d53139f31626b David Mosberger 2014-04-28 1439 * IOPINS1/IOPINS2 do not auto-increment, so we can't 2d53139f31626b David Mosberger 2014-04-28 1440 * use spi_wr_buf(). 2d53139f31626b David Mosberger 2014-04-28 1441 */ 2d53139f31626b David Mosberger 2014-04-28 1442 for (i = 0; i < ARRAY_SIZE(max3421_hcd->iopins); ++i) { 2d53139f31626b David Mosberger 2014-04-28 1443 u8 val = spi_rd8(hcd, MAX3421_REG_IOPINS1); 2d53139f31626b David Mosberger 2014-04-28 1444 2d53139f31626b David Mosberger 2014-04-28 1445 val = ((val & 0xf0) | 2d53139f31626b David Mosberger 2014-04-28 1446 (max3421_hcd->iopins[i] & 0x0f)); 2d53139f31626b David Mosberger 2014-04-28 1447 spi_wr8(hcd, MAX3421_REG_IOPINS1 + i, val); 2d53139f31626b David Mosberger 2014-04-28 1448 max3421_hcd->iopins[i] = val; 2d53139f31626b David Mosberger 2014-04-28 1449 } 2d53139f31626b David Mosberger 2014-04-28 1450 i_worked = 1; 2d53139f31626b David Mosberger 2014-04-28 1451 } 2d53139f31626b David Mosberger 2014-04-28 1452 } 2d53139f31626b David Mosberger 2014-04-28 1453 set_current_state(TASK_RUNNING); 2d53139f31626b David Mosberger 2014-04-28 1454 dev_info(&spi->dev, "SPI thread exiting"); 2d53139f31626b David Mosberger 2014-04-28 1455 return 0; 2d53139f31626b David Mosberger 2014-04-28 1456 } 2d53139f31626b David Mosberger 2014-04-28 1457
:::::: The code at line 1378 was first introduced by commit :::::: 2d53139f31626bad6f8983d8e519ddde2cbba921 Add support for using a MAX3421E chip as a host driver.
:::::: TO: David Mosberger <davidm@egauge.net> :::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-- 0-DAY CI Kernel Test Service https://01.org/lkp
| |