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.
thanks,
greg k-h
--
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