On 24 February 2020 18:54:27 EET, Stephen Hemminger <stephen@xxxxxxxxxxxxxxxxxx> wrote: >On Mon, 24 Feb 2020 18:46:22 +0200 >Nikolay Aleksandrov <nikolay@xxxxxxxxxxxxxxxxxxx> wrote: > >> - eth = eth_hdr(skb); >> skb_pull(skb, ETH_HLEN) > >you could just swap these two lines. Can't, still caching the wrong mac_header offset. br_allowed_ingress() can change the head pointer and also the offsets inside.