Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 8 Sep 2015 19:13:14 -0400
From:      Jung-uk Kim <jkim@FreeBSD.org>
To:        Adrian Chadd <adrian.chadd@gmail.com>
Cc:        "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r287578 - in head/usr.sbin/wpa: hostapd wpa_supplicant
Message-ID:  <55EF6B8A.2050109@FreeBSD.org>
In-Reply-To: <CAJ-VmomDo7j_gy9NWiW-qmY9xL=k1GSU5wmnkjJ=XP_8k2-=sg@mail.gmail.com>
References:  <201509082104.t88L4XaR095063@repo.freebsd.org> <CAJ-VmomDo7j_gy9NWiW-qmY9xL=k1GSU5wmnkjJ=XP_8k2-=sg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

On 09/08/2015 18:44, Adrian Chadd wrote:
> Wait a sec, did anyone review and test that this works out
> correctly?

I tested hostapd(8) with ath(4) myself and it is working flawlessly
for me.  I didn't get any review because I thought it wouldn't change
any default behavior.

> We already have working 802.11n; it's done inside net80211.

Yes, I know.  Basically, this commit let me set 802.11n via
hostapd.conf.  Previously, I had to manually set it via ifconfig(8).

> It also mostly adds 20/40 coexistence and channel width changing
> on the fly, which is absolutely going to break absolutely
> everything as we don't /have/ stable support for that in any
> driver.

Hmm...  I thought it has very low risk because user has to enable
802.11n for the driver (e.g., ATH_ENABLE_11N) and set it via
configuration (e.g., setting 'ieee80211n=1' in /etc/hostapd.conf).

> Not only that, but mwl firmware for freebsd doesn't even fully
> work with 802.11n out of the box anyway, it has bugs with dealing
> with aggregation. sigh.

Why don't we disable 802.11n support for mwl(4) by default and let the
user decide, just like we do for ath, then?

> So - please back this out and let's get this more thoroughly
> tested first. I don't want this update causing unpredictable
> behaviour.

Okay, I'll back it out.

Jung-uk Kim

> Thanks,
> 
> 
> 
> 
> -adrian
> 
> On 8 September 2015 at 14:04, Jung-uk Kim <jkim@freebsd.org>
> wrote:
>> Author: jkim Date: Tue Sep  8 21:04:33 2015 New Revision: 287578 
>> URL: https://svnweb.freebsd.org/changeset/base/287578
>> 
>> Log: Enable 802.11n support.
>> 
>> PR:           164102
>> 
>> Modified: head/usr.sbin/wpa/hostapd/Makefile 
>> head/usr.sbin/wpa/wpa_supplicant/Makefile
>> 
>> Modified: head/usr.sbin/wpa/hostapd/Makefile 
>> =====================================================================
=========
>>
>> 
- --- head/usr.sbin/wpa/hostapd/Makefile  Tue Sep  8 20:41:47 2015
(r287577)
>> +++ head/usr.sbin/wpa/hostapd/Makefile  Tue Sep  8 21:04:33 2015
>> (r287578) @@ -15,8 +15,8 @@ SRCS= accounting.c aes-omac1.c
>> ap_config eap_register.c eap_server.c eap_server_methods.c
>> eap_user_db.c \ eapol_auth_dump.c eapol_auth_sm.c eloop.c gas.c
>> gas_serv.c hostapd.c \ hs20.c http_client.c http_server.c
>> httpread.c \ -       hw_features_common.c ieee802_11_auth.c \ -
>> ieee802_11_common.c ieee802_11_shared.c ieee802_1x.c ip_addr.c \ 
>> +       hw_features_common.c ieee802_11_auth.c
>> ieee802_11_common.c \ +       ieee802_11_ht.c ieee802_11_shared.c
>> ieee802_1x.c ip_addr.c \ main.c ms_funcs.c os_unix.c
>> peerkey_auth.c pmksa_cache_auth.c \ preauth_auth.c radius.c
>> radius_client.c radius_das.c sta_info.c \ tkip_countermeasures.c
>> upnp_xml.c utils.c uuid.c vlan_init.c \ @@ -37,6 +37,7 @@ FILES=
>> hostapd.conf hostapd.eap_user hos CFLAGS+=-DCONFIG_DRIVER_BSD \ 
>> -DCONFIG_DRIVER_RADIUS_ACL \ -DCONFIG_HS20 \ +
>> -DCONFIG_IEEE80211N \ -DCONFIG_INTERWORKING \ -DCONFIG_PEERKEY \ 
>> -DCONFIG_RSN_PREAUTH \
>> 
>> Modified: head/usr.sbin/wpa/wpa_supplicant/Makefile 
>> =====================================================================
=========
>>
>> 
- --- head/usr.sbin/wpa/wpa_supplicant/Makefile   Tue Sep  8 20:41:47 2015
       (r287577)
>> +++ head/usr.sbin/wpa/wpa_supplicant/Makefile   Tue Sep  8
>> 21:04:33 2015        (r287578) @@ -40,6 +40,7 @@
>> CFLAGS+=-DCONFIG_BACKEND_FILE \ -DCONFIG_DRIVER_WIRED \ 
>> -DCONFIG_GAS \ -DCONFIG_HS20 \ +       -DCONFIG_IEEE80211N \ 
>> -DCONFIG_IEEE80211R \ -DCONFIG_INTERWORKING \ -DCONFIG_PEERKEY \
>> 
> .
> 

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQEcBAEBCAAGBQJV72uCAAoJEHyflib82/FGxgEH/jDCH6eSN5Go9tTEXgfofLxX
12DDoQjxMh2G5SPzkanYhjjljN2dYPt0+vaKUYELIdIcXblbcsXda81y2NjGX7lt
tu+L73cIfjmSvEo+vs/FJ9ERTk8DBTLSq2bU2I43gurhusqkAGnEeLGfrGchD7Dw
RDZnU99nLDYemr6NNzA8uIvLReIw8FTyg6Nsj6su9pvNvYh2x5YsNnXOBeGIi9Vj
2wDflU8tJ8WHYyjfHoSG790FiufEsW3A4O++YihzYHPODHsVN7qFCjgpQggti6Wy
fmYIOd9uEI/M0HT1CQ6sRgDxIqZr/ttG/1/iOQ8q8RhWrEBqhcBfQyYc1T+YLUc=
=UCLu
-----END PGP SIGNATURE-----



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?55EF6B8A.2050109>