package/kernel/mac80211/patches/351-0005-brcmfmac-rework-function-picking-free-BSS-index.patch

   1 From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
   2 Date: Thu, 26 May 2016 01:44:27 +0200
   3 Subject: [PATCH] brcmfmac: rework function picking free BSS index
   4 MIME-Version: 1.0
   5 Content-Type: text/plain; charset=UTF-8
   6 Content-Transfer-Encoding: 8bit
   7
   8 The old implementation was overcomplicated and slightly bugged in some
   9 corner cases.
  10
  11 Consider following state of BSS-es (limited to 6 for simplification):
  12 drvr->iflist[0]: { bsscfgidx:0, ndev->name:wlan1, }
  13 drvr->iflist[1]:  (null)
  14 drvr->iflist[2]: { bsscfgidx:2, ndev->name:wlan1-1, }
  15 drvr->iflist[3]: { bsscfgidx:3, ndev->name:wlan1-2, }
  16 drvr->iflist[4]:  (null)
  17 drvr->iflist[5]:  (null)
  18 In such case the next AP interface should bsscfgidx 4 (we don't use 1 as
  19 it's reserved for P2P).
  20
  21 With old code the loop iterations were following:
  22 [ifidx = 0] [bsscfgidx = 2] [highest = 2]
  23 [ifidx = 1] [bsscfgidx = 2] [highest = 2] available = true
  24 [ifidx = 2] [bsscfgidx = 2] [highest = 2] bsscfgidx = highest + 1
  25 [ifidx = 3] [bsscfgidx = 3] [highest = 2] bsscfgidx = highest + 1
  26 [ifidx = 4] [bsscfgidx = 3] [highest = 2] available = true
  27 [ifidx = 5] [bsscfgidx = 3] [highest = 2] available = true
  28 There were 2 obvious problems:
  29 1) Having empty BSS at index 1 was resulting in available being always
  30    set to true, even if we would run out of BSS-es.
  31 2) Calculated bsscfgidx was invalid (3 instead of 4) resulting in driver
  32    not being able to create the 4th AP interface.
  33
  34 New code is simpler, placed in file where it's really used, handles
  35 running out of free BSS-es and allows using 4 interfaces at the same
  36 time. It also looks for the first free BSS instead of one after the last
  37 in use. It works well with current driver (which doesn't allow deleting
  38 interfaces) and should be future proof (if we ever allow deleting).
  39
  40 Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
  41 Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
  42 ---
  43
  44 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
  45 +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
  46 @@ -527,6 +527,21 @@ brcmf_cfg80211_update_proto_addr_mode(st
  47                                                 ADDR_INDIRECT);
  48  }
  49
  50 +static int brcmf_get_first_free_bsscfgidx(struct brcmf_pub *drvr)
  51 +{
  52 +       int bsscfgidx;
  53 +
  54 +       for (bsscfgidx = 0; bsscfgidx < BRCMF_MAX_IFS; bsscfgidx++) {
  55 +               /* bsscfgidx 1 is reserved for legacy P2P */
  56 +               if (bsscfgidx == 1)
  57 +                       continue;
  58 +               if (!drvr->iflist[bsscfgidx])
  59 +                       return bsscfgidx;
  60 +       }
  61 +
  62 +       return -ENOMEM;
  63 +}
  64 +
  65  static int brcmf_cfg80211_request_ap_if(struct brcmf_if *ifp)
  66  {
  67         struct brcmf_mbss_ssid_le mbss_ssid_le;
  68 @@ -534,7 +549,7 @@ static int brcmf_cfg80211_request_ap_if(
  69         int err;
  70
  71         memset(&mbss_ssid_le, 0, sizeof(mbss_ssid_le));
  72 -       bsscfgidx = brcmf_get_next_free_bsscfgidx(ifp->drvr);
  73 +       bsscfgidx = brcmf_get_first_free_bsscfgidx(ifp->drvr);
  74         if (bsscfgidx < 0)
  75                 return bsscfgidx;
  76
  77 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c
  78 +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c
  79 @@ -753,30 +753,6 @@ void brcmf_remove_interface(struct brcmf
  80         brcmf_del_if(ifp->drvr, ifp->bsscfgidx);
  81  }
  82
  83 -int brcmf_get_next_free_bsscfgidx(struct brcmf_pub *drvr)
  84 -{
  85 -       int ifidx;
  86 -       int bsscfgidx;
  87 -       bool available;
  88 -       int highest;
  89 -
  90 -       available = false;
  91 -       bsscfgidx = 2;
  92 -       highest = 2;
  93 -       for (ifidx = 0; ifidx < BRCMF_MAX_IFS; ifidx++) {
  94 -               if (drvr->iflist[ifidx]) {
  95 -                       if (drvr->iflist[ifidx]->bsscfgidx == bsscfgidx)
  96 -                               bsscfgidx = highest + 1;
  97 -                       else if (drvr->iflist[ifidx]->bsscfgidx > highest)
  98 -                               highest = drvr->iflist[ifidx]->bsscfgidx;
  99 -               } else {
 100 -                       available = true;
 101 -               }
 102 -       }
 103 -
 104 -       return available ? bsscfgidx : -ENOMEM;
 105 -}
 106 -
 107  #ifdef CONFIG_INET
 108  #define ARPOL_MAX_ENTRIES      8
 109  static int brcmf_inetaddr_changed(struct notifier_block *nb,
 110 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h
 111 +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h
 112 @@ -217,7 +217,6 @@ int brcmf_net_attach(struct brcmf_if *if
 113  struct brcmf_if *brcmf_add_if(struct brcmf_pub *drvr, s32 bsscfgidx, s32 ifidx,
 114                               bool is_p2pdev, char *name, u8 *mac_addr);
 115  void brcmf_remove_interface(struct brcmf_if *ifp);
 116 -int brcmf_get_next_free_bsscfgidx(struct brcmf_pub *drvr);
 117  void brcmf_txflowblock_if(struct brcmf_if *ifp,
 118                           enum brcmf_netif_stop_reason reason, bool state);
 119  void brcmf_txfinalize(struct brcmf_if *ifp, struct sk_buff *txp, bool success);