diff options
| author | Johannes Berg | 2008-10-10 13:21:59 +0200 |
|---|---|---|
| committer | John W. Linville | 2008-11-01 00:00:12 +0100 |
| commit | 94778280fabdb6bc76db5509bd95859f1141385b (patch) | |
| tree | 795a20eed458dacab69a345c33467523a2a8dbe1 /net/mac80211/tx.c | |
| parent | p54: Move rx_mtu to struct bootrec_desc (diff) | |
| download | kernel-qcow2-linux-94778280fabdb6bc76db5509bd95859f1141385b.tar.gz kernel-qcow2-linux-94778280fabdb6bc76db5509bd95859f1141385b.tar.xz kernel-qcow2-linux-94778280fabdb6bc76db5509bd95859f1141385b.zip | |
mac80211: provide sequence numbers
I've come to think that not providing sequence numbers for
the normal STA mode case was a mistake, at least two drivers
now had to implement code they wouldn't otherwise need, and
I believe at76_usb and adm8211 might be broken.
This patch makes mac80211 assign a sequence number to all
those frames that need one except beacons. That means that
if a driver only implements modes that do not do beaconing
it need not worry about the sequence number.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/tx.c')
| -rw-r--r-- | net/mac80211/tx.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/net/mac80211/tx.c b/net/mac80211/tx.c index 22494bcf488a..8bcabefa86e0 100644 --- a/net/mac80211/tx.c +++ b/net/mac80211/tx.c @@ -600,8 +600,18 @@ ieee80211_tx_h_sequence(struct ieee80211_tx_data *tx) if (ieee80211_hdrlen(hdr->frame_control) < 24) return TX_CONTINUE; + /* + * Anything but QoS data that has a sequence number field + * (is long enough) gets a sequence number from the global + * counter. + */ if (!ieee80211_is_data_qos(hdr->frame_control)) { + /* driver should assign sequence number */ info->flags |= IEEE80211_TX_CTL_ASSIGN_SEQ; + /* for pure STA mode without beacons, we can do it */ + hdr->seq_ctrl = cpu_to_le16(tx->sdata->sequence_number); + tx->sdata->sequence_number += 0x10; + tx->sdata->sequence_number &= IEEE80211_SCTL_SEQ; return TX_CONTINUE; } |