All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jouni Malinen <jouni.malinen@atheros.com>
To: ath9k-devel@lists.ath9k.org
Subject: [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

WARNING: multiple messages have this Message-ID (diff)
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:42 UTC|newest]

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