brcmfmac: support 4-way handshake offloading for 802.1X
authorArend van Spriel <arend.vanspriel@broadcom.com>
Fri, 9 Jun 2017 12:08:48 +0000 (13:08 +0100)
committerKalle Valo <kvalo@codeaurora.org>
Fri, 30 Jun 2017 06:38:22 +0000 (09:38 +0300)
Adding callbacks for PMK provisioning. If firmware supports offloading
it is indicated to user-space that 802.1X offload is supported.

Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.h

index 36945e0c8923275f6a91ac003ee4a98ab376be0d..0287db38fb9f4004e96b6dc35611a9307c4b011f 100644 (file)
@@ -1717,6 +1717,7 @@ static s32
 brcmf_set_key_mgmt(struct net_device *ndev, struct cfg80211_connect_params *sme)
 {
        struct brcmf_if *ifp = netdev_priv(ndev);
+       struct brcmf_cfg80211_profile *profile = &ifp->vif->profile;
        s32 val;
        s32 err;
        const struct brcmf_tlv *rsn_ie;
@@ -1727,6 +1728,8 @@ brcmf_set_key_mgmt(struct net_device *ndev, struct cfg80211_connect_params *sme)
        u32 mfp;
        u16 count;
 
+       profile->use_fwsup = BRCMF_PROFILE_FWSUP_NONE;
+
        if (!sme->crypto.n_akm_suites)
                return 0;
 
@@ -1739,6 +1742,8 @@ brcmf_set_key_mgmt(struct net_device *ndev, struct cfg80211_connect_params *sme)
                switch (sme->crypto.akm_suites[0]) {
                case WLAN_AKM_SUITE_8021X:
                        val = WPA_AUTH_UNSPECIFIED;
+                       if (sme->want_1x)
+                               profile->use_fwsup = BRCMF_PROFILE_FWSUP_1X;
                        break;
                case WLAN_AKM_SUITE_PSK:
                        val = WPA_AUTH_PSK;
@@ -1752,9 +1757,13 @@ brcmf_set_key_mgmt(struct net_device *ndev, struct cfg80211_connect_params *sme)
                switch (sme->crypto.akm_suites[0]) {
                case WLAN_AKM_SUITE_8021X:
                        val = WPA2_AUTH_UNSPECIFIED;
+                       if (sme->want_1x)
+                               profile->use_fwsup = BRCMF_PROFILE_FWSUP_1X;
                        break;
                case WLAN_AKM_SUITE_8021X_SHA256:
                        val = WPA2_AUTH_1X_SHA256;
+                       if (sme->want_1x)
+                               profile->use_fwsup = BRCMF_PROFILE_FWSUP_1X;
                        break;
                case WLAN_AKM_SUITE_PSK_SHA256:
                        val = WPA2_AUTH_PSK_SHA256;
@@ -1769,6 +1778,9 @@ brcmf_set_key_mgmt(struct net_device *ndev, struct cfg80211_connect_params *sme)
                }
        }
 
+       if (profile->use_fwsup == BRCMF_PROFILE_FWSUP_1X)
+               brcmf_dbg(INFO, "using 1X offload\n");
+
        if (!brcmf_feat_is_enabled(ifp, BRCMF_FEAT_MFP))
                goto skip_mfp_config;
        /* The MFP mode (1 or 2) needs to be determined, parse IEs. The
@@ -1941,6 +1953,7 @@ brcmf_cfg80211_connect(struct wiphy *wiphy, struct net_device *ndev,
 {
        struct brcmf_cfg80211_info *cfg = wiphy_to_cfg(wiphy);
        struct brcmf_if *ifp = netdev_priv(ndev);
+       struct brcmf_cfg80211_profile *profile = &ifp->vif->profile;
        struct ieee80211_channel *chan = sme->channel;
        struct brcmf_join_params join_params;
        size_t join_params_size;
@@ -2038,16 +2051,24 @@ brcmf_cfg80211_connect(struct wiphy *wiphy, struct net_device *ndev,
        }
 
        if (sme->crypto.psk) {
+               if (WARN_ON(profile->use_fwsup != BRCMF_PROFILE_FWSUP_NONE)) {
+                       err = -EINVAL;
+                       goto done;
+               }
                brcmf_dbg(INFO, "using PSK offload\n");
+               profile->use_fwsup = BRCMF_PROFILE_FWSUP_PSK;
+       }
 
+       if (profile->use_fwsup != BRCMF_PROFILE_FWSUP_NONE) {
                /* enable firmware supplicant for this interface */
                err = brcmf_fil_iovar_int_set(ifp, "sup_wpa", 1);
                if (err < 0) {
                        brcmf_err("failed to enable fw supplicant\n");
                        goto done;
                }
-               ifp->vif->profile.use_fwsup = BRCMF_PROFILE_FWSUP_PSK;
+       }
 
