Commit 7bedd0cf authored by Johannes Berg's avatar Johannes Berg

mac80211: use rhashtable for station table

We currently have a hand-rolled table with 256 entries and are
using the last byte of the MAC address as the hash. This hash
is obviously very fast, but collisions are easily created and
we waste a lot of space in the common case of just connecting
as a client to an AP where we just have a single station. The
other common case of an AP is also suboptimal due to the size
of the hash table and the ease of causing collisions.

Convert all of this to use rhashtable with jhash, which gives
us the advantage of a far better hash function (with random
perturbation to avoid hash collision attacks) and of course
that the hash table grows and shrinks dynamically with chain
length, improving both cases above.

Use a specialised hash function (using jhash, but with fixed
length) to achieve better compiler optimisation as suggested
by Sergey Ryazanov.
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 9911674f
...@@ -26,6 +26,7 @@ ...@@ -26,6 +26,7 @@
#include <linux/etherdevice.h> #include <linux/etherdevice.h>
#include <linux/leds.h> #include <linux/leds.h>
#include <linux/idr.h> #include <linux/idr.h>
#include <linux/rhashtable.h>
#include <net/ieee80211_radiotap.h> #include <net/ieee80211_radiotap.h>
#include <net/cfg80211.h> #include <net/cfg80211.h>
#include <net/mac80211.h> #include <net/mac80211.h>
...@@ -1187,7 +1188,7 @@ struct ieee80211_local { ...@@ -1187,7 +1188,7 @@ struct ieee80211_local {
spinlock_t tim_lock; spinlock_t tim_lock;
unsigned long num_sta; unsigned long num_sta;
struct list_head sta_list; struct list_head sta_list;
struct sta_info __rcu *sta_hash[STA_HASH_SIZE]; struct rhashtable sta_hash;
struct timer_list sta_cleanup; struct timer_list sta_cleanup;
int sta_generation; int sta_generation;
......
...@@ -557,6 +557,9 @@ struct ieee80211_hw *ieee80211_alloc_hw_nm(size_t priv_data_len, ...@@ -557,6 +557,9 @@ struct ieee80211_hw *ieee80211_alloc_hw_nm(size_t priv_data_len,
local = wiphy_priv(wiphy); local = wiphy_priv(wiphy);
if (sta_info_init(local))
goto err_free;
local->hw.wiphy = wiphy; local->hw.wiphy = wiphy;
local->hw.priv = (char *)local + ALIGN(sizeof(*local), NETDEV_ALIGN); local->hw.priv = (char *)local + ALIGN(sizeof(*local), NETDEV_ALIGN);
...@@ -629,8 +632,6 @@ struct ieee80211_hw *ieee80211_alloc_hw_nm(size_t priv_data_len, ...@@ -629,8 +632,6 @@ struct ieee80211_hw *ieee80211_alloc_hw_nm(size_t priv_data_len,
spin_lock_init(&local->ack_status_lock); spin_lock_init(&local->ack_status_lock);
idr_init(&local->ack_status_frames); idr_init(&local->ack_status_frames);
sta_info_init(local);
for (i = 0; i < IEEE80211_MAX_QUEUES; i++) { for (i = 0; i < IEEE80211_MAX_QUEUES; i++) {
skb_queue_head_init(&local->pending[i]); skb_queue_head_init(&local->pending[i]);
atomic_set(&local->agg_queue_stop[i], 0); atomic_set(&local->agg_queue_stop[i], 0);
...@@ -650,6 +651,9 @@ struct ieee80211_hw *ieee80211_alloc_hw_nm(size_t priv_data_len, ...@@ -650,6 +651,9 @@ struct ieee80211_hw *ieee80211_alloc_hw_nm(size_t priv_data_len,
ieee80211_roc_setup(local); ieee80211_roc_setup(local);
return &local->hw; return &local->hw;
err_free:
wiphy_free(wiphy);
return NULL;
} }
EXPORT_SYMBOL(ieee80211_alloc_hw_nm); EXPORT_SYMBOL(ieee80211_alloc_hw_nm);
...@@ -1173,7 +1177,6 @@ void ieee80211_unregister_hw(struct ieee80211_hw *hw) ...@@ -1173,7 +1177,6 @@ void ieee80211_unregister_hw(struct ieee80211_hw *hw)
destroy_workqueue(local->workqueue); destroy_workqueue(local->workqueue);
wiphy_unregister(local->hw.wiphy); wiphy_unregister(local->hw.wiphy);
sta_info_stop(local);
ieee80211_wep_free(local); ieee80211_wep_free(local);
ieee80211_led_exit(local); ieee80211_led_exit(local);
kfree(local->int_scan_req); kfree(local->int_scan_req);
......
...@@ -3423,7 +3423,8 @@ static void __ieee80211_rx_handle_packet(struct ieee80211_hw *hw, ...@@ -3423,7 +3423,8 @@ static void __ieee80211_rx_handle_packet(struct ieee80211_hw *hw,
__le16 fc; __le16 fc;
struct ieee80211_rx_data rx; struct ieee80211_rx_data rx;
struct ieee80211_sub_if_data *prev; struct ieee80211_sub_if_data *prev;
struct sta_info *sta, *tmp, *prev_sta; struct sta_info *sta, *prev_sta;
struct rhash_head *tmp;
int err = 0; int err = 0;
fc = ((struct ieee80211_hdr *)skb->data)->frame_control; fc = ((struct ieee80211_hdr *)skb->data)->frame_control;
...@@ -3458,9 +3459,13 @@ static void __ieee80211_rx_handle_packet(struct ieee80211_hw *hw, ...@@ -3458,9 +3459,13 @@ static void __ieee80211_rx_handle_packet(struct ieee80211_hw *hw,
ieee80211_scan_rx(local, skb); ieee80211_scan_rx(local, skb);
if (ieee80211_is_data(fc)) { if (ieee80211_is_data(fc)) {
const struct bucket_table *tbl;
prev_sta = NULL; prev_sta = NULL;
for_each_sta_info(local, hdr->addr2, sta, tmp) { tbl = rht_dereference_rcu(local->sta_hash.tbl, &local->sta_hash);
for_each_sta_info(local, tbl, hdr->addr2, sta, tmp) {
if (!prev_sta) { if (!prev_sta) {
prev_sta = sta; prev_sta = sta;
continue; continue;
......
...@@ -64,32 +64,20 @@ ...@@ -64,32 +64,20 @@
* freed before they are done using it. * freed before they are done using it.
*/ */
static const struct rhashtable_params sta_rht_params = {
.nelem_hint = 3, /* start small */
.head_offset = offsetof(struct sta_info, hash_node),
.key_offset = offsetof(struct sta_info, sta.addr),
.key_len = ETH_ALEN,
.hashfn = sta_addr_hash,
};
/* Caller must hold local->sta_mtx */ /* Caller must hold local->sta_mtx */
static int sta_info_hash_del(struct ieee80211_local *local, static int sta_info_hash_del(struct ieee80211_local *local,
struct sta_info *sta) struct sta_info *sta)
{ {
struct sta_info *s; return rhashtable_remove_fast(&local->sta_hash, &sta->hash_node,
sta_rht_params);
s = rcu_dereference_protected(local->sta_hash[STA_HASH(sta->sta.addr)],
lockdep_is_held(&local->sta_mtx));
if (!s)
return -ENOENT;
if (s == sta) {
rcu_assign_pointer(local->sta_hash[STA_HASH(sta->sta.addr)],
s->hnext);
return 0;
}
while (rcu_access_pointer(s->hnext) &&
rcu_access_pointer(s->hnext) != sta)
s = rcu_dereference_protected(s->hnext,
lockdep_is_held(&local->sta_mtx));
if (rcu_access_pointer(s->hnext)) {
rcu_assign_pointer(s->hnext, sta->hnext);
return 0;
}
return -ENOENT;
} }
static void __cleanup_single_sta(struct sta_info *sta) static void __cleanup_single_sta(struct sta_info *sta)
...@@ -159,18 +147,8 @@ struct sta_info *sta_info_get(struct ieee80211_sub_if_data *sdata, ...@@ -159,18 +147,8 @@ struct sta_info *sta_info_get(struct ieee80211_sub_if_data *sdata,
const u8 *addr) const u8 *addr)
{ {
struct ieee80211_local *local = sdata->local; struct ieee80211_local *local = sdata->local;
struct sta_info *sta;
sta = rcu_dereference_check(local->sta_hash[STA_HASH(addr)], return rhashtable_lookup_fast(&local->sta_hash, addr, sta_rht_params);
lockdep_is_held(&local->sta_mtx));
while (sta) {
if (sta->sdata == sdata &&
ether_addr_equal(sta->sta.addr, addr))
break;
sta = rcu_dereference_check(sta->hnext,
lockdep_is_held(&local->sta_mtx));
}
return sta;
} }
/* /*
...@@ -182,18 +160,24 @@ struct sta_info *sta_info_get_bss(struct ieee80211_sub_if_data *sdata, ...@@ -182,18 +160,24 @@ struct sta_info *sta_info_get_bss(struct ieee80211_sub_if_data *sdata,
{ {
struct ieee80211_local *local = sdata->local; struct ieee80211_local *local = sdata->local;
struct sta_info *sta; struct sta_info *sta;
struct rhash_head *tmp;
const struct bucket_table *tbl;
sta = rcu_dereference_check(local->sta_hash[STA_HASH(addr)], rcu_read_lock();
lockdep_is_held(&local->sta_mtx)); tbl = rht_dereference_rcu(local->sta_hash.tbl, &local->sta_hash);
while (sta) {
if ((sta->sdata == sdata || for_each_sta_info(local, tbl, addr, sta, tmp) {
(sta->sdata->bss && sta->sdata->bss == sdata->bss)) && if (sta->sdata == sdata ||
ether_addr_equal(sta->sta.addr, addr)) (sta->sdata->bss && sta->sdata->bss == sdata->bss)) {
break; rcu_read_unlock();
sta = rcu_dereference_check(sta->hnext, /* this is safe as the caller must already hold
lockdep_is_held(&local->sta_mtx)); * another rcu read section or the mutex
} */
return sta; return sta;
}
}
rcu_read_unlock();
return NULL;
} }
struct sta_info *sta_info_get_by_idx(struct ieee80211_sub_if_data *sdata, struct sta_info *sta_info_get_by_idx(struct ieee80211_sub_if_data *sdata,
...@@ -242,9 +226,8 @@ void sta_info_free(struct ieee80211_local *local, struct sta_info *sta) ...@@ -242,9 +226,8 @@ void sta_info_free(struct ieee80211_local *local, struct sta_info *sta)
static void sta_info_hash_add(struct ieee80211_local *local, static void sta_info_hash_add(struct ieee80211_local *local,
struct sta_info *sta) struct sta_info *sta)
{ {
lockdep_assert_held(&local->sta_mtx); rhashtable_insert_fast(&local->sta_hash, &sta->hash_node,
sta->hnext = local->sta_hash[STA_HASH(sta->sta.addr)]; sta_rht_params);
rcu_assign_pointer(local->sta_hash[STA_HASH(sta->sta.addr)], sta);
} }
static void sta_deliver_ps_frames(struct work_struct *wk) static void sta_deliver_ps_frames(struct work_struct *wk)
...@@ -948,19 +931,32 @@ static void sta_info_cleanup(unsigned long data) ...@@ -948,19 +931,32 @@ static void sta_info_cleanup(unsigned long data)
round_jiffies(jiffies + STA_INFO_CLEANUP_INTERVAL)); round_jiffies(jiffies + STA_INFO_CLEANUP_INTERVAL));
} }
void sta_info_init(struct ieee80211_local *local) u32 sta_addr_hash(const void *key, u32 length, u32 seed)
{
return jhash(key, ETH_ALEN, seed);
}
int sta_info_init(struct ieee80211_local *local)
{ {
int err;
err = rhashtable_init(&local->sta_hash, &sta_rht_params);
if (err)
return err;
spin_lock_init(&local->tim_lock); spin_lock_init(&local->tim_lock);
mutex_init(&local->sta_mtx); mutex_init(&local->sta_mtx);
INIT_LIST_HEAD(&local->sta_list); INIT_LIST_HEAD(&local->sta_list);
setup_timer(&local->sta_cleanup, sta_info_cleanup, setup_timer(&local->sta_cleanup, sta_info_cleanup,
(unsigned long)local); (unsigned long)local);
return 0;
} }
void sta_info_stop(struct ieee80211_local *local) void sta_info_stop(struct ieee80211_local *local)
{ {
del_timer_sync(&local->sta_cleanup); del_timer_sync(&local->sta_cleanup);
rhashtable_destroy(&local->sta_hash);
} }
...@@ -1027,13 +1023,18 @@ struct ieee80211_sta *ieee80211_find_sta_by_ifaddr(struct ieee80211_hw *hw, ...@@ -1027,13 +1023,18 @@ struct ieee80211_sta *ieee80211_find_sta_by_ifaddr(struct ieee80211_hw *hw,
const u8 *addr, const u8 *addr,
const u8 *localaddr) const u8 *localaddr)
{ {
struct sta_info *sta, *nxt; struct ieee80211_local *local = hw_to_local(hw);
struct sta_info *sta;
struct rhash_head *tmp;
const struct bucket_table *tbl;
tbl = rht_dereference_rcu(local->sta_hash.tbl, &local->sta_hash);
/* /*
* Just return a random station if localaddr is NULL * Just return a random station if localaddr is NULL
* ... first in list. * ... first in list.
*/ */
for_each_sta_info(hw_to_local(hw), addr, sta, nxt) { for_each_sta_info(local, tbl, addr, sta, tmp) {
if (localaddr && if (localaddr &&
!ether_addr_equal(sta->sdata->vif.addr, localaddr)) !ether_addr_equal(sta->sdata->vif.addr, localaddr))
continue; continue;
......
...@@ -16,6 +16,7 @@ ...@@ -16,6 +16,7 @@
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <linux/average.h> #include <linux/average.h>
#include <linux/etherdevice.h> #include <linux/etherdevice.h>
#include <linux/rhashtable.h>
#include "key.h" #include "key.h"
/** /**
...@@ -246,7 +247,7 @@ struct sta_ampdu_mlme { ...@@ -246,7 +247,7 @@ struct sta_ampdu_mlme {
* *
* @list: global linked list entry * @list: global linked list entry
* @free_list: list entry for keeping track of stations to free * @free_list: list entry for keeping track of stations to free
* @hnext: hash table linked list pointer * @hash_node: hash node for rhashtable
* @local: pointer to the global information * @local: pointer to the global information
* @sdata: virtual interface this station belongs to * @sdata: virtual interface this station belongs to
* @ptk: peer keys negotiated with this station, if any * @ptk: peer keys negotiated with this station, if any
...@@ -339,7 +340,7 @@ struct sta_info { ...@@ -339,7 +340,7 @@ struct sta_info {
/* General information, mostly static */ /* General information, mostly static */
struct list_head list, free_list; struct list_head list, free_list;
struct rcu_head rcu_head; struct rcu_head rcu_head;
struct sta_info __rcu *hnext; struct rhash_head hash_node;
struct ieee80211_local *local; struct ieee80211_local *local;
struct ieee80211_sub_if_data *sdata; struct ieee80211_sub_if_data *sdata;
struct ieee80211_key __rcu *gtk[NUM_DEFAULT_KEYS + NUM_DEFAULT_MGMT_KEYS]; struct ieee80211_key __rcu *gtk[NUM_DEFAULT_KEYS + NUM_DEFAULT_MGMT_KEYS];
...@@ -535,10 +536,6 @@ rcu_dereference_protected_tid_tx(struct sta_info *sta, int tid) ...@@ -535,10 +536,6 @@ rcu_dereference_protected_tid_tx(struct sta_info *sta, int tid)
lockdep_is_held(&sta->ampdu_mlme.mtx)); lockdep_is_held(&sta->ampdu_mlme.mtx));
} }
#define STA_HASH_SIZE 256
#define STA_HASH(sta) (sta[5])
/* Maximum number of frames to buffer per power saving station per AC */ /* Maximum number of frames to buffer per power saving station per AC */
#define STA_MAX_TX_BUFFER 64 #define STA_MAX_TX_BUFFER 64
...@@ -559,26 +556,15 @@ struct sta_info *sta_info_get(struct ieee80211_sub_if_data *sdata, ...@@ -559,26 +556,15 @@ struct sta_info *sta_info_get(struct ieee80211_sub_if_data *sdata,
struct sta_info *sta_info_get_bss(struct ieee80211_sub_if_data *sdata, struct sta_info *sta_info_get_bss(struct ieee80211_sub_if_data *sdata,
const u8 *addr); const u8 *addr);
static inline u32 sta_addr_hash(const void *key, u32 length, u32 seed);
void for_each_sta_info_type_check(struct ieee80211_local *local,
const u8 *addr, #define _sta_bucket_idx(_tbl, _a) \
struct sta_info *sta, rht_bucket_index(_tbl, sta_addr_hash(_a, ETH_ALEN, (_tbl)->hash_rnd))
struct sta_info *nxt)
{
}
#define for_each_sta_info(local, _addr, _sta, nxt) \ #define for_each_sta_info(local, tbl, _addr, _sta, _tmp) \
for ( /* initialise loop */ \ rht_for_each_entry_rcu(_sta, _tmp, tbl, \
_sta = rcu_dereference(local->sta_hash[STA_HASH(_addr)]),\ _sta_bucket_idx(tbl, _addr), \
nxt = _sta ? rcu_dereference(_sta->hnext) : NULL; \ hash_node) \
/* typecheck */ \
for_each_sta_info_type_check(local, (_addr), _sta, nxt),\
/* continue condition */ \
_sta; \
/* advance loop */ \
_sta = nxt, \
nxt = _sta ? rcu_dereference(_sta->hnext) : NULL \
) \
/* compare address and run code only if it matches */ \ /* compare address and run code only if it matches */ \
if (ether_addr_equal(_sta->sta.addr, (_addr))) if (ether_addr_equal(_sta->sta.addr, (_addr)))
...@@ -615,7 +601,7 @@ int sta_info_destroy_addr_bss(struct ieee80211_sub_if_data *sdata, ...@@ -615,7 +601,7 @@ int sta_info_destroy_addr_bss(struct ieee80211_sub_if_data *sdata,
void sta_info_recalc_tim(struct sta_info *sta); void sta_info_recalc_tim(struct sta_info *sta);
void sta_info_init(struct ieee80211_local *local); int sta_info_init(struct ieee80211_local *local);
void sta_info_stop(struct ieee80211_local *local); void sta_info_stop(struct ieee80211_local *local);
/** /**
......
...@@ -654,7 +654,8 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -654,7 +654,8 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
struct ieee80211_supported_band *sband; struct ieee80211_supported_band *sband;
struct ieee80211_sub_if_data *sdata; struct ieee80211_sub_if_data *sdata;
struct net_device *prev_dev = NULL; struct net_device *prev_dev = NULL;
struct sta_info *sta, *tmp; struct sta_info *sta;
struct rhash_head *tmp;
int retry_count; int retry_count;
int rates_idx; int rates_idx;
bool send_to_cooked; bool send_to_cooked;
...@@ -663,6 +664,7 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -663,6 +664,7 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
int rtap_len; int rtap_len;
int shift = 0; int shift = 0;
int tid = IEEE80211_NUM_TIDS; int tid = IEEE80211_NUM_TIDS;
const struct bucket_table *tbl;
rates_idx = ieee80211_tx_get_rates(hw, info, &retry_count); rates_idx = ieee80211_tx_get_rates(hw, info, &retry_count);
...@@ -671,7 +673,9 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -671,7 +673,9 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
sband = local->hw.wiphy->bands[info->band]; sband = local->hw.wiphy->bands[info->band];
fc = hdr->frame_control; fc = hdr->frame_control;
for_each_sta_info(local, hdr->addr1, sta, tmp) { tbl = rht_dereference_rcu(local->sta_hash.tbl, &local->sta_hash);
for_each_sta_info(local, tbl, hdr->addr1, sta, tmp) {
/* skip wrong virtual interface */ /* skip wrong virtual interface */
if (!ether_addr_equal(hdr->addr2, sta->sdata->vif.addr)) if (!ether_addr_equal(hdr->addr2, sta->sdata->vif.addr))
continue; continue;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment