On Tue, Mar 15, 2022 at 11:56:44AM -0700, Jakub Kicinski wrote: > On Tue, 15 Mar 2022 10:15:09 +0100 Pablo Neira Ayuso wrote: > > + return false > > +#ifdef CONFIG_NF_TABLES_INET > > + || family == NFPROTO_INET > > +#endif > > +#ifdef CONFIG_NF_TABLES_IPV4 > > + || family == NFPROTO_IPV4 > > +#endif > > +#ifdef CONFIG_NF_TABLES_ARP > > + || family == NFPROTO_ARP > > +#endif > > +#ifdef CONFIG_NF_TABLES_NETDEV > > + || family == NFPROTO_NETDEV > > +#endif > > +#if IS_ENABLED(CONFIG_NF_TABLES_BRIDGE) > > is there a reason this one is IS_ENABLED() and everything else is ifdef? bridge might be compiled as a module, if the bridge infrastructure also comes a module as well. Anything else is either built-in or off.
