summaryrefslogtreecommitdiffstats
path: root/crypto
diff options
context:
space:
mode:
authorHoria Geanta <horia.geanta@freescale.com>2013-11-13 11:20:37 +0100
committerHerbert Xu <herbert@gondor.apana.org.au>2013-12-05 14:28:41 +0100
commitb62ffd8c72c2f04ac737ef3ff0f7ba37d2013633 (patch)
treec84520c93e902b7bc95617e42b79b320d01bde68 /crypto
parentcrypto: omap-sham - Only release DMA channel if successfully requested (diff)
downloadlinux-b62ffd8c72c2f04ac737ef3ff0f7ba37d2013633.tar.xz
linux-b62ffd8c72c2f04ac737ef3ff0f7ba37d2013633.zip
crypto: talitos - fix locating offending descriptor in error path
Commit 3e721aeb3df3816e283ab18e327cd4652972e213 ("crypto: talitos - handle descriptor not found in error path") tried to address the fact that CDPR (Current Descriptor Pointer Register) is unreliable. As it turns out, there are still issues in the function detecting the offending descriptor: -only 32 bits of the descriptor address are read, however the address is 36-bit - since reset_channel() initializes channels with EAE (extended address) bit set -reading CDPR can return zero in cur_desc; when searching the channel fifo for this address, cur_desc == dma_desc (= 0) case might happen, leading to an oops when trying to return desc->hdr (desc is zero) -read channel's .tail only once; the tail is a moving target; use a local variable for the end of search condition Signed-off-by: Lei Xu <Lei.Xu@freescale.com> Signed-off-by: Horia Geanta <horia.geanta@freescale.com> Tested-by: Kalyani Chowdhury <Kalyani.Chowdhury@freescale.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto')
0 files changed, 0 insertions, 0 deletions