From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Subject: Re: [PATCHv2] ipvs: Cleanup sync daemon code Date: Mon, 30 Jun 2008 14:51:56 +1000 Message-ID: <20080630045153.GA25021@verge.net.au> References: <1214760766-1526-1-git-send-email-sven.wegener@stealer.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: wensong@linux-vs.org, ja@ssi.bg, netdev@vger.kernel.org To: Sven Wegener Return-path: Received: from kirsty.vergenet.net ([202.4.237.240]:42236 "EHLO kirsty.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751344AbYF3Ev7 (ORCPT ); Mon, 30 Jun 2008 00:51:59 -0400 Content-Disposition: inline In-Reply-To: <1214760766-1526-1-git-send-email-sven.wegener@stealer.net> Sender: netdev-owner@vger.kernel.org List-ID: On Sun, Jun 29, 2008 at 07:32:41PM +0200, Sven Wegener wrote: > It's been a while since my last post of this patch series, but here we go gain. > > Changes since last post: > - Dropped the master daemon sleep change for now, that needs some more work. > - Splitted up the patch into a couple of logical chunks. > > The patches are > > Sven Wegener (5): > ipvs: Initialize mcast addr at compile time > ipvs: Use ERR_PTR for returning errors from make_receive_sock() and make_send_sock() > ipvs: Use kthread_run() instead of doing a double-fork via kernel_thread() > ipvs: Put backup thread on mcast socket wait queue > ipvs: Use schedule_timeout_interruptible() instead of msleep_interruptible() > > The overall difstat looks like > > ip_vs_sync.c | 431 +++++++++++++++++++++++------------------------------------ > 1 file changed, 172 insertions(+), 259 deletions(-) > > The patches are based on v2.6.26-rc8. If someone wants to pull them, that can > be done from > > git://git.stealer.net/linux-2.6.git stealer/ipvs/sync-daemon-cleanup These patches look good to me. -- Horms