Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 25 May 2017 22:57:10 +0200
From:      Vincenzo Maffione <v.maffione@gmail.com>
To:        Harry Schmalzbauer <freebsd@omnilan.de>
Cc:        FreeBSD Net <freebsd-net@freebsd.org>
Subject:   Re: [panic] netmap(4) and if_lagg(4)
Message-ID:  <CA%2B_eA9jg-xXJpeyQkcFzA%2B-y=qbwROww2FRWzCN=dkmfkaQivQ@mail.gmail.com>
In-Reply-To: <592742A8.4010207@omnilan.de>
References:  <58CBCD7A.8060301@omnilan.de> <CA%2B_eA9iCT7evWUcZMA_ViKfrZnSHp3OpBTS5c4iJ9=ZjO-Pfgw@mail.gmail.com> <58CC23F5.7060507@omnilan.de> <CA%2B_eA9iajZOUFsnWKdodN7zMvst8wn0xViM4xxEx%2B41jw_0B3g@mail.gmail.com> <58CFA394.8070901@omnilan.de> <CA%2B_eA9jf_VjcYUZynu5CXx-ps5z7PSxOky=QLpR-miqtxmMU5w@mail.gmail.com> <5926EE96.1010000@omnilan.de> <CA%2B_eA9gcMD1QyG35p5UY8kz6n1kF%2B56Arz3q1T8HVBfMv3XN0w@mail.gmail.com> <5926F9F9.4040706@omnilan.de> <CA%2B_eA9hripEStmfJhj_Q93OF5jFf2nfQXeuA2DnuSjBmJi6j4g@mail.gmail.com> <592701D6.7030301@omnilan.de> <592742A8.4010207@omnilan.de>

next in thread | previous in thread | raw e-mail | index | archive | help
No, the thing is that I misinterpreted your stack trace. The patch is ok
for a different bug. It seems that the problem are vlans more than lagg.
Which interface did you put in netmap mode, em or em.345?

Il 25 mag 2017 10:46 PM, "Harry Schmalzbauer" <freebsd@omnilan.de> ha
scritto:

 Bez=C3=BCglich Harry Schmalzbauer's Nachricht vom 25.05.2017 18:09 (localt=
ime):
> Bez=C3=BCglich Vincenzo Maffione's Nachricht vom 25.05.2017 17:56 (localt=
ime):
>> I see the bug is in FreeBSD 11. I attached the simple patch to fix it.
>> Can someone commit the patch to 11/stable?
>>
>> Harry: You should be able to workaround the bug by setting
>>
>> # sysctl dev.netmap.generic_rings=3D1
> I'll recompile with your patch, thanks a lot!

Hi, unfortunately I can't confirm it to be fixed.

The kgdb output is exactly the same:

(kgdb) list *0xffffffff80426714
0xffffffff80426714 is in generic_rx_handler
(/usr/local/share/deploy-tools/RELENG_11/src/sys/dev/netmap/
netmap_generic.c:628).
623             struct netmap_adapter *na =3D NA(ifp);
624             struct netmap_generic_adapter *gna =3D (struct
netmap_generic_adapter *)na;
625             u_int work_done;
626             u_int rr =3D MBUF_RXQ(m); // receive ring number
627
628             if (rr >=3D na->num_rx_rings) {
629                     rr =3D rr % na->num_rx_rings; // XXX expensive...
630             }
631
632             /* limit the size of the queue */
Current language:  auto; currently minimal

Will double-check tomorrow that the binary really includes your patch.
Ofcourse I checked briefly and revision and buildtime check affirms it.

best,

-harry



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CA%2B_eA9jg-xXJpeyQkcFzA%2B-y=qbwROww2FRWzCN=dkmfkaQivQ>