On 1/8/2018 11:03 AM, De Lara Guarch, Pablo wrote:
If unsigned then line <for (cdev_id = rte_cryptodev_count() - 1; cdev_id >= 0; cdev_id--)> will not work correctly.-----Original Message----- From: Nicolau, Radu Sent: Tuesday, December 12, 2017 12:50 PM To: dev@dpdk.org Cc: Ananyev, Konstantin <konstantin.anan...@intel.com>; Yigit, Ferruh <ferruh.yi...@intel.com>; Gonzalez Monroy, Sergio <sergio.gonzalez.mon...@intel.com>; De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; Doherty, Declan <declan.dohe...@intel.com>; akhil.go...@nxp.com; Nicolau, Radu <radu.nico...@intel.com> Subject: [PATCH 3/3] examples/ipsec_secgw: create session mempools for ethdevs Also moved offloaded packets from cryptodev queues Signed-off-by: Radu Nicolau <radu.nico...@intel.com> --- examples/ipsec-secgw/ipsec-secgw.c | 42 +++++++++++++++++++++++++++++++++++++- examples/ipsec-secgw/ipsec.c | 31 ++++++++++++++-------------- examples/ipsec-secgw/ipsec.h | 4 ++-- 3 files changed, 58 insertions(+), 19 deletions(-) diff --git a/examples/ipsec-secgw/ipsec-secgw.c b/examples/ipsec- secgw/ipsec-secgw.c index c98454a..08d5b5a 100644 --- a/examples/ipsec-secgw/ipsec-secgw.c +++ b/examples/ipsec-secgw/ipsec-secgw.c @@ -1244,7 +1244,7 @@ cryptodevs_init(void) struct rte_cryptodev_config dev_conf; struct rte_cryptodev_qp_conf qp_conf; uint16_t idx, max_nb_qps, qp, i; - int16_t cdev_id; + int16_t cdev_id, port_id;These two variables should be uint16_t (port_id is actually uint8_t).
For the rest of the patch: Akhil, could you review it? Thanks, Pablo
