<div dir="ltr"><div dir="ltr">Hi David,</div><div dir="ltr"><br></div><div>Thanks for pointing out this problem, I'll send a fix and v2.</div><div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">пн, 30 сент. 2024 г. в 16:07, David Marchand <<a href="mailto:david.marchand@redhat.com">david.marchand@redhat.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On Sat, Sep 28, 2024 at 1:51 AM David Marchand<br>
<<a href="mailto:david.marchand@redhat.com" target="_blank">david.marchand@redhat.com</a>> wrote:<br>
><br>
> On Fri, Sep 6, 2024 at 1:07 PM Vladimir Medvedkin<br>
> <<a href="mailto:vladimir.medvedkin@intel.com" target="_blank">vladimir.medvedkin@intel.com</a>> wrote:<br>
> ><br>
> > Previously when running rte_fib_lookup IPv4 addresses must have been in<br>
> > host byte order.<br>
> ><br>
> > This patch adds a new flag RTE_FIB_FLAG_LOOKUP_BE that can be passed on<br>
> > fib create, which will allow to have IPv4 in network byte order on<br>
> > lookup.<br>
> ><br>
> > Signed-off-by: Vladimir Medvedkin <<a href="mailto:vladimir.medvedkin@intel.com" target="_blank">vladimir.medvedkin@intel.com</a>><br>
><br>
> _mm512_shuffle_epi8 requires avx512bw, so the dir24_8 AVX512<br>
> implementation can only compile when such feature is supported and<br>
> enabled.<br>
> Like the trie AVX512 implem.<br>
><br>
> IOW, we are missing an update of the library meson.build.<br>
<br>
I also suspect that both this added support in the dir24_8<br>
implementation and the existing trie are missing some runtime check on<br>
RTE_CPUFLAG_AVX512BW in get_vector_fn.<br>
Can we get a fix for trie and a respin of this current patch?<br>
<br>
Thanks.<br>
<br>
<br>
-- <br>
David Marchand<br>
<br>
</blockquote></div><br clear="all"><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div>Vladimir<br></div></div></div>