Search Linux Wireless

Re: [PATCH 3/6] Add a function for evaluating rate changes for possible notification

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2010-10-14 at 07:24 -0700, Paul Stewart wrote:

> >> +             if (!(ifmgd->flags & IEEE80211_STA_TX_RATE_CHANGED) ||
> >> +                 bss_conf->cqm_bitrate_thold == 0)
> >> +                     continue;
> >
> > why would this work item even be running with a threshold of zero?
> 
> This would be in the case where the userspace reset the threshold back
> to zero (no longer interested in bitrate change events) but a work
> task was already scheduled it's a small window and I can remove that
> conditional if it suits.

Makes sense, I was just wondering. Maybe just add a comment?

> > However, I think all of this is really expensive. For many devices,
> > you'll effectively be triggering an evaluation of the bitrate for every
> > packet. This will work well with minstrel, since it properly uses and
> > advertises the "best throughput" rate, but not necessarily with other
> > algorithms.
> 
> I could throttle-down the execution rate of this work proc to a
> minimum periodicity, at the expense of timely information.  Is this
> along the lines of what you'd like done?

If I'd had a good idea I probably would've mentioned it. I guess that
would work, but I don't really know. I'm just worried that there is a
lot of calculation going on all the time, like all those divisions.
Those are _really_ expensive on embedded systems.

I wonder if "rate" is really the best interface for this. Since it's a
threshold event, does it matter? Maybe there should instead be events
for MCS changes and legacy rate changes so we can avoid all those
calculations?

johannes

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux