Re: [PATCH 06/12] crypto: caam - change return code in caam_jr_enqueue function

From: Horia Geanta
Date: Tue Nov 19 2019 - 10:21:41 EST


On 11/18/2019 12:31 AM, Iuliana Prodan wrote:
> Change the return code of caam_jr_enqueue function to -EINPROGRESS, in
> case of success, -ENOSPC in case the CAAM is busy (has no space left
> in job ring queue), -EIO if it cannot map the caller's descriptor.
>
> Update, also, the cases for resource-freeing for each algorithm type.
>
It probably would've been worth saying *why* these changes are needed.

Even though the patch is part of a patch set adding "backlogging support",
this grouping won't be visible in git log.

There's another reason however for the -EBUSY -> -ENOSPC change,
i.e. commit 6b80ea389a0b ("crypto: change transient busy return code to -ENOSPC")

> Signed-off-by: Iuliana Prodan <iuliana.prodan@xxxxxxx>
Reviewed-by: Horia Geantă <horia.geanta@xxxxxxx>

Thanks,
Horia