summaryrefslogtreecommitdiff
path: root/net/mac80211
diff options
context:
space:
mode:
authorJohannes Berg <johannes.berg@intel.com>2017-12-19 12:10:48 +0300
committerJohannes Berg <johannes.berg@intel.com>2017-12-19 12:12:26 +0300
commit0973dd45ecefd746569d414406f5733062fe2817 (patch)
tree7a7d3eb5261a47160fef831203a0c90fca6bee8d /net/mac80211
parentb0d52ad821843a6c5badebd80feef9f871904fa6 (diff)
downloadlinux-0973dd45ecefd746569d414406f5733062fe2817.tar.xz
Revert "mac80211: Add airtime account and scheduling to TXQs"
This reverts commit b0d52ad821843a6c5badebd80feef9f871904fa6. We need to revert the TXQ scheduling API due to conflicts with a new driver, and this depends on that API. Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211')
-rw-r--r--net/mac80211/debugfs.c1
-rw-r--r--net/mac80211/debugfs_sta.c29
-rw-r--r--net/mac80211/ieee80211_i.h8
-rw-r--r--net/mac80211/main.c3
-rw-r--r--net/mac80211/rx.c8
-rw-r--r--net/mac80211/sta_info.c2
-rw-r--r--net/mac80211/sta_info.h7
-rw-r--r--net/mac80211/status.c16
-rw-r--r--net/mac80211/tx.c31
9 files changed, 8 insertions, 97 deletions
diff --git a/net/mac80211/debugfs.c b/net/mac80211/debugfs.c
index d6b87a4ec3e9..1f466d12a6bc 100644
--- a/net/mac80211/debugfs.c
+++ b/net/mac80211/debugfs.c
@@ -212,7 +212,6 @@ static const char *hw_flag_names[] = {
FLAG(REPORTS_LOW_ACK),
FLAG(SUPPORTS_TX_FRAG),
FLAG(SUPPORTS_TDLS_BUFFER_STA),
- FLAG(AIRTIME_ACCOUNTING),
#undef FLAG
};
diff --git a/net/mac80211/debugfs_sta.c b/net/mac80211/debugfs_sta.c
index 40dba446836f..b15412c21ac9 100644
--- a/net/mac80211/debugfs_sta.c
+++ b/net/mac80211/debugfs_sta.c
@@ -188,32 +188,6 @@ static ssize_t sta_aqm_read(struct file *file, char __user *userbuf,
}
STA_OPS(aqm);
-static ssize_t sta_airtime_read(struct file *file, char __user *userbuf,
- size_t count, loff_t *ppos)
-{
- struct sta_info *sta = file->private_data;
- size_t bufsz = 200;
- char *buf = kzalloc(bufsz, GFP_KERNEL), *p = buf;
- ssize_t rv;
-
- if (!buf)
- return -ENOMEM;
-
- spin_lock_bh(&sta->lock);
-
- p += scnprintf(p, bufsz + buf - p,
- "RX: %llu us\nTX: %llu us\nDeficit: %lld us\n",
- sta->airtime_stats.rx_airtime,
- sta->airtime_stats.tx_airtime,
- sta->airtime_deficit);
-
- spin_unlock_bh(&sta->lock);
- rv = simple_read_from_buffer(userbuf, count, ppos, buf, p - buf);
- kfree(buf);
- return rv;
-}
-STA_OPS(airtime);
-
static ssize_t sta_agg_status_read(struct file *file, char __user *userbuf,
size_t count, loff_t *ppos)
{
@@ -568,9 +542,6 @@ void ieee80211_sta_debugfs_add(struct sta_info *sta)
if (local->ops->wake_tx_queue)
DEBUGFS_ADD(aqm);
- if (ieee80211_hw_check(&local->hw, AIRTIME_ACCOUNTING))
- DEBUGFS_ADD(airtime);
-
if (sizeof(sta->driver_buffered_tids) == sizeof(u32))
debugfs_create_x32("driver_buffered_tids", 0400,
sta->debugfs_dir,
diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h
index 120c516851cf..4155838c7bef 100644
--- a/net/mac80211/ieee80211_i.h
+++ b/net/mac80211/ieee80211_i.h
@@ -90,9 +90,6 @@ extern const u8 ieee80211_ac_to_qos_mask[IEEE80211_NUM_ACS];
#define IEEE80211_MAX_NAN_INSTANCE_ID 255
-/* How much to increase airtime deficit on each scheduling round */
-#define IEEE80211_AIRTIME_QUANTUM 1000 /* usec */
-
struct ieee80211_fragment_entry {
struct sk_buff_head skb_list;
unsigned long first_frag_time;
@@ -1126,10 +1123,9 @@ struct ieee80211_local {
struct codel_vars *cvars;
struct codel_params cparams;
- /* protects active_txqs_{new,old} and txqi->schedule_order */
+ /* protects active_txqs and txqi->schedule_order */
spinlock_t active_txq_lock;
- struct list_head active_txqs_new;
- struct list_head active_txqs_old;
+ struct list_head active_txqs;
const struct ieee80211_ops *ops;
diff --git a/net/mac80211/main.c b/net/mac80211/main.c
index b7142f8491d0..935d6e2491b1 100644
--- a/net/mac80211/main.c
+++ b/net/mac80211/main.c
@@ -619,8 +619,7 @@ struct ieee80211_hw *ieee80211_alloc_hw_nm(size_t priv_data_len,
spin_lock_init(&local->rx_path_lock);
spin_lock_init(&local->queue_stop_reason_lock);
- INIT_LIST_HEAD(&local->active_txqs_new);
- INIT_LIST_HEAD(&local->active_txqs_old);
+ INIT_LIST_HEAD(&local->active_txqs);
spin_lock_init(&local->active_txq_lock);
INIT_LIST_HEAD(&local->chanctx_list);
diff --git a/net/mac80211/rx.c b/net/mac80211/rx.c
index 808f41fb536a..b3cff69bfd66 100644
--- a/net/mac80211/rx.c
+++ b/net/mac80211/rx.c
@@ -1630,14 +1630,6 @@ ieee80211_rx_h_sta_process(struct ieee80211_rx_data *rx)
if (ieee80211_vif_is_mesh(&rx->sdata->vif))
ieee80211_mps_rx_h_sta_process(sta, hdr);
- /* airtime accounting */
- if (status->airtime) {
- spin_lock_bh(&sta->lock);
- sta->airtime_stats.rx_airtime += status->airtime;
- sta->airtime_deficit -= status->airtime;
- spin_unlock_bh(&sta->lock);
- }
-
/*
* Drop (qos-)data::nullfunc frames silently, since they
* are used only to control station power saving mode.
diff --git a/net/mac80211/sta_info.c b/net/mac80211/sta_info.c
index ed5500e8aafb..e0bcf16df494 100644
--- a/net/mac80211/sta_info.c
+++ b/net/mac80211/sta_info.c
@@ -425,8 +425,6 @@ struct sta_info *sta_info_alloc(struct ieee80211_sub_if_data *sdata,
sta->cparams.interval = MS2TIME(100);
sta->cparams.ecn = true;
- sta->airtime_deficit = IEEE80211_AIRTIME_QUANTUM;
-
sta_dbg(sdata, "Allocated STA %pM\n", sta->sta.addr);
return sta;
diff --git a/net/mac80211/sta_info.h b/net/mac80211/sta_info.h
index e356f2f85e12..cd53619435b6 100644
--- a/net/mac80211/sta_info.h
+++ b/net/mac80211/sta_info.h
@@ -559,13 +559,6 @@ struct sta_info {
} tx_stats;
u16 tid_seq[IEEE80211_QOS_CTL_TID_MASK + 1];
- /* Airtime stats and deficit, protected by lock */
- struct {
- u64 rx_airtime;
- u64 tx_airtime;
- } airtime_stats;
- s64 airtime_deficit;
-
/*
* Aggregation information, locked with lock.
*/
diff --git a/net/mac80211/status.c b/net/mac80211/status.c
index b044dbed2bb1..da7427a41529 100644
--- a/net/mac80211/status.c
+++ b/net/mac80211/status.c
@@ -823,14 +823,6 @@ static void __ieee80211_tx_status(struct ieee80211_hw *hw,
ieee80211_lost_packet(sta, info);
}
}
-
- if (info->status.tx_time &&
- ieee80211_hw_check(&local->hw, AIRTIME_ACCOUNTING)) {
- spin_lock_bh(&sta->lock);
- sta->airtime_stats.tx_airtime += info->status.tx_time;
- sta->airtime_deficit -= info->status.tx_time;
- spin_unlock_bh(&sta->lock);
- }
}
/* SNMP counters
@@ -955,14 +947,6 @@ void ieee80211_tx_status_ext(struct ieee80211_hw *hw,
sta->status_stats.retry_failed++;
sta->status_stats.retry_count += retry_count;
- if (info->status.tx_time &&
- ieee80211_hw_check(&local->hw, AIRTIME_ACCOUNTING)) {
- spin_lock_bh(&sta->lock);
- sta->airtime_stats.tx_airtime += info->status.tx_time;
- sta->airtime_deficit -= info->status.tx_time;
- spin_unlock_bh(&sta->lock);
- }
-
if (acked) {
sta->status_stats.last_ack = jiffies;
diff --git a/net/mac80211/tx.c b/net/mac80211/tx.c
index 18381581b5e9..842881ca8f20 100644
--- a/net/mac80211/tx.c
+++ b/net/mac80211/tx.c
@@ -3566,7 +3566,7 @@ bool ieee80211_schedule_txq(struct ieee80211_hw *hw,
spin_lock_bh(&local->active_txq_lock);
if (list_empty(&txqi->schedule_order)) {
- list_add_tail(&txqi->schedule_order, &local->active_txqs_new);
+ list_add_tail(&txqi->schedule_order, &local->active_txqs);
ret = true;
}
@@ -3580,35 +3580,14 @@ struct ieee80211_txq *ieee80211_next_txq(struct ieee80211_hw *hw)
{
struct ieee80211_local *local = hw_to_local(hw);
struct txq_info *txqi = NULL;
- struct list_head *head;
spin_lock_bh(&local->active_txq_lock);
-begin:
- head = &local->active_txqs_new;
- if (list_empty(head)) {
- head = &local->active_txqs_old;
- if (list_empty(head))
- goto out;
- }
-
- txqi = list_first_entry(head, struct txq_info, schedule_order);
-
- if (txqi->txq.sta) {
- struct sta_info *sta = container_of(txqi->txq.sta,
- struct sta_info, sta);
-
- spin_lock_bh(&sta->lock);
- if (sta->airtime_deficit < 0) {
- sta->airtime_deficit += IEEE80211_AIRTIME_QUANTUM;
- list_move_tail(&txqi->schedule_order,
- &local->active_txqs_old);
- spin_unlock_bh(&sta->lock);
- goto begin;
- }
- spin_unlock_bh(&sta->lock);
- }
+ if (list_empty(&local->active_txqs))
+ goto out;
+ txqi = list_first_entry(&local->active_txqs,
+ struct txq_info, schedule_order);
list_del_init(&txqi->schedule_order);
out: