rtw89: add addr_cam field to sta to support AP mode
authorPing-Ke Shih <pkshih@realtek.com>
Mon, 7 Feb 2022 06:38:55 +0000 (14:38 +0800)
committerKalle Valo <kvalo@kernel.org>
Thu, 10 Feb 2022 08:40:49 +0000 (10:40 +0200)
In AP mode, each connected station needs an entry of address CAM. The
address CAM of vif is still needed to assit in AP itself.

For station mode, it still uses vif's address CAM.

Add a help macro rtw89_get_addr_cam_of() to get addr_cam from vif or sta
for all use cases.

Signed-off-by: Ping-Ke Shih <pkshih@realtek.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220207063900.43643-3-pkshih@realtek.com
drivers/net/wireless/realtek/rtw89/cam.c
drivers/net/wireless/realtek/rtw89/core.c
drivers/net/wireless/realtek/rtw89/core.h

index 2114d117b603d14e379484130cafd42438de72e1..305dbbebff6bbc10d28d01619667ea8c8956ff70 100644 (file)
@@ -231,7 +231,7 @@ static int rtw89_cam_attach_sec_cam(struct rtw89_dev *rtwdev,
        }
 
        rtwvif = (struct rtw89_vif *)vif->drv_priv;
-       addr_cam = &rtwvif->addr_cam;
+       addr_cam = rtw89_get_addr_cam_of(rtwvif, rtwsta);
        ret = rtw89_cam_get_addr_cam_key_idx(addr_cam, sec_cam, key, &key_idx);
        if (ret) {
                rtw89_err(rtwdev, "failed to get addr cam key idx %d, %d\n",
@@ -387,7 +387,7 @@ int rtw89_cam_sec_key_del(struct rtw89_dev *rtwdev,
        }
 
        rtwvif = (struct rtw89_vif *)vif->drv_priv;
-       addr_cam = &rtwvif->addr_cam;
+       addr_cam = rtw89_get_addr_cam_of(rtwvif, rtwsta);
        sec_cam = addr_cam->sec_entries[key_idx];
        if (!sec_cam)
                return -EINVAL;
@@ -617,7 +617,7 @@ void rtw89_cam_fill_addr_cam_info(struct rtw89_dev *rtwdev,
                                  u8 *cmd)
 {
        struct ieee80211_vif *vif = rtwvif_to_vif(rtwvif);
-       struct rtw89_addr_cam_entry *addr_cam = &rtwvif->addr_cam;
+       struct rtw89_addr_cam_entry *addr_cam = rtw89_get_addr_cam_of(rtwvif, rtwsta);
        struct ieee80211_sta *sta = rtwsta_to_sta_safe(rtwsta);
        const u8 *sma = scan_mac_addr ? scan_mac_addr : rtwvif->mac_addr;
        u8 sma_hash, tma_hash, addr_msk_start;
index 490f5d86eeb2e3f377f71544a6053aa4e45f4a4e..81575463a833f2447637436d68d5ea2a9c2a046c 100644 (file)
@@ -4,6 +4,7 @@
 #include <linux/ip.h>
 #include <linux/udp.h>
 
+#include "cam.h"
 #include "coex.h"
 #include "core.h"
 #include "efuse.h"
@@ -399,9 +400,11 @@ rtw89_core_tx_update_sec_key(struct rtw89_dev *rtwdev,
                             struct rtw89_core_tx_request *tx_req)
 {
        struct ieee80211_vif *vif = tx_req->vif;
+       struct ieee80211_sta *sta = tx_req->sta;
        struct ieee80211_tx_info *info;
        struct ieee80211_key_conf *key;
        struct rtw89_vif *rtwvif;
+       struct rtw89_sta *rtwsta = sta_to_rtwsta_safe(sta);
        struct rtw89_addr_cam_entry *addr_cam;
        struct rtw89_sec_cam_entry *sec_cam;
        struct rtw89_tx_desc_info *desc_info = &tx_req->desc_info;
@@ -414,7 +417,7 @@ rtw89_core_tx_update_sec_key(struct rtw89_dev *rtwdev,
        }
 
        rtwvif = (struct rtw89_vif *)vif->drv_priv;
-       addr_cam = &rtwvif->addr_cam;
+       addr_cam = rtw89_get_addr_cam_of(rtwvif, rtwsta);
 
        info = IEEE80211_SKB_CB(skb);
        key = info->control.hw_key;
index cb71dfca9341c5a88ce22cc85f010bd1aa18f93b..deb91f6b67375be2cfddec48c137fd978ae3c39d 100644 (file)
@@ -1842,32 +1842,6 @@ struct rtw89_ba_cam_entry {
        u8 tid;
 };
 
-struct rtw89_sta {
-       u8 mac_id;
-       bool disassoc;
-       struct rtw89_vif *rtwvif;
-       struct rtw89_ra_info ra;
-       struct rtw89_ra_report ra_report;
-       int max_agg_wait;
-       u8 prev_rssi;
-       struct ewma_rssi avg_rssi;
-       struct rtw89_ampdu_params ampdu_params[IEEE80211_NUM_TIDS];
-       struct ieee80211_rx_status rx_status;
-       u16 rx_hw_rate;
-       __le32 htc_template;
-
-       bool use_cfg_mask;
-       struct cfg80211_bitrate_mask mask;
-
-       bool cctl_tx_time;
-       u32 ampdu_max_time:4;
-       bool cctl_tx_retry_limit;
-       u32 data_tx_cnt_lmt:6;
-
-       DECLARE_BITMAP(ba_cam_map, RTW89_BA_CAM_NUM);
-       struct rtw89_ba_cam_entry ba_cam_entry[RTW89_BA_CAM_NUM];
-};
-
 #define RTW89_MAX_ADDR_CAM_NUM         128
 #define RTW89_MAX_BSSID_CAM_NUM                20
 #define RTW89_MAX_SEC_CAM_NUM          128
@@ -1911,6 +1885,33 @@ struct rtw89_sec_cam_entry {
        u8 key[32];
 };
 
+struct rtw89_sta {
+       u8 mac_id;
+       bool disassoc;
+       struct rtw89_vif *rtwvif;
+       struct rtw89_ra_info ra;
+       struct rtw89_ra_report ra_report;
+       int max_agg_wait;
+       u8 prev_rssi;
+       struct ewma_rssi avg_rssi;
+       struct rtw89_ampdu_params ampdu_params[IEEE80211_NUM_TIDS];
+       struct ieee80211_rx_status rx_status;
+       u16 rx_hw_rate;
+       __le32 htc_template;
+       struct rtw89_addr_cam_entry addr_cam; /* AP mode only */
+
+       bool use_cfg_mask;
+       struct cfg80211_bitrate_mask mask;
+
+       bool cctl_tx_time;
+       u32 ampdu_max_time:4;
+       bool cctl_tx_retry_limit;
+       u32 data_tx_cnt_lmt:6;
+
+       DECLARE_BITMAP(ba_cam_map, RTW89_BA_CAM_NUM);
+       struct rtw89_ba_cam_entry ba_cam_entry[RTW89_BA_CAM_NUM];
+};
+
 struct rtw89_efuse {
        bool valid;
        u8 xtal_cap;
@@ -3149,6 +3150,15 @@ static inline struct rtw89_sta *sta_to_rtwsta_safe(struct ieee80211_sta *sta)
        return sta ? (struct rtw89_sta *)sta->drv_priv : NULL;
 }
 
+static inline
+struct rtw89_addr_cam_entry *rtw89_get_addr_cam_of(struct rtw89_vif *rtwvif,
+                                                  struct rtw89_sta *rtwsta)
+{
+       if (rtwvif->net_type == RTW89_NET_TYPE_AP_MODE && rtwsta)
+               return &rtwsta->addr_cam;
+       return &rtwvif->addr_cam;
+}
+
 static inline
 void rtw89_chip_set_channel_prepare(struct rtw89_dev *rtwdev,
                                    struct rtw89_channel_help_params *p)