|
On Tue, Oct 04, 2011 at 10:40:29AM +0300, Julian Anastasov wrote:
>
> Hello,
>
> On Tue, 4 Oct 2011, Hans Schillstrom wrote:
>
> > From: Hans Schillstrom <hans@xxxxxxxxxxxxxxx>
> >
> > ip_vs_mutext is used by both netns shutdown code and startup
> > and both implicit uses sk_lock-AF_INET mutex.
> >
> > cleanup CPU-1 startup CPU-2
> > ip_vs_dst_event() ip_vs_genl_set_cmd()
> > sk_lock-AF_INET __ip_vs_mutex
> > sk_lock-AF_INET
> > __ip_vs_mutex
> > * DEAD LOCK *
> >
> > A new mutex placed in ip_vs netns struct called sync_mutex is added.
> >
> > Comments from Julian and Simon added.
> > This patch has been running for more than 3 month now and it seems to work.
> >
> > Ver. 3
> > IP_VS_SO_GET_DAEMON in do_ip_vs_get_ctl protected by sync_mutex
> > instead of __ip_vs_mutex as sugested by Julian.
> >
> > Signed-off-by: Hans Schillstrom <hans@xxxxxxxxxxxxxxx>
>
> Looks good to me, with one extra empty line in
> start_sync_thread
>
> Acked-by: Julian Anastasov <ja@xxxxxx>
Thanks, I will apply this change.
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|
