netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 04/31 DO NOT APPLY] enable netfilter in netns
@ 2008-07-21 16:33 Alexey Dobriyan
  2008-07-21 17:01 ` Patrick McHardy
  0 siblings, 1 reply; 5+ messages in thread
From: Alexey Dobriyan @ 2008-07-21 16:33 UTC (permalink / raw)
  To: kaber
  Cc: netdev, netfilter-devel, xemul, den, ebiederm, dlezcano,
	benjamin.thery


	Only for testing.

--- a/net/netfilter/core.c
+++ b/net/netfilter/core.c
@@ -165,14 +165,6 @@ int nf_hook_slow(int pf, unsigned int hook, struct sk_buff *skb,
 	unsigned int verdict;
 	int ret = 0;
 
-#ifdef CONFIG_NET_NS
-	struct net *net;
-
-	net = indev == NULL ? dev_net(outdev) : dev_net(indev);
-	if (net != &init_net)
-		return 1;
-#endif
-
 	/* We may already have this, but read-locks nest anyway */
 	rcu_read_lock();
 
--- a/net/netfilter/nf_sockopt.c
+++ b/net/netfilter/nf_sockopt.c
@@ -65,9 +65,6 @@ static struct nf_sockopt_ops *nf_sockopt_find(struct sock *sk, int pf,
 {
 	struct nf_sockopt_ops *ops;
 
-	if (!net_eq(sock_net(sk), &init_net))
-		return ERR_PTR(-ENOPROTOOPT);
-
 	if (mutex_lock_interruptible(&nf_sockopt_mutex) != 0)
 		return ERR_PTR(-EINTR);
 
-- 
1.5.4.5



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 04/31 DO NOT APPLY] enable netfilter in netns
  2008-07-21 16:33 [PATCH 04/31 DO NOT APPLY] enable netfilter in netns Alexey Dobriyan
@ 2008-07-21 17:01 ` Patrick McHardy
  2008-07-21 17:07   ` David Miller
  2008-07-21 17:14   ` Alexey Dobriyan
  0 siblings, 2 replies; 5+ messages in thread
From: Patrick McHardy @ 2008-07-21 17:01 UTC (permalink / raw)
  To: Alexey Dobriyan
  Cc: netdev, netfilter-devel, xemul, den, ebiederm, dlezcano,
	benjamin.thery

Alexey Dobriyan wrote:
> 	Only for testing.

Thanks for posting this, I'll try to get this reviewed and
merged before the merge window closes.

Just one question since this patch is marked DO NO APPLY -
are the remaining patches meant for applying? Is there
any further work left until netfilter is fully namespace
aware?


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 04/31 DO NOT APPLY] enable netfilter in netns
  2008-07-21 17:01 ` Patrick McHardy
@ 2008-07-21 17:07   ` David Miller
  2008-07-21 17:17     ` Alexey Dobriyan
  2008-07-21 17:14   ` Alexey Dobriyan
  1 sibling, 1 reply; 5+ messages in thread
From: David Miller @ 2008-07-21 17:07 UTC (permalink / raw)
  To: kaber
  Cc: adobriyan, netdev, netfilter-devel, xemul, den, ebiederm,
	dlezcano, benjamin.thery

From: Patrick McHardy <kaber@trash.net>
Date: Mon, 21 Jul 2008 19:01:32 +0200

> Alexey Dobriyan wrote:
> > 	Only for testing.
> 
> Thanks for posting this, I'll try to get this reviewed and
> merged before the merge window closes.

For networking it closed yesterday:

--------------------
Subject: net-2.6 merged to Linus...
From: David Miller <davem@davemloft.net>
To: netdev@vger.kernel.org
Date: Sun, 20 Jul 2008 22:05:06 -0700 (PDT)
X-Mailer: Mew version 5.2 on Emacs 22.1 / Mule 5.0 (SAKAKI)


I would like to thank everyone for their work and I ask that folks
concentrate on bug fixing now.

Thanks a lot!
--------------------

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 04/31 DO NOT APPLY] enable netfilter in netns
  2008-07-21 17:01 ` Patrick McHardy
  2008-07-21 17:07   ` David Miller
@ 2008-07-21 17:14   ` Alexey Dobriyan
  1 sibling, 0 replies; 5+ messages in thread
From: Alexey Dobriyan @ 2008-07-21 17:14 UTC (permalink / raw)
  To: Patrick McHardy
  Cc: netdev, netfilter-devel, xemul, den, ebiederm, dlezcano,
	benjamin.thery

On Mon, Jul 21, 2008 at 07:01:32PM +0200, Patrick McHardy wrote:
> Alexey Dobriyan wrote:
>> 	Only for testing.
>
> Thanks for posting this, I'll try to get this reviewed and
> merged before the merge window closes.
>
> Just one question since this patch is marked DO NO APPLY -
> are the remaining patches meant for applying?

Yes!

Let's ask Den about this particular patch, though. :-)

> Is there any further work left until netfilter is fully namespace aware?

We discussed this in "unlock iptables in netns" thread
http://marc.info/?l=linux-netdev&m=121311890005554&w=2

With this patchset conntracking and NAT are largely netns-ready (I hope),
the list shrinks.

	ebtable_filter
	ebtable_nat
	selinux
	decnet
	IPv6 REJECT
	+ several individual matches/etc 

As for other stuff,
	conntrack statistics (trivial)
	ctnetlink (stubbed to init_net, dunno what it is)
	NF_LOG, NF_QUEUE (deprecated? don't bother at all?)



grep init_net -w -n -r net/netfilter/ net/ipv4/netfilter/ net/ipv6/netfilter/
shows more or less what works, and what doesn't.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 04/31 DO NOT APPLY] enable netfilter in netns
  2008-07-21 17:07   ` David Miller
@ 2008-07-21 17:17     ` Alexey Dobriyan
  0 siblings, 0 replies; 5+ messages in thread
From: Alexey Dobriyan @ 2008-07-21 17:17 UTC (permalink / raw)
  To: David Miller
  Cc: kaber, netdev, netfilter-devel, xemul, den, ebiederm, dlezcano,
	benjamin.thery

On Mon, Jul 21, 2008 at 10:07:22AM -0700, David Miller wrote:
> From: Patrick McHardy <kaber@trash.net>
> Date: Mon, 21 Jul 2008 19:01:32 +0200
> 
> > Alexey Dobriyan wrote:
> > > 	Only for testing.
> > 
> > Thanks for posting this, I'll try to get this reviewed and
> > merged before the merge window closes.
> 
> For networking it closed yesterday:

That's fine, if it waits for next one.
As they say in Russia, who understood life, doesn't hurry. :^)

> --------------------
> Subject: net-2.6 merged to Linus...
> From: David Miller <davem@davemloft.net>
> To: netdev@vger.kernel.org
> Date: Sun, 20 Jul 2008 22:05:06 -0700 (PDT)
> X-Mailer: Mew version 5.2 on Emacs 22.1 / Mule 5.0 (SAKAKI)
> 
> 
> I would like to thank everyone for their work and I ask that folks
> concentrate on bug fixing now.
> 
> Thanks a lot!
> --------------------


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2008-07-21 17:17 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-07-21 16:33 [PATCH 04/31 DO NOT APPLY] enable netfilter in netns Alexey Dobriyan
2008-07-21 17:01 ` Patrick McHardy
2008-07-21 17:07   ` David Miller
2008-07-21 17:17     ` Alexey Dobriyan
2008-07-21 17:14   ` Alexey Dobriyan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).