Date: Sat, 27 May 2006 18:00:42 -0700 (PDT) From: =?ISO-8859-1?Q?Mikko_Ty=F6l=E4j=E4rvi?= <mbsd@pacbell.net> To: Mike Silbersack <silby@silby.com> Cc: Makoto Matsushita <matusita@jp.FreeBSD.org>, mobile@freebsd.org, Ben Kaduk <minimarmot@gmail.com> Subject: Re: Making progress on if_bfe Message-ID: <20060527174928.V18218@antec.home> In-Reply-To: <20060527135922.C7405@odysseus.silby.com> References: <20060527135922.C7405@odysseus.silby.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 27 May 2006, Mike Silbersack wrote:
>
> I started working on if_bfe again, and I am able to reproduce the problems
> that people with 512MB of ram started to report after I fixed the driver for
> people with > 1G of ram. I have some ideas on what is going on, but no fix
> yet.
>
> For anyone who has time (no matter what size ram you have), I'd like you to
> try the attached patch. The patch is very simple - instead of interrupt
> storming, it tells you why the bfe card is unhappy, and downs the interface
> instead of continually attempting to restart it. The interrupt storm was a
> side-effect of the card complaining, and the driver continually trying to
> restart the card.
>
> So, apply the patch, then try with 128 as the values for BFE_TX_LIST_CNT and
> BFE_RX_LIST_CNT, see what happens, then try with 511 and see what happens.
> Then e-mail me the results. This will confirm that you are seeing the same
> errors from the card that I am.
>
> Oh, and once you're done with that, go back to using the values for
> BFE_TX_LIST_CNT and BFE_RX_LIST_CNT that work for you. :)
As I don't have the interrupt storm problem (I'm one of the "people
with > 1G of ram" :), the patch produces no output for me. The
interface still silently hangs with *_CNT set to 511 and still works
with *_CNT at 128 (seems to work at 256 as well, btw).
While on the subject of if_bfe; does your activity light work? Mine
just stays on solid unless I apply a patch like this:
--- if_bfe.c.orig Sat May 27 17:33:10 2006
+++ if_bfe.c Sat May 27 17:46:18 2006
@@ -1460,10 +1460,12 @@
miisc = LIST_NEXT(miisc, mii_list))
mii_phy_reset(miisc);
}
mii_mediachg(mii);
+ bfe_setupphy(sc);
+
return (0);
}
/*
* Report current media status.
I guess the settings are clobbered by mii_phy_reset(), and have to be
re-applied. The real fix might be to configure the LED settings in
bmtphy, but this works for me.
$.02,
/Mikko
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060527174928.V18218>