From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Nikita V. Shirokov" Subject: [PATCH bpf-next 09/10] [bpf]: make tun compatible w/ bpf_xdp_adjust_tail Date: Mon, 16 Apr 2018 23:51:30 -0700 Message-ID: <20180417065131.3632-10-tehnerd@tehnerd.com> References: <20180417065131.3632-1-tehnerd@tehnerd.com> Cc: netdev@vger.kernel.org, "Nikita V. Shirokov" To: Alexei Starovoitov , Daniel Borkmann , Jason Wang , mst@redhat.com Return-path: Received: from mail-pl0-f65.google.com ([209.85.160.65]:39583 "EHLO mail-pl0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751147AbeDQGyT (ORCPT ); Tue, 17 Apr 2018 02:54:19 -0400 Received: by mail-pl0-f65.google.com with SMTP id e7-v6so11458720plt.6 for ; Mon, 16 Apr 2018 23:54:19 -0700 (PDT) In-Reply-To: <20180417065131.3632-1-tehnerd@tehnerd.com> Sender: netdev-owner@vger.kernel.org List-ID: w/ bpf_xdp_adjust_tail helper xdp's data_end pointer could be changed as well (only "decrease" of pointer's location is going to be supported). changing of this pointer will change packet's size. for tun driver we need to adjust XDP_PASS handling by recalculating length of the packet if it was passed to the TCP/IP stack (in case if after xdp's prog run data_end pointer was adjusted) Signed-off-by: Nikita V. Shirokov --- drivers/net/tun.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/tun.c b/drivers/net/tun.c index 28583aa0c17d..0b488a958076 100644 --- a/drivers/net/tun.c +++ b/drivers/net/tun.c @@ -1688,6 +1688,7 @@ static struct sk_buff *tun_build_skb(struct tun_struct *tun, return NULL; case XDP_PASS: delta = orig_data - xdp.data; + len = xdp.data_end - xdp.data; break; default: bpf_warn_invalid_xdp_action(act); @@ -1708,7 +1709,7 @@ static struct sk_buff *tun_build_skb(struct tun_struct *tun, } skb_reserve(skb, pad - delta); - skb_put(skb, len + delta); + skb_put(skb, len); get_page(alloc_frag->page); alloc_frag->offset += buflen; -- 2.15.1