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  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <202202260711.HoB6xaby-lkp@intel.com>
Date:   Sat, 26 Feb 2022 07:09:23 +0800
From:   kernel test robot <lkp@...el.com>
To:     Jiri Pirko <jiri@...dia.com>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org
Subject: [jpirko-mlxsw:jiri_devel_linecards 38/39]
 drivers/net/netdevsim/dev.c:119:57: error: passing argument 4 of
 'devlink_linecard_device_create' from incompatible pointer type

tree:   https://github.com/jpirko/linux_mlxsw jiri_devel_linecards
head:   24e52724258ba5d7b58b5a4a86640dadb30b07c4
commit: 36f00f891469372ccec5b7d6442e13e284ceca13 [38/39] TMP: devlink device type
config: arc-allyesconfig (https://download.01.org/0day-ci/archive/20220226/202202260711.HoB6xaby-lkp@intel.com/config)
compiler: arceb-elf-gcc (GCC) 11.2.0
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://github.com/jpirko/linux_mlxsw/commit/36f00f891469372ccec5b7d6442e13e284ceca13
        git remote add jpirko-mlxsw https://github.com/jpirko/linux_mlxsw
        git fetch --no-tags jpirko-mlxsw jiri_devel_linecards
        git checkout 36f00f891469372ccec5b7d6442e13e284ceca13
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arc SHELL=/bin/bash

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

All errors (new ones prefixed by >>):

   drivers/net/netdevsim/dev.c: In function 'nsim_dev_linecard_devices_create':
>> drivers/net/netdevsim/dev.c:119:57: error: passing argument 4 of 'devlink_linecard_device_create' from incompatible pointer type [-Werror=incompatible-pointer-types]
     119 |                                                         nsim_dev_linecard);
         |                                                         ^~~~~~~~~~~~~~~~~
         |                                                         |
         |                                                         struct nsim_dev_linecard *
   In file included from drivers/net/netdevsim/dev.c:29:
   include/net/devlink.h:1580:44: note: expected 'const char *' but argument is of type 'struct nsim_dev_linecard *'
    1580 |                                const char *type, void *priv);
         |                                ~~~~~~~~~~~~^~~~
>> drivers/net/netdevsim/dev.c:117:26: error: too few arguments to function 'devlink_linecard_device_create'
     117 |                 device = devlink_linecard_device_create(nsim_dev_linecard->devlink_linecard,
         |                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/net/netdevsim/dev.c:29:
   include/net/devlink.h:1577:1: note: declared here
    1577 | devlink_linecard_device_create(struct devlink_linecard *linecard,
         | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors


vim +/devlink_linecard_device_create +119 drivers/net/netdevsim/dev.c

e8caa751f8b7e3 Jiri Pirko 2021-05-05  100  
e8caa751f8b7e3 Jiri Pirko 2021-05-05  101  static int
e8caa751f8b7e3 Jiri Pirko 2021-05-05  102  nsim_dev_linecard_devices_create(struct nsim_dev_linecard *nsim_dev_linecard)
e8caa751f8b7e3 Jiri Pirko 2021-05-05  103  {
e8caa751f8b7e3 Jiri Pirko 2021-05-05  104  	struct devlink_linecard_device *device;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  105  	char *component_name;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  106  	int err;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  107  	int i;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  108  
e8caa751f8b7e3 Jiri Pirko 2021-05-05  109  	snprintf(nsim_dev_linecard->device_component_name,
e8caa751f8b7e3 Jiri Pirko 2021-05-05  110  		 sizeof(nsim_dev_linecard->device_component_name), "lc%u_dev0",
e8caa751f8b7e3 Jiri Pirko 2021-05-05  111  		 nsim_dev_linecard->linecard_index);
e8caa751f8b7e3 Jiri Pirko 2021-05-05  112  	component_name = nsim_dev_linecard->device_component_name;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  113  
e8caa751f8b7e3 Jiri Pirko 2021-05-05  114  	for (i = 0; i < NSIM_DEV_LINECARD_DEVICE_COUNT; i++) {
e8caa751f8b7e3 Jiri Pirko 2021-05-05  115  		if (i > 0)
e8caa751f8b7e3 Jiri Pirko 2021-05-05  116  			component_name = NULL;
e8caa751f8b7e3 Jiri Pirko 2021-05-05 @117  		device = devlink_linecard_device_create(nsim_dev_linecard->devlink_linecard,
e8caa751f8b7e3 Jiri Pirko 2021-05-05  118  							i, component_name,
e8caa751f8b7e3 Jiri Pirko 2021-05-05 @119  							nsim_dev_linecard);
e8caa751f8b7e3 Jiri Pirko 2021-05-05  120  		if (IS_ERR(device)) {
e8caa751f8b7e3 Jiri Pirko 2021-05-05  121  			err = PTR_ERR(device);
e8caa751f8b7e3 Jiri Pirko 2021-05-05  122  			goto rollback;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  123  		}
e8caa751f8b7e3 Jiri Pirko 2021-05-05  124  		nsim_dev_linecard->devlink_device[i] = device;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  125  	}
e8caa751f8b7e3 Jiri Pirko 2021-05-05  126  	return 0;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  127  
e8caa751f8b7e3 Jiri Pirko 2021-05-05  128  rollback:
e8caa751f8b7e3 Jiri Pirko 2021-05-05  129  	for (i--; i >= 0; i--) {
e8caa751f8b7e3 Jiri Pirko 2021-05-05  130  		device = nsim_dev_linecard->devlink_device[i];
e8caa751f8b7e3 Jiri Pirko 2021-05-05  131  		devlink_linecard_device_destroy(nsim_dev_linecard->devlink_linecard,
e8caa751f8b7e3 Jiri Pirko 2021-05-05  132  						device);
e8caa751f8b7e3 Jiri Pirko 2021-05-05  133  	}
e8caa751f8b7e3 Jiri Pirko 2021-05-05  134  	return err;
e8caa751f8b7e3 Jiri Pirko 2021-05-05  135  }
e8caa751f8b7e3 Jiri Pirko 2021-05-05  136  

:::::: The code at line 119 was first introduced by commit
:::::: e8caa751f8b7e39b8286205755f61701496d4c79 netdevsim: create couple of devices on each linecard

:::::: TO: Jiri Pirko <jiri@...dia.com>
:::::: CC: Jiri Pirko <jiri@...dia.com>

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