Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 19 Aug 2001 10:39:55 -0700
From:      Sam Habash <the@llama.com>
To:        Jonathan Chen <jon@FreeBSD.ORG>
Cc:        freebsd-gnats-submit@FreeBSD.ORG, mobile@FreeBSD.ORG, hackers@FreeBSD.ORG
Subject:   Re: kern/24854: NEWCARD support for aironet driver an(4)
Message-ID:  <20010819103955.A35427@llama.com>
In-Reply-To: <20010819091452.A86632@enterprise.spock.org>; from jon@FreeBSD.ORG on Sun, Aug 19, 2001 at 09:14:52AM -0400
References:  <20010817011103.A25841@llama.com> <20010818110728.A20719@enterprise.spock.org> <20010818092654.C28694@llama.com> <20010818124503.A26350@enterprise.spock.org> <20010818212011.A33879@llama.com> <20010819091452.A86632@enterprise.spock.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Aug 19, 2001 at 09:14:52AM -0400, Jonathan Chen wrote:
> 
> [summary of past events for the benefit of gnats]
>  - original PR filed for Cisco aironet not working under newcard
>  - My LMC342 works just fine, and I though this might be a bug in newcard 
>    (as opposed to the an driver as suggested in the pr), suggested Sam to 
>    try the updated newcard (http://people.freebsd.org/~jon/newcard.diff.3)
>  - this patch makes an work under newcard without the proposed changes in 
>    the pr.
> 
> > I believe I already tried just uncommenting 'optional ata pccard'
> > with the -current source, without any luck. 
> 
> Sam, I'm not sure what you mean here.

What I mean here is that an unpatched -current (as of mid-August)
was not resulting in detection of my LMC352.  In the current sources, 
the following in src/sys/conf/files has been commented out:

#dev/an/if_an_pccard.c   optional an pccard

Commenting this out and recompiling did NOT result in a successful
attach, at least when I tested this in early August.  

I'm testing again by backing out of your patch and recompiling, and 
it turns out that it -does- work now, most likely this was operator 
error in not uncommenting the the conf/files entry, since I don't 
see any likely commits that would have changed anything...sigh,
sorry for the waste of time.

http://www.freebsd.org/cgi/cvsweb.cgi/src/sys/conf/files.diff?r1=1.483&r2=1.484&f=h

documents when the line was commented out...it's been that way for
a while...

> > The first time I tried it (not in X) removal and reinsertion of the 
> > Aironet card was fine.
> > 
> > The second time (while in X), removal was ok, but card reinsertion caused 
> > a reboot (no panic)...
> > 
> > Let me know what you want me to do exactly, in terms of what kind of 
> > debugging you need...once I hear back I'll build a debug kernel.
> 
> Are you sure there is no panic while in X?  Your X server might have 
> futzed with the video card so you might not actually see the panic.  If you 
> can try to reproduce this in text mode, then the panic message as well as a 
> traceback would be a very good start.

-nod-

> But before you do that, I believe I may have found a fix for a possible 
> panic situation.  This panic occurs in witness_destroy as a supervisor read 
> page not present error (I presume while trying to dereference lock).  This 
> panic can be easily reproduced by running dhclient an0, removing and 
> reinserting the card, then killing dhclient.  The fix appears to be 
> shockingly simple:

I'll see if I can reproduce the problem in CLI mode with your patch
to if_an_pccard.c

> 
> diff -u -r1.8 if_an_pccard.c
> --- sys/dev/an/if_an_pccard.c	2001/05/26 09:26:58	1.8
> +++ sys/dev/an/if_an_pccard.c	2001/08/19 12:57:14
> @@ -118,6 +118,7 @@
>  	sc->an_gone = 1;
>  	bus_teardown_intr(dev, sc->irq_res, sc->irq_handle);
>  	an_release_resources(dev);
> +	mtx_destroy(&sc->an_mtx);
>  	return (0);
>  }

Take care,

--Sam

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-mobile" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010819103955.A35427>