netdev - Re: [PATCHv2] ipvs: Cleanup sync daemon code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 17 Jul 2008 12:48:50 +1000
From:	Simon Horman <horms@...ge.net.au>
To:	Sven Wegener <sven.wegener@...aler.net>
Cc:	wensong@...ux-vs.org, ja@....bg, netdev@...r.kernel.org,
	David Miller <davem@...emloft.net>
Subject: Re: [PATCHv2] ipvs: Cleanup sync daemon code

On Thu, Jul 17, 2008 at 01:33:44AM +0200, Sven Wegener wrote:
> On Mon, 30 Jun 2008, Simon Horman wrote:
> 
> > 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.
> 
> 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

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html