On 9/26/19 12:09 PM, Florian Westphal wrote: > Eric Dumazet <eric.duma...@gmail.com> wrote: >>> - secpath_reset(skb); >>> + skb_ext_reset(skb); >>> nf_reset(skb); >>> nf_reset_trace(skb); >> >> >> It is unfortunate nf_reset(skb) will call skb_ext_del(skb, >> SKB_EXT_BRIDGE_NF), >> which is useless after skb_ext_reset(skb) >> >> Maybe time for a nf_ct_reset() helper only dealing with nfct. > > Agree, but that seems more like -next material? Sure.
- [PATCH v2 net] sk_buff: drop all skb extensions on free a... Florian Westphal
- Re: [PATCH v2 net] sk_buff: drop all skb extensions ... Eric Dumazet
- Re: [PATCH v2 net] sk_buff: drop all skb extensi... Florian Westphal
- Re: [PATCH v2 net] sk_buff: drop all skb ext... Eric Dumazet
- Re: [PATCH v2 net] sk_buff: drop all skb extensions ... David Miller