| 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
| ||
|
Message-ID: <201602090402.fmI4HN3p%fengguang.wu@intel.com>
Date: Tue, 9 Feb 2016 04:30:31 +0800
From: kbuild test robot <lkp@...el.com>
To: Jacob Keller <jacob.e.keller@...el.com>
Cc: kbuild-all@...org, netdev@...r.kernel.org, mooray3@...pl,
davem@...emloft.net, Jacob Keller <jacob.e.keller@...el.com>
Subject: Re: [PATCH 3/4] ethtool: can't set combined and tx/rx channel counts
at the same time
Hi Jacob,
[auto build test WARNING on net/master]
[also build test WARNING on v4.5-rc3 next-20160208]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
url: https://github.com/0day-ci/linux/commits/Jacob-Keller/ethtool-correctly-ensure-GS-CHANNELS-doesn-t-conflict-with-GS-RXFH/20160209-041131
config: x86_64-randconfig-x012-201606 (attached as .config)
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
All warnings (new ones prefixed by >>):
net/core/ethtool.c: In function 'ethtool_set_channels':
net/core/ethtool.c:1289:6: error: void value not ignored as it ought to be
ret = dev->ethtool_ops->get_channels(dev, &max);
^
net/core/ethtool.c:1303:6: error: expected ')' before 'return'
return -EINVAL;
^
net/core/ethtool.c:2159:1: error: expected declaration or statement at end of input
}
^
>> net/core/ethtool.c:1280:6: warning: unused variable 'max_rx' [-Wunused-variable]
u32 max_rx = 0;
^
net/core/ethtool.c: At top level:
net/core/ethtool.c:116:12: warning: 'ethtool_get_features' defined but not used [-Wunused-function]
static int ethtool_get_features(struct net_device *dev, void __user *useraddr)
^
net/core/ethtool.c:154:12: warning: 'ethtool_set_features' defined but not used [-Wunused-function]
static int ethtool_set_features(struct net_device *dev, void __user *useraddr)
^
net/core/ethtool.c:237:13: warning: '__ethtool_get_strings' defined but not used [-Wunused-function]
static void __ethtool_get_strings(struct net_device *dev,
^
net/core/ethtool.c:296:12: warning: 'ethtool_get_one_feature' defined but not used [-Wunused-function]
static int ethtool_get_one_feature(struct net_device *dev,
^
net/core/ethtool.c:310:12: warning: 'ethtool_set_one_feature' defined but not used [-Wunused-function]
static int ethtool_set_one_feature(struct net_device *dev,
^
net/core/ethtool.c:340:12: warning: '__ethtool_get_flags' defined but not used [-Wunused-function]
static u32 __ethtool_get_flags(struct net_device *dev)
^
net/core/ethtool.c:358:12: warning: '__ethtool_set_flags' defined but not used [-Wunused-function]
static int __ethtool_set_flags(struct net_device *dev, u32 data)
^
net/core/ethtool.c:402:12: warning: 'ethtool_get_settings' defined but not used [-Wunused-function]
static int ethtool_get_settings(struct net_device *dev, void __user *useraddr)
^
net/core/ethtool.c:416:12: warning: 'ethtool_set_settings' defined but not used [-Wunused-function]
static int ethtool_set_settings(struct net_device *dev, void __user *useraddr)
^
net/core/ethtool.c:429:31: warning: 'ethtool_get_drvinfo' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_get_drvinfo(struct net_device *dev,
^
net/core/ethtool.c:475:31: warning: 'ethtool_get_sset_info' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_get_sset_info(struct net_device *dev,
^
net/core/ethtool.c:531:31: warning: 'ethtool_set_rxnfc' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_set_rxnfc(struct net_device *dev,
^
net/core/ethtool.c:563:31: warning: 'ethtool_get_rxnfc' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_get_rxnfc(struct net_device *dev,
^
net/core/ethtool.c:678:31: warning: 'ethtool_get_rxfh_indir' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_get_rxfh_indir(struct net_device *dev,
^
net/core/ethtool.c:726:31: warning: 'ethtool_set_rxfh_indir' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_set_rxfh_indir(struct net_device *dev,
^
net/core/ethtool.c:788:31: warning: 'ethtool_get_rxfh' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_get_rxfh(struct net_device *dev,
^
net/core/ethtool.c:860:31: warning: 'ethtool_set_rxfh' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_set_rxfh(struct net_device *dev,
^
net/core/ethtool.c:955:12: warning: 'ethtool_get_regs' defined but not used [-Wunused-function]
static int ethtool_get_regs(struct net_device *dev, char __user *useraddr)
^
net/core/ethtool.c:991:12: warning: 'ethtool_reset' defined but not used [-Wunused-function]
static int ethtool_reset(struct net_device *dev, char __user *useraddr)
^
net/core/ethtool.c:1011:12: warning: 'ethtool_get_wol' defined but not used [-Wunused-function]
static int ethtool_get_wol(struct net_device *dev, char __user *useraddr)
^
net/core/ethtool.c:1025:12: warning: 'ethtool_set_wol' defined but not used [-Wunused-function]
static int ethtool_set_wol(struct net_device *dev, char __user *useraddr)
^
net/core/ethtool.c:1038:12: warning: 'ethtool_get_eee' defined but not used [-Wunused-function]
static int ethtool_get_eee(struct net_device *dev, char __user *useraddr)
^
net/core/ethtool.c:1059:12: warning: 'ethtool_set_eee' defined but not used [-Wunused-function]
static int ethtool_set_eee(struct net_device *dev, char __user *useraddr)
^
net/core/ethtool.c:1072:12: warning: 'ethtool_nway_reset' defined but not used [-Wunused-function]
static int ethtool_nway_reset(struct net_device *dev)
^
net/core/ethtool.c:1080:12: warning: 'ethtool_get_link' defined but not used [-Wunused-function]
static int ethtool_get_link(struct net_device *dev, char __user *useraddr)
^
net/core/ethtool.c:1145:12: warning: 'ethtool_get_eeprom' defined but not used [-Wunused-function]
static int ethtool_get_eeprom(struct net_device *dev, void __user *useraddr)
^
net/core/ethtool.c:1157:12: warning: 'ethtool_set_eeprom' defined but not used [-Wunused-function]
static int ethtool_set_eeprom(struct net_device *dev, void __user *useraddr)
^
net/core/ethtool.c:1205:31: warning: 'ethtool_get_coalesce' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_get_coalesce(struct net_device *dev,
^
net/core/ethtool.c:1220:31: warning: 'ethtool_set_coalesce' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_set_coalesce(struct net_device *dev,
^
net/core/ethtool.c:1234:12: warning: 'ethtool_get_ringparam' defined but not used [-Wunused-function]
static int ethtool_get_ringparam(struct net_device *dev, void __user *useraddr)
^
net/core/ethtool.c:1248:12: warning: 'ethtool_set_ringparam' defined but not used [-Wunused-function]
static int ethtool_set_ringparam(struct net_device *dev, void __user *useraddr)
^
net/core/ethtool.c:1261:31: warning: 'ethtool_get_channels' defined but not used [-Wunused-function]
static noinline_for_stack int ethtool_get_channels(struct net_device *dev,
^
net/core/ethtool.c:1276:31: warning: 'ethtool_set_channels' defined but not used [-Wunused-function]
vim +/max_rx +1280 net/core/ethtool.c
8b5933c38 amit salecha 2011-04-07 1274 }
8b5933c38 amit salecha 2011-04-07 1275
8b5933c38 amit salecha 2011-04-07 1276 static noinline_for_stack int ethtool_set_channels(struct net_device *dev,
8b5933c38 amit salecha 2011-04-07 1277 void __user *useraddr)
8b5933c38 amit salecha 2011-04-07 1278 {
214881553 Jacob Keller 2016-02-08 1279 struct ethtool_channels channels, max;
bb15c47a7 Jacob Keller 2016-02-08 @1280 u32 max_rx = 0;
bb15c47a7 Jacob Keller 2016-02-08 1281 int ret;
8b5933c38 amit salecha 2011-04-07 1282
214881553 Jacob Keller 2016-02-08 1283 if (!dev->ethtool_ops->set_channels || !dev->ethtool_ops->get_channels)
8b5933c38 amit salecha 2011-04-07 1284 return -EOPNOTSUPP;
8b5933c38 amit salecha 2011-04-07 1285
8b5933c38 amit salecha 2011-04-07 1286 if (copy_from_user(&channels, useraddr, sizeof(channels)))
8b5933c38 amit salecha 2011-04-07 1287 return -EFAULT;
8b5933c38 amit salecha 2011-04-07 1288
214881553 Jacob Keller 2016-02-08 @1289 ret = dev->ethtool_ops->get_channels(dev, &max);
214881553 Jacob Keller 2016-02-08 1290 if (ret)
214881553 Jacob Keller 2016-02-08 1291 return ret;
214881553 Jacob Keller 2016-02-08 1292
:::::: The code at line 1280 was first introduced by commit
:::::: bb15c47a78c2d9aab399f5dae676b03fefda79ca ethtool: correctly ensure {GS}CHANNELS doesn't conflict with GS{RXFH}
:::::: TO: Jacob Keller <jacob.e.keller@...el.com>
:::::: CC: 0day robot <fengguang.wu@...el.com>
---
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/octet-stream" (30591 bytes)
Powered by blists - more mailing lists