Re: [PATCHv2] ipvs: Cleanup sync daemon code
From: Simon Horman <horms@verge.net.au>
Date: 2008-07-17 02:48:52
On Thu, Jul 17, 2008 at 01:33:44AM +0200, Sven Wegener wrote:
On Mon, 30 Jun 2008, Simon Horman wrote:quoted
On Sun, Jun 29, 2008 at 07:32:41PM +0200, Sven Wegener wrote:quoted
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-cleanupThese patches look good to me.So, who is going to take these patches? I expect them to go through davem's net-next tree, so I've rebased them onto his tree and appended your acked-by. The hton -> __constant_hton change is included. The changes 0b57664..375c6bb are available at git://git.stealer.net/linux-2.6.git stealer/ipvs/sync-daemon-cleanup-for-next
Thanks, verified, they look good to me :-) Dave, is there anything I can do to make merging these patches easier for you? -- Horms