Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-14 Thread Alexei Starovoitov
On Tue, Oct 13, 2020 at 04:07:26PM -0700, Jakub Kicinski wrote: > On Tue, 13 Oct 2020 22:40:09 +0200 Jesper Dangaard Brouer wrote: > > > FWIW I took a quick swing at testing it with the HW I have and it did > > > exactly what hardware should do. The TX unit entered an error state > > > and then th

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-13 Thread Jakub Kicinski
On Tue, 13 Oct 2020 22:40:09 +0200 Jesper Dangaard Brouer wrote: > > FWIW I took a quick swing at testing it with the HW I have and it did > > exactly what hardware should do. The TX unit entered an error state > > and then the driver detected that and reset it a few seconds later. > > The driv

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-13 Thread Jesper Dangaard Brouer
On Sat, 10 Oct 2020 09:32:12 -0700 Jakub Kicinski wrote: > On Sat, 10 Oct 2020 12:44:02 +0200 Jesper Dangaard Brouer wrote: > > > > > We will not be sprinkling validation checks across the drivers because > > > > > some reconfiguration path may occasionally yield a bad packet, or it's > > > > > h

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-11 Thread Jakub Kicinski
On Sat, 10 Oct 2020 16:52:48 -0700 John Fastabend wrote: > Jakub Kicinski wrote: > > FWIW I took a quick swing at testing it with the HW I have and it did > > exactly what hardware should do. The TX unit entered an error state > > and then the driver detected that and reset it a few seconds later.

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-10 Thread John Fastabend
Jakub Kicinski wrote: > On Sat, 10 Oct 2020 12:44:02 +0200 Jesper Dangaard Brouer wrote: > > > > > We will not be sprinkling validation checks across the drivers because > > > > > some reconfiguration path may occasionally yield a bad packet, or it's > > > > > hard to do something right with BPF.

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-10 Thread Jesper Dangaard Brouer
On Fri, 9 Oct 2020 16:00:10 -0700 Jakub Kicinski wrote: > On Fri, 09 Oct 2020 13:49:14 -0700 John Fastabend wrote: > > Jakub Kicinski wrote: > > > On Thu, 08 Oct 2020 16:08:57 +0200 Jesper Dangaard Brouer wrote: > > > > V3: Drop enforcement of MTU in net-core, leave it to drivers > > >

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-10 Thread Jakub Kicinski
On Sat, 10 Oct 2020 12:44:02 +0200 Jesper Dangaard Brouer wrote: > > > > We will not be sprinkling validation checks across the drivers because > > > > some reconfiguration path may occasionally yield a bad packet, or it's > > > > hard to do something right with BPF. > > > > > > This is a dr

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-09 Thread Jakub Kicinski
On Fri, 09 Oct 2020 13:49:14 -0700 John Fastabend wrote: > Jakub Kicinski wrote: > > On Thu, 08 Oct 2020 16:08:57 +0200 Jesper Dangaard Brouer wrote: > > > V3: Drop enforcement of MTU in net-core, leave it to drivers > > > > Sorry for being late to the discussion. > > > > I absolutely disagre

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-09 Thread Alexei Starovoitov
On Fri, Oct 09, 2020 at 01:49:14PM -0700, John Fastabend wrote: > Jakub Kicinski wrote: > > On Thu, 08 Oct 2020 16:08:57 +0200 Jesper Dangaard Brouer wrote: > > > V3: Drop enforcement of MTU in net-core, leave it to drivers > > > > Sorry for being late to the discussion. > > > > I absolutely disa

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-09 Thread John Fastabend
Jakub Kicinski wrote: > On Thu, 08 Oct 2020 16:08:57 +0200 Jesper Dangaard Brouer wrote: > > V3: Drop enforcement of MTU in net-core, leave it to drivers > > Sorry for being late to the discussion. > > I absolutely disagree. We had cases in the past where HW would lock up > if it was sent a frame

Re: [PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-09 Thread Jakub Kicinski
On Thu, 08 Oct 2020 16:08:57 +0200 Jesper Dangaard Brouer wrote: > V3: Drop enforcement of MTU in net-core, leave it to drivers Sorry for being late to the discussion. I absolutely disagree. We had cases in the past where HW would lock up if it was sent a frame with bad geometry. We will not be

[PATCH bpf-next V3 0/6] bpf: New approach for BPF MTU handling

2020-10-08 Thread Jesper Dangaard Brouer
This patchset drops all the MTU checks in TC BPF-helpers that limits growing the packet size. This is done because these BPF-helpers doesn't take redirect into account, which can result in their MTU check being done against the wrong netdev. The new approach is to give BPF-programs knowledge about