On Wednesday 13 February 2008 13:07:53 Johannes Berg wrote:
>
> > + if (queue >= ARRAY_SIZE(wl->qos_params)) {
> > + /* Queue not available or don't support setting
> > + * params on this queue. Return success to not
> > + * confuse mac80211. */
> > + return 0;
> > + }
>
> Is that really necessary? If so, I'll probably remove it when doing the
> QoS rework, it shouldn't be, imho you should return an error if mac80211
> tries to configure a queue >= hw.queues.
Yep, mac80211 walks the whole queue array without looking at hw->queues.
I think this needs fixing in mac80211.
--
Greetings Michael.
_______________________________________________
Bcm43xx-dev mailing list
Bcm43xx-dev@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
