[PATCH] iproute2: Fix no default display of ip4ip6 tunnels

[PATCH] iproute2: Fix no default display of ip4ip6 tunnels

[Srivats P]

From: Srivats P. <srivats.p@conexant.com>

"ip -6 tunnel show" displays only ip6ip6 tunnels not ip4ip6 tunnels - it should display all irrespective of proto.

This is because the default tunnel proto is initialized to IPPROTO_IPV6 in ip6_tnl_parm_init() which is fine for a 'add' command but not for 'show'. This patch overrides proto with 0 signifying 'mode any' as the default in case of a 'show'.

Signed-off-by: Srivats P. <srivats.p@conexant.com>
---

diff --git a/ip/ip6tunnel.c b/ip/ip6tunnel.c
index 8421983..0122974 100644
--- a/ip/ip6tunnel.c
+++ b/ip/ip6tunnel.c
@@ -335,6 +335,7 @@ static int do_show(int argc, char **argv)
         struct ip6_tnl_parm p;

        ip6_tnl_parm_init(&p, 0);
+       p.proto = 0;

         if (parse_args(argc, argv, &p) < 0)
                 return -1;

Conexant E-mail Firewall (Conexant.Com) made the following annotations
---------------------------------------------------------------------
********************** Legal Disclaimer **************************** 

"This email may contain confidential and privileged material for the sole use of the intended recipient. Any unauthorized review, use or distribution by others is strictly prohibited. If you have received the message in error, please advise the sender by reply email and delete the message. Thank you." 

********************************************************************** 

---------------------------------------------------------------------

Re: [PATCH] iproute2: Fix no default display of ip4ip6 tunnels

[Stephen Hemminger]

On Thu, 19 Feb 2009 12:06:33 +0530
"Srivats P" <Srivats.P@conexant.com> wrote:

> From: Srivats P. <srivats.p@conexant.com>
> 
> "ip -6 tunnel show" displays only ip6ip6 tunnels not ip4ip6 tunnels - it should display all irrespective of proto.
> 
> This is because the default tunnel proto is initialized to IPPROTO_IPV6 in ip6_tnl_parm_init() which is fine for a 'add' command but not for 'show'. This patch overrides proto with 0 signifying 'mode any' as the default in case of a 'show'.
> 
> Signed-off-by: Srivats P. <srivats.p@conexant.com>
> ---
> 
> diff --git a/ip/ip6tunnel.c b/ip/ip6tunnel.c
> index 8421983..0122974 100644
> --- a/ip/ip6tunnel.c
> +++ b/ip/ip6tunnel.c
> @@ -335,6 +335,7 @@ static int do_show(int argc, char **argv)
>          struct ip6_tnl_parm p;
> 
>         ip6_tnl_parm_init(&p, 0);
> +       p.proto = 0;
> 
>          if (parse_args(argc, argv, &p) < 0)
>                  return -1;
> 

it is possible to see all with
	ip -6 tunnel show mode all

RE: [PATCH] iproute2: Fix no default display of ip4ip6 tunnels

[Srivats P]

Stephen,

>it is possible to see all with
>	ip -6 tunnel show mode all

I guess you meant "mode any" instead of "mode all". And yes, you are right - "mode any" does display all tunnels.

However the equivalent INET4 command - "ip tunnel show" displays all tunnels (gre/ipip/sit) irrespective of mode and from a end user point of view he would probably expect the same behaviour with the INET6 tunnels without having to specify "mode any"

If you agree, I'll submit a replacement patch for my earlier one which does the same thing.

Regards,
Srivats


Conexant E-mail Firewall (Conexant.Com) made the following annotations
---------------------------------------------------------------------
********************** Legal Disclaimer **************************** 

"This email may contain confidential and privileged material for the sole use of the intended recipient. Any unauthorized review, use or distribution by others is strictly prohibited. If you have received the message in error, please advise the sender by reply email and delete the message. Thank you." 

********************************************************************** 

---------------------------------------------------------------------

RE: [PATCH] iproute2: Fix no default display of ip4ip6 tunnels

[Srivats P]

 
Stephen,

Awaiting your response.

Regards,
Srivats

-----Original Message-----
From: Srivats P 
Sent: Friday, February 20, 2009 10:20 AM
To: 'Stephen Hemminger'
Cc: netdev@vger.kernel.org
Subject: RE: [PATCH] iproute2: Fix no default display of ip4ip6 tunnels

Stephen,

>it is possible to see all with
>	ip -6 tunnel show mode all

I guess you meant "mode any" instead of "mode all". And yes, you are right - "mode any" does display all tunnels.

However the equivalent INET4 command - "ip tunnel show" displays all tunnels (gre/ipip/sit) irrespective of mode and from a end user point of view he would probably expect the same behaviour with the INET6 tunnels without having to specify "mode any"

If you agree, I'll submit a replacement patch for my earlier one which does the same thing.

Regards,
Srivats


Conexant E-mail Firewall (Conexant.Com) made the following annotations
---------------------------------------------------------------------
********************** Legal Disclaimer **************************** 

"This email may contain confidential and privileged material for the sole use of the intended recipient. Any unauthorized review, use or distribution by others is strictly prohibited. If you have received the message in error, please advise the sender by reply email and delete the message. Thank you." 

********************************************************************** 

---------------------------------------------------------------------