Um, sorry about that subject line.
On Fri, May 09, 2008 at 03:23:19PM -0400, J. Bruce Fields wrote:
> From: J. Bruce Fields <bfields@xxxxxxxxxxxxxx>
>
> Commit f15364bd4cf8799a7677b6daeed7b67d9139d974 ("IPv6 support for NFS
> server export caches") dropped a couple spaces, rendering the output
> here difficult to read.
>
> (However note that we expect the output to be parsed only by humans, not
> machines, so this shouldn't have broken any userland software.)
>
> Signed-off-by: J. Bruce Fields <bfields@xxxxxxxxxxxxxx>
> Cc: Aurelien Charbon <aurelien.charbon@xxxxxxxx>
> Cc: Neil Brown <neilb@xxxxxxx>
> Cc: Brian Haley <brian.haley@xxxxxx>
> Cc: YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@xxxxxxxxxxxxxx>
>
> ---
> net/sunrpc/svcauth_unix.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> I just noticed this while doing some debugging at Connectathon. I'm
> undecided whether to take it seriously. Does anyone other than me use
> these /content files? I suppose it should probably go into 2.6.26 but
> not 2.6.25.x.
>
> --b.
>
> diff --git a/net/sunrpc/svcauth_unix.c b/net/sunrpc/svcauth_unix.c
> index 3f30ee6..f24800f 100644
> --- a/net/sunrpc/svcauth_unix.c
> +++ b/net/sunrpc/svcauth_unix.c
> @@ -278,7 +278,7 @@ static int ip_map_show(struct seq_file *m,
> dom = im->m_client->h.name;
>
> if (ipv6_addr_v4mapped(&addr)) {
> - seq_printf(m, "%s" NIPQUAD_FMT "%s\n",
> + seq_printf(m, "%s " NIPQUAD_FMT " %s\n",
> im->m_class,
> ntohl(addr.s6_addr32[3]) >> 24 & 0xff,
> ntohl(addr.s6_addr32[3]) >> 16 & 0xff,
> @@ -286,7 +286,7 @@ static int ip_map_show(struct seq_file *m,
> ntohl(addr.s6_addr32[3]) >> 0 & 0xff,
> dom);
> } else {
> - seq_printf(m, "%s" NIP6_FMT "%s\n",
> + seq_printf(m, "%s " NIP6_FMT " %s\n",
> im->m_class, NIP6(addr), dom);
> }
> return 0;
> --
> 1.5.5.rc1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
