Re: [PATCH v8 05/25] dept: Tie to Lockdep and IRQ tracing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Byungchul,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on tip/locking/core]
[also build test WARNING on tip/sched/core drm-misc/drm-misc-next linus/master v6.2-rc5]
[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#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Byungchul-Park/llist-Move-llist_-head-node-definition-to-types-h/20230128-102456
patch link:    https://lore.kernel.org/r/1674782358-25542-6-git-send-email-max.byungchul.park%40gmail.com
patch subject: [PATCH v8 05/25] dept: Tie to Lockdep and IRQ tracing
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20230129/202301291601.7591xziv-lkp@xxxxxxxxx/config)
compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
reproduce (this is a W=1 build):
        # https://github.com/intel-lab-lkp/linux/commit/aed5169e3b6767146ee602447fcf75b4c734d2db
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Byungchul-Park/llist-Move-llist_-head-node-definition-to-types-h/20230128-102456
        git checkout aed5169e3b6767146ee602447fcf75b4c734d2db
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 O=build_dir ARCH=x86_64 olddefconfig
        make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/net/ethernet/microchip/vcap/ drivers/platform/chrome/

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/platform/chrome/cros_ec_proto_test.c: In function 'cros_ec_proto_test_get_sensor_count_xfer_error':
>> drivers/platform/chrome/cros_ec_proto_test.c:2528:1: warning: the frame size of 2072 bytes is larger than 2048 bytes [-Wframe-larger-than=]
    2528 | }
         | ^
   drivers/platform/chrome/cros_ec_proto_test.c: In function 'cros_ec_proto_test_get_sensor_count_normal':
   drivers/platform/chrome/cros_ec_proto_test.c:2488:1: warning: the frame size of 2072 bytes is larger than 2048 bytes [-Wframe-larger-than=]
    2488 | }
         | ^
   drivers/platform/chrome/cros_ec_proto_test.c: In function 'cros_ec_proto_test_get_sensor_count_legacy':
   drivers/platform/chrome/cros_ec_proto_test.c:2593:1: warning: the frame size of 2136 bytes is larger than 2048 bytes [-Wframe-larger-than=]
    2593 | }
         | ^
   drivers/platform/chrome/cros_ec_proto_test.c: In function 'cros_ec_proto_test_check_features_not_cached':
   drivers/platform/chrome/cros_ec_proto_test.c:2443:1: warning: the frame size of 2056 bytes is larger than 2048 bytes [-Wframe-larger-than=]
    2443 | }
         | ^
--
   In file included from drivers/net/ethernet/microchip/vcap/vcap_api.c:2882:
   drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c: In function 'vcap_api_next_lookup_advanced_test':
>> drivers/net/ethernet/microchip/vcap/vcap_api_kunit.c:1955:1: warning: the frame size of 2240 bytes is larger than 2048 bytes [-Wframe-larger-than=]
    1955 | }
         | ^


vim +2528 drivers/platform/chrome/cros_ec_proto_test.c

33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2489  
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2490  static void cros_ec_proto_test_get_sensor_count_xfer_error(struct kunit *test)
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2491  {
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2492  	struct cros_ec_proto_test_priv *priv = test->priv;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2493  	struct cros_ec_device *ec_dev = &priv->ec_dev;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2494  	struct ec_xfer_mock *mock;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2495  	int ret;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2496  	struct cros_ec_dev ec;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2497  
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2498  	ec_dev->max_request = 0xff;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2499  	ec_dev->max_response = 0xee;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2500  	ec.ec_dev = ec_dev;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2501  	ec.dev = ec_dev->dev;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2502  	ec.cmd_offset = 0;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2503  
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2504  	/* For EC_CMD_MOTION_SENSE_CMD. */
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2505  	{
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2506  		mock = cros_kunit_ec_xfer_mock_addx(test, -EPROTO, EC_RES_SUCCESS, 0);
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2507  		KUNIT_ASSERT_PTR_NE(test, mock, NULL);
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2508  	}
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2509  
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2510  	ret = cros_ec_get_sensor_count(&ec);
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2511  	KUNIT_EXPECT_EQ(test, ret, -EPROTO);
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2512  
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2513  	/* For EC_CMD_MOTION_SENSE_CMD. */
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2514  	{
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2515  		struct ec_params_motion_sense *data;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2516  
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2517  		mock = cros_kunit_ec_xfer_mock_next();
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2518  		KUNIT_EXPECT_PTR_NE(test, mock, NULL);
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2519  
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2520  		KUNIT_EXPECT_EQ(test, mock->msg.version, 1);
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2521  		KUNIT_EXPECT_EQ(test, mock->msg.command, EC_CMD_MOTION_SENSE_CMD);
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2522  		KUNIT_EXPECT_EQ(test, mock->msg.insize, sizeof(struct ec_response_motion_sense));
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2523  		KUNIT_EXPECT_EQ(test, mock->msg.outsize, sizeof(*data));
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2524  
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2525  		data = (struct ec_params_motion_sense *)mock->i_data;
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2526  		KUNIT_EXPECT_EQ(test, data->cmd, MOTIONSENSE_CMD_DUMP);
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2527  	}
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22 @2528  }
33f0fdba6066b5 Tzung-Bi Shih 2022-06-22  2529  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests



[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux