Skip to contentSkip to conversation

: Fix compiler error with CONFIG_PROC_FS=n

PATCH In review Johann Felix Soden 3 participants 3 messages Feb 5, 2008 – Feb 6, 2008

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 : Fix compiler error with CONFIG_PROC_FS=n

From: Johann Felix Soden <[email protected]>

Handle CONFIG_PROC_FS=n in net/ipv4/fib_frontend.c because:

net/ipv4/fib_frontend.c: In function 'fib_net_init':
net/ipv4/fib_frontend.c:1032: error: implicit declaration of function 'fib_proc_init' net/ipv4/fib_frontend.c: In function 'fib_net_exit':
net/ipv4/fib_frontend.c:1047: error: implicit declaration of function 'fib_proc_exit'

---
 net/ipv4/fib_frontend.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
net/ipv4/fib_frontend.c +5 -0
index 86ff271..581d588 100644
@@ -1029,9 +1029,12 @@ static int __net_init fib_net_init(struct net *net)
 	error = nl_fib_lookup_init(net);
 	if (error < 0)
 		goto out_nlfl;
+#ifdef CONFIG_PROC_FS
 	error = fib_proc_init(net);
 	if (error < 0)
 		goto out_proc;
+#endif
+
 out:
 	return error;
 
@@ -1044,7 +1047,9 @@ out_nlfl:
 
 static void __net_exit fib_net_exit(struct net *net)
 {
+#ifdef CONFIG_PROC_FS
 	fib_proc_exit(net);
+#endif
 	nl_fib_lookup_exit(net);
 	ip_fib_net_exit(net);
 }
Signature · 1 line
1.5.4
#1 lore raw
Quoted 10 lines · Johann Felix Soden

From: Johann Felix Soden <[email protected]>

Handle CONFIG_PROC_FS=n in net/ipv4/fib_frontend.c because:

net/ipv4/fib_frontend.c: In function 'fib_net_init':
net/ipv4/fib_frontend.c:1032: error: implicit declaration of function 'fib_proc_init' net/ipv4/fib_frontend.c: In function 'fib_net_exit':
net/ipv4/fib_frontend.c:1047: error: implicit declaration of function 'fib_proc_exit'

Fix from Li Zefan is already commited.
(cc8274f50f2ad9a97a837451f63a0a3e65f7f490)

maintainer
#2 lore raw
In reply to Johann Felix Soden

From: Johann Felix Soden <[email protected]>
Date: Tue, 05 Feb 2008 19:43:54 +0100

Quoted 10 lines

From: Johann Felix Soden <[email protected]>

Handle CONFIG_PROC_FS=n in net/ipv4/fib_frontend.c because:

net/ipv4/fib_frontend.c: In function 'fib_net_init':
net/ipv4/fib_frontend.c:1032: error: implicit declaration of function 'fib_proc_init' net/ipv4/fib_frontend.c: In function 'fib_net_exit':
net/ipv4/fib_frontend.c:1047: error: implicit declaration of function 'fib_proc_exit'

I'm pretty sure I merged in a change this morning which fixes this.

BTW, in general we do not put ifdefs into *.c code to fix stuff like this, instead we add empty implementations into a header file which is a better place for ifdef tests.

And that's how the patch which was merged handles this problem.