[Differential] [Commented On] D1711: Changes to the callout code to restore active semantics and also add a test-framework and test to validate thecallout code (and potentially for use by other tests).

[hiren (hiren panchasara)]

hiren added a comment.

>>! In D1711#88, @rrs wrote:
> Hans:
> 
> I don't get your call sequence, I sent you an email on it..
> 
> Hiren:
> 
> Can you go up the call chain and dump the callout structure
> c in
> 0xffffffff80760064 in callout_lock (c=0xfffff8000d81dc98) at /usr/src/sys/kern/kern_timeout.c:530



  (kgdb) frame 4 
  #4  0xffffffff80760064 in callout_lock (c=0xfffff8000d81dc98) at /usr/src/sys/kern/kern_timeout.c:530
  530                     CC_LOCK(cc);
  (kgdb) p *c 
  $1 = {c_links = {le = {le_next = 0x0, le_prev = 0xfffffe0000804db8}, sle = {sle_next = 0x0}, tqe = {tqe_next = 0x0, tqe_prev = 0xfffffe0000804db8}}, 
    c_time = 903238992575048, c_precision = 241591893750, c_arg = 0xfffff8000d81dc00, c_func = 0xffffffff8082a610 <arptimer>, c_lock = 0x0, c_flags = 22, 
    c_cpu = 0}



> 
> There is something funny here, because the lock's listed (which is what the spin lock panic was on)
> are all mtx_lock=4.. which means they are un-locked.

REVISION DETAIL
  https://reviews.freebsd.org/D1711

To: rrs, gnn, rwatson, lstewart, jhb, kostikbel, sbruno, imp, adrian, hselasky
Cc: julian, hiren, jhb, kostikbel, emaste, delphij, neel, erj, freebsd-net