+       if (profile->use_fwsup == BRCMF_PROFILE_FWSUP_PSK) {
                err = brcmf_set_pmk(ifp, sme->crypto.psk,
                                    BRCMF_WSEC_MAX_PSK_LEN);
                if (err)
@@ -5193,6 +5214,34 @@ brcmf_cfg80211_set_rekey_data(struct wiphy *wiphy, struct net_device *ndev,
 }
 #endif
 
+static int brcmf_cfg80211_set_pmk(struct wiphy *wiphy, struct net_device *dev,
+                                 const struct cfg80211_pmk_conf *conf)
+{
+       struct brcmf_if *ifp;
+
+       brcmf_dbg(TRACE, "enter\n");
+
+       /* expect using firmware supplicant for 1X */
+       ifp = netdev_priv(dev);
+       if (WARN_ON(ifp->vif->profile.use_fwsup != BRCMF_PROFILE_FWSUP_1X))
+               return -EINVAL;
+
+       return brcmf_set_pmk(ifp, conf->pmk, conf->pmk_len);
+}
+
+static int brcmf_cfg80211_del_pmk(struct wiphy *wiphy, struct net_device *dev,
+                                 const u8 *aa)
+{
+       struct brcmf_if *ifp;
+
+       brcmf_dbg(TRACE, "enter\n");
+       ifp = netdev_priv(dev);
+       if (WARN_ON(ifp->vif->profile.use_fwsup != BRCMF_PROFILE_FWSUP_1X))
+               return -EINVAL;
+
+       return brcmf_set_pmk(ifp, NULL, 0);
+}
+
 static struct cfg80211_ops brcmf_cfg80211_ops = {
        .add_virtual_intf = brcmf_cfg80211_add_iface,
        .del_virtual_intf = brcmf_cfg80211_del_iface,
@@ -5236,6 +5285,8 @@ static struct cfg80211_ops brcmf_cfg80211_ops = {
        .crit_proto_stop = brcmf_cfg80211_crit_proto_stop,
        .tdls_oper = brcmf_cfg80211_tdls_oper,
        .update_connect_params = brcmf_cfg80211_update_conn_params,
+       .set_pmk = brcmf_cfg80211_set_pmk,
+       .del_pmk = brcmf_cfg80211_del_pmk,
 };
 
 struct brcmf_cfg80211_vif *brcmf_alloc_vif(struct brcmf_cfg80211_info *cfg,
@@ -5295,7 +5346,8 @@ static bool brcmf_is_linkup(struct brcmf_cfg80211_vif *vif,
        u32 event = e->event_code;
        u32 status = e->status;
 
-       if (event == BRCMF_E_PSK_SUP &&
+       if (vif->profile.use_fwsup == BRCMF_PROFILE_FWSUP_PSK &&
+           event == BRCMF_E_PSK_SUP &&
            status == BRCMF_E_STATUS_FWSUP_COMPLETED)
                set_bit(BRCMF_VIF_STATUS_EAP_SUCCESS, &vif->sme_state);
        if (event == BRCMF_E_SET_SSID && status == BRCMF_E_STATUS_SUCCESS) {
@@ -6555,9 +6607,12 @@ static int brcmf_setup_wiphy(struct wiphy *wiphy, struct brcmf_if *ifp)
                wiphy->flags |= WIPHY_FLAG_SUPPORTS_TDLS;
        if (!ifp->drvr->settings->roamoff)
                wiphy->flags |= WIPHY_FLAG_SUPPORTS_FW_ROAM;
-       if (brcmf_feat_is_enabled(ifp, BRCMF_FEAT_FWSUP))
+       if (brcmf_feat_is_enabled(ifp, BRCMF_FEAT_FWSUP)) {
                wiphy_ext_feature_set(wiphy,
                                      NL80211_EXT_FEATURE_4WAY_HANDSHAKE_STA_PSK);
+               wiphy_ext_feature_set(wiphy,
+                                     NL80211_EXT_FEATURE_4WAY_HANDSHAKE_STA_1X);
+       }
        wiphy->mgmt_stypes = brcmf_txrx_stypes;
        wiphy->max_remain_on_channel_duration = 5000;
        if (brcmf_feat_is_enabled(ifp, BRCMF_FEAT_PNO)) {
index 9e7d3c6f625ee7017455af55c56474f0bda69b8c..7b2835e5e434474970e84468726d985e89fd519b 100644 (file)
@@ -117,7 +117,8 @@ struct brcmf_cfg80211_security {
 
 enum brcmf_profile_fwsup {
        BRCMF_PROFILE_FWSUP_NONE,
-       BRCMF_PROFILE_FWSUP_PSK
+       BRCMF_PROFILE_FWSUP_PSK,
+       BRCMF_PROFILE_FWSUP_1X
 };
 
 /**