On Mon, Nov 21, 2016 at 12:01:51AM +0100, Stefan Sperling wrote:
> On Sat, Nov 19, 2016 at 02:07:08PM -0700, Stefan Sperling wrote:
> > CVSROOT:    /cvs
> > Module name:        src
> > Changes by: [email protected]    2016/11/19 14:07:08
> > 
> > Modified files:
> >     sys/dev/pci    : if_iwm.c 
> > 
> > Log message:
> > While setting up the basic rate bitmask for iwm's firmware, if the AP does
> > not specify basic rates for either the CCK or OFDM set, add just the most
> > basic rate to that set (1 Mbit/s in case of CCK, 6 Mbit/s in case of OFDM).
> > This behaviour matches what code comments seem to imply.
> > 
> > The previous code would add all possible basic rates in such cases.
> > So if all basic rates were CCK only, the code would add all possible OFDM 
> > basic
> > rates on top. Then the firmware would send some frames at too high rates, 
> > e.g.
> > RTS frames would be sent at 24Mbit/s which is a bit risky on noisy channels.
> > 
> > ok tb@
> > 
> 
> and also ok phessler@ mpi@

I replied to the wrong commit mail here so the additional two OKs
were added in error. Sorry about that.

Reply via email to