On Sun, Jun 19, 2016 at 9:23 PM, Michael van Elst <mlel...@serpens.de> wrote: > brad.har...@gmail.com (bch) writes: > >>kernel (adjusted from GENNERIC to allow dtrace support) from latest src >>panics: > >>(transcription): > >>reboot after panic: panic: kernel diagnostic assertion "M_GETCTX(m, >>struct ieee80211_node *) == NULL)" failed: file >>"/usr/src/sys/80211/ieee80211_output.c", line 1347 > > > That assertion seems to be bogus. It checks a field in an mbuf > that was just allocated in ieee80211_getmgtframe using m_getcl > and that may contain random data in the ctx pointer.
Indeed. > > Another similar assertion in the same file is #ifdef __FreeBSD__. > > Looking at the current FreeBSD code, it still abuses the rcvif > pointer for local data. But there are no such assertions, which > would be bogus in FreeBSD either. Thanks. I think we can remove the assertion(s) safely. (I'm not sure why the assertion hadn't failed ever. I guess my changes broke some implicit zeroing rcvif somewhere.) ozaki-r