On 20/09/2016 6:40 PM, Alexei Starovoitov wrote:
On Tue, Sep 20, 2016 at 03:53:10PM +0300, Tariq Toukan wrote:
+       case XDP_ABORTED:
It is not clearly defined, but I believe XDP_ABORTED should also result
in a warning (calling bpf_warn_invalid_xdp_action(act)).
I'll add this.
Certainly NOT.
XDP_ABORTED is an exception case when program does divide by zero.
It should NOT do bpf_warn. It must drop the packet.
We discussed it several months ago.
See mlx4/en_rx.c for canonical implementation.

This is also the example given here:

I prefer to align with the documentation (and with current mlx4 driver code), which means keeping the XDP_ABORTED w/o a warning. Anyway, I don't think this should block the coming V2. If you decide to change documentation/specification, we will simply adjust our drivers accordingly.


Reply via email to