On 26.04.2015 09:19, Greg KH wrote:
On Sun, Apr 26, 2015 at 09:11:03AM +0200, Dirk Behme wrote:
Back porting the upstream commit b1cb59cf2efe7 ("net: sysctl_net_core: check
SNDBUF and RCVBUF for min length") to -stable introduces a compiler warning
net/core/sysctl_net_core.c:26:12: warning: ‘one’ defined but not used [-Wunused-variable]
static int one = 1;
While upstream has an additional user of the variable 'one' after that commit
("tstamp_allow_data"), -stable doesn't have any user of 'one' any more, resulting
in above compiler warning. Or in other words: -stable doesn't have the
commit b245be1f4db1a0 ("net-timestamp: no-payload only sysctl") introducing the
additional user of 'one'.
Fix the compiler warning by removing the unused variable.
No, this was done on purpose, sorry, I can't take this patch. The
network maintainer wants it this way to be easier to backport things.
We can live with the compiler warning.
Would it be an option to pick
b245be1f4db1a0 ("net-timestamp: no-payload only sysctl")
then, too?
Thanks
Dirk
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html