On 12 Jul 2021, at 7:51, kumar Amber wrote:

> This commits add the mfex optimized options to be
> executed as part of scalar DPIF.
>
> Signed-off-by: kumar Amber <[email protected]>
> Acked-by: Flavio Leitner <[email protected]>

The patch in general looks good to me, however, the only concern is the 
performance impact when AVX512 is not enabled compared to the master branch 
without the MFEX patch.

Up until this patch, the performance compared to the master branch is actually 
up about .14 ~ .5% for the PVP use case. After applying this patch the 
performance is down compared to the master branch, -.24 ~ -.6%.

So the question to the upstream community is, are we ok with the performance 
drop for all the users who do not want to enable AVX512 or don’t have AVX512?

//Eelco

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to