[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'

From: kernel test robot
Date: Sun Aug 14 2022 - 20:00:51 EST


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@xxxxxxxxx/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@xxxxxxxxx>

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@xxxxxxxxxx>
:::::: CC: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>

--
0-DAY CI Kernel Test Service
https://01.org/lkp