Commit 5fe49a9d authored by Felix Fietkau's avatar Felix Fietkau Committed by Johannes Berg

mac80211: add ieee80211_tx_status_ext

This allows the driver to pass in struct ieee80211_tx_status directly.
Make ieee80211_tx_status_noskb a wrapper around it.

As with ieee80211_tx_status_noskb, there is no _ni variant of this call,
because it probably won't be needed.

Even if the driver won't provide any extra status info other than what's
in struct ieee80211_tx_info already, it can optimize status reporting
this way by passing in the station pointer.
Signed-off-by: default avatarFelix Fietkau <nbd@nbd.name>
[use C99 initializers]
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent eefebd31
...@@ -4213,6 +4213,23 @@ void ieee80211_get_tx_rates(struct ieee80211_vif *vif, ...@@ -4213,6 +4213,23 @@ void ieee80211_get_tx_rates(struct ieee80211_vif *vif,
void ieee80211_tx_status(struct ieee80211_hw *hw, void ieee80211_tx_status(struct ieee80211_hw *hw,
struct sk_buff *skb); struct sk_buff *skb);
/**
* ieee80211_tx_status_ext - extended transmit status callback
*
* This function can be used as a replacement for ieee80211_tx_status
* in drivers that may want to provide extra information that does not
* fit into &struct ieee80211_tx_info.
*
* Calls to this function for a single hardware must be synchronized
* against each other. Calls to this function, ieee80211_tx_status_ni()
* and ieee80211_tx_status_irqsafe() may not be mixed for a single hardware.
*
* @hw: the hardware the frame was transmitted by
* @status: tx status information
*/
void ieee80211_tx_status_ext(struct ieee80211_hw *hw,
struct ieee80211_tx_status *status);
/** /**
* ieee80211_tx_status_noskb - transmit status callback without skb * ieee80211_tx_status_noskb - transmit status callback without skb
* *
...@@ -4229,9 +4246,17 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, ...@@ -4229,9 +4246,17 @@ void ieee80211_tx_status(struct ieee80211_hw *hw,
* (NULL for multicast packets) * (NULL for multicast packets)
* @info: tx status information * @info: tx status information
*/ */
void ieee80211_tx_status_noskb(struct ieee80211_hw *hw, static inline void ieee80211_tx_status_noskb(struct ieee80211_hw *hw,
struct ieee80211_sta *sta, struct ieee80211_sta *sta,
struct ieee80211_tx_info *info); struct ieee80211_tx_info *info)
{
struct ieee80211_tx_status status = {
.sta = sta,
.info = info,
};
ieee80211_tx_status_ext(hw, &status);
}
/** /**
* ieee80211_tx_status_ni - transmit status callback (in process context) * ieee80211_tx_status_ni - transmit status callback (in process context)
......
...@@ -688,16 +688,16 @@ void ieee80211_tx_monitor(struct ieee80211_local *local, struct sk_buff *skb, ...@@ -688,16 +688,16 @@ void ieee80211_tx_monitor(struct ieee80211_local *local, struct sk_buff *skb,
dev_kfree_skb(skb); dev_kfree_skb(skb);
} }
void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) static void __ieee80211_tx_status(struct ieee80211_hw *hw,
struct ieee80211_tx_status *status)
{ {
struct sk_buff *skb = status->skb;
struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data; struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;
struct ieee80211_local *local = hw_to_local(hw); struct ieee80211_local *local = hw_to_local(hw);
struct ieee80211_tx_info *info = IEEE80211_SKB_CB(skb); struct ieee80211_tx_info *info = status->info;
struct ieee80211_tx_status status = {}; struct sta_info *sta;
__le16 fc; __le16 fc;
struct ieee80211_supported_band *sband; struct ieee80211_supported_band *sband;
struct rhlist_head *tmp;
struct sta_info *sta;
int retry_count; int retry_count;
int rates_idx; int rates_idx;
bool send_to_cooked; bool send_to_cooked;
...@@ -708,16 +708,11 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -708,16 +708,11 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
rates_idx = ieee80211_tx_get_rates(hw, info, &retry_count); rates_idx = ieee80211_tx_get_rates(hw, info, &retry_count);
rcu_read_lock();
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) { if (status->sta) {
/* skip wrong virtual interface */ sta = container_of(status->sta, struct sta_info, sta);
if (!ether_addr_equal(hdr->addr2, sta->sdata->vif.addr))
continue;
shift = ieee80211_vif_get_shift(&sta->sdata->vif); shift = ieee80211_vif_get_shift(&sta->sdata->vif);
if (info->flags & IEEE80211_TX_STATUS_EOSP) if (info->flags & IEEE80211_TX_STATUS_EOSP)
...@@ -737,7 +732,6 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -737,7 +732,6 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
* that this TX packet failed because of that. * that this TX packet failed because of that.
*/ */
ieee80211_handle_filtered_frame(local, sta, skb); ieee80211_handle_filtered_frame(local, sta, skb);
rcu_read_unlock();
return; return;
} }
...@@ -787,7 +781,6 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -787,7 +781,6 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
if (info->flags & IEEE80211_TX_STAT_TX_FILTERED) { if (info->flags & IEEE80211_TX_STAT_TX_FILTERED) {
ieee80211_handle_filtered_frame(local, sta, skb); ieee80211_handle_filtered_frame(local, sta, skb);
rcu_read_unlock();
return; return;
} else { } else {
if (!acked) if (!acked)
...@@ -803,10 +796,7 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -803,10 +796,7 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
} }
} }
status.sta = &sta->sta; rate_control_tx_status(local, sband, status);
status.skb = skb;
status.info = info;
rate_control_tx_status(local, sband, &status);
if (ieee80211_vif_is_mesh(&sta->sdata->vif)) if (ieee80211_vif_is_mesh(&sta->sdata->vif))
ieee80211s_update_metric(local, sta, skb); ieee80211s_update_metric(local, sta, skb);
...@@ -833,8 +823,6 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -833,8 +823,6 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
} }
} }
rcu_read_unlock();
ieee80211_led_tx(local); ieee80211_led_tx(local);
/* SNMP counters /* SNMP counters
...@@ -899,18 +887,50 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -899,18 +887,50 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
/* send to monitor interfaces */ /* send to monitor interfaces */
ieee80211_tx_monitor(local, skb, sband, retry_count, shift, send_to_cooked); ieee80211_tx_monitor(local, skb, sband, retry_count, shift, send_to_cooked);
} }
void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
{
struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;
struct ieee80211_local *local = hw_to_local(hw);
struct ieee80211_tx_status status = {
.skb = skb,
.info = IEEE80211_SKB_CB(skb),
};
struct rhlist_head *tmp;
struct sta_info *sta;
rcu_read_lock();
for_each_sta_info(local, hdr->addr1, sta, tmp) {
/* skip wrong virtual interface */
if (!ether_addr_equal(hdr->addr2, sta->sdata->vif.addr))
continue;
status.sta = &sta->sta;
break;
}
__ieee80211_tx_status(hw, &status);
rcu_read_unlock();
}
EXPORT_SYMBOL(ieee80211_tx_status); EXPORT_SYMBOL(ieee80211_tx_status);
void ieee80211_tx_status_noskb(struct ieee80211_hw *hw, void ieee80211_tx_status_ext(struct ieee80211_hw *hw,
struct ieee80211_sta *pubsta, struct ieee80211_tx_status *status)
struct ieee80211_tx_info *info)
{ {
struct ieee80211_local *local = hw_to_local(hw); struct ieee80211_local *local = hw_to_local(hw);
struct ieee80211_tx_info *info = status->info;
struct ieee80211_sta *pubsta = status->sta;
struct ieee80211_supported_band *sband; struct ieee80211_supported_band *sband;
struct ieee80211_tx_status status = {};
int retry_count; int retry_count;
bool acked, noack_success; bool acked, noack_success;
if (status->skb)
return __ieee80211_tx_status(hw, status);
if (!status->sta)
return;
ieee80211_tx_get_rates(hw, info, &retry_count); ieee80211_tx_get_rates(hw, info, &retry_count);
sband = hw->wiphy->bands[info->band]; sband = hw->wiphy->bands[info->band];
...@@ -940,9 +960,7 @@ void ieee80211_tx_status_noskb(struct ieee80211_hw *hw, ...@@ -940,9 +960,7 @@ void ieee80211_tx_status_noskb(struct ieee80211_hw *hw,
ieee80211_lost_packet(sta, info); ieee80211_lost_packet(sta, info);
} }
status.sta = pubsta; rate_control_tx_status(local, sband, status);
status.info = info;
rate_control_tx_status(local, sband, &status);
} }
if (acked || noack_success) { if (acked || noack_success) {
...@@ -957,7 +975,7 @@ void ieee80211_tx_status_noskb(struct ieee80211_hw *hw, ...@@ -957,7 +975,7 @@ void ieee80211_tx_status_noskb(struct ieee80211_hw *hw,
I802_DEBUG_INC(local->dot11FailedCount); I802_DEBUG_INC(local->dot11FailedCount);
} }
} }
EXPORT_SYMBOL(ieee80211_tx_status_noskb); EXPORT_SYMBOL(ieee80211_tx_status_ext);
void ieee80211_report_low_ack(struct ieee80211_sta *pubsta, u32 num_packets) void ieee80211_report_low_ack(struct ieee80211_sta *pubsta, u32 num_packets)
{ {
......
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