Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> wrote: > > OTOH I think that trace support is a very important thing to have and I > > doubt that compiling it out would save a lot of size. > > > > But sure, if you think it makes sense I can add CONFIG_NF_TRACE and > > make nf_tables tracing conditional. > > This is just a cosmetic comment, but I can live with that long #if > line. Leave it up to you to decide. I see. I that case please consider applying the present patch. I think adding extra kconfig symbol just for the sake of avoiding two longer IS_ENABLED() statements is plain overkill :-) -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html