linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jouni Malinen <jouni.malinen@atheros.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Luis Rodriguez <Luis.Rodriguez@Atheros.com>,
	Zhu Yi <yi.zhu@intel.com>,
	Chittajit Mitra <Chittajit.Mitra@Atheros.com>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"linville@tuxdriver.com" <linville@tuxdriver.com>,
	"ipw3945-devel@lists.sourceforge.net"
	<ipw3945-devel@lists.sourceforge.net>,
	Derek Smithies <derek@indranet.co.nz>,
	"ath9k-devel@lists.ath9k.org" <ath9k-devel@lists.ath9k.org>,
	Reinette Chatre <reinette.chatre@intel.com>
Subject: Re: [ath9k-devel] [PATCH 14/15] mac80211: move management / no-ack frame rate decision to mac80211
Date: Fri, 5 Jun 2009 10:42:58 +0300	[thread overview]
Message-ID: <1244187778.3952.2.camel@jm-desktop> (raw)
In-Reply-To: <1244183288.22576.93.camel@johannes.local>

On Thu, 2009-06-04 at 23:28 -0700, Johannes Berg wrote:
> On Fri, 2009-06-05 at 01:41 -0400, Luis R. Rodriguez wrote:
> > All rate control algorithms agree to send management and no-ack
> > frames at the lowest rate. They also agree to do this when sta
> > and the private rate control data is NULL. We move this to mac80211
> > and simplify the rate control algorithm code.
> 
> We previously thought this would constrain our rate control algorithms
> more than we would like for future experimentation with higher multicast
> rates. Has that changed?

I hope not. I don't like the idea of forcing the lowest rate in
mac80211; we should allow rate control algorithms to be improved to do
more intelligent selection of rate at least for multicast/broadcast
frames. That might also be of use for some (likely post-association)
unicast management frames since their use is going to increase in the
future.

- Jouni



  reply	other threads:[~2009-06-05  7:43 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-05  5:41 [PATCH 00/15] ath9k/mac80211/iwlwifi: rate control cleanup Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 01/15] ath9k: fix oops by downgrading assert in rc.c Luis R. Rodriguez
2009-06-05  6:30   ` Vasanthakumar Thiagarajan
2009-06-05  7:47     ` Luis R. Rodriguez
2009-06-05  7:52       ` Vasanthakumar Thiagarajan
2009-06-05 18:29         ` Luis R. Rodriguez
2009-06-06  6:55           ` Vasanth Thiagarajan
2009-06-09  0:26             ` Luis R. Rodriguez
2009-06-09  0:58               ` Luis R. Rodriguez
2009-06-09  5:24                 ` Vasanthakumar Thiagarajan
2009-06-09  5:27                   ` Vasanthakumar Thiagarajan
2009-06-09  5:35                   ` Vasanthakumar Thiagarajan
2009-06-09  5:19               ` Vasanthakumar Thiagarajan
2009-06-05 22:16   ` Gábor Stefanik
2009-06-05 22:27     ` Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 02/15] ath9k: cleanup try count for MRR in rate control Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 03/15] ath9k: remove unused min rate calculation code Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 04/15] ath9k: remove unused stepdown when looking for the next rate Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 05/15] ath9k: remove pointless wrapper ath_rc_rate_getidx() Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 06/15] ath9k: rename ath_rc_get_nextlowervalid_txrate() Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 07/15] ath9k: remove unused ath_rc_isvalid_txmask() Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 08/15] ath9k: remove ATH9K_MODE_11B Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 09/15] ath9k: remap ATH9K_MODE_* Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 10/15] ath9k: rename ath_rc_ratefind_ht() to ath_rc_get_highest_rix() Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 11/15] ath9k: remove unnecessary IEEE80211_TX_CTL_NO_ACK checks Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 12/15] mac80211: make minstrel/pid RC use ieee80211_is_data(fc) Luis R. Rodriguez
2009-06-05  6:24   ` Johannes Berg
2009-06-05  6:34     ` Luis R. Rodriguez
2009-06-05  6:52       ` Johannes Berg
2009-06-05  5:41 ` [PATCH 13/15] iwlwifi: " Luis R. Rodriguez
2009-06-05 14:56   ` Help ath5k José Luis Vargas Araoz
2009-06-05  5:41 ` [PATCH 14/15] mac80211: move management / no-ack frame rate decision to mac80211 Luis R. Rodriguez
2009-06-05  6:28   ` Johannes Berg
2009-06-05  7:42     ` Jouni Malinen [this message]
2009-06-05  7:45       ` [ath9k-devel] " Luis R. Rodriguez
2009-06-05  5:41 ` [PATCH 15/15] ath9k: remove rate control wraper Luis R. Rodriguez
2009-06-05  5:52 ` [PATCH 00/15] ath9k/mac80211/iwlwifi: rate control cleanup Luis R. Rodriguez

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=1244187778.3952.2.camel@jm-desktop \
    --to=jouni.malinen@atheros.com \
    --cc=Chittajit.Mitra@Atheros.com \
    --cc=Luis.Rodriguez@Atheros.com \
    --cc=ath9k-devel@lists.ath9k.org \
    --cc=derek@indranet.co.nz \
    --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 \
    --cc=yi.zhu@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;
as well as URLs for NNTP newsgroup(s).