On 2 November 2016 at 07:39, Roopa Prabhu <ro...@cumulusnetworks.com> wrote: >> diff --git a/net/core/Makefile b/net/core/Makefile >> index d6508c2..a675fd3 100644 >> --- a/net/core/Makefile >> +++ b/net/core/Makefile >> @@ -23,7 +23,7 @@ obj-$(CONFIG_NETWORK_PHY_TIMESTAMPING) += timestamping.o >> obj-$(CONFIG_NET_PTP_CLASSIFY) += ptp_classifier.o >> obj-$(CONFIG_CGROUP_NET_PRIO) += netprio_cgroup.o >> obj-$(CONFIG_CGROUP_NET_CLASSID) += netclassid_cgroup.o >> -obj-$(CONFIG_LWTUNNEL) += lwtunnel.o >> +obj-$(CONFIG_LWTUNNEL) += lwtunnel.o lwt_bpf.o > > Any reason you want to keep lwt bpf under the main CONFIG_LWTUNNEL infra > config ?. > since it is defined as yet another plug-gable encap function, seems like it > will be better under a separate > CONFIG_LWTUNNEL_BPF or CONFIG_LWT_BPF that depends on CONFIG_LWTUNNEL
The code was so minimal with no additional dependencies that I didn't see a need for a separate Kconfig. I'm fine adding that in the next iteration though. No objections.