Re: [PATCH] crypto: caam - fix rfc4106 encap shared descriptor

2015-07-30 Thread Horia Geantă
On 7/30/2015 4:46 PM, Herbert Xu wrote: On Thu, Jul 30, 2015 at 04:39:26PM +0300, Horia Geantă wrote: The encap shared descriptor was changed to use the new IV convention. In the process some commands were shifted, making the output length zero, caam effectively writing garbage in dst.

[PATCH] crypto: caam - fix rfc4106 encap shared descriptor

2015-07-30 Thread Horia Geantă
The encap shared descriptor was changed to use the new IV convention. In the process some commands were shifted, making the output length zero, caam effectively writing garbage in dst. While here, update the decap descriptor to execute the write commands before the reads (as it previously was).

Re: [PATCH] crypto: caam - fix rfc4106 encap shared descriptor

2015-07-30 Thread Herbert Xu
On Thu, Jul 30, 2015 at 04:59:01PM +0300, Horia Geantă wrote: Indeed, there is: A-005473 - Using SEQ FIFO LOAD SKIP and SEQ FIFO STORE SKIP simultaneously will cause the DECO to hang. However, the skip commands are not consecutive, there's a math command between them (both for encap and

Re: [PATCH] crypto: caam - fix rfc4106 encap shared descriptor

2015-07-30 Thread Horia Geantă
On 7/30/2015 5:03 PM, Herbert Xu wrote: On Thu, Jul 30, 2015 at 04:59:01PM +0300, Horia Geantă wrote: Indeed, there is: A-005473 - Using SEQ FIFO LOAD SKIP and SEQ FIFO STORE SKIP simultaneously will cause the DECO to hang. However, the skip commands are not consecutive, there's a math