Re: Making progress on if_bfe

Mikko Työläjärvi <[email protected]>
Newsgroups gmane.os.freebsd.devel.mobile
Message-ID <[email protected]>
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
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-mobile
To unsubscribe, send any mail to "[email protected]"
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.