From: Jouni Malinen <j@w1.fi>
To: Arik Nemtsov <arik@wizery.com>
Cc: linux-wireless@vger.kernel.org, Luciano Coelho <coelho@ti.com>,
Johannes Berg <johannes@sipsolutions.net>,
"John W. Linville" <linville@tuxdriver.com>
Subject: Re: [PATCH v2 0/6] Probe-resp offloading support
Date: Tue, 25 Jan 2011 11:41:30 +0200 [thread overview]
Message-ID: <20110125094130.GA25962@jm.kir.nu> (raw)
In-Reply-To: <AANLkTimYZN5438DOGNOE9Ltny5M-qg-K1ShFKToRMyuw@mail.gmail.com>
On Mon, Jan 24, 2011 at 11:16:43PM +0200, Arik Nemtsov wrote:
> The SSID-as-attr not used for the probe-resp template. It is used by
> FW to filter out which probe-reqs should be responded to when
> operating with a hidden SSID.
OK, that would be useful to mention in nl80211.h comments and
cfg80211.h, too, I'd guess.
> The wl12xx FW updates the timestamp and DA. I mentioned in the
> documentation of ieee80211_proberesp_get (and various other parts)
> that the DA should be set manually and that this should be used as a
> template (which also implies timestamp should be set). If some part is
> unclear can you point out a specific patch?
DA should be pretty obvious. Timestamp is something that I hope is clear
for whoever is implementing a driver, but it could be mentioned
somewhere.
> For WPS (without p2p), it appears the probe-resp doesn't depend on the
> probe-req (in hostapd code).
This is the case for WPS 1.0, but the protocol update in WSC 2.0 does
introduce a dependency.
> Perhaps we can start with these AP-mode only patches and expand the
> functionality when it is needed for p2p?
I'm fine with them if there is a capability flag that allows user space
to know that the driver does not use user space -based Probe Request
processing. With that, hostapd/wpa_supplicant can disable functionality
like multi-SSID support, WPS 2.0, P2P until we get more complete
capability information on what devices that process Probe Request frames
internally can do.
> Note that the current patch doesn't remove existing functionality - a
> driver/FW operating in p2p mode can simply choose not to use the
> template and just pass probe-requests up.
Until now, I've been assuming that nl80211-based interface means that
Probe Request frames are handled in user space. Since this set of
patches makes it very obvious that that will not be the case, I do want
to be able to figure that out in user space easily. All it really takes
is to add a capability flag indicating whether user space -based Probe
Request processing is used or not.
--
Jouni Malinen PGP id EFC895FA
next prev parent reply other threads:[~2011-01-25 9:41 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-23 21:02 [PATCH v2 0/6] Probe-resp offloading support Arik Nemtsov
2011-01-23 21:02 ` [PATCH v2 1/6] nl80211: allow passing SSID in nl80211_set_bss Arik Nemtsov
2011-01-23 21:02 ` [PATCH v2 2/6] nl80211: Pass probe response data to drivers Arik Nemtsov
2011-01-24 11:35 ` Johannes Berg
2011-01-24 20:47 ` Arik Nemtsov
2011-01-25 10:12 ` Johannes Berg
2011-01-26 6:00 ` Arik Nemtsov
2011-01-26 8:32 ` Johannes Berg
2011-01-26 21:51 ` Arik Nemtsov
2011-01-26 21:55 ` Johannes Berg
2011-01-27 13:29 ` Johannes Berg
2011-01-30 10:34 ` Arik Nemtsov
2011-01-30 10:44 ` Johannes Berg
2011-01-23 21:02 ` [PATCH v2 3/6] mac80211: add SSID for AP mode with change notification Arik Nemtsov
2011-01-23 21:02 ` [PATCH v2 4/6] mac80211: Save probe response data for BSS Arik Nemtsov
2011-01-24 11:38 ` Johannes Berg
2011-01-26 5:54 ` Arik Nemtsov
2011-01-23 21:02 ` [PATCH v2 5/6] wl12xx: AP mode - support hidden SSID Arik Nemtsov
2011-01-23 21:02 ` [PATCH v2 6/6] wl12xx: configure probe-resp template according to notification Arik Nemtsov
2011-01-24 9:44 ` [PATCH v2 0/6] Probe-resp offloading support Jouni Malinen
2011-01-24 21:16 ` Arik Nemtsov
2011-01-25 9:41 ` Jouni Malinen [this message]
2011-01-26 6:08 ` Arik Nemtsov
2011-01-26 6:24 ` Arik Nemtsov
2011-01-26 9:46 ` Jouni Malinen
2011-01-24 11:41 ` Johannes Berg
2011-01-24 21:21 ` Arik Nemtsov
2011-01-25 10:10 ` Johannes Berg
2011-01-25 10:42 ` Johannes Berg
2011-01-26 6:38 ` Arik Nemtsov
2011-01-26 6:35 ` Arik Nemtsov
2011-01-26 8:44 ` Johannes Berg
2011-01-26 14:04 ` Jouni Malinen
2011-01-30 11:24 ` Arik Nemtsov
2011-01-30 18:37 ` Jouni Malinen
2011-01-30 22:19 ` Arik Nemtsov
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=20110125094130.GA25962@jm.kir.nu \
--to=j@w1.fi \
--cc=arik@wizery.com \
--cc=coelho@ti.com \
--cc=johannes@sipsolutions.net \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.