On 5/11/2018 4:36 PM, Iremonger, Bernard wrote:
>> -----Original Message-----
>> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Shahaf Shuler
>> Sent: Thursday, May 10, 2018 5:39 AM
>> To: Yigit, Ferruh <ferruh.yi...@intel.com>; Lu, Wenzhuo
>> <wenzhuo...@intel.com>; Wu, Jingjing <jingjing...@intel.com>
>> Cc: dev@dpdk.org; Dai, Wei <wei....@intel.com>; Zhang, Qi Z
>> <qi.z.zh...@intel.com>
>> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: check if CRC strip offload
>> supported
>>
>> Thursday, May 10, 2018 1:09 AM, Ferruh Yigit:
>>> Subject: [PATCH] app/testpmd: check if CRC strip offload supported
>>>
>>> Testpmd set CRC_STRIP offload blindly, this is wrong according offload
>>> API definition, and will cause error for the PMDs that doesn't support
>>> CRC_STRIP like virtual PMDs.
>>>
>>> Check if underlying device report this capability and don't set it if
>>> not supported.
>>>
>>> Fixes: 0074d02fca21 ("app/testpmd: convert to new Rx offloads API")
>>> Cc: shah...@mellanox.com
>>>
>>> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com>
>>
>> Yes this is the correct behavior. I hope all the PMDs report the CRC strip
>> correctly.
>>
>> Acked-by: Shahaf Shuler < shah...@mellanox.com>
> 
> Acked-by: Bernard Iremonger <bernard.iremon...@intel.com>

Applied to dpdk-next-net/master, thanks.

Reply via email to