Qasim, While reviewing your patch I also noticed that the flags HWSIM_TX_CTL_NO_ACK and HWSIM_TX_CTL_REQ_TX_STATUS are not converted back to IEEE80211_TX_CTL_NO_ACK and IEEE80211_TX_CTL_REQ_TX_STATUS on frames received from userspace. Do you think that might affect rate adaptation or the failed average?
Cheers, Javier On Tue, Jun 5, 2012 at 9:39 AM, Javier Cardona <[email protected]> wrote: > On Mon, Jun 4, 2012 at 11:25 PM, Qasim Javed <[email protected]> wrote: >> The station fail average is not updated correctly since the >> IEEE80211_STAT_ACK flag is not set when using wmediumd with mac80211_hwsim. >> Set this flag when wmediumd indicates that the frame was successfully >> transmitted (eventually). >> >> Signed-off-by: Qasim Javed <[email protected]> > Signed-off-by: Javier Cardona <[email protected]> > >> --- >> drivers/net/wireless/mac80211_hwsim.c | 1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/net/wireless/mac80211_hwsim.c >> b/drivers/net/wireless/mac80211_hwsim.c >> index 4c9336c..a0b7cfd 100644 >> --- a/drivers/net/wireless/mac80211_hwsim.c >> +++ b/drivers/net/wireless/mac80211_hwsim.c >> @@ -1555,6 +1555,7 @@ static int hwsim_tx_info_frame_received_nl(struct >> sk_buff *skb_2, >> hdr = (struct ieee80211_hdr *) skb->data; >> mac80211_hwsim_monitor_ack(data2->hw, hdr->addr2); >> } >> + txi->flags |= IEEE80211_TX_STAT_ACK; >> } >> ieee80211_tx_status_irqsafe(data2->hw, skb); >> return 0; >> -- >> 1.7.1 >> > > > > -- > Javier Cardona > cozybit Inc. > http://www.cozybit.com -- Javier Cardona cozybit Inc. http://www.cozybit.com _______________________________________________ Devel mailing list [email protected] http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel
