panic: mb_dtor_pack: ref_cnt != 1
Brian Fundakowski Feldman
green at freebsd.org
Mon Nov 7 12:54:24 PST 2005
On Sat, Nov 05, 2005 at 08:47:30PM +0100, Andre Oppermann wrote:
> Gleb Smirnoff wrote:
> >
> > On Sat, Nov 05, 2005 at 11:01:16AM +0300, Gleb Smirnoff wrote:
> > T> On Sat, Nov 05, 2005 at 05:41:05AM +0200, Giorgos Keramidas wrote:
> > T> G> On 2005-11-05 03:34, Gleb Smirnoff <glebius at freebsd.org> wrote:
> > T> G> > Andre, Thierry, Sam,
> > T> G> >
> > T> G> > this patch should fix the problems
> > T> G>
> > T> G> But it panics in mb_dtor_pack() because ext_type != EXT_CLUSTER
> > T> G> when my ath0 interface tries to associate with an AP.
> > T> G>
> > T> G> I had to change this too, to make things work:
> > T>
> > T> Updated patch.
> >
> > One more update. Since I have removed this block:
> >
> > if (*(m->m_ext.ref_cnt) == 0)
> > *(m->m_ext.ref_cnt) = 1;
> >
> > I have also altered KASSERT in mb_dtor_pack(). I don't like
> > inventing an incorrect invariant check and then adding helpers
> > to avoid it being triggered.
>
> This block is still required for the packet zone as the refcount
> is not re-initialized for mbuf+cluster out of the packet zone.
> You may up with wrong refcounts if you remove this check. It
> is not directly related to the KASSERT in mb_dtor_pack() but to
> the way atomic_fetchadd_int() works in concurrency situations.
>
> I have committed a fix to the same effect as your proposed patch.
> When UMA is fixed this may be reverted again.
Could you submit a UMA bug report? There are a reasonable number
of hackers around that can fix UMA/VM bugs. I wouldn't mind at
trying my hand at one again these days.
--
Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\
<> green at FreeBSD.org \ The Power to Serve! \
Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\
More information about the freebsd-current
mailing list