; Jieqiang
> Wang ; Honnappa Nagarahalli
> ; nd
> Subject: Re: [vpp-dev] ACL plugin optimization
>
> Hi Govind,
>
> 1) According to Jenkins, this patch permits some of the packets that should
> be denied, hence JJB voted "-1".
>
> 2) If you suspect merely th
> ; nd
> Subject: Re: [vpp-dev] ACL plugin optimization
>
>
> Hi Govind,
>
> As well as removing the prefetches, you've also removed the per packet call
> to acl_fa_find_session_with_hash(). So IIUC you've removed the per-packet
> session lookup and inste
Hi Govind,
As well as removing the prefetches, you've also removed the per packet call to
acl_fa_find_session_with_hash(). So IIUC you've removed the per-packet session
lookup and instead re-use the lookup of packet 0 each time. that'll make things
quicker but it's not functionally correct.
/
Hi Govind,
1) According to Jenkins, this patch permits some of the packets that
should be denied, hence JJB voted "-1".
2) If you suspect merely the prefetches are the issue, just commenting
out the body of prefetch_session_entry() in the original code should
turn it into a no-op that doesn't bre
Hi Andrew,
While profiling the ACL plugin node using perf tool in ARM Neoverse platform,
Bihash related prefetches were shown as bottleneck.
Performance improvement is seen in ARM N1, TX2 and Intel Skylake servers after
removing those prefetches. Testing is done with Ingress ACL/IPv4 forwardi
