Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1665288

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

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] Add printk for bonding module packets_per_slave parameter Michael Dilmore <michael.j.dilmore@gmail.com> - 2017-06-13 15:50 +0200
  Re: [PATCH] Add printk for bonding module packets_per_slave  parameter David Miller <davem@davemloft.net> - 2017-06-13 17:40 +0200
    Re: [PATCH] Add printk for bonding module packets_per_slave  parameter Joe Perches <joe@perches.com> - 2017-06-13 18:30 +0200
      Re: [PATCH] Add printk for bonding module packets_per_slave parameter Jonathan Toppins <jtoppins@redhat.com> - 2017-06-13 18:50 +0200
        Re: [PATCH] Add printk for bonding module packets_per_slave  parameter Joe Perches <joe@perches.com> - 2017-06-13 19:10 +0200
          Re: [PATCH] Add printk for bonding module packets_per_slave parameter Nikolay Aleksandrov <nikolay@cumulusnetworks.com> - 2017-06-13 20:00 +0200
      Re: [PATCH] Add printk for bonding module packets_per_slave  parameter David Miller <davem@davemloft.net> - 2017-06-13 18:50 +0200
  Re: [PATCH] Add printk for bonding module packets_per_slave parameter kbuild test robot <lkp@intel.com> - 2017-06-14 01:20 +0200

csiph-web