netdev - Re: [RFC PATCH 5/9] ipvs network name space aware

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]

Message-ID: <201010211342.42833.hans.schillstrom@ericsson.com>
Date:	Thu, 21 Oct 2010 13:42:41 +0200
From:	Hans Schillstrom <hans.schillstrom@...csson.com>
To:	Simon Horman <horms@...ge.net.au>
CC:	"lvs-devel@...r.kernel.org" <lvs-devel@...r.kernel.org>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"netfilter-devel@...r.kernel.org" <netfilter-devel@...r.kernel.org>,
	"ja@....bg" <ja@....bg>,
	"wensong@...ux-vs.org" <wensong@...ux-vs.org>,
	"daniel.lezcano@...e.fr" <daniel.lezcano@...e.fr>
Subject: Re: [RFC PATCH 5/9] ipvs network name space aware

On Thursday 21 October 2010 13:16:47 Simon Horman wrote:
> On Fri, Oct 08, 2010 at 01:17:02PM +0200, Hans Schillstrom wrote:
> > This patch just contains ip_vs_ctl
> >
> > Signed-off-by:Hans Schillstrom <hans.schillstrom@...csson.com>
> >
> > diff --git a/net/netfilter/ipvs/ip_vs_ctl.c b/net/netfilter/ipvs/ip_vs_ctl.c
> > index ca8ec8c..7e99cbc 100644
> > --- a/net/netfilter/ipvs/ip_vs_ctl.c
> > +++ b/net/netfilter/ipvs/ip_vs_ctl.c
>
> [ snip ]
>
> > @@ -2680,10 +2664,15 @@ static int ip_vs_genl_dump_services(struct sk_buff *skb,
> >  	int idx = 0, i;
> >  	int start = cb->args[0];
> >  	struct ip_vs_service *svc;
> > -
> > +	struct net *net = skb->sk->sk_net;
>
> skb->sk->sk_net needs CONFIG_NS_NET.
> Is your plan for IPVS to unconditionally depend on CONFIG_NS_NET?
No.
> It would be nice to avoid that, but I fear it will be too messy.

Crap, I missed that.
I think a couple of inlines in ip_vs.h can do the job like;

static inline struct net * ipvs_sknet(struct sock *sk)
{
#ifdef CONFIG_NS_NET
	return sk->sk_net;
#else
	return init_net;
#endif
}

static inline struct net * ipvs_devnet(struct skb *skb)
{
#ifdef CONFIG_NS_NET
	return dev_net(skb->dev);
#else
	return init_net;
#endif
}

and use it like this
	struct net *net = ipvs_sknet(skb->sk);


> > +	struct netns_ipvs *ipvs;
> > +	if (!net)
> > +		net = dev_net(dev_net(skb->dev););

		net = ipvs_devnet(skb->dev);

> > +	BUG_ON(!net);
> > +	ipvs = net->ipvs;
> >  	mutex_lock(&__ip_vs_mutex);
> >  	for (i = 0; i < IP_VS_SVC_TAB_SIZE; i++) {
> > -		list_for_each_entry(svc, &ip_vs_svc_table[i], s_list) {
> > +		list_for_each_entry(svc, &ipvs->ctl_svc_table[i], s_list) {
> >  			if (++idx <= start)
> >  				continue;
> >  			if (ip_vs_genl_dump_service(skb, svc, cb) < 0) {
>
> [ snip ]
> --


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