From: Herbert Xu
Sent: 1/18/2006 12:26:12 PM
> On Wed, Jan 18, 2006 at 06:42:06AM -0800, Kris Katterjohn wrote:
> > 
> > Since it can't be a module and is always built in, we can just remove the
> > EXPORT_SYMBOL()s, right?
> 
> No, these are there so that other modules can use these symboles.

Okay, gotcha.

I went through again and I'm pretty confident in this patch. I added a few 
headers, some of which whose #includes were nested 2/3 files into others.

Let me know what you think (or if you see any obvious screwups). Thanks!

--- x/net/core/filter.c 2006-01-18 16:08:42.000000000 -0600
+++ y/net/core/filter.c 2006-01-18 16:08:36.000000000 -0600
@@ -18,21 +18,12 @@
 
 #include <linux/module.h>
 #include <linux/types.h>
-#include <linux/sched.h>
-#include <linux/mm.h>
-#include <linux/fcntl.h>
-#include <linux/socket.h>
-#include <linux/in.h>
-#include <linux/inet.h>
-#include <linux/netdevice.h>
-#include <linux/if_packet.h>
-#include <net/ip.h>
-#include <net/protocol.h>
 #include <linux/skbuff.h>
 #include <net/sock.h>
 #include <linux/errno.h>
-#include <linux/timer.h>
-#include <asm/system.h>
+#include <asm/atomic.h>
+#include <asm/bug.h>
+#include <asm/byteorder.h>
 #include <asm/uaccess.h>
 #include <linux/filter.h>
 


-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to