From: Eric Dumazet <[EMAIL PROTECTED]>
Date: Mon, 30 Apr 2007 16:28:51 +0200

> On 64bit arches, we can speedup some IPV6 addresses compares, using 64 bits 
> fields in struct in6_addr.
> 
> I am not sure if this patch wont break some user ABI, maybe we should use 
> some ifdef(KERNEL) ?
> 
> This patch saves some space, and also reduce number of conditional branches 
> in ipv6_addr_equal()
> 
> # size vmlinux*
>    text    data     bss     dec     hex filename
> 6257978  692044  660472 7610494  74207e vmlinux
> 6259178  692044  660472 7611694  74252e vmlinux.before_patch
> 
> Note : Maybe ipv6_addr_cmp() could be defined on top of ipv6_addr_equal() ?
> 
> Signed-off-by: Eric Dumazet <[EMAIL PROTECTED]>

Unfortunately, as mentioned elsewhere, we're only able to assume
32-bit alignment of ipv6 packet headers and that isn't likely to
change any time soon.

So, for example, you patch would fill my log file with unaligned trap
messages on my sparc64 workstation.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to