RE: [PATCH v7 12/12] wifi: rtlwifi: Enable the new rtl8192du driver

[Ping-Ke Shih <pkshih@realtek.com>]

Bitterblue Smith <rtl8821cerfe2@gmail.com> wrote:
> Johannes suggested that we should set hw->wiphy->retry_{long,short}
> before ieee80211_register_hw(). So that would go in
> _rtl_init_mac80211(). That has the added benefit of making the
> true retry limits visible to userspace ("iw phy").
> 
> The problem is that setting hw->wiphy->retry_{long,short} is
> not enough. rtlwifi still gets the default retry limits of 4
> and 7, because ieee80211_register_hw() doesn't set
> hw->conf.long_frame_max_tx_count (and hw->conf.short_frame_max_tx_count).
> Johannes suggested moving this code from ieee80211_alloc_hw_nm()
> to ieee80211_register_hw():
> 
>         local->hw.conf.long_frame_max_tx_count = wiphy->retry_long;
>         local->hw.conf.short_frame_max_tx_count = wiphy->retry_short;
> 
> I didn't do this yet partly because I don't want to compile
> the entire kernel, and partly because I'm not sure how to handle
> the different retry limits for AP/IBSS mode and station mode.
> 
> Can we change hw->wiphy->retry_{long,short} any time, not just
> before ieee80211_register_hw()? If yes, what is even the point
> of hw->conf.{short,long}_frame_max_tx_count ? It would be simpler
> if we can ignore them and use hw->wiphy->retry_{long,short}
> directly.
> 
> What do y'all think?

Logically I think you can change hw->wiphy->retry_{long,short} any time,
because cfg80211/mac80211 seemingly just bypass the values to driver. 

One thing is that should we honor the values set by user space?
But we can't know if user space has set the value, right?

If user space has not set, driver wants to control this value by itself
according to AP/IBSS/station modes.
If user space has set, driver fully follows the value from user space.
Is above the behavior you want?