SunFire X2200 ilo's bge1 DOWN/UP

[John Baldwin]

On Thursday, May 30, 2013 2:44:35 am Daniel Braniss wrote:
> > --/04w6evG8XlLl3ft
> > Content-Type: text/x-diff; charset=us-ascii
> > Content-Disposition: attachment; filename="bge.media_sts.diff"
> > 
> > Index: sys/dev/bge/if_bge.c
> > ===================================================================
> > --- sys/dev/bge/if_bge.c	(revision 251021)
> > +++ sys/dev/bge/if_bge.c	(working copy)
> > @@ -5583,6 +5583,10 @@ bge_ifmedia_sts(struct ifnet *ifp, struct ifmediar
> >  
> >  	BGE_LOCK(sc);
> >  
> > +	if ((ifp->if_flags & IFF_UP) == 0) {
> > +		BGE_UNLOCK(sc);
> > +		return;
> > +	}
> >  	if (sc->bge_flags & BGE_FLAG_TBI) {
> >  		ifmr->ifm_status = IFM_AVALID;
> >  		ifmr->ifm_active = IFM_ETHER;
> > 
> > --/04w6evG8XlLl3ft--
> after 18hs, the logs are empty!
> it seems the patch fixes the problem.
> 
> now maybe it's time to hunt for who is randomly calling for bge_ifmedia_sts
> ...

It could be any number of daemons that query interface state such as an
SNMP server, ladvd, etc.

If you wanted help you could modify the patch so that it does something like 
this:

	if (/* test for IFF_UP */) {
		BGE_UNLOCK(sc);
		if_printf(ifp, "state queried on down interface by pid %d (%s)",
		    curthread->td_proc->p_pid, curthread->td_proc->p_comm);
		return;
	}

-- 
John Baldwin