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
