linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stefano Brivio <stefano.brivio@polimi.it>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: mattias.nissler@gmx.de, linville@tuxdriver.com,
	linux-wireless@vger.kernel.org
Subject: Re: [patch 4/9] mac80211: Make PID rate control algorithm the default
Date: Mon, 17 Dec 2007 01:57:48 +0100	[thread overview]
Message-ID: <20071217015748.5bb3f7fc@morte> (raw)
In-Reply-To: <1197852873.4885.2.camel@johannes.berg>

On Mon, 17 Dec 2007 01:54:33 +0100
Johannes Berg <johannes@sipsolutions.net> wrote:

> 
> > > > +What:	rc80211-simple rate control algorithm for mac80211
> > > > +When:	2.6.26
> > > > +Files:	net/mac80211/rc80211-simple.c
> > > > +Why:	This algorithm was provided for reference but always exhibited bad
> > > > +	responsiveness and performance and has some serious flaws. It has been
> > > > +	replaced by rc80211-pid.
> > > > +Who:	Stefano Brivio <stefano.brivio@polimi.it>
> > > 
> > > I don't think we really want to do this. We can't even change the
> > > algorithm anyway except by recompiling with a different default.
> > 
> > What do you suggest instead?
> 
> Just kick it out right away.

That was my original proposal, but during a recent discussion John
suggested not to remove rc80211-simple right now, as it's currently
almost working for a lot of people. John, any suggestions?


-- 
Ciao
Stefano

  reply	other threads:[~2007-12-17  1:01 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-17  0:19 [patch 0/9] Rate control rework mattias.nissler
2007-12-17  0:20 ` [patch 5/9] mac80211: Introduce rate behaviour learning algorithm mattias.nissler
2007-12-17  0:20 ` [patch 9/9] mac80211: Publish rc80211_pid parameters in debugfs mattias.nissler
     [not found] ` <20071217002056.351743954@gmx.de>
2007-12-17  0:49   ` [patch 4/9] mac80211: Make PID rate control algorithm the default Johannes Berg
2007-12-17  0:50     ` Stefano Brivio
2007-12-17  0:54       ` Johannes Berg
2007-12-17  0:57         ` Stefano Brivio [this message]
2007-12-17  1:09           ` Johannes Berg
2007-12-17  9:56             ` Mattias Nissler
2007-12-17 14:48               ` John W. Linville
2007-12-17 17:42                 ` Johannes Berg
2007-12-17  9:51       ` Mattias Nissler
2007-12-17  9:54 ` [patch 0/9] Rate control rework Mattias Nissler
2007-12-17 14:05 ` Johannes Berg
2007-12-17 20:59   ` Mattias Nissler
2007-12-18 13:18     ` Johannes Berg
     [not found] ` <20071217002056.677829348@gmx.de>
2007-12-17 16:43   ` [patch 8/9] debugfs: Revamp debugfs_create_{u,x,s}{8,16,32,64} to support signed integers Johannes Berg
2007-12-17 21:05     ` Mattias Nissler
     [not found] <20071217012517.882216322@gmx.de>
     [not found] ` <20071217012550.213561122@gmx.de>
2007-12-17 11:52   ` [patch 4/9] mac80211: Make PID rate control algorithm the default Johannes Berg
2007-12-17 21:12     ` Mattias Nissler
2007-12-17 15:02   ` John W. Linville
2007-12-17 20:06     ` Stefano Brivio
2007-12-17 20:41       ` John W. Linville
2007-12-17 21:16         ` Stefano Brivio
2007-12-18 12:49           ` Johannes Berg
2007-12-18 20:01             ` Mattias Nissler
2007-12-19 16:05               ` Johannes Berg

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=20071217015748.5bb3f7fc@morte \
    --to=stefano.brivio@polimi.it \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=mattias.nissler@gmx.de \
    /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).