Re: [PATCH net v2 00/10] bpf: offload: check netdev pointer in the drivers and namespace trouble

2017-11-29 Thread Daniel Borkmann
On 11/29/2017 09:52 AM, Jiri Pirko wrote: > Wed, Nov 29, 2017 at 09:18:29AM CET, j...@resnulli.us wrote: >> Tue, Nov 21, 2017 at 01:25:56AM CET, dan...@iogearbox.net wrote: >>> On 11/21/2017 12:21 AM, Jakub Kicinski wrote: Hi! This series addresses some late comments and moves

Re: [PATCH net v2 00/10] bpf: offload: check netdev pointer in the drivers and namespace trouble

2017-11-29 Thread Jiri Pirko
Wed, Nov 29, 2017 at 09:18:29AM CET, j...@resnulli.us wrote: >Tue, Nov 21, 2017 at 01:25:56AM CET, dan...@iogearbox.net wrote: >>On 11/21/2017 12:21 AM, Jakub Kicinski wrote: >>> Hi! >>> >>> This series addresses some late comments and moves checking if program >>> has been loaded for the correct

Re: [PATCH net v2 00/10] bpf: offload: check netdev pointer in the drivers and namespace trouble

2017-11-29 Thread Jiri Pirko
Tue, Nov 21, 2017 at 01:25:56AM CET, dan...@iogearbox.net wrote: >On 11/21/2017 12:21 AM, Jakub Kicinski wrote: >> Hi! >> >> This series addresses some late comments and moves checking if program >> has been loaded for the correct device to the drivers. There are also >> some problems with net

Re: [PATCH net v2 00/10] bpf: offload: check netdev pointer in the drivers and namespace trouble

2017-11-20 Thread Daniel Borkmann
On 11/21/2017 12:21 AM, Jakub Kicinski wrote: > Hi! > > This series addresses some late comments and moves checking if program > has been loaded for the correct device to the drivers. There are also > some problems with net namespaces which I didn't take into consideration. > On the kernel side

[PATCH net v2 00/10] bpf: offload: check netdev pointer in the drivers and namespace trouble

2017-11-20 Thread Jakub Kicinski
Hi! This series addresses some late comments and moves checking if program has been loaded for the correct device to the drivers. There are also some problems with net namespaces which I didn't take into consideration. On the kernel side we will now simply ignore namespace moves. Since the user