linux-kernel - Re: [PATCH v2 2/3]mac80211: improve PID rate control mechanism by avoiding rate oscillation problem

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Mon, 12 Mar 2012 12:20:54 +1100
From:	Julian Calaby <julian.calaby@...il.com>
To:	wei <yinwei168@...il.com>
Cc:	linux-wireless@...r.kernel.org, johannes@...solutions.net,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 2/3]mac80211: improve PID rate control mechanism by
 avoiding rate oscillation problem

Hi Wei,

Again, a couple of comments

On Mon, Mar 12, 2012 at 12:10, wei <yinwei168@...il.com> wrote:
> From Wei YIN <Wei.Yin@...ta.com.au>
> Improve PID rate control mechanism by avoiding rate oscillation problem
>
> Signed-off-by: Wei YIN <Wei.Yin@...ta.com.au>
> ---
> kernel 3.3.0
> net/mac80211/rc80211_pid.h | 38 ++++++++++++++++++++++++++++++++++++++
> 1 file changed, 38 insertions(+), 0 deletions(-)

You shouldn't split this change up into separate files. When I said
"this should be in a separate patch" - I meant that the change I was
referring to was unrelated to what your patch was about - most of
these were minor cleanups or things like that. You should write
patches that do one thing and do it properly, and if you find any
places that could use work unrelated to that "one thing", those
changes should go in a separate patch.

Also, this patch looks like it adds all the data structures that the
first patch in this series would depend on - this should be part of
that patch.

Thanks,

-- 
Julian Calaby

Email: julian.calaby@...il.com
Profile: http://www.google.com/profiles/julian.calaby/
.Plan: http://sites.google.com/site/juliancalaby/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/