linux-kernel - Re: [PATCH v2] arm: eBPF JIT compiler

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAHgaXd+HyzFLcQ8Lwam0N2TOVz_=JpQ+m2Myhs_+gZjdsHUdVg@mail.gmail.com>
Date:   Fri, 26 May 2017 22:27:02 +0530
From:   Shubham Bansal <illusionist.neo@...il.com>
To:     Andrew Lunn <andrew@...n.ch>
Cc:     Russell King - ARM Linux <linux@...linux.org.uk>,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
        Kees Cook <keescook@...omium.org>
Subject: Re: [PATCH v2] arm: eBPF JIT compiler

Hi Andrew,

Did you get the time to test the code with CONFIG_FRAME_POINTER? It
would be great if you could check if it works on ARMv5 so that Russell
can look at the patch.

Thanks.
Best,
Shubham Bansal


On Fri, May 26, 2017 at 5:06 AM, Shubham Bansal
<illusionist.neo@...il.com> wrote:
> Just to add. It a very very small part which deal with
> CONFIG_FRAME_POINTER just one move instruction.
> Best,
> Shubham Bansal
>
>
> On Fri, May 26, 2017 at 5:04 AM, Shubham Bansal
> <illusionist.neo@...il.com> wrote:
>> Hi Andrew,
>>
>> Oh. I didn't knew. Can you test it now and confirm it? I think it will
>> work and wouldn't take much of the time.
>>
>> Please.
>> Best,
>> Shubham Bansal
>>
>>
>> On Fri, May 26, 2017 at 4:53 AM, Andrew Lunn <andrew@...n.ch> wrote:
>>>> Tested on ARMv7 with QEMU by me (Shubham Bansal).
>>>> Tested on ARMv5 by Andrew Lunn (andrew@...n.ch).
>>>> Expected to work on ARMv6 as well, as its a part ARMv7 and part ARMv5.
>>>> Although, a proper testing is not done for ARMv6.
>>>>
>>>> Both of these testing are done with and without CONFIG_FRAME_POINTER
>>>> separately for LITTLE ENDIAN machine.
>>>
>>> Nope. I only tested it in the default configuration of mvebu_v5_defconfig.
>>> Please change the 'Both' to 'Some'.
>>>
>>>       Andrew