On Mon, 2007-09-10 at 10:09 +0800, Zhu Yi wrote: > On Fri, 2007-09-07 at 15:40 +0200, Johannes Berg wrote: > > But you need to have another place where this flag is set based on the > > equivalent mac80211 flag, so this is not necessary. > > OK. Will use mac80211's indication if mac80211 already takes care about > the "trick". If not I'd consider it a bug. But I don't see a "trick" there anyhow. What's non-standard in this behaviour? Almost every unicast packet is expected to be acknowledged. > > > BTW, the whole 802.11 qdisc hack will be removed with > > > the multiqueue supported in .24. > If nobody will work on that, we Intel will do. I remembered Tomas > promised John in this OLS ;) :) Sounds good. Do you guys have contact to whoever wrote it? ISTR it came from Intel in the first place. > It should be 31, not 32 (shift = 31 - bits). Will use your suggestion. Huh. /me double-checks. Am I doing this bit-calculation totally wrong? I thought you had a "bits"-bit number that included the sign in the bit "bits-1" (if you number 0 through "bits-1"). Then you have to shift up by "32 - bits" so that the "bits" bits including the sign live in the upper "bits" bits of the 32-bit number, and then shift down again by the same amount to do the sign extension. I wonder if there's a good way to ask gcc to do this :) > OK. Let me remove the 11N stuff in iwlwifi for .24 and then we do > another round of mac80211 11N patches submission/review later. That would be much appreciated. We really also should have rate control be able to influence that generically and lots more things that have to be figured out but that are quite hard to work with when the 802.11n drafts are not available to many. > I believe > you can do a better job than Jiri did. Thanks. I'm not sure I can live up to that though; for example I'll have to mostly disappear for the next 10 days or so to prepare for an exam. I'll be around, but not patching quite as much ;) > I thought John would take my .24 iwlwifi patch to wireless-dev GIT > directly since he just rebased the tree (at that time) and all the > commits for iwlwifi were merged into one big patch. Ah ok. > Anyway, I will > repost it explicitly for wireless-dev next time: iwlwifi driver + HT > features for the iwlwifi (or everything) branch; and iwlwifi driver > without HT features for pending-upstream branch. We can work on iwlwifi > driver to make it in .24 first and sort out HT things in .25. Sounds good. I'll take another look then. Thanks, johannes
Attachment:
signature.asc
Description: This is a digitally signed message part