Re: [PATCH] Add printk for bonding module packets_per_slave parameter

From: kbuild test robot
Date: Tue Jun 13 2017 - 19:13:05 EST


Hi Michael,

[auto build test WARNING on net-next/master]
[also build test WARNING on v4.12-rc5 next-20170613]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Michael-Dilmore/Add-printk-for-bonding-module-packets_per_slave-parameter/20170614-045412
config: tile-tilegx_defconfig (attached as .config)
compiler: tilegx-linux-gcc (GCC) 4.6.2
reproduce:
wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=tile

All warnings (new ones prefixed by >>):

drivers/net/bonding/bond_options.c: In function 'bond_option_pps_set':
>> drivers/net/bonding/bond_options.c:1260:7: warning: format '%d' expects argument of type 'int', but argument 3 has type 'u64' [-Wformat]

vim +1260 drivers/net/bonding/bond_options.c

1244
1245 return 0;
1246 }
1247
1248 static int bond_option_lp_interval_set(struct bonding *bond,
1249 const struct bond_opt_value *newval)
1250 {
1251 bond->params.lp_interval = newval->value;
1252
1253 return 0;
1254 }
1255
1256 static int bond_option_pps_set(struct bonding *bond,
1257 const struct bond_opt_value *newval)
1258 {
1259 netdev_info(bond->dev, "Setting packets per slave to %d\n",
> 1260 newval->value);
1261 bond->params.packets_per_slave = newval->value;
1262 if (newval->value > 0) {
1263 bond->params.reciprocal_packets_per_slave =
1264 reciprocal_value(newval->value);
1265 } else {
1266 /* reciprocal_packets_per_slave is unused if
1267 * packets_per_slave is 0 or 1, just initialize it
1268 */

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: application/gzip