[iovisor-dev] [PATCH RFC] bpf: add connection tracking helper functions

William Tu u9012063 at gmail.com
Sun Sep 3 22:18:29 UTC 2017


Hi Alexei,
Thanks, I do see the lockdep complain now. I will switch to use GFP_ATOMIC.
William

On Sat, Sep 2, 2017 at 8:20 AM, William Tu <u9012063 at gmail.com> wrote:

>
>
> On Fri, Sep 1, 2017 at 10:53 PM, Alexei Starovoitov <
> alexei.starovoitov at gmail.com> wrote:
>
>> On Fri, Sep 1, 2017 at 4:30 AM, William Tu <u9012063 at gmail.com> wrote:
>> > +
>> > +       /* TODO: conntrack expectation */
>> > +
>> > +       nf_ct_zone_init(&zone, info->zone_id,
>> > +                       NF_CT_DEFAULT_ZONE_DIR, 0);
>> > +       tmpl = nf_ct_tmpl_alloc(net, &zone, GFP_KERNEL);
>>
>> did you test with lockdep?
>> I think above should complain.
>>
>
> I didn't see any complain. At least after system boots up, the
> "Lock dependency validator: Copyright (c) 2006 Red Hat, Inc., Ingo Molnar"
> shows all 256 testcases passed.
>
> I have lock debugging as below
> #
> # Lock Debugging (spinlocks, mutexes, etc...)
> #
> CONFIG_DEBUG_RT_MUTEXES=y
> CONFIG_DEBUG_SPINLOCK=y
> CONFIG_DEBUG_MUTEXES=y
> CONFIG_DEBUG_WW_MUTEX_SLOWPATH=y
> CONFIG_DEBUG_LOCK_ALLOC=y
> CONFIG_PROVE_LOCKING=y
> CONFIG_LOCKDEP=y
> CONFIG_LOCK_STAT=y
> CONFIG_DEBUG_LOCKDEP=y
> CONFIG_DEBUG_ATOMIC_SLEEP=y
> CONFIG_DEBUG_LOCKING_API_SELFTESTS=y
> CONFIG_LOCK_TORTURE_TEST=m
> CONFIG_WW_MUTEX_SELFTEST=m
> CONFIG_TRACE_IRQFLAGS=y
> CONFIG_STACKTRACE=y
>
> Thanks,
> William
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.iovisor.org/pipermail/iovisor-dev/attachments/20170903/21dcd57c/attachment.html>


More information about the iovisor-dev mailing list