On 29/03/17 14:42, Sergio Gonzalez Monroy wrote:
There is bug when more crypto ops are enqueued than dequeued. The return value is not checked when trying to enqueue the processed crypto op into the internal ring, which in the case of being full will results in crypto ops and mbufs being leaked. The issue is more obvious with different cores doing enqueue/dequeue.This patch moves the crypto operation to the dequeue function which fixes the above issue without having to check for the number of free entries in the ring. Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto operations") Signed-off-by: Sergio Gonzalez Monroy <[email protected]> ---
...
Hey Sergio, this looks good but it needs to be rebased for the rte_ring API change. I also see a marginally performance increase with this change using the crypto-perf app which wasn't expected.
Acked-by: Declan Doherty <[email protected]>
