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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 20 Nov 2007 17:44:24 -0800 (PST) From: David Miller <davem@...emloft.net> To: horms@...ge.net.au Cc: xemul@...nvz.org, netdev@...r.kernel.org, devel@...nvz.org Subject: Re: [PATCH][IPVS] Fix compiler warning about unused register_ip_vs_protocol From: Simon Horman <horms@...ge.net.au> Date: Wed, 21 Nov 2007 10:38:02 +0900 > On Tue, Nov 20, 2007 at 03:08:42PM +0300, Pavel Emelyanov wrote: > > This is silly, but I have turned the CONFIG_IP_VS to m, > > to check the compilation of one (recently sent) fix > > and set all the CONFIG_IP_VS_PROTO_XXX options to n to > > speed up the compilation. > > > > In this configuration the compiler warns me about > > > > CC [M] net/ipv4/ipvs/ip_vs_proto.o > > net/ipv4/ipvs/ip_vs_proto.c:49: warning: ‘register_ip_vs_protocol’ defined but not used > > > > Indeed. With no protocols selected there are no > > calls to this function - all are compiled out with > > ifdefs. > > > > Maybe the best fix would be to surround this call with > > ifdef-s or tune the Kconfig dependences, but I think that > > marking this register function as __used is enough. No? > > > > Signed-off-by: Pavel Emelyanov <xemul@...nvz.org> > > That is kind of interesting. > > On the one hand it might be more correct in some way to use some Kconfig > foo to get rid of register_ip_vs_protocol, and also most of > the body of ‘register_ip_vs_protocol() if no protocols are defined. > > But on the other hand, its really a corner case, because in practice if > you are going to use IPVS then you are going to need at least one > protocol. So I'm happy with the simplicity of the __used solution, even > though strictly speaking it is a lie. > > Acked-by: Simon Horman <horms@...ge.net.au> Ok, I've applied Pavel's patch. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists