Making progress on if_bfe

[Mikko Työläjärvi]

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