On 5/28/19, 1:43 PM, "Eric Dumazet" <eric.duma...@gmail.com> wrote:
On 5/28/19 11:54 AM, Lawrence Brakmo wrote: > On 5/28/19, 6:43 AM, "netdev-ow...@vger.kernel.org on behalf of Eric Dumazet" <netdev-ow...@vger.kernel.org on behalf of eric.duma...@gmail.com> wrote: > > Why are you using preempt_enable_no_resched() here ? > > Because that is what __BPF_PROG_RUN_ARRAY() calls and the macro > BPF_PROG_CGROUP_INET_EGRESS_RUN_ARRAY() is an instantiation of it > (with minor changes in the return value). I do not see this in my tree. Please rebase your tree, do not bring back an issue that was solved already. My mistake. My tree is up to date, I looked in the wrong place and did not realized the call had changed. I will fix it and resubmit again. Thank you for the feedback.