]> git.baikalelectronics.ru Git - kernel.git/commit
crypto: cavium - Fix fallout from CONFIG_VMAP_STACK
authorJan Glauber <jglauber@cavium.com>
Mon, 9 Apr 2018 15:45:50 +0000 (17:45 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 20 Apr 2018 16:58:34 +0000 (00:58 +0800)
commit94e779fbcc509dd4c07960aa6df1b269c01bf1b9
tree74c47cac0489b75f909976fd450454d9acb59d20
parentd7a53f8ceec4bed20ede19155e61722d753ecff5
crypto: cavium - Fix fallout from CONFIG_VMAP_STACK

Enabling virtual mapped kernel stacks breaks the thunderx_zip
driver. On compression or decompression the executing CPU hangs
in an endless loop. The reason for this is the usage of __pa
by the driver which does no longer work for an address that is
not part of the 1:1 mapping.

The zip driver allocates a result struct on the stack and needs
to tell the hardware the physical address within this struct
that is used to signal the completion of the request.

As the hardware gets the wrong address after the broken __pa
conversion it writes to an arbitrary address. The zip driver then
waits forever for the completion byte to contain a non-zero value.

Allocating the result struct from 1:1 mapped memory resolves this
bug.

Signed-off-by: Jan Glauber <jglauber@cavium.com>
Reviewed-by: Robert Richter <rrichter@cavium.com>
Cc: stable <stable@vger.kernel.org> # 4.14
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/cavium/zip/zip_crypto.c