mac80211: merge the new version of the throughput led trigger patch
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@24182 3c298f89-4303-0410-b956-a3cf2f4a3e73master
parent
5ac80d9e71
commit
9419300d8a
|
@ -14,20 +14,23 @@ device, best with blink hardware offload.
|
|||
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
|
||||
---
|
||||
v2: turn off LED when turning off radio
|
||||
v3: - use only data frames
|
||||
- fix update frequency to 1 second
|
||||
- use DIV_ROUND_UP
|
||||
|
||||
include/net/mac80211.h | 42 +++++++++++++++
|
||||
net/mac80211/ieee80211_i.h | 14 +++++
|
||||
include/net/mac80211.h | 38 ++++++++++++++
|
||||
net/mac80211/ieee80211_i.h | 13 ++++
|
||||
net/mac80211/iface.c | 1
|
||||
net/mac80211/led.c | 120 +++++++++++++++++++++++++++++++++++++++++++++
|
||||
net/mac80211/led.c | 119 +++++++++++++++++++++++++++++++++++++++++++++
|
||||
net/mac80211/led.h | 44 +++++++++++++---
|
||||
net/mac80211/rx.c | 1
|
||||
net/mac80211/tx.c | 1
|
||||
net/mac80211/rx.c | 3 +
|
||||
net/mac80211/tx.c | 3 +
|
||||
net/mac80211/util.c | 2
|
||||
8 files changed, 216 insertions(+), 9 deletions(-)
|
||||
8 files changed, 214 insertions(+), 9 deletions(-)
|
||||
|
||||
--- a/include/net/mac80211.h
|
||||
+++ b/include/net/mac80211.h
|
||||
@@ -1849,11 +1849,27 @@ struct ieee80211_hw *ieee80211_alloc_hw(
|
||||
@@ -1849,11 +1849,26 @@ struct ieee80211_hw *ieee80211_alloc_hw(
|
||||
*/
|
||||
int ieee80211_register_hw(struct ieee80211_hw *hw);
|
||||
|
||||
|
@ -49,19 +52,17 @@ v2: turn off LED when turning off radio
|
|||
extern char *__ieee80211_get_radio_led_name(struct ieee80211_hw *hw);
|
||||
+extern char *__ieee80211_create_tpt_led_trigger(
|
||||
+ struct ieee80211_hw *hw,
|
||||
+ unsigned long update_timeout,
|
||||
+ const struct ieee80211_tpt_blink *blink_table,
|
||||
+ unsigned int blink_table_len);
|
||||
#endif
|
||||
/**
|
||||
* ieee80211_get_tx_led_name - get name of TX LED
|
||||
@@ -1932,6 +1948,32 @@ static inline char *ieee80211_get_radio_
|
||||
@@ -1932,6 +1947,29 @@ static inline char *ieee80211_get_radio_
|
||||
}
|
||||
|
||||
/**
|
||||
+ * ieee80211_create_tpt_led_trigger - create throughput LED trigger
|
||||
+ * @hw: the hardware to create the trigger for
|
||||
+ * @update_timeout: the update timeout (in jiffies)
|
||||
+ * @blink_table: the blink table -- needs to be ordered by throughput
|
||||
+ * @blink_table_len: size of the blink table
|
||||
+ *
|
||||
|
@ -71,13 +72,11 @@ v2: turn off LED when turning off radio
|
|||
+ */
|
||||
+static inline char *
|
||||
+ieee80211_create_tpt_led_trigger(struct ieee80211_hw *hw,
|
||||
+ unsigned long update_timeout,
|
||||
+ const struct ieee80211_tpt_blink *blink_table,
|
||||
+ unsigned int blink_table_len)
|
||||
+{
|
||||
+#ifdef CONFIG_MAC80211_LEDS
|
||||
+ return __ieee80211_create_tpt_led_trigger(hw, update_timeout,
|
||||
+ blink_table,
|
||||
+ return __ieee80211_create_tpt_led_trigger(hw, blink_table,
|
||||
+ blink_table_len);
|
||||
+#else
|
||||
+ return NULL;
|
||||
|
@ -98,7 +97,7 @@ v2: turn off LED when turning off radio
|
|||
#include <net/ieee80211_radiotap.h>
|
||||
#include <net/cfg80211.h>
|
||||
#include <net/mac80211.h>
|
||||
@@ -636,6 +637,18 @@ enum queue_stop_reason {
|
||||
@@ -636,6 +637,17 @@ enum queue_stop_reason {
|
||||
IEEE80211_QUEUE_STOP_REASON_SKB_ADD,
|
||||
};
|
||||
|
||||
|
@ -107,7 +106,6 @@ v2: turn off LED when turning off radio
|
|||
+ char name[32];
|
||||
+ const struct ieee80211_tpt_blink *blink_table;
|
||||
+ unsigned int blink_table_len;
|
||||
+ unsigned long update_timeout;
|
||||
+ struct timer_list timer;
|
||||
+ bool running;
|
||||
+ unsigned long prev_traffic;
|
||||
|
@ -117,7 +115,7 @@ v2: turn off LED when turning off radio
|
|||
/**
|
||||
* mac80211 scan flags - currently active scan mode
|
||||
*
|
||||
@@ -849,6 +862,7 @@ struct ieee80211_local {
|
||||
@@ -849,6 +861,7 @@ struct ieee80211_local {
|
||||
#ifdef CONFIG_MAC80211_LEDS
|
||||
int tx_led_counter, rx_led_counter;
|
||||
struct led_trigger *tx_led, *rx_led, *assoc_led, *radio_led;
|
||||
|
@ -153,7 +151,7 @@ v2: turn off LED when turning off radio
|
|||
}
|
||||
|
||||
char *__ieee80211_get_radio_led_name(struct ieee80211_hw *hw)
|
||||
@@ -156,3 +168,111 @@ char *__ieee80211_get_rx_led_name(struct
|
||||
@@ -156,3 +168,110 @@ char *__ieee80211_get_rx_led_name(struct
|
||||
return local->rx_led_name;
|
||||
}
|
||||
EXPORT_SYMBOL(__ieee80211_get_rx_led_name);
|
||||
|
@ -167,7 +165,7 @@ v2: turn off LED when turning off radio
|
|||
+
|
||||
+ delta = traffic - tpt_trig->prev_traffic;
|
||||
+ tpt_trig->prev_traffic = traffic;
|
||||
+ return delta / (1024 / 8);
|
||||
+ return DIV_ROUND_UP(delta, 1024 / 8);
|
||||
+}
|
||||
+
|
||||
+static void tpt_trig_timer(unsigned long data)
|
||||
|
@ -181,7 +179,7 @@ v2: turn off LED when turning off radio
|
|||
+ if (!tpt_trig->running)
|
||||
+ return;
|
||||
+
|
||||
+ mod_timer(&tpt_trig->timer, jiffies + tpt_trig->update_timeout);
|
||||
+ mod_timer(&tpt_trig->timer, round_jiffies(jiffies + HZ));
|
||||
+
|
||||
+ tpt = tpt_trig_traffic(local, tpt_trig);
|
||||
+
|
||||
|
@ -190,7 +188,7 @@ v2: turn off LED when turning off radio
|
|||
+ off = 0;
|
||||
+
|
||||
+ for (i = tpt_trig->blink_table_len - 1; i >= 0; i--) {
|
||||
+ if (tpt >= tpt_trig->blink_table[i].throughput) {
|
||||
+ if (tpt > tpt_trig->blink_table[i].throughput) {
|
||||
+ off = tpt_trig->blink_table[i].blink_time / 2;
|
||||
+ on = tpt_trig->blink_table[i].blink_time - off;
|
||||
+ break;
|
||||
|
@ -205,7 +203,6 @@ v2: turn off LED when turning off radio
|
|||
+
|
||||
+extern char *__ieee80211_create_tpt_led_trigger(
|
||||
+ struct ieee80211_hw *hw,
|
||||
+ unsigned long update_timeout,
|
||||
+ const struct ieee80211_tpt_blink *blink_table,
|
||||
+ unsigned int blink_table_len)
|
||||
+{
|
||||
|
@ -224,7 +221,6 @@ v2: turn off LED when turning off radio
|
|||
+
|
||||
+ tpt_trig->trig.name = tpt_trig->name;
|
||||
+
|
||||
+ tpt_trig->update_timeout = update_timeout;
|
||||
+ tpt_trig->blink_table = blink_table;
|
||||
+ tpt_trig->blink_table_len = blink_table_len;
|
||||
+
|
||||
|
@ -246,7 +242,8 @@ v2: turn off LED when turning off radio
|
|||
+ /* reset traffic */
|
||||
+ tpt_trig_traffic(local, tpt_trig);
|
||||
+ tpt_trig->running = true;
|
||||
+ mod_timer(&tpt_trig->timer, jiffies + tpt_trig->update_timeout);
|
||||
+
|
||||
+ mod_timer(&tpt_trig->timer, round_jiffies(jiffies + HZ));
|
||||
+}
|
||||
+
|
||||
+void ieee80211_stop_tpt_led_trig(struct ieee80211_local *local)
|
||||
|
@ -307,19 +304,19 @@ v2: turn off LED when turning off radio
|
|||
+#endif
|
||||
+
|
||||
+static inline void
|
||||
+ieee80211_tpt_led_trig_tx(struct ieee80211_local *local, int bytes)
|
||||
+ieee80211_tpt_led_trig_tx(struct ieee80211_local *local, __le16 fc, int bytes)
|
||||
+{
|
||||
+#ifdef CONFIG_MAC80211_LEDS
|
||||
+ if (local->tpt_led_trigger)
|
||||
+ if (local->tpt_led_trigger && ieee80211_is_data(fc))
|
||||
+ local->tpt_led_trigger->tx_bytes += bytes;
|
||||
#endif
|
||||
+}
|
||||
+
|
||||
+static inline void
|
||||
+ieee80211_tpt_led_trig_rx(struct ieee80211_local *local, int bytes)
|
||||
+ieee80211_tpt_led_trig_rx(struct ieee80211_local *local, __le16 fc, int bytes)
|
||||
+{
|
||||
+#ifdef CONFIG_MAC80211_LEDS
|
||||
+ if (local->tpt_led_trigger)
|
||||
+ if (local->tpt_led_trigger && ieee80211_is_data(fc))
|
||||
+ local->tpt_led_trigger->rx_bytes += bytes;
|
||||
+#endif
|
||||
+}
|
||||
|
@ -353,21 +350,39 @@ v2: turn off LED when turning off radio
|
|||
/* add interfaces */
|
||||
--- a/net/mac80211/rx.c
|
||||
+++ b/net/mac80211/rx.c
|
||||
@@ -2873,6 +2873,7 @@ void ieee80211_rx(struct ieee80211_hw *h
|
||||
@@ -2873,6 +2873,9 @@ void ieee80211_rx(struct ieee80211_hw *h
|
||||
return;
|
||||
}
|
||||
|
||||
+ ieee80211_tpt_led_trig_rx(local, skb->len);
|
||||
+ ieee80211_tpt_led_trig_rx(local,
|
||||
+ ((struct ieee80211_hdr *)skb->data)->frame_control,
|
||||
+ skb->len);
|
||||
__ieee80211_rx_handle_packet(hw, skb);
|
||||
|
||||
rcu_read_unlock();
|
||||
--- a/net/mac80211/tx.c
|
||||
+++ b/net/mac80211/tx.c
|
||||
@@ -1344,6 +1344,7 @@ static int __ieee80211_tx(struct ieee802
|
||||
@@ -1292,6 +1292,7 @@ static int __ieee80211_tx(struct ieee802
|
||||
|
||||
while (skb) {
|
||||
int q = skb_get_queue_mapping(skb);
|
||||
+ __le16 fc;
|
||||
|
||||
spin_lock_irqsave(&local->queue_stop_reason_lock, flags);
|
||||
ret = IEEE80211_TX_OK;
|
||||
@@ -1334,6 +1335,7 @@ static int __ieee80211_tx(struct ieee802
|
||||
else
|
||||
info->control.sta = NULL;
|
||||
|
||||
+ fc = ((struct ieee80211_hdr *)skb->data)->frame_control;
|
||||
ret = drv_tx(local, skb);
|
||||
if (WARN_ON(ret != NETDEV_TX_OK && skb->len != len)) {
|
||||
dev_kfree_skb(skb);
|
||||
@@ -1344,6 +1346,7 @@ static int __ieee80211_tx(struct ieee802
|
||||
return IEEE80211_TX_AGAIN;
|
||||
}
|
||||
|
||||
+ ieee80211_tpt_led_trig_tx(local, len);
|
||||
+ ieee80211_tpt_led_trig_tx(local, fc, len);
|
||||
*skbp = skb = next;
|
||||
ieee80211_led_tx(local, 1);
|
||||
fragm = true;
|
||||
|
|
|
@ -61,7 +61,7 @@
|
|||
|
||||
--- a/drivers/net/wireless/ath/ath9k/gpio.c
|
||||
+++ b/drivers/net/wireless/ath/ath9k/gpio.c
|
||||
@@ -20,115 +20,34 @@
|
||||
@@ -20,120 +20,25 @@
|
||||
/* LED functions */
|
||||
/********************************/
|
||||
|
||||
|
@ -98,20 +98,8 @@
|
|||
- else
|
||||
- sc->sc_flags |= SC_OP_LED_ON;
|
||||
-}
|
||||
-
|
||||
+#ifdef CONFIG_MAC80211_LEDS
|
||||
+static const struct ieee80211_tpt_blink ath9k_blink[] = {
|
||||
+ { .throughput = 0 * 1024, .blink_time = 334 },
|
||||
+ { .throughput = 1 * 1024, .blink_time = 260 },
|
||||
+ { .throughput = 5 * 1024, .blink_time = 220 },
|
||||
+ { .throughput = 10 * 1024, .blink_time = 190 },
|
||||
+ { .throughput = 20 * 1024, .blink_time = 170 },
|
||||
+ { .throughput = 50 * 1024, .blink_time = 150 },
|
||||
+ { .throughput = 70 * 1024, .blink_time = 130 },
|
||||
+ { .throughput = 100 * 1024, .blink_time = 110 },
|
||||
+ { .throughput = 200 * 1024, .blink_time = 80 },
|
||||
+ { .throughput = 300 * 1024, .blink_time = 50 },
|
||||
+};
|
||||
|
||||
static void ath_led_brightness(struct led_classdev *led_cdev,
|
||||
enum led_brightness brightness)
|
||||
{
|
||||
|
@ -195,25 +183,24 @@
|
|||
}
|
||||
|
||||
void ath_init_leds(struct ath_softc *sc)
|
||||
@@ -152,48 +71,28 @@ void ath_init_leds(struct ath_softc *sc)
|
||||
{
|
||||
- char *trigger;
|
||||
int ret;
|
||||
|
||||
if (AR_SREV_9100(sc->sc_ah))
|
||||
@@ -152,48 +57,22 @@ void ath_init_leds(struct ath_softc *sc)
|
||||
/* LED off, active low */
|
||||
ath9k_hw_set_gpio(sc->sc_ah, sc->sc_ah->led_pin, 1);
|
||||
|
||||
- if (led_blink)
|
||||
- INIT_DELAYED_WORK(&sc->ath_led_blink_work, ath_led_blink_work);
|
||||
+ /*
|
||||
+ * create the tpt trigger even if led_blink is disabled, so that
|
||||
+ * user space can still make use of it
|
||||
+ */
|
||||
+ trigger = ieee80211_create_tpt_led_trigger(sc->hw, HZ, ath9k_blink,
|
||||
+ ARRAY_SIZE(ath9k_blink));
|
||||
+ if (!led_blink)
|
||||
+ trigger = ieee80211_get_radio_led_name(sc->hw);
|
||||
+ sc->led_cdev.default_trigger =
|
||||
+ ieee80211_get_radio_led_name(sc->hw);
|
||||
+
|
||||
+ snprintf(sc->led_name, sizeof(sc->led_name),
|
||||
+ "ath9k-%s", wiphy_name(sc->hw->wiphy));
|
||||
+ sc->led_cdev.name = sc->led_name;
|
||||
+ sc->led_cdev.default_trigger = trigger;
|
||||
+ sc->led_cdev.brightness_set = ath_led_brightness;
|
||||
+
|
||||
+ ret = led_classdev_register(wiphy_dev(sc->hw->wiphy), &sc->led_cdev);
|
||||
|
@ -276,3 +263,37 @@
|
|||
cancel_delayed_work_sync(&sc->tx_complete_work);
|
||||
cancel_work_sync(&sc->paprd_work);
|
||||
cancel_work_sync(&sc->hw_check_work);
|
||||
--- a/drivers/net/wireless/ath/ath9k/init.c
|
||||
+++ b/drivers/net/wireless/ath/ath9k/init.c
|
||||
@@ -136,6 +136,20 @@ static struct ieee80211_rate ath9k_legac
|
||||
RATE(540, 0x0c, 0),
|
||||
};
|
||||
|
||||
+static const struct ieee80211_tpt_blink ath9k_tpt_blink[] = {
|
||||
+ { .throughput = 0 * 1024, .blink_time = 334 },
|
||||
+ { .throughput = 1 * 1024, .blink_time = 260 },
|
||||
+ { .throughput = 5 * 1024, .blink_time = 220 },
|
||||
+ { .throughput = 10 * 1024, .blink_time = 190 },
|
||||
+ { .throughput = 20 * 1024, .blink_time = 170 },
|
||||
+ { .throughput = 50 * 1024, .blink_time = 150 },
|
||||
+ { .throughput = 70 * 1024, .blink_time = 130 },
|
||||
+ { .throughput = 100 * 1024, .blink_time = 110 },
|
||||
+ { .throughput = 200 * 1024, .blink_time = 80 },
|
||||
+ { .throughput = 300 * 1024, .blink_time = 50 },
|
||||
+};
|
||||
+
|
||||
+
|
||||
static void ath9k_deinit_softc(struct ath_softc *sc);
|
||||
|
||||
/*
|
||||
@@ -746,6 +760,10 @@ int ath9k_init_device(u16 devid, struct
|
||||
|
||||
ath9k_init_txpower_limits(sc);
|
||||
|
||||
+ /* must be initialized before ieee80211_register_hw */
|
||||
+ sc->led_cdev.default_trigger = ieee80211_create_tpt_led_trigger(sc->hw,
|
||||
+ ath9k_tpt_blink, ARRAY_SIZE(ath9k_tpt_blink));
|
||||
+
|
||||
/* Register with mac80211 */
|
||||
error = ieee80211_register_hw(hw);
|
||||
if (error)
|
|
@ -1,29 +0,0 @@
|
|||
--- a/net/mac80211/led.c
|
||||
+++ b/net/mac80211/led.c
|
||||
@@ -103,13 +103,6 @@ void ieee80211_led_init(struct ieee80211
|
||||
local->radio_led = NULL;
|
||||
}
|
||||
}
|
||||
-
|
||||
- if (local->tpt_led_trigger) {
|
||||
- if (led_trigger_register(&local->tpt_led_trigger->trig)) {
|
||||
- kfree(local->tpt_led_trigger);
|
||||
- local->tpt_led_trigger = NULL;
|
||||
- }
|
||||
- }
|
||||
}
|
||||
|
||||
void ieee80211_led_exit(struct ieee80211_local *local)
|
||||
@@ -243,6 +236,12 @@ extern char *__ieee80211_create_tpt_led_
|
||||
|
||||
local->tpt_led_trigger = tpt_trig;
|
||||
|
||||
+ if (led_trigger_register(&local->tpt_led_trigger->trig)) {
|
||||
+ kfree(local->tpt_led_trigger);
|
||||
+ local->tpt_led_trigger = NULL;
|
||||
+ return NULL;
|
||||
+ }
|
||||
+
|
||||
return tpt_trig->name;
|
||||
}
|
||||
EXPORT_SYMBOL(__ieee80211_create_tpt_led_trigger);
|
|
@ -1,29 +0,0 @@
|
|||
--- a/net/mac80211/led.c
|
||||
+++ b/net/mac80211/led.c
|
||||
@@ -194,7 +194,7 @@ static void tpt_trig_timer(unsigned long
|
||||
off = 0;
|
||||
|
||||
for (i = tpt_trig->blink_table_len - 1; i >= 0; i--) {
|
||||
- if (tpt >= tpt_trig->blink_table[i].throughput) {
|
||||
+ if (tpt > tpt_trig->blink_table[i].throughput) {
|
||||
off = tpt_trig->blink_table[i].blink_time / 2;
|
||||
on = tpt_trig->blink_table[i].blink_time - off;
|
||||
break;
|
||||
--- a/net/mac80211/rx.c
|
||||
+++ b/net/mac80211/rx.c
|
||||
@@ -1877,6 +1877,7 @@ ieee80211_rx_h_data(struct ieee80211_rx_
|
||||
msecs_to_jiffies(local->hw.conf.dynamic_ps_timeout));
|
||||
}
|
||||
|
||||
+ ieee80211_tpt_led_trig_rx(local, rx->skb->len);
|
||||
ieee80211_deliver_skb(rx);
|
||||
|
||||
return RX_QUEUED;
|
||||
@@ -2873,7 +2874,6 @@ void ieee80211_rx(struct ieee80211_hw *h
|
||||
return;
|
||||
}
|
||||
|
||||
- ieee80211_tpt_led_trig_rx(local, skb->len);
|
||||
__ieee80211_rx_handle_packet(hw, skb);
|
||||
|
||||
rcu_read_unlock();
|
Loading…
Reference in New Issue