On 3/3/22 11:40, Ferriter, Cian wrote:
> 
> 
>> -----Original Message-----
>> From: Amber, Kumar <[email protected]>
>> Sent: Tuesday 1 March 2022 15:08
>> To: [email protected]
>> Cc: Ferriter, Cian <[email protected]>; [email protected]; Stokes, 
>> Ian <[email protected]>;
>> Van Haaren, Harry <[email protected]>; Amber, Kumar 
>> <[email protected]>
>> Subject: [PATCH v2] system-dpdk: Fix mfex autovalidator tests.
>>
>> AVX512 DPIF must be active in order for the MFEX AutoValidator to be 
>> executed.
>> If the DPIF-AVX512 is not available, the unit test is skipped, as the
>> scalar DPIF does not use the MFEX function-pointer based optimizations.
>>
>> Fixes: 50be6715c0 ("test/sytem-dpdk: Add unit test for mfex autovalidator")
>> Suggested-by: Cian Ferriter <[email protected]>
>> Signed-off-by: Kumar Amber <[email protected]>
> 
> Hi Amber,
> 
> Thanks for addressing the changes.
> 
> This patch LGTM. I have also tested that the unit tests are being run by 
> introducing an error in the MFEX C code and running the unit tests. The 
> errors are caught and the MFEX autovalidator tests fail.
> 
> Acked-by: Cian Ferriter <[email protected]>

Thanks!  Applied and backported down to 2.16.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to