Re: [ovs-dev] [v4 10/12] dpif-netdev/mfex: Add AVX512 based optimized miniflow extract

[Eelco Chaudron]

On 1 Jul 2021, at 15:26, Flavio Leitner wrote:

<SNIP>

>>>> Yes, these are the standard gcc/clang etc compiler -m <isa level> switches.
>>>>
>>>> Search for "-mavx512bw" on e.g. this GCC page, lists them all;
>>>> https://gcc.gnu.org/onlinedocs/gcc/x86-Options.html
>>>>
>>>> If a compiler does not understand them, we will have to #ifdef that 
>>>> compiler out,
>>>> as it just doesn't support the ISA.
>>>
>>> Guess my concern is with the windows/Microsoft compiler, as I have no 
>>> windows
>>> setup, I can not verify this.
>>
>> Me neither. Flavio you mentioned a windows compiler issue on the DPIF 
>> patchset,
>> would you test compile here please?
>
> I used the DPIF v13 and MFEX v4 with a quick fix replacing __builtin_ctz()
> with raw_ctz() and it builds fine on AppVeyor:
> https://ci.appveyor.com/project/fleitner/ovs/build/job/svobt835gl8wrm8q

Interesting, I just created an appveyor account (using GitHub) and did a window 
build, by adding my GitHub repo. That was too easy ;)

Thanks for sharing!

//Eelco

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev