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

From: kbuild test robot
Date: Tue Jun 13 2017 - 17:18:42 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: i386-randconfig-x014-06122022 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All warnings (new ones prefixed by >>):

drivers/net//bonding/bond_options.c: In function 'bond_option_pps_set':
>> drivers/net//bonding/bond_options.c:1259:56: warning: format '%d' expects argument of type 'int', but argument 3 has type 'u64 {aka const long long unsigned int}' [-Wformat=]
netdev_info(bond->dev, "Setting packets per slave to %d\n",
^

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

1243 bond_set_carrier(bond);
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

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

Attachment: .config.gz
Description: application/gzip