From: Maxim Levitsky <maximlevitsky@gmail.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Reinette Chatre <reinette.chatre@intel.com>,
linville@tuxdriver.com, linux-wireless@vger.kernel.org,
ipw3945-devel@lists.sourceforge.net
Subject: Re: [PATCH 15/16] iwlwifi: handle unicast PS buffering
Date: Thu, 19 Nov 2009 01:34:10 +0200 [thread overview]
Message-ID: <1258587250.8981.11.camel@maxim-laptop> (raw)
In-Reply-To: <1258150857.6167.28.camel@johannes.local>
On Fri, 2009-11-13 at 23:20 +0100, Johannes Berg wrote:
> On Sat, 2009-11-14 at 00:14 +0200, Maxim Levitsky wrote:
>
> > Does this cover iwl3945?
>
> No, and I don't know if this functionality is present in 3945 devices...
> Maybe, but I think station handling is somewhat different... You can
> test it by allowing AP mode, connecting a PS station like an N810 (good
> test device!) and adding its mac addr with arp _manually_, see below.
>
> > If so, AP mode would be possible?
>
> No, because multicast buffering isn't working properly and I can't
> figure out how to do it. John's patch doesn't work at all. And if you
> want to test PS you need to use arp to add the mac addr manually since
> the arps either don't or do go out etc.
I understand now.
Still let me ask if I understand correctly this:
An AP has to do following things:
1) - send beacons (with DTIM data embedded)
2) - be capable sending frames that target several different stations.
3) - buffer frames for stations that are in sleep, and buffer multicast
frames, as long as any station is sleeping.
4) - handle regular association, handshaking, etc
1) & 2) are have to be done in hardware, but IBSS requires this feature.
I have never seen my iwl3945 sending beacons, but it should do so.
4) also needs hardware support, but any card that supports injection,
will work. My iwl3945 supports injection perfectly.
So, the multicast/unicast buffering is the sole problem that forbids AP
mode, right?
Now, there is need to know if any station is in PS mode, but to enter PS
mode, station should send something, right?
If it does, why not to implement multi-cast buffering in software?
In addition to that, I think it might be OK for users to use AP mode,
even if it doesn't work with power saving, it can be ensured that all
stations aren't using this feature. (Its usually broken anyway...)
Best regards,
Maxim Levitsky
next prev parent reply other threads:[~2009-11-18 23:34 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-13 19:56 [PATCH 00/16] iwlwifi updates 11/13/2009 Reinette Chatre
2009-11-13 19:56 ` [PATCH 01/16] iwlwifi: validate enhanced tx power entry Reinette Chatre
2009-11-13 19:56 ` [PATCH 02/16] iwlwifi: disable coex until implementation ready for 6x50 Reinette Chatre
2009-11-13 19:56 ` [PATCH 03/16] iwlwifi: remove unused parameter from iwl_channel_info Reinette Chatre
2009-11-13 19:56 ` [PATCH 04/16] iwlwifi: drop non-production PCI-IDs for 6x50 series Reinette Chatre
2009-11-14 15:26 ` Gábor Stefanik
2009-11-16 14:24 ` Guy, Wey-Yi
2009-11-13 19:56 ` [PATCH 05/16] iwlwifi: remove external reference for non-exist data structure Reinette Chatre
2009-11-13 19:56 ` [PATCH 06/16] iwlwifi: update reply_statistics_cmd with 'clear' parameter Reinette Chatre
2009-11-13 19:56 ` [PATCH 07/16] iwl3945: Reset saved POWER_TABLE_CMD in "up" Reinette Chatre
2009-11-13 19:56 ` [PATCH 08/16] iwlwifi: eliminate the possible 1/2 dBm tx power loss in 6x00 & 6x50 series Reinette Chatre
2009-11-13 19:56 ` [PATCH 09/16] iwlwifi: align tx/rx statistics debugfs format Reinette Chatre
2009-11-13 19:56 ` [PATCH 10/16] iwlagn: power up device before initializing EEPROM Reinette Chatre
2009-11-13 19:56 ` [PATCH 11/16] iwlwifi: fix bugs in beacon configuration Reinette Chatre
2009-11-13 19:56 ` [PATCH 12/16] iwlwifi: make iwlwifi send beacons Reinette Chatre
2009-11-13 19:56 ` [PATCH 13/16] iwlwifi: report PS filtered status Reinette Chatre
2009-11-13 19:56 ` [PATCH 14/16] iwlwifi: add sleep_tx_count ucode station API Reinette Chatre
2009-11-13 19:56 ` [PATCH 15/16] iwlwifi: handle unicast PS buffering Reinette Chatre
2009-11-13 22:14 ` Maxim Levitsky
2009-11-13 22:20 ` Johannes Berg
2009-11-18 23:34 ` Maxim Levitsky [this message]
2009-11-18 23:46 ` Johannes Berg
2009-11-13 19:56 ` [PATCH 16/16] iwlwifi: Add comments about CSR registers Reinette Chatre
2009-11-13 20:12 ` [PATCH 00/16] iwlwifi updates 11/13/2009 Luis R. Rodriguez
2009-11-13 20:41 ` reinette chatre
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1258587250.8981.11.camel@maxim-laptop \
--to=maximlevitsky@gmail.com \
--cc=ipw3945-devel@lists.sourceforge.net \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=reinette.chatre@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox