Skip to content

Commit

Permalink
bpf: Add ETH_HLEN=0 to stress test complexity
Browse files Browse the repository at this point in the history
The PR "datapath: Support NodePort BPF on L2-less devices" [1] has
increased the complexity of bpf_host and bpf_lxc by introducing a
support for ETH_HLEN=0.

Extend the base options by adding ETH_HLEN=0 to stress test the verifier
complexity when running on net-next (ETH_HLEN=0 depends on the
skb_change_head helper which was introduced in 5.8).

[1]: #14858

Suggested-by: Paul Chaignon <paul@cilium.io>
Signed-off-by: Martynas Pumputis <m@lambda.lt>
  • Loading branch information
brb committed Apr 13, 2021
1 parent 8a80550 commit 1a699db
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion bpf/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -97,8 +97,10 @@ ifeq ("$(KERNEL)","netnext")
# We need to define ENABLE_HOST_SERVICES_FULL to work around complexity issue #14234.
# We define ENABLE_CUSTOM_CALLS only for net-next (vs. net-next and 4.19) to work
# around program size issue #15539.
# We define ETH_HLEN only for net-next, as bpf_skb_change_head is non-available
# on 4.{9,19}.
MAX_BASE_OPTIONS += -DENABLE_TPROXY=1 -DENABLE_REDIRECT_FAST=1 -DENABLE_BANDWIDTH_MANAGER=1 \
-DENABLE_HOST_SERVICES_FULL=1 -DENABLE_CUSTOM_CALLS=1
-DENABLE_HOST_SERVICES_FULL=1 -DENABLE_CUSTOM_CALLS=1 -DETH_HLEN=0
endif
endif

Expand Down

0 comments on commit 1a699db

Please sign in to comment.