Date: Tue, 22 Nov 2011 06:56:10 +0800 From: Adrian Chadd <adrian@freebsd.org> To: Alexander Best <arundel@freebsd.org> Cc: freebsd-wireless@freebsd.org Subject: Re: issues with the recent ath commits (r227740, r227651, ...) Message-ID: <CAJ-Vmo=XbxbPwwQse73w19hyy-gEXOjogWZrmoPuzm85hY=hYw@mail.gmail.com> In-Reply-To: <20111121225343.GA19155@freebsd.org> References: <20111121131730.GA3599@freebsd.org> <alpine.BSF.2.00.1111210719430.30516@wonkity.com> <20111121152054.GA22220@freebsd.org> <CAJ-VmomzY6Arh_=nPihdH=RhNyQXV7jgdvV7oyGKfqzb1BQ6sw@mail.gmail.com> <alpine.BSF.2.00.1111211433100.33629@wonkity.com> <CAJ-VmomO-7-TsT1b_=2FBu5rnYf7Z5UhQyuRF95iupiL2xGTnA@mail.gmail.com> <20111121221224.GA11508@freebsd.org> <20111121225343.GA19155@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 22 November 2011 06:53, Alexander Best <arundel@freebsd.org> wrote: > ok. after > 45 minutes uptime i can confirm that the patch fixes the issues i > experienced beforehand. unless i didn't get a boot-up-panic, my system locked > up after < 5 minutes. Sweet, I'll commit that to -HEAD now. > however using your patch i noticed the following console output: > > ath0: ath_start: sc_inreset_cnt > 0; bailing > > ... only once, however. i don't believe i've seen this one ever beforehand. That's intentional. That means that a frame was queued for TX whilst the hardware was busy being reset. It may be because I'm now processing the frames on an interface reset, rather than just deleting them. But on SMP machines it's quite plausible that a reset on one thread would run in parallel with if_start on another thread, causing these to interfere. My eventual aim is to have TX/RX properly suspended before a reset occurs, so these messages will go away (and their appearance will only be when there's a real problem.) But that's going to take time. Thanks for getting back to me about this issue! Adrian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-Vmo=XbxbPwwQse73w19hyy-gEXOjogWZrmoPuzm85hY=hYw>