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 <[email protected]> wrote: > > > On Fri, Sep 1, 2017 at 10:53 PM, Alexei Starovoitov < > [email protected]> wrote: > >> On Fri, Sep 1, 2017 at 4:30 AM, William Tu <[email protected]> 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 >
_______________________________________________ iovisor-dev mailing list [email protected] https://lists.iovisor.org/mailman/listinfo/iovisor-dev
