Skip to content
  • Jouni Malinen's avatar
    ath9k: Revert fix to TX status reporting for retries and MCS index · 157ec876
    Jouni Malinen authored
    
    
    This patch reverts "ath9k: Fix TX status reporting for retries and MCS index"
    because that change ended up breaking ath9k rate control. While the
    MCS index reporting to mac80211 was indeed fixed by the patch, it did
    not take into account that the ath9k rate control algorithm was
    updating private tables based on this index and the index comes
    through the rate control API call, i.e., based on mac80211 TX status
    call. In addition, it looks like the "fix" to remove +1 from TX status
    'count' field was not correct based on ieee80211_tx_status()
    implementation that counts the total of count values, but starting
    from -1, not 0.
    
    The TX status reporting for frames using MCS needs to be fixed
    somehow, but it does not look like there is any easy fix for the ath9k
    rate control algorithm, so the best option now seems to be to revert the
    change and bring it back once the rate control code is cleaned up to
    handle this better.
    
    Signed-off-by: default avatarJouni Malinen <jouni.malinen@atheros.com>
    Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
    157ec876