On Fri, 2008-10-10 at 14:45 +0200, Tomas Winkler wrote:
> On Fri, Oct 10, 2008 at 11:11 AM, Johannes Berg
> <johannes@xxxxxxxxxxxxxxxx> wrote:
> > sta_id = iwl_get_ra_sta_id(priv, hdr);
> > if (txq->sched_retry && unlikely(sta_id == IWL_INVALID_STATION)) {
> > IWL_ERROR("Station not known\n");
> > return;
> > }
> >
> > if (txq->sched_retry) {
> > [...]
> > } else {
> > [...]
> > if (index != -1) {
> > int freed = iwl_tx_queue_reclaim(priv, txq_id, index);
> > if (tid != MAX_TID_COUNT)
> > priv->stations[sta_id].tid[tid].tfds_in_queue -= freed;
> > [...]
> >
> >
> > priv->stations has 32 entries, IWL_INVALID_STATION is 255.
>
> Don't see a problem this is what iwl_find_station returns if it cannot find
The last line I quoted uses the sta_id regardless of what was returned.
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
