powerpc: fix csum_ipv6_magic() on little endian platforms
authorChristophe Leroy <christophe.leroy@c-s.fr>
Mon, 10 Sep 2018 06:09:04 +0000 (06:09 +0000)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 20 Sep 2018 11:12:28 +0000 (21:12 +1000)
commit85682a7e3b9c664995ad477520f917039afdc330
tree3e88a96623dba2fab54fcbff79f49e1483b548c7
parent7233b8cab39014620ac9534da11f0f3e506d8fd8
powerpc: fix csum_ipv6_magic() on little endian platforms

On little endian platforms, csum_ipv6_magic() keeps len and proto in
CPU byte order. This generates a bad results leading to ICMPv6 packets
from other hosts being dropped by powerpc64le platforms.

In order to fix this, len and proto should be converted to network
byte order ie bigendian byte order. However checksumming 0x12345678
and 0x56341278 provide the exact same result so it is enough to
rotate the sum of len and proto by 1 byte.

PPC32 only support bigendian so the fix is needed for PPC64 only

Fixes: e9c4943a107b ("powerpc: Implement csum_ipv6_magic in assembly")
Reported-by: Jianlin Shi <jishi@redhat.com>
Reported-by: Xin Long <lucien.xin@gmail.com>
Cc: <stable@vger.kernel.org> # 4.18+
Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
Tested-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/lib/checksum_64.S