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