Introduce debugfs entry to select beacon tx mode to staggered or bursted mode. In staggered mode, software beacon alert will be evenly distributed for each beaconing interfaces by (beacon intval / number of vif). In bursted mode, batch of beacons will be sent out as batch at beacon interval. By default, staggered mode is selected. To select bursted mode echo 1 > /sys/kernel/debug/ieee80211/phyX/ath10k/beacon_tx_mode For staggered mode, echo 0 > /sys/kernel/debug/ieee80211/phyX/ath10k/beacon_tx_mode Signed-off-by: Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx> --- drivers/net/wireless/ath/ath10k/core.h | 1 + drivers/net/wireless/ath/ath10k/debug.c | 60 +++++++++++++++++++++++++++++++++ drivers/net/wireless/ath/ath10k/mac.c | 9 ++--- drivers/net/wireless/ath/ath10k/mac.h | 1 + 4 files changed, 67 insertions(+), 4 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/core.h b/drivers/net/wireless/ath/ath10k/core.h index 827b3d7..be85b1f 100644 --- a/drivers/net/wireless/ath/ath10k/core.h +++ b/drivers/net/wireless/ath/ath10k/core.h @@ -394,6 +394,7 @@ struct ath10k_debug { u8 htt_max_ampdu; struct ath10k_fw_crash_data *fw_crash_data; + enum wmi_beacon_gen_mode beacon_tx_mode; }; enum ath10k_state { diff --git a/drivers/net/wireless/ath/ath10k/debug.c b/drivers/net/wireless/ath/ath10k/debug.c index 8fa606a..de63465 100644 --- a/drivers/net/wireless/ath/ath10k/debug.c +++ b/drivers/net/wireless/ath/ath10k/debug.c @@ -24,6 +24,7 @@ #include "debug.h" #include "hif.h" #include "wmi-ops.h" +#include "mac.h" /* ms */ #define ATH10K_DEBUG_HTT_STATS_INTERVAL 1000 @@ -1885,6 +1886,7 @@ int ath10k_debug_start(struct ath10k *ar) ath10k_warn(ar, "cal period cfg failed from debug start: %d\n", ret); } + ar->debug.beacon_tx_mode = WMI_BEACON_STAGGERED_MODE; return ret; } @@ -2092,6 +2094,61 @@ static const struct file_operations fops_quiet_period = { .open = simple_open }; +static ssize_t ath10k_write_beacon_tx_mode(struct file *file, + const char __user *ubuf, + size_t count, loff_t *ppos) +{ + struct ath10k *ar = file->private_data; + u32 beacon_tx_mode; + int ret; + + if (kstrtouint_from_user(ubuf, count, 0, &beacon_tx_mode)) + return -EINVAL; + + if (beacon_tx_mode > WMI_BEACON_BURST_MODE) { + ath10k_warn(ar, "Invalid beaconing mode configured: %d\n", + beacon_tx_mode); + return -EINVAL; + } + mutex_lock(&ar->conf_mutex); + ret = ath10k_wmi_pdev_set_param(ar, ar->wmi.pdev_param->beacon_tx_mode, + beacon_tx_mode); + if (ret) { + ath10k_warn(ar, "failed to set beacon tx mode: %d\n", ret); + mutex_unlock(&ar->conf_mutex); + return ret; + } + + ath10k_mac_chan_reconfigure(ar); + ar->debug.beacon_tx_mode = beacon_tx_mode; + mutex_unlock(&ar->conf_mutex); + + return count; +} + +static ssize_t ath10k_read_beacon_tx_mode(struct file *file, char __user *ubuf, + size_t count, loff_t *ppos) +{ + char buf[100]; + struct ath10k *ar = file->private_data; + int len = 0; + + mutex_lock(&ar->conf_mutex); + len = scnprintf(buf + len, sizeof(buf) - len, + "available beaconing modes: 0 - staggered 1 - bursted\n"); + len += scnprintf(buf + len, sizeof(buf) - len, "current mode: %s\n", + (ar->debug.beacon_tx_mode) ? "bursted" : "staggered"); + mutex_unlock(&ar->conf_mutex); + + return simple_read_from_buffer(ubuf, count, ppos, buf, len); +} + +static const struct file_operations fops_beacon_tx_mode = { + .read = ath10k_read_beacon_tx_mode, + .write = ath10k_write_beacon_tx_mode, + .open = simple_open +}; + int ath10k_debug_create(struct ath10k *ar) { ar->debug.fw_crash_data = vzalloc(sizeof(*ar->debug.fw_crash_data)); @@ -2195,6 +2252,9 @@ int ath10k_debug_register(struct ath10k *ar) debugfs_create_file("quiet_period", S_IRUGO | S_IWUSR, ar->debug.debugfs_phy, ar, &fops_quiet_period); + debugfs_create_file("beacon_tx_mode", S_IRUGO | S_IWUSR, + ar->debug.debugfs_phy, ar, &fops_beacon_tx_mode); + return 0; } diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c index 425dbe2..898b821 100644 --- a/drivers/net/wireless/ath/ath10k/mac.c +++ b/drivers/net/wireless/ath/ath10k/mac.c @@ -3969,7 +3969,7 @@ static int ath10k_config_ps(struct ath10k *ar) return ret; } -static void ath10k_mac_chan_reconfigure(struct ath10k *ar) +void ath10k_mac_chan_reconfigure(struct ath10k *ar) { struct ath10k_vif *arvif; struct cfg80211_chan_def def; @@ -4556,12 +4556,13 @@ static void ath10k_bss_info_changed(struct ieee80211_hw *hw, if (changed & BSS_CHANGED_BEACON) { ath10k_dbg(ar, ATH10K_DBG_MAC, - "vdev %d set beacon tx mode to staggered\n", - arvif->vdev_id); + "vdev %d set beacon tx mode to %s\n", + arvif->vdev_id, + ar->debug.beacon_tx_mode ? "bursted" : "staggered"); pdev_param = ar->wmi.pdev_param->beacon_tx_mode; ret = ath10k_wmi_pdev_set_param(ar, pdev_param, - WMI_BEACON_STAGGERED_MODE); + ar->debug.beacon_tx_mode); if (ret) ath10k_warn(ar, "failed to set beacon mode for vdev %d: %i\n", arvif->vdev_id, ret); diff --git a/drivers/net/wireless/ath/ath10k/mac.h b/drivers/net/wireless/ath/ath10k/mac.h index b291f06..a765ae8 100644 --- a/drivers/net/wireless/ath/ath10k/mac.h +++ b/drivers/net/wireless/ath/ath10k/mac.h @@ -74,6 +74,7 @@ void ath10k_mac_tx_lock(struct ath10k *ar, int reason); void ath10k_mac_tx_unlock(struct ath10k *ar, int reason); void ath10k_mac_vif_tx_lock(struct ath10k_vif *arvif, int reason); void ath10k_mac_vif_tx_unlock(struct ath10k_vif *arvif, int reason); +void ath10k_mac_chan_reconfigure(struct ath10k *ar); static inline struct ath10k_vif *ath10k_vif_to_arvif(struct ieee80211_vif *vif) { -- 2.4.0 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html