linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jouni Malinen <j@w1.fi>
To: Arend van Spriel <arend@broadcom.com>
Cc: "hostap@lists.shmoo.com" <hostap@lists.shmoo.com>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	Kalle Valo <kvalo@codeaurora.org>,
	"Gautam (Gautam Kumar) Shukla" <gautams@broadcom.com>
Subject: Re: hostapd and 11h support
Date: Thu, 25 Dec 2014 11:10:59 +0200	[thread overview]
Message-ID: <20141225091059.GA3436@w1.fi> (raw)
In-Reply-To: <549B24A9.4080001@broadcom.com>

On Wed, Dec 24, 2014 at 09:40:09PM +0100, Arend van Spriel wrote:
> On 12/24/14 20:04, Jouni Malinen wrote:
> >I see nothing odd in other drivers using the already defined QCA vendor
> >specific nl80211 command.
> 
> Maybe not odd but counter-intuitive to some people.

Maybe so, but certainly not on purpose. It has to be a specific OUI and
as such, assigned to someone.. This one happens to be coordinated in a
way that all (non-MAC address) assignments get done by upstream
hostap.git commits. In other words, there is no intent of trying to
limit it to a single vendor, but to have something that is easily usable
for hostapd/wpa_supplicant extensions in cases where upstream nl80211
extensions for needed functionality are not acceptable for some reason
at the time.

> Anyway, I was
> looking at the code in driver_nl80211_capa.c and stumbled upon
> WPA_DRIVER_FLAGS_KEY_MGMT_OFFLOAD. I introduced entensible feature
> flags upstream just yesterday to accomodate this type of offload.
> Just curious but are those QCA vendor specific nl80211 commands used
> by the upstream QCA drivers or is this for supporting QCA
> proprietary drivers. I guess the latter as that would be the only
> reason I can think of to have vendor specific commands.

This is currently used by an open source, but not upstream, driver.
There was actually an attempt to get this done in standard nl80211
commands/events, but the response was not exactly encouraging with no
existing upstream driver to show as a user for the new functionality.
I'd still like to get this done cleanly in nl80211 at some point in
time, though, since extensions to Connect command/event are not possible
using the vendor specific commands/events, i.e., you end up having to
fully override these commands/events which gets ugly pretty quickly.

-- 
Jouni Malinen                                            PGP id EFC895FA

      reply	other threads:[~2014-12-25  9:11 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-24  9:32 hostapd and 11h support Arend van Spriel
2014-12-24 17:32 ` Jouni Malinen
2014-12-24 18:29   ` Arend van Spriel
2014-12-24 19:04     ` Jouni Malinen
2014-12-24 20:40       ` Arend van Spriel
2014-12-25  9:10         ` Jouni Malinen [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=20141225091059.GA3436@w1.fi \
    --to=j@w1.fi \
    --cc=arend@broadcom.com \
    --cc=gautams@broadcom.com \
    --cc=hostap@lists.shmoo.com \
    --cc=kvalo@codeaurora.org \
    --cc=linux-wireless@vger.kernel.org \
    /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).