linux-kernel - Re: [PATCH] Add printk for bonding module packets_per

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: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <201706140550.vFu82C0V%fengguang.wu@intel.com>
Date:   Wed, 14 Jun 2017 05:18:01 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Michael Dilmore <michael.j.dilmore@...il.com>
Cc:     kbuild-all@...org, Jay Vosburgh <j.vosburgh@...il.com>,
        Michael Dilmore <michael.j.dilmore@...il.com>,
        Veaceslav Falico <vfalico@...il.com>,
        Andy Gospodarek <andy@...yhouse.net>, netdev@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Add printk for bonding module packets_per_slave parameter

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

Download attachment ".config.gz" of type "application/gzip" (31513 bytes)