[lvs-devel] Sync connection timeout patch by Andy Gospodarek - made to w

Subject: [lvs-devel] Sync connection timeout patch by Andy Gospodarek - made to work better.
From: rumen at (Rumen Bogdanovski)
Date: Wed, 31 Oct 2007 11:43:28 +0200
On Wed, 2007-10-31 at 11:35 +0200, Julian Anastasov wrote:
>       Hello,
> On Wed, 31 Oct 2007, Simon Horman wrote:
> > For reference, I have attached my version of your two patches,
> > which I have rediffed a little. Could you provide an appropriate
> > comment and signoff for the second patch once you are happy with it?
>       The ip_vs_find_dest function is not safe. dest is valid only
> while user holds reference to svc. That is how the locking works
> currently, dest reference is hidden behind svc reference.
> ip_vs_service_put must be called after ip_vs_conn_new. Another option
> is ip_vs_find_dest to use ip_vs_dest_get, a new function that increments
> dest->refcnt while we hold svc. In this case corresponding ip_vs_dest_put 
> is needed. It will just use atomic_dec(&dest->refcnt). My preference is 
> the first option.
Thank you I will fix it!

>       Off-topic: is lvs-devel archive on web?
> Regards
> --
> Julian Anastasov <ja at>

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