From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH 0/7] Final ADSL-optimizer patch series Date: Thu, 17 Apr 2008 10:10:30 -0700 Message-ID: <20080417101030.6807d577@extreme> References: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Stephen Hemminger , netdev , "David S. Miller" , Patrick McHardy To: Jesper Dangaard Brouer Return-path: Received: from mail.vyatta.com ([216.93.170.194]:51061 "EHLO mail.vyatta.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751243AbYDQRKk (ORCPT ); Thu, 17 Apr 2008 13:10:40 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Wed, 9 Apr 2008 22:57:25 +0200 (CEST) Jesper Dangaard Brouer wrote: > > This is the final part of the ADSL-optimizer patch, which is the real > "holy grail" of packet scheduling on ADSL/ATM lines. (Referring to > Wondershaper claim of the "holy grail".) > > Shaping on ADSL has always been surrounded with mystique. People > reduce and tweek the upstream bandwidth, but how much and why does it > not work all the time? With this change, the tweeking and bandwidth > waste is gone, simply specify the bandwidth you bought. > > This patch series introduces a parameter called "linklayer", which > currently supports "ethernet" and "atm". Simply, what happens is, > that the rate table is aligned for ATM cells. > > This is a general implementation for all shapers, except HFSC which > does not use rate table lookups. Earlier (around Sep.2007), Patrick > McHardy wanted to make an even more general patch, that also included > HFSC. Nothing has happened since... > > After talking with DaveM (at his house during an icehocky match), I've > come to the conclusion that we have something that works now (and has > been since Oct.2004) and we should use it! Everybody is allowed to > change and improve upon that. Its should not mean that we keep > something like this back, which will allow packet scheduling to > actually work on ADSL. One should also realize that different shapers > have different properties. > > Patrick's further improvements can use the same userspace parameter > "linklayer" to allow userspace parameter compatibility. > > This patch series is ABI (Application Binary Interface) compatible. > > See you around, > Jesper Brouer Applied but: * Used constants from atm.h, pulled in include/linux/atm.h etc. * rolled 2-7 together in one patch. All though small patches are preferred it is more important that the build doesn't break after each patch so if an API changes (in this case calc_rtable) fix all the functions that use it in one patch. * Some mailer damage, so the patches would not apply automatically, had to do hand edits.