[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1250198022.28285.133.camel@Joe-Laptop.home>
Date: Thu, 13 Aug 2009 14:13:42 -0700
From: Joe Perches <joe@...ches.com>
To: Chuck Lever <chuck.lever@...cle.com>
Cc: Brian Haley <brian.haley@...com>, Jens Rosenboom <jens@...one.net>,
Linux Network Developers <netdev@...r.kernel.org>
Subject: Re: [RFC] ipv6: Change %pI6 format to output compacted addresses?
On Thu, 2009-08-13 at 17:02 -0400, Chuck Lever wrote:
> On Aug 13, 2009, at 4:34 PM, Joe Perches wrote:
> > net/sunrpc/svcauth_unix.c: seq_printf(m, "%s %pI6 %s\n", im-
> > >m_class, &addr, dom);
>
> This one might be a bad example.
There are 9 of them in net
$ grep -rP --include=*.[ch] "%pI6" net | grep seq_
net/sctp/ipv6.c: seq_printf(seq, "%pI6 ", &addr->v6.sin6_addr);
net/ipv6/netfilter/nf_conntrack_l3proto_ipv6.c: return seq_printf(s, "src=%pI6 dst=%pI6 ",
net/ipv6/ip6mr.c: seq_printf(seq, "%pI6 %pI6 %-3hd",
net/netfilter/xt_hashlimit.c: return seq_printf(s, "%ld %pI6:%u->%pI6:%u %u %u %u\n",
net/netfilter/xt_recent.c: seq_printf(seq, "src=%pI6 ttl: %u last_seen: %lu oldest_pkt: %u",
net/netfilter/ipvs/ip_vs_ctl.c: seq_printf(seq, "%s [%pI6]:%04X %s ",
net/netfilter/ipvs/ip_vs_conn.c: seq_printf(seq, "%-3s %pI6 %04X %pI6 %04X %pI6 %04X %-11s %7lu\n",
net/netfilter/ipvs/ip_vs_conn.c: seq_printf(seq, "%-3s %pI6 %04X %pI6 %04X %pI6 %04X %-11s %-6s %7lu\n",
net/sunrpc/svcauth_unix.c: seq_printf(m, "%s %pI6 %s\n", im->m_class, &addr, dom);
> [ I would think user space in general should be using inet_pton(3)
> everywhere for such interfaces, so the format of these addresses
> wouldn't matter so much. Probably impossible at this point. ]
David Miller is authoritative here.
> I'm not arguing one way or the other, but it would be useful if
> someone could check exactly what the dependencies are right now. It
> seems like we're speculating a bit.
cheers, Joe
--
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
Powered by blists - more mailing lists