0b96ef630b
is queued to the hardware. Because multiple concurrent paths can execute ath_start(), multiple concurrent paths can push frames into the software/hardware TX queue and since preemption/interrupting can occur, there's the possibility that a gap in time will occur between allocating the sequence number and queuing it to the hardware. Because of this, it's possible that a thread will have allocated a sequence number and then be preempted by another thread doing the same. If the second thread sneaks the frame into the BAW, the (earlier) sequence number of the first frame will be now outside the BAW and will result in the frame being constantly re-added to the tail of the queue. There it will live until the sequence numbers cycle around again. This also creates a hole in the RX BAW tracking which can also cause issues. This patch delays the sequence number allocation to occur only just before the frame is going to be added to the BAW. I've been wanting to do this anyway as part of a general code tidyup but I've not gotten around to it. This fixes the PR. However, it still makes it quite difficult to try and ensure in-order queuing and dequeuing of frames. Since multiple copies of ath_start() can be run at the same time (eg one TXing process thread, one TX completion task/one RX task) the driver may end up having frames dequeued and pushed into the hardware slightly/occasionally out of order. And, to make matters more annoying, net80211 may have the same behaviour - in the non-aggregation case, the TX code allocates sequence numbers before it's thrown to the driver. I'll open another PR to investigate this and potentially introduce some kind of final-pass TX serialisation before frames are thrown to the hardware. It's also very likely worthwhile adding some debugging code into ath(4) and net80211 to catch when/if this does occur. PR: kern/166190
130 lines
5.0 KiB
C
130 lines
5.0 KiB
C
/*-
|
|
* Copyright (c) 2002-2009 Sam Leffler, Errno Consulting
|
|
* All rights reserved.
|
|
*
|
|
* Redistribution and use in source and binary forms, with or without
|
|
* modification, are permitted provided that the following conditions
|
|
* are met:
|
|
* 1. Redistributions of source code must retain the above copyright
|
|
* notice, this list of conditions and the following disclaimer,
|
|
* without modification.
|
|
* 2. Redistributions in binary form must reproduce at minimum a disclaimer
|
|
* similar to the "NO WARRANTY" disclaimer below ("Disclaimer") and any
|
|
* redistribution must be conditioned upon including a substantially
|
|
* similar Disclaimer requirement for further binary redistribution.
|
|
*
|
|
* NO WARRANTY
|
|
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
|
|
* ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
|
|
* LIMITED TO, THE IMPLIED WARRANTIES OF NONINFRINGEMENT, MERCHANTIBILITY
|
|
* AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
|
|
* THE COPYRIGHT HOLDERS OR CONTRIBUTORS BE LIABLE FOR SPECIAL, EXEMPLARY,
|
|
* OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
|
|
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
|
|
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER
|
|
* IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
|
|
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
|
|
* THE POSSIBILITY OF SUCH DAMAGES.
|
|
*
|
|
* $FreeBSD$
|
|
*/
|
|
#ifndef __IF_ATH_TX_H__
|
|
#define __IF_ATH_TX_H__
|
|
|
|
/*
|
|
* some general macros
|
|
*/
|
|
#define INCR(_l, _sz) (_l) ++; (_l) &= ((_sz) - 1)
|
|
/*
|
|
* return block-ack bitmap index given sequence and starting sequence
|
|
*/
|
|
#define ATH_BA_INDEX(_st, _seq) (((_seq) - (_st)) & (IEEE80211_SEQ_RANGE - 1))
|
|
|
|
#define WME_BA_BMP_SIZE 64
|
|
#define WME_MAX_BA WME_BA_BMP_SIZE
|
|
|
|
/*
|
|
* How 'busy' to try and keep the hardware txq
|
|
*/
|
|
#define ATH_AGGR_MIN_QDEPTH 2
|
|
|
|
/*
|
|
* Watermark for scheduling TIDs in order to maximise aggregation.
|
|
*
|
|
* If hwq_depth is greater than this, don't schedule the TID
|
|
* for packet scheduling - the hardware is already busy servicing
|
|
* this TID.
|
|
*
|
|
* If hwq_depth is less than this, schedule the TID for packet
|
|
* scheduling in the completion handler.
|
|
*/
|
|
#define ATH_AGGR_SCHED_HIGH 4
|
|
#define ATH_AGGR_SCHED_LOW 2
|
|
|
|
/*
|
|
* return whether a bit at index _n in bitmap _bm is set
|
|
* _sz is the size of the bitmap
|
|
*/
|
|
#define ATH_BA_ISSET(_bm, _n) (((_n) < (WME_BA_BMP_SIZE)) && \
|
|
((_bm)[(_n) >> 5] & (1 << ((_n) & 31))))
|
|
|
|
|
|
/* extracting the seqno from buffer seqno */
|
|
#define SEQNO(_a) ((_a) >> IEEE80211_SEQ_SEQ_SHIFT)
|
|
|
|
/*
|
|
* Whether the current sequence number is within the
|
|
* BAW.
|
|
*/
|
|
#define BAW_WITHIN(_start, _bawsz, _seqno) \
|
|
((((_seqno) - (_start)) & 4095) < (_bawsz))
|
|
|
|
extern void ath_txq_restart_dma(struct ath_softc *sc, struct ath_txq *txq);
|
|
extern void ath_freetx(struct mbuf *m);
|
|
extern void ath_tx_node_flush(struct ath_softc *sc, struct ath_node *an);
|
|
extern void ath_tx_txq_drain(struct ath_softc *sc, struct ath_txq *txq);
|
|
extern void ath_txfrag_cleanup(struct ath_softc *sc, ath_bufhead *frags,
|
|
struct ieee80211_node *ni);
|
|
extern int ath_txfrag_setup(struct ath_softc *sc, ath_bufhead *frags,
|
|
struct mbuf *m0, struct ieee80211_node *ni);
|
|
extern int ath_tx_start(struct ath_softc *sc, struct ieee80211_node *ni,
|
|
struct ath_buf *bf, struct mbuf *m0);
|
|
extern int ath_raw_xmit(struct ieee80211_node *ni, struct mbuf *m,
|
|
const struct ieee80211_bpf_params *params);
|
|
|
|
/* software queue stuff */
|
|
extern void ath_tx_swq(struct ath_softc *sc, struct ieee80211_node *ni,
|
|
struct ath_txq *txq, struct ath_buf *bf);
|
|
extern void ath_tx_tid_init(struct ath_softc *sc, struct ath_node *an);
|
|
extern void ath_tx_tid_hw_queue_aggr(struct ath_softc *sc, struct ath_node *an,
|
|
struct ath_tid *tid);
|
|
extern void ath_tx_tid_hw_queue_norm(struct ath_softc *sc, struct ath_node *an,
|
|
struct ath_tid *tid);
|
|
extern void ath_txq_sched(struct ath_softc *sc, struct ath_txq *txq);
|
|
extern void ath_tx_normal_comp(struct ath_softc *sc, struct ath_buf *bf,
|
|
int fail);
|
|
extern void ath_tx_aggr_comp(struct ath_softc *sc, struct ath_buf *bf,
|
|
int fail);
|
|
extern void ath_tx_addto_baw(struct ath_softc *sc, struct ath_node *an,
|
|
struct ath_tid *tid, struct ath_buf *bf);
|
|
extern struct ieee80211_tx_ampdu * ath_tx_get_tx_tid(struct ath_node *an,
|
|
int tid);
|
|
extern int ath_tx_tid_seqno_assign(struct ath_softc *sc,
|
|
struct ieee80211_node *ni, struct ath_buf *bf, struct mbuf *m0);
|
|
|
|
/* TX addba handling */
|
|
extern int ath_addba_request(struct ieee80211_node *ni,
|
|
struct ieee80211_tx_ampdu *tap, int dialogtoken,
|
|
int baparamset, int batimeout);
|
|
extern int ath_addba_response(struct ieee80211_node *ni,
|
|
struct ieee80211_tx_ampdu *tap, int dialogtoken,
|
|
int code, int batimeout);
|
|
extern void ath_addba_stop(struct ieee80211_node *ni,
|
|
struct ieee80211_tx_ampdu *tap);
|
|
extern void ath_bar_response(struct ieee80211_node *ni,
|
|
struct ieee80211_tx_ampdu *tap, int status);
|
|
extern void ath_addba_response_timeout(struct ieee80211_node *ni,
|
|
struct ieee80211_tx_ampdu *tap);
|
|
|
|
#endif
|