lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Date:   Sat, 17 Oct 2020 13:45:13 +0800
From:   kernel test robot <lkp@...el.com>
To:     glider <glider@...gle.com>
Cc:     kbuild-all@...ts.01.org, clang-built-linux@...glegroups.com,
        linux-kernel@...r.kernel.org, Kees Cook <keescook@...omium.org>,
        Maciej ┼╗enczykowski <maze@...gle.com>
Subject: drivers/media/i2c/mt9t112.c:670:12: warning: stack frame size of
 8344 bytes in function 'mt9t112_init_camera'

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   071a0578b0ce0b0e543d1e38ee6926b9cc21c198
commit: f0fe00d4972a8cd4b98cc2c29758615e4d51cdfe security: allow using Clang's zero initialization for stack variables
date:   4 months ago
config: x86_64-randconfig-a012-20201017 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project efd02c1548ee458d59063f6393e94e972b5c3d50)
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
        # install x86_64 cross compiling tool for clang build
        # apt-get install binutils-x86-64-linux-gnu
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f0fe00d4972a8cd4b98cc2c29758615e4d51cdfe
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout f0fe00d4972a8cd4b98cc2c29758615e4d51cdfe
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>

All warnings (new ones prefixed by >>):

>> drivers/media/i2c/mt9t112.c:670:12: warning: stack frame size of 8344 bytes in function 'mt9t112_init_camera' [-Wframe-larger-than=]
   static int mt9t112_init_camera(const struct i2c_client *client)
              ^
   1 warning generated.

vim +/mt9t112_init_camera +670 drivers/media/i2c/mt9t112.c

7641b04421954ea Jacopo Mondi 2018-03-12  669  
7641b04421954ea Jacopo Mondi 2018-03-12 @670  static int mt9t112_init_camera(const struct i2c_client *client)
7641b04421954ea Jacopo Mondi 2018-03-12  671  {
7641b04421954ea Jacopo Mondi 2018-03-12  672  	int ret;
7641b04421954ea Jacopo Mondi 2018-03-12  673  
7641b04421954ea Jacopo Mondi 2018-03-12  674  	ECHECKER(ret, mt9t112_reset(client));
7641b04421954ea Jacopo Mondi 2018-03-12  675  	ECHECKER(ret, mt9t112_init_pll(client));
7641b04421954ea Jacopo Mondi 2018-03-12  676  	ECHECKER(ret, mt9t112_init_setting(client));
7641b04421954ea Jacopo Mondi 2018-03-12  677  	ECHECKER(ret, mt9t112_auto_focus_setting(client));
7641b04421954ea Jacopo Mondi 2018-03-12  678  
7641b04421954ea Jacopo Mondi 2018-03-12  679  	mt9t112_reg_mask_set(ret, client, 0x0018, 0x0004, 0);
7641b04421954ea Jacopo Mondi 2018-03-12  680  
6a26f141bf6200a Jacopo Mondi 2018-03-12  681  	/* Analog setting B.*/
7641b04421954ea Jacopo Mondi 2018-03-12  682  	mt9t112_reg_write(ret, client, 0x3084, 0x2409);
7641b04421954ea Jacopo Mondi 2018-03-12  683  	mt9t112_reg_write(ret, client, 0x3092, 0x0A49);
7641b04421954ea Jacopo Mondi 2018-03-12  684  	mt9t112_reg_write(ret, client, 0x3094, 0x4949);
7641b04421954ea Jacopo Mondi 2018-03-12  685  	mt9t112_reg_write(ret, client, 0x3096, 0x4950);
7641b04421954ea Jacopo Mondi 2018-03-12  686  
7641b04421954ea Jacopo Mondi 2018-03-12  687  	/*
6a26f141bf6200a Jacopo Mondi 2018-03-12  688  	 * Disable adaptive clock.
7641b04421954ea Jacopo Mondi 2018-03-12  689  	 * PRI_A_CONFIG_JPEG_OB_TX_CONTROL_VAR
7641b04421954ea Jacopo Mondi 2018-03-12  690  	 * PRI_B_CONFIG_JPEG_OB_TX_CONTROL_VAR
7641b04421954ea Jacopo Mondi 2018-03-12  691  	 */
7641b04421954ea Jacopo Mondi 2018-03-12  692  	mt9t112_mcu_write(ret, client, VAR(26, 160), 0x0A2E);
7641b04421954ea Jacopo Mondi 2018-03-12  693  	mt9t112_mcu_write(ret, client, VAR(27, 160), 0x0A2E);
7641b04421954ea Jacopo Mondi 2018-03-12  694  
6a26f141bf6200a Jacopo Mondi 2018-03-12  695  	/*
6a26f141bf6200a Jacopo Mondi 2018-03-12  696  	 * Configure Status in Status_before_length Format and enable header.
6a26f141bf6200a Jacopo Mondi 2018-03-12  697  	 * PRI_B_CONFIG_JPEG_OB_TX_CONTROL_VAR
6a26f141bf6200a Jacopo Mondi 2018-03-12  698  	 */
7641b04421954ea Jacopo Mondi 2018-03-12  699  	mt9t112_mcu_write(ret, client, VAR(27, 144), 0x0CB4);
7641b04421954ea Jacopo Mondi 2018-03-12  700  
6a26f141bf6200a Jacopo Mondi 2018-03-12  701  	/*
6a26f141bf6200a Jacopo Mondi 2018-03-12  702  	 * Enable JPEG in context B.
6a26f141bf6200a Jacopo Mondi 2018-03-12  703  	 * PRI_B_CONFIG_JPEG_OB_TX_CONTROL_VAR
6a26f141bf6200a Jacopo Mondi 2018-03-12  704  	 */
7641b04421954ea Jacopo Mondi 2018-03-12  705  	mt9t112_mcu_write(ret, client, VAR8(27, 142), 0x01);
7641b04421954ea Jacopo Mondi 2018-03-12  706  
6a26f141bf6200a Jacopo Mondi 2018-03-12  707  	/* Disable Dac_TXLO. */
7641b04421954ea Jacopo Mondi 2018-03-12  708  	mt9t112_reg_write(ret, client, 0x316C, 0x350F);
7641b04421954ea Jacopo Mondi 2018-03-12  709  
6a26f141bf6200a Jacopo Mondi 2018-03-12  710  	/* Set max slew rates. */
7641b04421954ea Jacopo Mondi 2018-03-12  711  	mt9t112_reg_write(ret, client, 0x1E, 0x777);
7641b04421954ea Jacopo Mondi 2018-03-12  712  
7641b04421954ea Jacopo Mondi 2018-03-12  713  	return ret;
7641b04421954ea Jacopo Mondi 2018-03-12  714  }
7641b04421954ea Jacopo Mondi 2018-03-12  715  

:::::: The code at line 670 was first introduced by commit
:::::: 7641b04421954ea092b8fa7edbf904c3852875b5 media: i2c: Copy mt9t112 soc_camera sensor driver

:::::: TO: Jacopo Mondi <jacopo+renesas@...ndi.org>
:::::: CC: Mauro Carvalho Chehab <mchehab@...pensource.com>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Download attachment ".config.gz" of type "application/gzip" (35695 bytes)

Powered by blists - more mailing lists