On 5 April 2017 at 04:22, Thomas Monjalon <thomas.monja...@6wind.com> wrote:
> 2017-04-04 10:30, Ferruh Yigit:
>> On 4/4/2017 2:55 AM, Qi Zhang wrote:
>> > The patch set create new APIs that help to change the mapping from hardware
>> > defined packet type to software defined packet type for i40e NICs.
>> > Keep these APIs private is because currently they are only meaningful for
>> > devices that support dynamic PTYPE configuration, which may not be a 
>> > general
>> > device feature.
> [...]
>> > Qi Zhang (3):
>> >   net/i40e: enable per dev PTYPE mapping table
>> >   net/i40e: configurable PTYPE mapping
>> >   app/testpmd: add CL for ptype mapping configure
>>
>> Series applied to dpdk-next-net/master, thanks.
>
> It does not compile for ARM (not tested for POWER):
>         drivers/net/i40e/i40e_rxtx_vec_neon.c: In function 
> '_recv_raw_pkts_vec':
>                 229:2: error: unknown type name 'uint32'
>                 uint32 *ptype_tbl = rxq->vsi->adapter->ptype_tbl;
>

It must be uint32_t.

> Given that
>         - it changes NEON and Altivec code
>         - it does not compile on ARM
>         - there is no Ack from NEON or Altivec maintainers (they were not 
> Cc'ed)
> I really doubt it has been tested.
> That's why it won't be in RC1.
>
> If NEON and Altivec maintainers agree, we can give it a chance for RC2.
>

Other than the above error on ARM:
Acked-by: Jianbo Liu <jianbo....@linaro.org>

> PS: please use --in-reply-to to let us check the discussion history.

Reply via email to