> +#ifdef CONFIG_PROC_FS > + /* Set non-zero value if this protocol manages its > + own /proc/net/PROTOCOL entry and the entry has inode column. */ > + int has_own_proc_entry; > +#endif
Is it really worth ifdefs in every file for what is (or should be) a bool value in the protocol structs of which there are not many > }; > > /* > diff --git a/net/socket.c b/net/socket.c > index dfe5b66..4044b58 100644 > --- a/net/socket.c > +++ b/net/socket.c > @@ -105,6 +105,10 @@ > #include <linux/sockios.h> > #include <linux/atalk.h> > > +#ifdef CONFIG_PROC_FS > +#include <linux/rwlock_types.h> > +#endif You can include it anyway and that makes it easier to avoid bugs -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
