lkml.org 
[lkml]   [2021]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.15 534/917] wcn36xx: Fix discarded frames due to wrong sequence number
    Date
    From: Loic Poulain <loic.poulain@linaro.org>

    [ Upstream commit 113f304dbc1627c6ec9d5329d839964095768980 ]

    The firmware is offering features such as ARP offload, for which
    firmware crafts its own (QoS)packets without waking up the host.
    Point is that the sequence numbers generated by the firmware are
    not in sync with the host mac80211 layer and can cause packets
    such as firmware ARP reponses to be dropped by the AP (too old SN).

    To fix this we need to let the firmware manages the sequence
    numbers by its own (except for QoS null frames). There is a SN
    counter for each QoS queue and one global/baseline counter for
    Non-QoS.

    Fixes: 84aff52e4f57 ("wcn36xx: Use sequence number allocated by mac80211")
    Signed-off-by: Loic Poulain <loic.poulain@linaro.org>
    Tested-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
    Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
    Link: https://lore.kernel.org/r/1635150336-18736-1-git-send-email-loic.poulain@linaro.org
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/wireless/ath/wcn36xx/txrx.c | 9 +++++++--
    1 file changed, 7 insertions(+), 2 deletions(-)

    diff --git a/drivers/net/wireless/ath/wcn36xx/txrx.c b/drivers/net/wireless/ath/wcn36xx/txrx.c
    index c0f51fa13dfa1..bbd7194c82e27 100644
    --- a/drivers/net/wireless/ath/wcn36xx/txrx.c
    +++ b/drivers/net/wireless/ath/wcn36xx/txrx.c
    @@ -344,8 +344,6 @@ static void wcn36xx_set_tx_pdu(struct wcn36xx_tx_bd *bd,
    bd->pdu.mpdu_header_off;
    bd->pdu.mpdu_len = len;
    bd->pdu.tid = tid;
    - /* Use seq number generated by mac80211 */
    - bd->pdu.bd_ssn = WCN36XX_TXBD_SSN_FILL_HOST;
    }

    static inline struct wcn36xx_vif *get_vif_by_addr(struct wcn36xx *wcn,
    @@ -442,6 +440,9 @@ static void wcn36xx_set_tx_data(struct wcn36xx_tx_bd *bd,
    tid = ieee80211_get_tid(hdr);
    /* TID->QID is one-to-one mapping */
    bd->queue_id = tid;
    + bd->pdu.bd_ssn = WCN36XX_TXBD_SSN_FILL_DPU_QOS;
    + } else {
    + bd->pdu.bd_ssn = WCN36XX_TXBD_SSN_FILL_DPU_NON_QOS;
    }

    if (info->flags & IEEE80211_TX_INTFL_DONT_ENCRYPT ||
    @@ -453,6 +454,8 @@ static void wcn36xx_set_tx_data(struct wcn36xx_tx_bd *bd,
    /* Don't use a regular queue for null packet (no ampdu) */
    bd->queue_id = WCN36XX_TX_U_WQ_ID;
    bd->bd_rate = WCN36XX_BD_RATE_CTRL;
    + if (ieee80211_is_qos_nullfunc(hdr->frame_control))
    + bd->pdu.bd_ssn = WCN36XX_TXBD_SSN_FILL_HOST;
    }

    if (bcast) {
    @@ -512,6 +515,8 @@ static void wcn36xx_set_tx_mgmt(struct wcn36xx_tx_bd *bd,
    bd->queue_id = WCN36XX_TX_U_WQ_ID;
    *vif_priv = __vif_priv;

    + bd->pdu.bd_ssn = WCN36XX_TXBD_SSN_FILL_DPU_NON_QOS;
    +
    wcn36xx_set_tx_pdu(bd,
    ieee80211_is_data_qos(hdr->frame_control) ?
    sizeof(struct ieee80211_qos_hdr) :
    --
    2.33.0


    \
     
     \ /
      Last update: 2021-11-16 00:48    [W:2.565 / U:0.168 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site