Date: Sat, 10 Mar 2012 14:50:31 -0800 From: Juli Mallett <jmallett@FreeBSD.org> To: Aleksandr Rybalko <ray@ddteam.net> Cc: freebsd-mips@freebsd.org Subject: Re: [PATCH] Fix for using NFS root with if_arge Message-ID: <CACVs6=9ZcuzZqf_4-DHk5Bp5AQwFaFUm9tWR=O9v50oYjuXtYg@mail.gmail.com> In-Reply-To: <20120311002908.58ea1513.ray@ddteam.net> References: <CAD44qMVKyL-Fdrp-Tf3o9HXMH_UudCSfEOKnHz3nK%2BaqMwW6Pw@mail.gmail.com> <20120310151128.647d4a18.ray@ddteam.net> <CACVs6=-=EW6-7rqxH3yLKLSGZDrXFqbkbhSVLQ7eaA5vd5J_kg@mail.gmail.com> <20120311002908.58ea1513.ray@ddteam.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Mar 10, 2012 at 14:29, Aleksandr Rybalko <ray@ddteam.net> wrote: > On Sat, 10 Mar 2012 14:24:09 -0800 > Juli Mallett <jmallett@FreeBSD.org> wrote: >> This patch seems to cause interrupt storms on rb450g's arge1 >> interface, at least if that interface is wired to the switch. =C2=A0I wo= uld >> guess it has to do with the MAC clock? > > Yeah, but not really this patch. This patch just workaround one problem > which is unhide second. I'm still not sure what is real problem > source, but think it is because clock for MAC output, or not run, or > run on incorrect frequency. I guess I really don't understand the nature of the problem. It looks like this relates to moving the ring initialization out to attach-time =E2=80=94 why not just make the stop function do the right thing and free allocated resources, instead of doing the latter in detach? Is it simply the lack of resetting the tx_prod, tx_cons, tx_cnt and rx_cons fields in the init function if you move the ring initialization to attach? It looks to me like that could be the problem.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACVs6=9ZcuzZqf_4-DHk5Bp5AQwFaFUm9tWR=O9v50oYjuXtYg>