Re: [PATCH bpf-next 02/12] net: Add BPF_XDP_EGRESS as a bpf_attach_type
From: Toke Høiland-Jørgensen <hidden>
Date: 2020-01-23 11:34:19
From: Toke Høiland-Jørgensen <hidden>
Date: 2020-01-23 11:34:19
David Ahern [off-list ref] writes:
From: Prashant Bhole <redacted> Add new bpf_attach_type, BPF_XDP_EGRESS, for BPF programs attached at the XDP layer, but the egress path. Since egress path does not have rx_queue_index and ingress_ifindex set, update xdp_is_valid_access to block access to these entries in the xdp context when a program is attached to egress path.
Isn't the whole point of this to be able to use unchanged XDP programs? But now you're introducing a semantic difference. Since supposedly only point-to-point links are going to be using this attach type, don't they know enough about their peer device to be able to populate those fields with meaningful values, instead of restricting access to them? -Toke