> -----Original Message-----
> From: Iremonger, Bernard
> Sent: Thursday, September 7, 2017 5:43 PM
> To: dev@dpdk.org; Yigit, Ferruh <ferruh.yi...@intel.com>; Ananyev,
> Konstantin <konstantin.anan...@intel.com>; Dumitrescu, Cristian
> <cristian.dumitre...@intel.com>; adrien.mazarg...@6wind.com
> Cc: Iremonger, Bernard <bernard.iremon...@intel.com>; sta...@dpdk.org
> Subject: [PATCH v5 2/6] librte_table: fix acl lookup function
>
> The rte_table_acl_lookup() function was returning data from acl_memory
> instead of acl_rule_memory.
>
> Fixes: 166923eb2f78 ("table: ACL")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Bernard Iremonger <bernard.iremon...@intel.com>
> ---
> lib/librte_table/rte_table_acl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/librte_table/rte_table_acl.c
> b/lib/librte_table/rte_table_acl.c
> index e84b437..258916d 100644
> --- a/lib/librte_table/rte_table_acl.c
> +++ b/lib/librte_table/rte_table_acl.c
> @@ -794,7 +794,7 @@ struct rte_table_acl {
> if (action_table_pos != 0) {
> pkts_out_mask |= pkt_mask;
> entries[pkt_pos] = (void *)
> - &acl->memory[action_table_pos *
> + &acl->acl_rule_memory[action_table_pos *
> acl->entry_size];
> rte_prefetch0(entries[pkt_pos]);
> }
> --
> 1.9.1
NACK
Existing code is correct, proposed code changes are wrong (for same reasons
described in patch 1 of this patch set).
