diff mbox

[iproute2,v2] man ip-netns: Fix syntax in default ns process, indent's

Message ID 1426960318-12044-1-git-send-email-vadim4j@gmail.com
State Rejected, archived
Delegated to: stephen hemminger
Headers show

Commit Message

Vadym Kochan March 21, 2015, 5:51 p.m. UTC
From: Vadim Kochan <vadim4j@gmail.com>

Signed-off-by: Vadim Kochan <vadim4j@gmail.com>
---
v2:
    Deleted leading spaces - suggested by Ben Hutchings <ben@decadent.org.uk>

 man/man8/ip-netns.8 | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

Comments

Simon Horman March 23, 2015, 1:11 a.m. UTC | #1
On Sat, Mar 21, 2015 at 07:51:58PM +0200, Vadim Kochan wrote:
> From: Vadim Kochan <vadim4j@gmail.com>
> 
> Signed-off-by: Vadim Kochan <vadim4j@gmail.com>
> ---
> v2:
>     Deleted leading spaces - suggested by Ben Hutchings <ben@decadent.org.uk>
> 
>  man/man8/ip-netns.8 | 18 +++++++++++++-----
>  1 file changed, 13 insertions(+), 5 deletions(-)
> 
> diff --git a/man/man8/ip-netns.8 b/man/man8/ip-netns.8
> index d496675..0c3dca8 100644
> --- a/man/man8/ip-netns.8
> +++ b/man/man8/ip-netns.8
> @@ -46,7 +46,7 @@ ip-netns \- process network namespace management
>  A network namespace is logically another copy of the network stack,
>  with its own routes, firewall rules, and network devices.
>  
> -By default process inherits network namespace from its parent. Initially all
> +By default a process inherits network namespace from its parent. Initially all

Perhaps also s/inherits/inherits its/

>  the processes share the same default network namespace from the init process.
>  
>  By convention a named network namespace is an object at
> @@ -103,10 +103,18 @@ option was specified then all the network namespace names will be removed.
>  It is possible to lose the physical device when it was moved to netns and
>  then this netns was deleted with a running process:
>  
> -    $ ip netns add net0
> -    $ ip link set dev eth0 netns net0
> -    $ ip netns exec net0 SOME_PROCESS_IN_BACKGROUND
> -    $ ip netns del net0
> +.RS 10
> +$ ip netns add net0
> +.RE
> +.RS 10
> +$ ip link set dev eth0 netns net0
> +.RE
> +.RS 10
> +$ ip netns exec net0 SOME_PROCESS_IN_BACKGROUND
> +.RE
> +.RS 10
> +$ ip netns del net0
> +.RE
>  
>  and eth0 will appear in the default netns only after SOME_PROCESS_IN_BACKGROUND
>  will exit or will be killed. To prevent this the processes running in net0
> -- 
> 2.3.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Vadym Kochan March 24, 2015, 8:29 a.m. UTC | #2
On Mon, Mar 23, 2015 at 10:11:24AM +0900, Simon Horman wrote:
> On Sat, Mar 21, 2015 at 07:51:58PM +0200, Vadim Kochan wrote:
> > From: Vadim Kochan <vadim4j@gmail.com>
> > 
> > Signed-off-by: Vadim Kochan <vadim4j@gmail.com>
> > ---
> > v2:
> >     Deleted leading spaces - suggested by Ben Hutchings <ben@decadent.org.uk>
> > 
> >  man/man8/ip-netns.8 | 18 +++++++++++++-----
> >  1 file changed, 13 insertions(+), 5 deletions(-)
> > 
> > diff --git a/man/man8/ip-netns.8 b/man/man8/ip-netns.8
> > index d496675..0c3dca8 100644
> > --- a/man/man8/ip-netns.8
> > +++ b/man/man8/ip-netns.8
> > @@ -46,7 +46,7 @@ ip-netns \- process network namespace management
> >  A network namespace is logically another copy of the network stack,
> >  with its own routes, firewall rules, and network devices.
> >  
> > -By default process inherits network namespace from its parent. Initially all
> > +By default a process inherits network namespace from its parent. Initially all
> 
> Perhaps also s/inherits/inherits its/
> 
> >  the processes share the same default network namespace from the init process.
> >  
> >  By convention a named network namespace is an object at
> > @@ -103,10 +103,18 @@ option was specified then all the network namespace names will be removed.
> >  It is possible to lose the physical device when it was moved to netns and
> >  then this netns was deleted with a running process:
> >  
> > -    $ ip netns add net0
> > -    $ ip link set dev eth0 netns net0
> > -    $ ip netns exec net0 SOME_PROCESS_IN_BACKGROUND
> > -    $ ip netns del net0
> > +.RS 10
> > +$ ip netns add net0
> > +.RE
> > +.RS 10
> > +$ ip link set dev eth0 netns net0
> > +.RE
> > +.RS 10
> > +$ ip netns exec net0 SOME_PROCESS_IN_BACKGROUND
> > +.RE
> > +.RS 10
> > +$ ip netns del net0
> > +.RE
> >  
> >  and eth0 will appear in the default netns only after SOME_PROCESS_IN_BACKGROUND
> >  will exit or will be killed. To prevent this the processes running in net0
> > -- 
> > 2.3.1
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe netdev" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 

Hi,

So may it be like this ?

	By default a process inherits its network namespace from the parent.

or like this ?

	By default a process inherits its network namespace from its parent.

Thanks,
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Simon Horman March 24, 2015, 9:20 a.m. UTC | #3
Hi Vadim,

On Tue, Mar 24, 2015 at 10:29:24AM +0200, Vadim Kochan wrote:
> On Mon, Mar 23, 2015 at 10:11:24AM +0900, Simon Horman wrote:
> > On Sat, Mar 21, 2015 at 07:51:58PM +0200, Vadim Kochan wrote:

[snip]

> Hi,
> 
> So may it be like this ?
> 
> 	By default a process inherits its network namespace from the parent.
> 
> or like this ?
> 
> 	By default a process inherits its network namespace from its parent.

The second option seems best to me.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/man/man8/ip-netns.8 b/man/man8/ip-netns.8
index d496675..0c3dca8 100644
--- a/man/man8/ip-netns.8
+++ b/man/man8/ip-netns.8
@@ -46,7 +46,7 @@  ip-netns \- process network namespace management
 A network namespace is logically another copy of the network stack,
 with its own routes, firewall rules, and network devices.
 
-By default process inherits network namespace from its parent. Initially all
+By default a process inherits network namespace from its parent. Initially all
 the processes share the same default network namespace from the init process.
 
 By convention a named network namespace is an object at
@@ -103,10 +103,18 @@  option was specified then all the network namespace names will be removed.
 It is possible to lose the physical device when it was moved to netns and
 then this netns was deleted with a running process:
 
-    $ ip netns add net0
-    $ ip link set dev eth0 netns net0
-    $ ip netns exec net0 SOME_PROCESS_IN_BACKGROUND
-    $ ip netns del net0
+.RS 10
+$ ip netns add net0
+.RE
+.RS 10
+$ ip link set dev eth0 netns net0
+.RE
+.RS 10
+$ ip netns exec net0 SOME_PROCESS_IN_BACKGROUND
+.RE
+.RS 10
+$ ip netns del net0
+.RE
 
 and eth0 will appear in the default netns only after SOME_PROCESS_IN_BACKGROUND
 will exit or will be killed. To prevent this the processes running in net0