From: "Luis R. Rodriguez" <lrodriguez@atheros.com>
To: "John W. Linville" <linville@tuxdriver.com>
Cc: linux-wireless@vger.kernel.org
Subject: Re: [PATCH v3] mac80211: redefine usage of the mac80211 workqueue
Date: Wed, 29 Jul 2009 14:13:58 -0700 [thread overview]
Message-ID: <43e72e890907291413u7dc005f7laab0a6f7a3d6d831@mail.gmail.com> (raw)
In-Reply-To: <43e72e890907291345u5de674fem7b0769f52a7251c5@mail.gmail.com>
On Wed, Jul 29, 2009 at 1:45 PM, Luis R.
Rodriguez<lrodriguez@atheros.com> wrote:
> On Wed, Jul 29, 2009 at 1:00 PM, John W. Linville<linville@tuxdriver.com> wrote:
>> On Mon, Jul 27, 2009 at 03:21:31PM -0700, Luis R. Rodriguez wrote:
>>> The mac80211 workqueue exists to enable mac80211 and drivers
>>> to queue their own work on a single threaded workqueue. mac80211
>>> takes care to flush the workqueue during suspend but we never
>>> really had requirements on drivers for how they should use
>>> the workqueue in consideration for suspend.
>>>
>>> We extend mac80211 to document how the mac80211 workqueue should
>>> be used, how it should not be used and finally move raw access to
>>> the workqueue to mac80211 only. Drivers and mac80211 use helpers
>>> to queue work onto the mac80211 workqueue:
>>>
>>> * ieee80211_queue_work()
>>> * ieee80211_queue_delayed_work()
>>>
>>> These helpers will now warn if mac80211 already completed its
>>> suspend cycle and someone is trying to queue work. mac80211
>>> flushes the mac80211 workqueue prior to suspend a few times,
>>> but we haven't taken the care to ensure drivers won't add more
>>> work after suspend. To help with this we add a warning when
>>> someone tries to add work and mac80211 already completed the
>>> suspend cycle.
>>>
>>> Drivers should ensure they cancel any work or delayed work
>>> in the mac80211 stop() callback.
>>
>> How are they supposed to do that after you've removed workqueue
>> from the ieee80211_hw structure?
>
> They do not refer to the mac80211 workqueue for that, they can cancel
> work using cancel_work_sync(their_foo_work_struct).
>
>> This breaks the compilation of
>> at76c50x-usb.c...
>
> Strange, I thought I tested that, will retest,
Oh ok, sorry about that this requires a patch which I thought I had
sent but it seems I didn't:
at76c50x-usb: remove unneeded flush_workqueue() at usb disconnect
Will send.
Luis
prev parent reply other threads:[~2009-07-29 21:47 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-27 22:21 [PATCH v3] mac80211: redefine usage of the mac80211 workqueue Luis R. Rodriguez
2009-07-29 20:00 ` John W. Linville
2009-07-29 20:45 ` Luis R. Rodriguez
2009-07-29 21:13 ` Luis R. Rodriguez [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=43e72e890907291413u7dc005f7laab0a6f7a3d6d831@mail.gmail.com \
--to=lrodriguez@atheros.com \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).