mirror of
https://github.com/torvalds/linux.git
synced 2024-11-07 04:32:03 +00:00
iwlwifi: remove extra statistics request from debugfs
When reading current ucode statistics information from debugfs, in current implementation, it will always send a new "statistics request" to uCode. In normal operation, uCode should report the statistics per beacon interval. Remove this extra request to reduce the additional command exchanges between driver and uCode. Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com> Signed-off-by: Reinette Chatre <reinette.chatre@intel.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
1b3eb8236a
commit
4e9772b0a9
@ -1109,16 +1109,6 @@ static ssize_t iwl_dbgfs_ucode_rx_stats_read(struct file *file,
|
|||||||
if (!iwl_is_alive(priv))
|
if (!iwl_is_alive(priv))
|
||||||
return -EAGAIN;
|
return -EAGAIN;
|
||||||
|
|
||||||
/* make request to uCode to retrieve statistics information */
|
|
||||||
mutex_lock(&priv->mutex);
|
|
||||||
ret = iwl_send_statistics_request(priv, CMD_SYNC, false);
|
|
||||||
mutex_unlock(&priv->mutex);
|
|
||||||
|
|
||||||
if (ret) {
|
|
||||||
IWL_ERR(priv,
|
|
||||||
"Error sending statistics request: %zd\n", ret);
|
|
||||||
return -EAGAIN;
|
|
||||||
}
|
|
||||||
buf = kzalloc(bufsz, GFP_KERNEL);
|
buf = kzalloc(bufsz, GFP_KERNEL);
|
||||||
if (!buf) {
|
if (!buf) {
|
||||||
IWL_ERR(priv, "Can not allocate Buffer\n");
|
IWL_ERR(priv, "Can not allocate Buffer\n");
|
||||||
@ -1552,16 +1542,6 @@ static ssize_t iwl_dbgfs_ucode_tx_stats_read(struct file *file,
|
|||||||
if (!iwl_is_alive(priv))
|
if (!iwl_is_alive(priv))
|
||||||
return -EAGAIN;
|
return -EAGAIN;
|
||||||
|
|
||||||
/* make request to uCode to retrieve statistics information */
|
|
||||||
mutex_lock(&priv->mutex);
|
|
||||||
ret = iwl_send_statistics_request(priv, CMD_SYNC, false);
|
|
||||||
mutex_unlock(&priv->mutex);
|
|
||||||
|
|
||||||
if (ret) {
|
|
||||||
IWL_ERR(priv,
|
|
||||||
"Error sending statistics request: %zd\n", ret);
|
|
||||||
return -EAGAIN;
|
|
||||||
}
|
|
||||||
buf = kzalloc(bufsz, GFP_KERNEL);
|
buf = kzalloc(bufsz, GFP_KERNEL);
|
||||||
if (!buf) {
|
if (!buf) {
|
||||||
IWL_ERR(priv, "Can not allocate Buffer\n");
|
IWL_ERR(priv, "Can not allocate Buffer\n");
|
||||||
@ -1739,16 +1719,6 @@ static ssize_t iwl_dbgfs_ucode_general_stats_read(struct file *file,
|
|||||||
if (!iwl_is_alive(priv))
|
if (!iwl_is_alive(priv))
|
||||||
return -EAGAIN;
|
return -EAGAIN;
|
||||||
|
|
||||||
/* make request to uCode to retrieve statistics information */
|
|
||||||
mutex_lock(&priv->mutex);
|
|
||||||
ret = iwl_send_statistics_request(priv, CMD_SYNC, false);
|
|
||||||
mutex_unlock(&priv->mutex);
|
|
||||||
|
|
||||||
if (ret) {
|
|
||||||
IWL_ERR(priv,
|
|
||||||
"Error sending statistics request: %zd\n", ret);
|
|
||||||
return -EAGAIN;
|
|
||||||
}
|
|
||||||
buf = kzalloc(bufsz, GFP_KERNEL);
|
buf = kzalloc(bufsz, GFP_KERNEL);
|
||||||
if (!buf) {
|
if (!buf) {
|
||||||
IWL_ERR(priv, "Can not allocate Buffer\n");
|
IWL_ERR(priv, "Can not allocate Buffer\n");
|
||||||
@ -1986,23 +1956,12 @@ static ssize_t iwl_dbgfs_tx_power_read(struct file *file,
|
|||||||
struct iwl_priv *priv = file->private_data;
|
struct iwl_priv *priv = file->private_data;
|
||||||
char buf[128];
|
char buf[128];
|
||||||
int pos = 0;
|
int pos = 0;
|
||||||
ssize_t ret;
|
|
||||||
const size_t bufsz = sizeof(buf);
|
const size_t bufsz = sizeof(buf);
|
||||||
struct statistics_tx *tx;
|
struct statistics_tx *tx;
|
||||||
|
|
||||||
if (!iwl_is_alive(priv))
|
if (!iwl_is_alive(priv))
|
||||||
pos += scnprintf(buf + pos, bufsz - pos, "N/A\n");
|
pos += scnprintf(buf + pos, bufsz - pos, "N/A\n");
|
||||||
else {
|
else {
|
||||||
/* make request to uCode to retrieve statistics information */
|
|
||||||
mutex_lock(&priv->mutex);
|
|
||||||
ret = iwl_send_statistics_request(priv, CMD_SYNC, false);
|
|
||||||
mutex_unlock(&priv->mutex);
|
|
||||||
|
|
||||||
if (ret) {
|
|
||||||
IWL_ERR(priv, "Error sending statistics request: %zd\n",
|
|
||||||
ret);
|
|
||||||
return -EAGAIN;
|
|
||||||
}
|
|
||||||
tx = &priv->statistics.tx;
|
tx = &priv->statistics.tx;
|
||||||
if (tx->tx_power.ant_a ||
|
if (tx->tx_power.ant_a ||
|
||||||
tx->tx_power.ant_b ||
|
tx->tx_power.ant_b ||
|
||||||
|
Loading…
Reference in New Issue
Block a user