Hi Ian,

> -----Original Message-----
> From: Stokes, Ian
> Sent: Tuesday, October 17, 2017 12:04 PM
> To: [email protected]
> Cc: De Lara Guarch, Pablo <[email protected]>; Stokes, Ian
> <[email protected]>
> Subject: [PATCH v1 1/1] rte_crypto.h: Fix compilation issue with Ofast.
> 
> When compiling with an application that includes rte_cryptodev.h with
> Ofast, an error is reported regarding enumeration
> RTE_CRYPTO_OP_TYPE_UNDEFINED not handled in switch case in function
> __rte_crypto_op_reset().
> 
> Fix this by adding case for RTE_OP_TYPE_UNDEFINED.
> 
> Signed-off-by: Ian Stokes <[email protected]>

Since this is a fix, you should add a fixes line with the commit id where the 
issue was introduced.
This case: Fixes: c0f87eb5252b ("cryptodev: change burst API to be crypto op 
oriented")

Also, the title should be: "cryptodev: fix build with Ofast"

Could you send a v2 with these changes?

Thanks,
Pablo

Reply via email to