Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 11 Apr 2018 10:26:36 -0700
From:      "K. Macy" <kmacy@freebsd.org>
To:        David Wolfskill <david@catwhisker.org>, current@freebsd.org
Subject:   Re: panic: mtx_lock() of spin mutex (null) @ /usr/src/sys/net/iflib.c:3716
Message-ID:  <CAHM0Q_MYRAX4BnYa-uKPfCZNuo15-eePDfoqWynPrwrG=QnTow@mail.gmail.com>
In-Reply-To: <CAHM0Q_OHm1pq4nQDpCr-gTdb=Cc2SBO=VSrjviV9Pv9x3CS=pA@mail.gmail.com>
References:  <20180411113958.GE1134@albert.catwhisker.org> <CAHM0Q_OHm1pq4nQDpCr-gTdb=Cc2SBO=VSrjviV9Pv9x3CS=pA@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Actually ctx lock is still a mutex. Just add the STATE_LOCK_INIT line.
-M

On Wed, Apr 11, 2018 at 10:24 AM, K. Macy <kmacy@freebsd.org> wrote:
> Sorry about that. It looks like my review must have been missing a line.
>
> @@ -4702,8 +4707,8 @@ iflib_register(if_ctx_t ctx)
>
>         _iflib_assert(sctx);
>
> -       CTX_LOCK_INIT(ctx, device_get_nameunit(ctx->ifc_dev));
> -
> +       CTX_LOCK_INIT(ctx);
> +       STATE_LOCK_INIT(ctx, device_get_nameunit(ctx->ifc_dev));
>         ifp = ctx->ifc_ifp = if_gethandle(IFT_ETHER);
>         if (ifp == NULL) {
>                 device_printf(dev, "can not allocate ifnet structure\n");
> @@ -5430,8 +5435,8 @@ iflib_io_tqg_attach(struct grouptask *gt, void
> *uniq, int cpu, char *name)
>  }
>
>  void
>
> On Wed, Apr 11, 2018 at 4:39 AM, David Wolfskill <david@catwhisker.org> wrote:
>> This was running:
>>
>> FreeBSD g1-215.catwhisker.org 12.0-CURRENT FreeBSD 12.0-CURRENT #156  r332399M/332400:1200061: Wed Apr 11 04:17:45 PDT 2018     root@g1-215.catwhisker.org:/common/S4/obj/usr/src/amd64.amd64/sys/CANARY  amd64
>>
>> during boot, after updating from:
>>
>> FreeBSD g1-215.catwhisker.org 12.0-CURRENT FreeBSD 12.0-CURRENT #155  r332354M/332357:1200061: Tue Apr 10 04:00:41 PDT 2018     root@g1-215.catwhisker.org:/common/S4/obj/usr/src/amd64.amd64/sys/CANARY  amd64
>>
>> (My build machine, which uses an re((4) NIC, did not encounter the issue.)
>>
>> It appears that r332389 is implicated.
>>
>> ...
>> Unread portion of the kernel message buffer:
>>
>> __curthread () at ./machine/pcpu.h:230
>> 230             __asm("movq %%gs:%1,%0" : "=r" (td)
>> (kgdb) #0  __curthread () at ./machine/pcpu.h:230
>> #1  doadump (textdump=3) at /usr/src/sys/kern/kern_shutdown.c:361
>> #2  0xffffffff80433f4c in db_fncall_generic (addr=<optimized out>,
>>     rv=<optimized out>, nargs=<optimized out>, args=<optimized out>)
>>     at /usr/src/sys/ddb/db_command.c:609
>> #3  db_fncall (dummy1=<optimized out>, dummy2=<optimized out>,
>>     dummy3=<optimized out>, dummy4=<optimized out>)
>>     at /usr/src/sys/ddb/db_command.c:657
>> #4  0xffffffff80433a99 in db_command (last_cmdp=<optimized out>,
>>     cmd_table=<optimized out>, dopager=<optimized out>)
>>     at /usr/src/sys/ddb/db_command.c:481
>> #5  0xffffffff80433814 in db_command_loop ()
>>     at /usr/src/sys/ddb/db_command.c:534
>> #6  0xffffffff80436a3f in db_trap (type=<optimized out>, code=<optimized out>)
>>     at /usr/src/sys/ddb/db_main.c:250
>> #7  0xffffffff80b753e3 in kdb_trap (type=3, code=-61456, tf=<optimized out>)
>>     at /usr/src/sys/kern/subr_kdb.c:697
>> #8  0xffffffff80f7eaa8 in trap (frame=0xfffffe00004377a0)
>>     at /usr/src/sys/amd64/amd64/trap.c:548
>> #9  <signal handler called>
>> #10 kdb_enter (why=0xffffffff811df9d4 "panic", msg=<optimized out>)
>>     at /usr/src/sys/kern/subr_kdb.c:479
>> #11 0xffffffff80b2feda in vpanic (fmt=<optimized out>, ap=0xfffffe0000437910)
>>     at /usr/src/sys/kern/kern_shutdown.c:826
>> #12 0xffffffff80b2fca0 in kassert_panic (
>>     fmt=0xffffffff811dadca "mtx_lock() of spin mutex %s @ %s:%d")
>>     at /usr/src/sys/kern/kern_shutdown.c:723
>> #13 0xffffffff80b0ec93 in __mtx_lock_flags (c=0xfffff80008c85d88, opts=0,
>>     file=0xffffffff81113c90 "/usr/src/sys/net/iflib.c", line=<optimized out>)
>>     at /usr/src/sys/kern/kern_mutex.c:246
>> #14 0xffffffff80c466e1 in _task_fn_admin (context=0xfffff80008c85c00)
>>     at /usr/src/sys/net/iflib.c:3716
>> #15 0xffffffff80b73849 in gtaskqueue_run_locked (queue=0xfffff80008489500)
>>     at /usr/src/sys/kern/subr_gtaskqueue.c:331
>> #16 0xffffffff80b735c8 in gtaskqueue_thread_loop (arg=<optimized out>)
>>     at /usr/src/sys/kern/subr_gtaskqueue.c:506
>> #17 0xffffffff80af0064 in fork_exit (
>>     callout=0xffffffff80b73540 <gtaskqueue_thread_loop>,
>>     arg=0xfffffe0844223008, frame=0xfffffe0000437ac0)
>>     at /usr/src/sys/kern/kern_fork.c:1039
>> #18 <signal handler called>
>> (kgdb)
>>
>> If the dump would be useful, I can put it up for access.
>>
>> Peace,
>> david
>> --
>> David H. Wolfskill                              david@catwhisker.org
>> Well, what did you EXPECT from Trump?  He has a history of breaking promises.
>>
>> See http://www.catwhisker.org/~david/publickey.gpg for my public key.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAHM0Q_MYRAX4BnYa-uKPfCZNuo15-eePDfoqWynPrwrG=QnTow>