if_run in hostap mode: issue with stations in the power save mode

[Alexander Zagrebin]

Hi!

On 04.02.2011 13:14:17 +0100, Bernhard Schmidt wrote:

> > Also I have to note, that it seems that other wlan drivers can has
> > this problem too: only ral's and ath's code uses
> > ieee80211_beacon_update(). 
> 
> Yeah, true.
> 
> > Also, I've found a kern/124753. One of
> > replies contains a log's fragment with many records, like following:
> >   kernel: ath0: [00:18:41:c0:06:54] power save mode on, 1 sta's in ps
> > mode kernel: ath0: [00:18:41:c0:06:54] save frame with age 0, 1 now
> > queued kernel: ath0: [00:18:41:c0:06:54] save frame with age 0, 2
> > now queued kernel: ath0: [00:18:41:c0:06:54] power save mode off, 0
> > sta's in ps mode kernel: ath0: [00:18:41:c0:06:54] flush ps queue, 2
> > packets queue But there are no records, which have to be for a PSM
> > enabled stations: When a beacon is properly updated, then we have to
> > see records about 1. TIM updating, like
> >      ieee80211_beacon_update: TIM updated, pending 1, off 0, len 1
> > 2. poll messages from a stations, like
> >      wlan0: [18:86:ac:10:4b:88] recv ps-poll, send packet, queue
> > empty
> > 
> > Thanks for your cooperation!
> 
> You mean there are missing debug messages in net80211/run?

I meant, that when stations in the PSM are handled correctly, then
the log contains a records like this:

kernel: wlan0: [18:86:ac:10:4b:88] power save mode on, 1 sta's in ps mode
kernel: wlan0: [18:86:ac:10:4b:88] save frame with age 8, 1 now queued
kernel: wlan0: ieee80211_beacon_update: TIM updated, pending 1, off 0, len 1
kernel: wlan0: [18:86:ac:10:4b:88] recv ps-poll, send packet, queue empty
kernel: wlan0: ieee80211_beacon_update: TIM updated, pending 0, off 0, len 1

But the log's fragment from the kern/124753 doesn't contain records
with the text "...TIM updated..." and "...recv ps-poll...".
I had the very similar records in the log with the unpatched if_run.c.

-- 
Alexander Zagrebin