Re: [PATCH net-next 1/2] bpf: improve htab inlining for future 32 bit jits

2017-08-18 Thread Alexei Starovoitov
On 8/18/17 5:21 PM, Daniel Borkmann wrote: On 08/19/2017 02:00 AM, Alexei Starovoitov wrote: On 8/18/17 4:51 PM, Daniel Borkmann wrote: Lets future proof htab lookup inlining, commit 9015d2f59535 ("bpf: inline htab_map_lookup_elem()") was making the assumption that a direct call emission to __h

Re: [PATCH net-next 1/2] bpf: improve htab inlining for future 32 bit jits

2017-08-18 Thread Daniel Borkmann
On 08/19/2017 02:00 AM, Alexei Starovoitov wrote: On 8/18/17 4:51 PM, Daniel Borkmann wrote: Lets future proof htab lookup inlining, commit 9015d2f59535 ("bpf: inline htab_map_lookup_elem()") was making the assumption that a direct call emission to __htab_map_lookup_elem() will always work out f

Re: [PATCH net-next 1/2] bpf: improve htab inlining for future 32 bit jits

2017-08-18 Thread Alexei Starovoitov
On 8/18/17 4:51 PM, Daniel Borkmann wrote: Lets future proof htab lookup inlining, commit 9015d2f59535 ("bpf: inline htab_map_lookup_elem()") was making the assumption that a direct call emission to __htab_map_lookup_elem() will always work out for JITs. This is currently true since all JITs we h

[PATCH net-next 1/2] bpf: improve htab inlining for future 32 bit jits

2017-08-18 Thread Daniel Borkmann
Lets future proof htab lookup inlining, commit 9015d2f59535 ("bpf: inline htab_map_lookup_elem()") was making the assumption that a direct call emission to __htab_map_lookup_elem() will always work out for JITs. This is currently true since all JITs we have are for 64 bit archs, but in case of 32 b