* [PATCH iproute2] ip: add support for IFA_F_DADFAILED
@ 2009-09-14 21:00 Brian Haley
2009-12-03 19:39 ` Brian Haley
0 siblings, 1 reply; 5+ messages in thread
From: Brian Haley @ 2009-09-14 21:00 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: netdev@vger.kernel.org
[I'm not sure if I needed to update if_addr.h, or if that will happen
automatically when you update the headers to 2.6.32]
Add support for IFA_F_DADFAILED and update ip.8 man page.
Signed-off-by: Brian Haley <brian.haley@hp.com>
---
diff --git a/include/linux/if_addr.h b/include/linux/if_addr.h
index 08ea980..41b0193 100644
--- a/include/linux/if_addr.h
+++ b/include/linux/if_addr.h
@@ -41,6 +41,7 @@ enum
#define IFA_F_NODAD 0x02
#define IFA_F_OPTIMISTIC 0x04
+#define IFA_F_DADFAILED 0x08
#define IFA_F_HOMEADDRESS 0x10
#define IFA_F_DEPRECATED 0x20
#define IFA_F_TENTATIVE 0x40
diff --git a/ip/ipaddress.c b/ip/ipaddress.c
index 267ecb3..32c9008 100644
--- a/ip/ipaddress.c
+++ b/ip/ipaddress.c
@@ -64,7 +64,7 @@ static void usage(void)
iplink_usage();
}
fprintf(stderr, "Usage: ip addr {add|change|replace} IFADDR dev STRING [ LIFETIME ]\n");
- fprintf(stderr, " [ CONFFLAG-LIST]\n");
+ fprintf(stderr, " [ CONFFLAG-LIST ]\n");
fprintf(stderr, " ip addr del IFADDR dev STRING\n");
fprintf(stderr, " ip addr {show|flush} [ dev STRING ] [ scope SCOPE-ID ]\n");
fprintf(stderr, " [ to PREFIX ] [ FLAG-LIST ] [ label PATTERN ]\n");
@@ -74,7 +74,8 @@ static void usage(void)
fprintf(stderr, "SCOPE-ID := [ host | link | global | NUMBER ]\n");
fprintf(stderr, "FLAG-LIST := [ FLAG-LIST ] FLAG\n");
fprintf(stderr, "FLAG := [ permanent | dynamic | secondary | primary |\n");
- fprintf(stderr, " tentative | deprecated | CONFFLAG-LIST ]\n");
+ fprintf(stderr, " tentative | deprecated | dadfailed |\n");
+ fprintf(stderr, " CONFFLAG-LIST ]\n");
fprintf(stderr, "CONFFLAG-LIST := [ CONFFLAG-LIST ] CONFFLAG\n");
fprintf(stderr, "CONFFLAG := [ home | nodad ]\n");
fprintf(stderr, "LIFETIME := [ valid_lft LFT ] [ preferred_lft LFT ]\n");
@@ -508,6 +509,10 @@ int print_addrinfo(const struct sockaddr_nl *who, struct nlmsghdr *n,
fprintf(fp, "dynamic ");
} else
ifa->ifa_flags &= ~IFA_F_PERMANENT;
+ if (ifa->ifa_flags&IFA_F_DADFAILED) {
+ ifa->ifa_flags &= ~IFA_F_DADFAILED;
+ fprintf(fp, "dadfailed ");
+ }
if (ifa->ifa_flags)
fprintf(fp, "flags %02x ", ifa->ifa_flags);
if (rta_tb[IFA_LABEL])
@@ -655,6 +660,9 @@ static int ipaddr_list_or_flush(int argc, char **argv, int flush)
} else if (strcmp(*argv, "nodad") == 0) {
filter.flags |= IFA_F_NODAD;
filter.flagmask |= IFA_F_NODAD;
+ } else if (strcmp(*argv, "dadfailed") == 0) {
+ filter.flags |= IFA_F_DADFAILED;
+ filter.flagmask |= IFA_F_DADFAILED;
} else if (strcmp(*argv, "label") == 0) {
NEXT_ARG();
filter.label = *argv;
diff --git a/man/man8/ip.8 b/man/man8/ip.8
index a8fccc4..1145b28 100644
--- a/man/man8/ip.8
+++ b/man/man8/ip.8
@@ -97,7 +97,7 @@ ip \- show / manipulate routing, devices, policy routing and tunnels
.ti -8
.IR FLAG " := "
.RB "[ " permanent " | " dynamic " | " secondary " | " primary " | "\
-tentative " | " deprecated " ]"
+tentative " | " deprecated " | " dadfailed " ]"
.ti -8
.BR "ip addrlabel" " { " add " | " del " } " prefix
@@ -1030,7 +1030,7 @@ addresses.
.TP
.B tentative
-(IPv6 only) only list addresses which did not pass duplicate
+(IPv6 only) only list addresses which have not yet passed duplicate
address detection.
.TP
@@ -1038,6 +1038,11 @@ address detection.
(IPv6 only) only list deprecated addresses.
.TP
+.B dadfailed
+(IPv6 only) only list addresses which have failed duplicate
+address detection.
+
+.TP
.BR primary " and " secondary
only list primary (or secondary) addresses.
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH iproute2] ip: add support for IFA_F_DADFAILED
2009-09-14 21:00 [PATCH iproute2] ip: add support for IFA_F_DADFAILED Brian Haley
@ 2009-12-03 19:39 ` Brian Haley
2009-12-03 20:10 ` Stephen Hemminger
0 siblings, 1 reply; 5+ messages in thread
From: Brian Haley @ 2009-12-03 19:39 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: netdev@vger.kernel.org
Hi Stephen,
Brian Haley wrote:
> [I'm not sure if I needed to update if_addr.h, or if that will happen
> automatically when you update the headers to 2.6.32]
>
>
> Add support for IFA_F_DADFAILED and update ip.8 man page.
It looks like you applied a test patch I had sent Jens instead of this
one. Can you revert that and apply this instead? Or I can send a new
one based on your current git tree.
Thanks,
-Brian
> Signed-off-by: Brian Haley <brian.haley@hp.com>
> ---
>
> diff --git a/include/linux/if_addr.h b/include/linux/if_addr.h
> index 08ea980..41b0193 100644
> --- a/include/linux/if_addr.h
> +++ b/include/linux/if_addr.h
> @@ -41,6 +41,7 @@ enum
>
> #define IFA_F_NODAD 0x02
> #define IFA_F_OPTIMISTIC 0x04
> +#define IFA_F_DADFAILED 0x08
> #define IFA_F_HOMEADDRESS 0x10
> #define IFA_F_DEPRECATED 0x20
> #define IFA_F_TENTATIVE 0x40
> diff --git a/ip/ipaddress.c b/ip/ipaddress.c
> index 267ecb3..32c9008 100644
> --- a/ip/ipaddress.c
> +++ b/ip/ipaddress.c
> @@ -64,7 +64,7 @@ static void usage(void)
> iplink_usage();
> }
> fprintf(stderr, "Usage: ip addr {add|change|replace} IFADDR dev STRING [ LIFETIME ]\n");
> - fprintf(stderr, " [ CONFFLAG-LIST]\n");
> + fprintf(stderr, " [ CONFFLAG-LIST ]\n");
> fprintf(stderr, " ip addr del IFADDR dev STRING\n");
> fprintf(stderr, " ip addr {show|flush} [ dev STRING ] [ scope SCOPE-ID ]\n");
> fprintf(stderr, " [ to PREFIX ] [ FLAG-LIST ] [ label PATTERN ]\n");
> @@ -74,7 +74,8 @@ static void usage(void)
> fprintf(stderr, "SCOPE-ID := [ host | link | global | NUMBER ]\n");
> fprintf(stderr, "FLAG-LIST := [ FLAG-LIST ] FLAG\n");
> fprintf(stderr, "FLAG := [ permanent | dynamic | secondary | primary |\n");
> - fprintf(stderr, " tentative | deprecated | CONFFLAG-LIST ]\n");
> + fprintf(stderr, " tentative | deprecated | dadfailed |\n");
> + fprintf(stderr, " CONFFLAG-LIST ]\n");
> fprintf(stderr, "CONFFLAG-LIST := [ CONFFLAG-LIST ] CONFFLAG\n");
> fprintf(stderr, "CONFFLAG := [ home | nodad ]\n");
> fprintf(stderr, "LIFETIME := [ valid_lft LFT ] [ preferred_lft LFT ]\n");
> @@ -508,6 +509,10 @@ int print_addrinfo(const struct sockaddr_nl *who, struct nlmsghdr *n,
> fprintf(fp, "dynamic ");
> } else
> ifa->ifa_flags &= ~IFA_F_PERMANENT;
> + if (ifa->ifa_flags&IFA_F_DADFAILED) {
> + ifa->ifa_flags &= ~IFA_F_DADFAILED;
> + fprintf(fp, "dadfailed ");
> + }
> if (ifa->ifa_flags)
> fprintf(fp, "flags %02x ", ifa->ifa_flags);
> if (rta_tb[IFA_LABEL])
> @@ -655,6 +660,9 @@ static int ipaddr_list_or_flush(int argc, char **argv, int flush)
> } else if (strcmp(*argv, "nodad") == 0) {
> filter.flags |= IFA_F_NODAD;
> filter.flagmask |= IFA_F_NODAD;
> + } else if (strcmp(*argv, "dadfailed") == 0) {
> + filter.flags |= IFA_F_DADFAILED;
> + filter.flagmask |= IFA_F_DADFAILED;
> } else if (strcmp(*argv, "label") == 0) {
> NEXT_ARG();
> filter.label = *argv;
> diff --git a/man/man8/ip.8 b/man/man8/ip.8
> index a8fccc4..1145b28 100644
> --- a/man/man8/ip.8
> +++ b/man/man8/ip.8
> @@ -97,7 +97,7 @@ ip \- show / manipulate routing, devices, policy routing and tunnels
> .ti -8
> .IR FLAG " := "
> .RB "[ " permanent " | " dynamic " | " secondary " | " primary " | "\
> -tentative " | " deprecated " ]"
> +tentative " | " deprecated " | " dadfailed " ]"
>
> .ti -8
> .BR "ip addrlabel" " { " add " | " del " } " prefix
> @@ -1030,7 +1030,7 @@ addresses.
>
> .TP
> .B tentative
> -(IPv6 only) only list addresses which did not pass duplicate
> +(IPv6 only) only list addresses which have not yet passed duplicate
> address detection.
>
> .TP
> @@ -1038,6 +1038,11 @@ address detection.
> (IPv6 only) only list deprecated addresses.
>
> .TP
> +.B dadfailed
> +(IPv6 only) only list addresses which have failed duplicate
> +address detection.
> +
> +.TP
> .BR primary " and " secondary
> only list primary (or secondary) addresses.
>
> --
> 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
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH iproute2] ip: add support for IFA_F_DADFAILED
2009-12-03 19:39 ` Brian Haley
@ 2009-12-03 20:10 ` Stephen Hemminger
2009-12-03 20:39 ` [iproute2]: fix " Brian Haley
0 siblings, 1 reply; 5+ messages in thread
From: Stephen Hemminger @ 2009-12-03 20:10 UTC (permalink / raw)
To: Brian Haley; +Cc: netdev@vger.kernel.org
On Thu, 03 Dec 2009 14:39:19 -0500
Brian Haley <brian.haley@hp.com> wrote:
> Hi Stephen,
>
> Brian Haley wrote:
> > [I'm not sure if I needed to update if_addr.h, or if that will happen
> > automatically when you update the headers to 2.6.32]
> >
> >
> > Add support for IFA_F_DADFAILED and update ip.8 man page.
>
> It looks like you applied a test patch I had sent Jens instead of this
> one. Can you revert that and apply this instead? Or I can send a new
> one based on your current git tree.
>
> Thanks,
Delta would be easier.
--
^ permalink raw reply [flat|nested] 5+ messages in thread
* [iproute2]: fix support for IFA_F_DADFAILED
2009-12-03 20:10 ` Stephen Hemminger
@ 2009-12-03 20:39 ` Brian Haley
2009-12-03 20:55 ` David Miller
0 siblings, 1 reply; 5+ messages in thread
From: Brian Haley @ 2009-12-03 20:39 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: netdev@vger.kernel.org
Stephen Hemminger wrote:
> Delta would be easier.
Ok, here it is.
-Brian
Fix support for IFA_F_DADFAILED and update ip.8 man page.
Signed-off-by: Brian Haley <brian.haley@hp.com>
---
diff --git a/ip/ipaddress.c b/ip/ipaddress.c
index f0add80..581722a 100644
--- a/ip/ipaddress.c
+++ b/ip/ipaddress.c
@@ -62,7 +62,7 @@ static void usage(void)
iplink_usage();
}
fprintf(stderr, "Usage: ip addr {add|change|replace} IFADDR dev STRING [ LIFETIME ]\n");
- fprintf(stderr, " [ CONFFLAG-LIST]\n");
+ fprintf(stderr, " [ CONFFLAG-LIST ]\n");
fprintf(stderr, " ip addr del IFADDR dev STRING\n");
fprintf(stderr, " ip addr {show|flush} [ dev STRING ] [ scope SCOPE-ID ]\n");
fprintf(stderr, " [ to PREFIX ] [ FLAG-LIST ] [ label PATTERN ]\n");
@@ -72,7 +72,8 @@ static void usage(void)
fprintf(stderr, "SCOPE-ID := [ host | link | global | NUMBER ]\n");
fprintf(stderr, "FLAG-LIST := [ FLAG-LIST ] FLAG\n");
fprintf(stderr, "FLAG := [ permanent | dynamic | secondary | primary |\n");
- fprintf(stderr, " tentative | deprecated | CONFFLAG-LIST ]\n");
+ fprintf(stderr, " tentative | deprecated | dadfailed |\n");
+ fprintf(stderr, " CONFFLAG-LIST ]\n");
fprintf(stderr, "CONFFLAG-LIST := [ CONFFLAG-LIST ] CONFFLAG\n");
fprintf(stderr, "CONFFLAG := [ home | nodad ]\n");
fprintf(stderr, "LIFETIME := [ valid_lft LFT ] [ preferred_lft LFT ]\n");
@@ -657,6 +658,9 @@ static int ipaddr_list_or_flush(int argc, char **argv, int flush)
} else if (strcmp(*argv, "nodad") == 0) {
filter.flags |= IFA_F_NODAD;
filter.flagmask |= IFA_F_NODAD;
+ } else if (strcmp(*argv, "dadfailed") == 0) {
+ filter.flags |= IFA_F_DADFAILED;
+ filter.flagmask |= IFA_F_DADFAILED;
} else if (strcmp(*argv, "label") == 0) {
NEXT_ARG();
filter.label = *argv;
diff --git a/man/man8/ip.8 b/man/man8/ip.8
index a8fccc4..1145b28 100644
--- a/man/man8/ip.8
+++ b/man/man8/ip.8
@@ -97,7 +97,7 @@ ip \- show / manipulate routing, devices, policy routing and tunnels
.ti -8
.IR FLAG " := "
.RB "[ " permanent " | " dynamic " | " secondary " | " primary " | "\
-tentative " | " deprecated " ]"
+tentative " | " deprecated " | " dadfailed " ]"
.ti -8
.BR "ip addrlabel" " { " add " | " del " } " prefix
@@ -1030,7 +1030,7 @@ addresses.
.TP
.B tentative
-(IPv6 only) only list addresses which did not pass duplicate
+(IPv6 only) only list addresses which have not yet passed duplicate
address detection.
.TP
@@ -1038,6 +1038,11 @@ address detection.
(IPv6 only) only list deprecated addresses.
.TP
+.B dadfailed
+(IPv6 only) only list addresses which have failed duplicate
+address detection.
+
+.TP
.BR primary " and " secondary
only list primary (or secondary) addresses.
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [iproute2]: fix support for IFA_F_DADFAILED
2009-12-03 20:39 ` [iproute2]: fix " Brian Haley
@ 2009-12-03 20:55 ` David Miller
0 siblings, 0 replies; 5+ messages in thread
From: David Miller @ 2009-12-03 20:55 UTC (permalink / raw)
To: brian.haley; +Cc: shemminger, netdev
From: Brian Haley <brian.haley@hp.com>
Date: Thu, 03 Dec 2009 15:39:36 -0500
> Stephen Hemminger wrote:
>> Delta would be easier.
>
> Ok, here it is.
This showed up in patchwork properly, FWIW.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-12-03 20:55 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-09-14 21:00 [PATCH iproute2] ip: add support for IFA_F_DADFAILED Brian Haley
2009-12-03 19:39 ` Brian Haley
2009-12-03 20:10 ` Stephen Hemminger
2009-12-03 20:39 ` [iproute2]: fix " Brian Haley
2009-12-03 20:55 ` David Miller
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).