LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [PATCH 2/2] ipvs: fix app registration in netns

To: Julian Anastasov <ja@xxxxxx>
Subject: Re: [PATCH 2/2] ipvs: fix app registration in netns
Cc: Simon Horman <horms@xxxxxxxxxxxx>, "lvs-devel@xxxxxxxxxxxxxxx" <lvs-devel@xxxxxxxxxxxxxxx>
From: Hans Schillstrom <hans.schillstrom@xxxxxxxxxxxx>
Date: Fri, 13 Apr 2012 11:47:48 +0200
Hello Julian
On Friday 13 April 2012 10:44:02 Julian Anastasov wrote:
>       Avoid crash when registering applications (ftp) when
> the IPVS core initialization for netns fails. Do this by
> checking for present core (net->ipvs) when registering apps.
> 
>       As result this gets rid of the ftp_app pointer and allows
> applications to be registered without adding fields in the
> netns_ipvs structure.
> 
>       Also, make sure applications are unregistered from the
> core automatically, should not be needed in theory.
> 

I think we can do this much simpler,
remove the auto init of ftp (ip_vs_ftp_init and ip_vs_ftp_exit) 
and let ip_vs_app call __ip_vs_ftp_init() as a we do with the protocols.

If the init fails before ip_vs_app_net_init() ftp init will not be called
and if it fails after a proper cleanup will be performed.

i.e. same design every where.

ex remove the
 ip_vs_ftp_init(void)
 ip_vs_ftp_exit(void)

-- 
Regards
Hans Schillstrom <hans.schillstrom@xxxxxxxxxxxx>
--
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

<Prev in Thread] Current Thread [Next in Thread>