[dpdk-dev] [PATCH] net/i40e/base: skip further adminq init for VF
Ye Xiaolong
xiaolong.ye at intel.com
Wed Jan 22 08:49:13 CET 2020
On 01/22, Xing, Beilei wrote:
>
>
>> -----Original Message-----
>> From: Ye, Xiaolong
>> Sent: Wednesday, January 22, 2020 11:58 AM
>> To: Xing, Beilei <beilei.xing at intel.com>; Zhang, Qi Z <qi.z.zhang at intel.com>
>> Cc: dev at dpdk.org; Ye, Xiaolong <xiaolong.ye at intel.com>
>> Subject: [PATCH] net/i40e/base: skip further adminq init for VF
>>
>> Since VF has no need of firmware, we can skip further adminq init which
>> involves firmware operation, this patch fixes the testpmd segfault issue when
>> starting with i40e VF.
>>
>> Fixes: d5e1a149362e ("net/i40e/base: check MAC type")
>>
>> Signed-off-by: Xiaolong Ye <xiaolong.ye at intel.com>
>> ---
>> drivers/net/i40e/base/i40e_adminq.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/net/i40e/base/i40e_adminq.c
>> b/drivers/net/i40e/base/i40e_adminq.c
>> index 221c327d1..bc9ef3c87 100644
>> --- a/drivers/net/i40e/base/i40e_adminq.c
>> +++ b/drivers/net/i40e/base/i40e_adminq.c
>> @@ -673,6 +673,9 @@ enum i40e_status_code i40e_init_adminq(struct
>> i40e_hw *hw)
>> if (ret_code != I40E_SUCCESS)
>> goto init_adminq_free_asq;
>>
>> + if (i40e_is_vf(hw))
>> + goto init_adminq_exit;
>> +
>> /* There are some cases where the firmware may not be quite ready
>> * for AdminQ operations, so we retry the AdminQ setup a few times
>> * if we see timeouts in this first AQ call.
>> --
>> 2.17.1
>
>Acked-by: Beilei Xing <beilei.xing at intel.com>
Applied to dpdk-next-net-intel, Thanks.
More information about the dev
mailing list