LVS
lvs-users
Google
 
Web LinuxVirtualServer.org

Re: [PATCH] wlib scheduler (fwd)

To: Wensong Zhang <wensong@xxxxxxxxxxxx>
Subject: Re: [PATCH] wlib scheduler (fwd)
Cc: lvs-users@xxxxxxxxxxxxxxxxxxxxxx
From: Chris Caputo <ccaputo@xxxxxxx>
Date: Sun, 16 Jan 2005 21:46:45 -0800 (PST)
Hi.  Would "Weighted Least Incoming Bps" be a better name?

I am not sure how the byte rate and packet rate could be combined
together.  Would you like to see a compile time configuration hook that
determines which one is used?

I believe the divide that is done in the code is okay in the kernel since
it is integer based rather than float based.

Thanks,
Chris

On Mon, 17 Jan 2005, Wensong Zhang wrote:
> On Fri, 14 Jan 2005, Julian Anastasov wrote:
> > On Thu, 13 Jan 2005, Chris Caputo wrote:
> >> Please consider the below patches against ipvsadm-1.24 and linux kernel
> >> 2.6.10 which add a "Weighted Least Incoming Bandwidth" scheduler to LVS.
> >> 
> >> I have also put these up at:
> >> 
> >>   http://www.caputo.com/foss/ipvsadm-1.24-wlib.patch
> >>   http://www.caputo.com/foss/lvs_wlib-2.6.10.patch
> >> 
> >> Comments and suggestions welcome.  I'd like to work towards this being
> >> included in future releases if you think it worthwhile.
> > 
> >     It looks good to me. Balancing based on outbps can
> > also be a good option for NAT. I assume there is one side effect,
> > one server can accept more load in short time while its inbps are
> > still low. As alternative, a wrr algorithm with weights updated
> > (frequently enough) based on inbps/outbps can better keep the
> > load according to the weights. But it depends on the content length per
> > connection, on the estimator's update period (isn't 2 seconds too
> > high for wlib) and only the traffic stats can make one happy :)
> >
> 
> Agreed.
> 
> As for "Bandwidth" in the name of this scheduler, shall we consider to
> combine byte rate and packet rate together? different applications may
> require different caculating method. we probably need to have some
> flexibility in configuration.
> 
> BTW, there is probably no float in kernel mode, it's good to transform
> the comparison formula.
> 
> Thanks,
> 
> Wensong


<Prev in Thread] Current Thread [Next in Thread>