Date: Mon, 22 May 2017 13:05:38 -0700 From: Kees Cook <keescook@...omium.org> To: Shubham Bansal <illusionist.neo@...il.com> Cc: Daniel Borkmann <daniel@...earbox.net>, David Miller <davem@...emloft.net>, Mircea Gherzan <mgherzan@...il.com>, Network Development <netdev@...r.kernel.org>, "kernel-hardening@...ts.openwall.com" <kernel-hardening@...ts.openwall.com>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, ast@...com Subject: Re: arch: arm: bpf: Converting cBPF to eBPF for arm 32 bit On Mon, May 22, 2017 at 10:04 AM, Shubham Bansal <illusionist.neo@...il.com> wrote: > These all benchmarks are for ARMv7. Thanks! In the future, try to avoid the white-space damage (line-wrapping). And it looks like you've still got debugging turned on in your jit code: [ 56.176033] test_bpf: #21 LD_CPU [ 56.176329] bpf_jit: *** NOT YET: opcode 85 *** [ 56.176565] jited:0 2639 702 PASS That breaks the test report line. After I cleaned these up and parsed the results, they look great. Most things are half the speed of the interpreter, if not better. Only the LD_ABS suffered, and that's mainly the const blinding, I assume. Please post your current patch. Thanks for this! -Kees -- Kees Cook Pixel Security View attachment "jitted.txt" of type "text/plain" (29420 bytes)
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.