Re: 7_STABLE panic()

[christos%astron.com@localhost (Christos Zoulas)]

In article <587366A8.4070504%egervary.hu@localhost>,
Egerváry Gergely  <gergely%egervary.hu@localhost> wrote:
>Before send_pr, let's have a look: probably GRE or IPFilter related?
>Upstream connection is on a GRE tunnel.
>
>0xffffffff804ce60f in cpu_reboot (howto=howto@entry=260,
>bootstr=bootstr@entry=0x0)
>    at ../../../../arch/amd64/amd64/machdep.c:671
>671                     dumpsys();
>(gdb) bt
>#0  0xffffffff804ce60f in cpu_reboot (howto=howto@entry=260,
>bootstr=bootstr@entry=0x0)
>    at ../../../../arch/amd64/amd64/machdep.c:671
>#1  0xffffffff8066c1e2 in vpanic (fmt=fmt@entry=0xffffffff80a35300
>"m_makewritable: length changed",
>    ap=ap@entry=0xfffffe8040005c70) at ../../../../kern/subr_prf.c:340
>#2  0xffffffff8066c29d in panic (fmt=fmt@entry=0xffffffff80a35300
>"m_makewritable: length changed")
>    at ../../../../kern/subr_prf.c:256
>#3  0xffffffff806f4d2c in m_makewritable
>(mp=mp@entry=0xfffffe8040005d30, off=off@entry=0, len=len@entry=-1,
>    how=how@entry=1) at ../../../../kern/uipc_mbuf.c:1355
>#4  0xffffffff8040893c in ipf_check_wrapper (arg=<optimized out>,
>mp=0xfffffe8040005d30, ifp=0xfffffe8116321008,
>    dir=2) at ../../../../external/bsd/ipf/netinet/ip_fil_netbsd.c:186
>#5  0xffffffff8059a2ee in pfil_run_hooks (ph=<optimized out>,
>mp=mp@entry=0xfffffe8040005dd0,
>    ifp=ifp@entry=0xfffffe8116321008, dir=dir@entry=2) at
>../../../../net/pfil.c:266
>#6  0xffffffff80425a3a in ip_output (m0=m0@entry=0xfffffe80a5751400) at
>../../../../netinet/ip_output.c:480
>#7  0xffffffff805c5ee2 in rip_output (m=m@entry=0xfffffe80a5751400) at
>../../../../netinet/raw_ip.c:389
>#8  0xffffffff805c603d in rip_send (l=<optimized out>,
>control=<optimized out>, nam=0x0, m=0xfffffe80a5751400,
>    so=0xfffffe810e4c2498) at ../../../../netinet/raw_ip.c:781
>#9  rip_send_wrapper (a=0xfffffe810e4c2498, b=0xfffffe80a5751400, c=0x0,
>d=<optimized out>, e=<optimized out>)
>    at ../../../../netinet/raw_ip.c:850
>#10 0xffffffff8033aa41 in gre_sosend (top=0xfffffe80a5751400,
>so=0xfffffe810e4c2498)
>    at ../../../../net/if_gre.c:536
>#11 greintr (arg=0xfffffe810ae10008) at ../../../../net/if_gre.c:221
>#12 0xffffffff804a3be4 in softint_execute (l=<optimized out>,
>s=<optimized out>, si=<optimized out>)
>    at ../../../../kern/kern_softint.c:589
>#13 softint_dispatch (pinned=<optimized out>, s=4) at
>../../../../kern/kern_softint.c:871
>#14 0xffffffff8011402f in Xsoftintr ()
>

                        if (off + len >= MINCLSIZE) {
                                n = m_getcl(how, m->m_type, 0);
                        } else {
                                n = m_get(how, m->m_type);
                        }
                        if (n == NULL) {
                                goto out;
                        }       

That should be 'goto enobufs;' I think. But even worse,  the debug code
does not check for error. I will fix the second while I wait for confirmation
of the first :-)

christos