On 5/23/2018 1:16 AM, Yao chuhong wrote:
> hi, Ferruh & Shijith
>
> Sorry, I can't reply the email in time, because it was late at night.
Hi Chuhong,
I was aware it is late for you so I updated the commit log for you.
> I read your email, Will use commit log suggested by Shijith and the patch
> Applied to dpdk-next-net/master.
> That means my contribution to this bug has been done, right?
Yes, fix included into rc5. Thank you for contribution.
> Thanks again for your help. Thanks!
>
>
>
> ------------------ Original ------------------
> *From: * "Ferruh Yigit"<ferruh.yi...@intel.com>;
> *Date: * Wed, May 23, 2018 05:12 AM
> *To: * "chuhong yao"<y...@panath.cn>;
> "shijith.thotton"<shijith.thot...@cavium.com>;
> *Cc: * "dev"<dev@dpdk.org>;
> *Subject: * Re: [dpdk-dev] [PATCH v2] net/liquidio fixes unable to update
> linfovar
>
> On 5/22/2018 10:07 PM, Ferruh Yigit wrote:
>> On 5/22/2018 3:33 PM, chuhong yao wrote:
>>> When I was using VPP +dpdk-18.02+liqudio CN23xx,
>>> I encountered such a bug.
>>> When VPP called dpdk_device_start to initialize DPDK liqudio drive,
>>> I found that initialization failed.
>>> The reason for the failure is that VF MTU > PF MTU,
>>> but PF MTU has been modified to 9600 (> VF MTU).
>>> Finally, I am location that DPDK liqudio drive
>>> cannot get the correct PF driver to liqudio network card.
>>> It is due to the fact that when VPP calls dpdk_device_start to
>>> initialize DPDK liqudio drive,this time,
>>> lio_dev->linfo.Link var already exists in the old value, not empty.
>>> Cause lio_dev->linfo.Link. Link_status64 != 0
>>> statement is set up,
>>> and the link info is stopped directly to liqudio card,
>>> resulting in no get accurate pf mtu.
>>> I did a test model to reproduce the bug,
>>> which is to add rte_eth_dev_set_mtu(portid, vf_mtu)
>>> to the rte_eth_dev_start function when using
>>> dpdk-18.02+liqudio CN23xx+l2fwd.
>>> You need to make sure that 1500 < vf_mtu < pf_mtu will be available.
>>> At this time, you will have
>>> net_liovf[04:00.3]ERROR: lio_dev_mtu_set()
>>> VF MTU should be >= 68 and <= 1500. Such a mistake.
>>>
>>> Signed-off-by: chuhong yao <y...@panath.cn>
>>
>> Will use commit log suggested by maintainer [1]. Thanks Shijith!
>>
>>
>> [1]
>> net/liquidio: fix link state fetching during device start
>>
>> Clear link state information during device start for fetching new state
>> before function completion.
>>
>> Fixes: 605164c8e79d ("net/liquidio: add API to validate VF MTU")
>> Cc: sta...@dpdk.org
>>
>> Signed-off-by: Chuhong Yao <y...@panath.cn>
>> Acked-by: Shijith Thotton <shijith.thot...@caviumnetworks.com>
>
> Applied to dpdk-next-net/master, thanks.
>
> 搜索
>
> 复制
>
