> +     sband = local->hw.wiphy->bands[band];
> +     bitrates = sband->bitrates;

It might be interesting to loop once here and build a bitmap (in an
unsigned long) of rates marked ERP_G. Something like

erp_rates = 0;
for (i = 0; i < sband->n_bitrates; i++)
        if (bitrates[i].flags & ...)
                erp_rates |= BIT(i);


> +     rcu_read_lock();
> +     list_for_each_entry_rcu(sta, &local->sta_list, list) {
> +             if (sdata != sta->sdata ||
> +                 sta->plink_state != NL80211_PLINK_ESTAB)
> +                     continue;
> +
> +             short_slot = false;
> +             rates = sta->sta.supp_rates[band];
> +             for_each_set_bit(i, (unsigned long *)&rates,
> +                              sizeof(rates) * BITS_PER_BYTE) {
> +                     if (bitrates[i].flags & IEEE80211_RATE_ERP_G) {
> +                             short_slot = true;
> +                             break;
> +                     }
> +             }
> +
> +             if (!short_slot)
> +                     break;

Then the inner bitrate loop here becomes just

                if (!(rates & erp_rates)) {
                        short_slot = false;
                        break;
                }

which assumes "short_slot = true" as a precondition to going into the
sta loop.

(although that does change the overall logic to enable short slot when
NO stations are present)

johannes

_______________________________________________
Devel mailing list
[email protected]
http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel

Reply via email to