Re: [PATCH v2] arm: eBPF JIT compiler

From: Shubham Bansal
Date: Fri May 26 2017 - 12:57:21 EST


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@xxxxxxxxx> 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@xxxxxxxxx> 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@xxxxxxx> wrote:
>>>> Tested on ARMv7 with QEMU by me (Shubham Bansal).
>>>> Tested on ARMv5 by Andrew Lunn (andrew@xxxxxxx).
>>>> 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