Hi Ian,

> -----Original Message-----
> From: Stokes, Ian
> Sent: Tuesday, October 17, 2017 12:04 PM
> To: dev@dpdk.org
> Cc: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; Stokes, Ian
> <ian.sto...@intel.com>
> 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 <ian.sto...@intel.com>

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