On Tue, Jul 8, 2014 at 2:24 AM, Alexei Starovoitov <a...@plumgrid.com> wrote:
> On Tue, Jul 8, 2014 at 12:06 AM, Zi Shen Lim <zlim....@gmail.com> wrote:
[...]
>>   Also, per discussion with Alexei, and additional suggestion from
>>   Daniel:
>>   - moved load_pointer() from net/core/filter.c into filter.h
>>     as bpf_load_pointer()
>>   which is done as a separate preparatory patch. [1]
>>
>> [1] http://patchwork.ozlabs.org/patch/366906/
>
> Just to clarify. This patch is on top of [1]

Yes, that's correct. This patch uses bpf_load_pointer() from [1].

BTW, looks like David has applied it in net-next 9f12fbe603f7
("net: filter: move load_pointer() into filter.h").

[...]
>> +               case BPF_DW:
>
> this case can be removed too.

Thanks for catching this :) I've removed it.

>
> Acked-by: Alexei Starovoitov <a...@plumgrid.com>
>
> Great work. Thanks.

Thanks for your time and feedback!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to