Date: 10 Aug 2002 22:21:09 -0400 From: Joe Marcus Clarke <marcus@marcuscom.com> To: Ian Dowse <iedowse@maths.tcd.ie> Cc: Mike Tancsa <mike@sentex.net>, stable@FreeBSD.ORG, luigi@FreeBSD.ORG Subject: Re: fxp problems with latest stable Re: panics after upgrading to -STABLE Aug 9, 2002 (still something up) Message-ID: <1029032469.61373.10.camel@shumai.marcuscom.com> In-Reply-To: <200208110304.aa47521@salmon.maths.tcd.ie> References: <200208110304.aa47521@salmon.maths.tcd.ie>
next in thread | previous in thread | raw e-mail | index | archive | help
--=-zdvq3NxOY3x4f+S4bYyV
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable
On Sat, 2002-08-10 at 22:04, Ian Dowse wrote:
> In message <5.1.0.14.0.20020810210400.0622d770@192.168.0.12>, Mike Tancsa=
write
> s:
> >I did some more tests with the new driver with 2 machines on a totally=20
> >different segment and the same sorts of packet loss on simple pings. If=
I=20
> >revert to the previous version, no packet loss at all.
>=20
> Hi Mike,
>=20
> thanks for tracking down the commit that caused this. I had a quick
> look at the changes in that revision, and although there were no
> obvious problems I could see, a few things stood out as possible
> candidates:
>=20
> The FXP_RFA_RNRMARK flag: Luigi, do you have hardware documentation
> for the Intel etherexpress chips that explicitly says that this bit
> is free for use by the device driver? If not, this would be my first
> guess at the problem. A patch that just #if 0's out this code
> (breaking the DEVICE_POLLING case) is below. It would be very useful
> to know if this makes a difference.
>=20
> Total packet size: the old code used rfa->actual_size & (MCLBYTES - 1)
> and the new version uses a mask of 0x3fff. I assume this is correct,
> but to test if it is the problem, change the 0x3fff to 0x1fff.
Actually, Ian, I noticed after your uipc_mbuf.c commit, my panics went
away. So far, I've upgraded two fxp-based machines to the latest
-stable, and neither of them have panicked. The same could not be said
of -stable from last night.
Joe
>=20
> Ian
>=20
> Index: if_fxp.c
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> RCS file: /home/iedowse/CVS/src/sys/dev/fxp/if_fxp.c,v
> retrieving revision 1.110.2.24
> diff -u -r1.110.2.24 if_fxp.c
> --- if_fxp.c 9 Aug 2002 02:04:20 -0000 1.110.2.24
> +++ if_fxp.c 11 Aug 2002 01:46:23 -0000
> @@ -1292,7 +1292,9 @@
> * info will be used in the subsequent polling cycle.
> */
> =20
> +#if 0
> #define FXP_RFA_RNRMARK 0x4000 /* used to mark a pending RNR intr */
> +#endif
> =20
> for (;;) {
> m =3D sc->rfa_headm;
> @@ -1307,8 +1309,10 @@
> if ( (rfa->rfa_status & FXP_RFA_STATUS_C) =3D=3D 0)
> break;
> =20
> +#if 0
> if (rfa->rfa_status & FXP_RFA_RNRMARK)
> rnr =3D 1;
> +#endif
> /*
> * Remove first packet from the chain.
> */
> @@ -1342,15 +1346,19 @@
> }
> }
> if (rnr) {
> +#if 0
> if (rfa->rfa_status & FXP_RFA_STATUS_C)
> rfa->rfa_status |=3D FXP_RFA_RNRMARK;
> else {
> +#endif
> fxp_scb_wait(sc);
> CSR_WRITE_4(sc, FXP_CSR_SCB_GENERAL,
> vtophys(sc->rfa_headm->m_ext.ext_buf) +
> RFA_ALIGNMENT_FUDGE);
> fxp_scb_cmd(sc, FXP_SCB_COMMAND_RU_START);
> +#if 0
> }
> +#endif
> }
> }
> =20
>=20
> To Unsubscribe: send mail to majordomo@FreeBSD.org
> with "unsubscribe freebsd-stable" in the body of the message
>=20
--=-zdvq3NxOY3x4f+S4bYyV
Content-Type: application/pgp-signature; name=signature.asc
Content-Description: This is a digitally signed message part
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (FreeBSD)
iD8DBQA9VcoVb2iPiv4Uz4cRAibYAJ44gvBZmyFX6qZH4wQuXctCXwXX9ACfaebp
GmVJuSbVdcvd1stJhNnq7kc=
=oKe8
-----END PGP SIGNATURE-----
--=-zdvq3NxOY3x4f+S4bYyV--
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-stable" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1029032469.61373.10.camel>