LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

[lvs-devel] Fixed: [PATCH 2.6.23.1] IPVS: fixes possible real server ove

Subject: [lvs-devel] Fixed: [PATCH 2.6.23.1] IPVS: fixes possible real server overcommit on director failover.
From: rumen at voicecho.com (Rumen Bogdanovski)
Date: Thu, 01 Nov 2007 10:20:36 +0200
Simon,
send me please the formatted by you version so I can work on it
directly.

On Thu, 2007-11-01 at 17:14 +0900, Simon Horman wrote:
> On Thu, Nov 01, 2007 at 12:51:00AM +0200, Julian Anastasov wrote:
> > 
> >     Hello,
> > 
> > On Wed, 31 Oct 2007, Rumen Bogdanovski wrote:
> > 
> > > Hi Simon,
> > > Here is the patch fixed as Julian proposed. I have used the second
> > > approach he offered since the first one required function redefinings
> > > with unnecessary service parameters. I hope ip_vs_find_dest() is safe
> > > now.
> > 
> >     Yep, dest usage looks valid now, thanks! I would be
> > satisfied if patch goes uplink with coding style fixed to
> > avoid rejection at higher level.
> 
> Thanks Rumen, thanks Julian. It looks good to me too.
> I've tided up a few of the formatting problems
> (well, as I see them, we will see what the netdev people think).
> 
> As both patches now seem to be in order I will forward them
> on shortly. I'll CC the people who are on this post.
> 
> 
> Lastly, I wonder if we should have ip_vs_dest_get and ipvs_vs_dest_put
> rather than having atomic increments and decrements scattered throughout
> the code.
> 



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