Re: [PATCH v6 0/8] crypto: qce - Fix crypto self-test failures
Eric Biggers <[email protected]> Thu, 23 Jul 2026 13:27:04 -0700
| Newsgroups | org.kernel.vger.linux-crypto,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel,org.kernel.vger.stable |
|---|---|
| Message-ID | <20260723202704.GB110634@quark> |
On Fri, Jul 17, 2026 at 05:53:29PM +0200, Bartosz Golaszewski wrote: > This extends the initial submission from Kuldeep. > > The QCE hardware crypto engine has several limitations that cause it to > produce incorrect results or stall on certain inputs. This series fixes > several bugs and adds workaround allowing the deiver to pass crypto > self-tests. > > The failures addressed are: I was also looking at how the request queueing works in this driver. qce_handle_queue() gets invoked synchronously from almost all the ahash/skcipher/aead API entry points. However, it takes a mutex, which can sleep. Not a great choice when these APIs can be called in softirq context. I strongly suspect this would crash if anyone actually tried to use this with IPsec, for example. This is interesting when we consider that the driver implements the "rfc4309(ccm(aes))" algorithm, whose exclusive purpose is IPsec. I'm not sure if there are some out of tree patches going on, or if this was just never actually tested with what it was theoretically supposed to have been useful for. But I'd guess the latter. - Eric