Date: Sun, 24 Jan 2016 14:01:05 -0800 From: Adrian Chadd <adrian@freebsd.org> To: Andriy Voskoboinyk <s3erios@gmail.com> Cc: "freebsd-wireless@freebsd.org" <freebsd-wireless@freebsd.org> Subject: Re: net80211: why we ca'nt use the net80211 taskqueue for everything Message-ID: <CAJ-VmomkvSWpBCY24EpgR6xXE8=OtMCYSa58qc=XHb_LCeBzLQ@mail.gmail.com> In-Reply-To: <op.ybrih2vkiew4ia@localhost> References: <CAJ-VmoneUBz4Vt3hFj8S4G_o8ptd3Z-NL5%2B6HuG33C3C_x-2jQ@mail.gmail.com> <op.ybrih2vkiew4ia@localhost>
next in thread | previous in thread | raw e-mail | index | archive | help
On 24 January 2016 at 13:40, Andriy Voskoboinyk <s3erios@gmail.com> wrote: > Sun, 24 Jan 2016 22:22:44 +0200 =D0=B1=D1=83=D0=BB=D0=BE =D0=BD=D0=B0=D0= =BF=D0=B8=D1=81=D0=B0=D0=BD=D0=BE Adrian Chadd > <adrian@freebsd.org>: > > Hi, > > is it actual for wpi(4) / iwn(4) ? If you defer things like tx, calibration, etc then yeah - you'll see things like software generated probe request frames and such be deferred. (I tried this with ath(4) about 5 years ago - hit the same issue. So I've put it off until the scan code is "fixed".) -adrian > > >> hi, >> >> andriy has a few reviews out that tidy up some things, which I'd reply >> to, but .. reviews is offline. So, here's the 30 second version: >> >> * the net80211 taskqueue runs the software scan engine, and the >> software scan engine currently sleeps whilst it's running. >> >> This means that if you put newstate, deferred transmit, etc into the >> net80211 taskqueue, then it just won't run during scan. >> >> The net80211 software scan thing should be modified to not sleep >> whilst it's waiting for scan results and instead just kick off another >> timer event to finish that part of the loop. Then yes, we can just >> migrate * to the net80211 task queue and use it for all serialisation >> of a wifi driver. >> >> (And yes, I'd like to see that done ASAP..) >> >> Thanks, >> >> >> -adrian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmomkvSWpBCY24EpgR6xXE8=OtMCYSa58qc=XHb_LCeBzLQ>