Skip to content

Commit 6103aa9

Browse files
jrfastabdavem330
authored andcommitted
net: implement XDP_REDIRECT for xdp generic
Add support for redirect to xdp generic creating a fall back for devices that do not yet have support and allowing test infrastructure using veth pairs to be built. Signed-off-by: John Fastabend <[email protected]> Tested-by: Andy Gospodarek <[email protected]> Acked-by: Daniel Borkmann <[email protected]> Acked-by: Jesper Dangaard Brouer <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 832622e commit 6103aa9

File tree

3 files changed

+47
-2
lines changed

3 files changed

+47
-2
lines changed

include/linux/filter.h

+1
Original file line numberDiff line numberDiff line change
@@ -712,6 +712,7 @@ bool bpf_helper_changes_pkt_data(void *func);
712712
struct bpf_prog *bpf_patch_insn_single(struct bpf_prog *prog, u32 off,
713713
const struct bpf_insn *patch, u32 len);
714714

715+
int xdp_do_generic_redirect(struct net_device *dev, struct sk_buff *skb);
715716
int xdp_do_redirect(struct net_device *dev, struct xdp_buff *xdp);
716717

717718
void bpf_warn_invalid_xdp_action(u32 act);

net/core/dev.c

+20-2
Original file line numberDiff line numberDiff line change
@@ -3902,6 +3902,7 @@ static u32 netif_receive_generic_xdp(struct sk_buff *skb,
39023902
__skb_push(skb, -off);
39033903

39043904
switch (act) {
3905+
case XDP_REDIRECT:
39053906
case XDP_TX:
39063907
__skb_push(skb, mac_len);
39073908
/* fall through */
@@ -3956,14 +3957,27 @@ static int do_xdp_generic(struct sk_buff *skb)
39563957

39573958
if (xdp_prog) {
39583959
u32 act = netif_receive_generic_xdp(skb, xdp_prog);
3960+
int err;
39593961

39603962
if (act != XDP_PASS) {
3961-
if (act == XDP_TX)
3963+
switch (act) {
3964+
case XDP_REDIRECT:
3965+
err = xdp_do_generic_redirect(skb->dev, skb);
3966+
if (err)
3967+
goto out_redir;
3968+
/* fallthru to submit skb */
3969+
case XDP_TX:
39623970
generic_xdp_tx(skb, xdp_prog);
3971+
break;
3972+
}
39633973
return XDP_DROP;
39643974
}
39653975
}
39663976
return XDP_PASS;
3977+
out_redir:
3978+
trace_xdp_exception(skb->dev, xdp_prog, XDP_REDIRECT);
3979+
kfree_skb(skb);
3980+
return XDP_DROP;
39673981
}
39683982

39693983
static int netif_rx_internal(struct sk_buff *skb)
@@ -3977,8 +3991,12 @@ static int netif_rx_internal(struct sk_buff *skb)
39773991
if (static_key_false(&generic_xdp_needed)) {
39783992
int ret = do_xdp_generic(skb);
39793993

3994+
/* Consider XDP consuming the packet a success from
3995+
* the netdev point of view we do not want to count
3996+
* this as an error.
3997+
*/
39803998
if (ret != XDP_PASS)
3981-
return NET_RX_DROP;
3999+
return NET_RX_SUCCESS;
39824000
}
39834001

39844002
#ifdef CONFIG_RPS

net/core/filter.c

+26
Original file line numberDiff line numberDiff line change
@@ -2437,6 +2437,32 @@ int xdp_do_redirect(struct net_device *dev, struct xdp_buff *xdp)
24372437
}
24382438
EXPORT_SYMBOL_GPL(xdp_do_redirect);
24392439

2440+
int xdp_do_generic_redirect(struct net_device *dev, struct sk_buff *skb)
2441+
{
2442+
struct redirect_info *ri = this_cpu_ptr(&redirect_info);
2443+
unsigned int len;
2444+
2445+
dev = dev_get_by_index_rcu(dev_net(dev), ri->ifindex);
2446+
ri->ifindex = 0;
2447+
if (unlikely(!dev)) {
2448+
bpf_warn_invalid_xdp_redirect(ri->ifindex);
2449+
goto err;
2450+
}
2451+
2452+
if (unlikely(!(dev->flags & IFF_UP)))
2453+
goto err;
2454+
2455+
len = dev->mtu + dev->hard_header_len + VLAN_HLEN;
2456+
if (skb->len > len)
2457+
goto err;
2458+
2459+
skb->dev = dev;
2460+
return 0;
2461+
err:
2462+
return -EINVAL;
2463+
}
2464+
EXPORT_SYMBOL_GPL(xdp_do_generic_redirect);
2465+
24402466
BPF_CALL_2(bpf_xdp_redirect, u32, ifindex, u64, flags)
24412467
{
24422468
struct redirect_info *ri = this_cpu_ptr(&redirect_info);

0 commit comments

Comments
 (0)