Re: [dpdk-dev] [PATCH v2] net/hns3: add runtime config to set MBX limit time

["Min Hu (Connor)" <humin29@huawei.com>]

Hi, Ferruh,
	V3 has been sent,please check it out, thanks.

在 2021/10/21 21:06, Ferruh Yigit 写道:
> On 10/21/2021 3:22 AM, Min Hu (Connor) wrote:
>> From: Chengchang Tang<tangchengchang@huawei.com>
>>
>> Current, the max waiting time for MBX response is 500ms, but in
>> some scenarios, it is not enough. Since it depends on the response
>> of the kernel mode driver, and its response time is related to the
>> scheduling of the system. In this special scenario, most of the
>> cores are isolated, and only a few cores are used for system
>> scheduling. When a large number of services are started, the
>> scheduling of the system will be very busy, and the reply of the
>> mbx message will time out, which will cause our PMD initialization
>> to fail.
>>
>> This patch add a runtime config to set the max wait time. For the
>> above scenes, users can adjust the waiting time to a suitable value
>> by themselves.
>>
>> Fixes: 463e748964f5 ("net/hns3: support mailbox")
>> Cc:stable@dpdk.org
>>
>> Signed-off-by: Chengchang Tang<tangchengchang@huawei.com>
>> Signed-off-by: Min Hu (Connor)<humin29@huawei.com>
>> ---
>> v2:
>> * add some comment for HNS3_MBX_DEF_TIME_LIMIT_MS.
>> * remove some check for mbx_time_limit.
>> ---
>>   drivers/net/hns3/hns3_ethdev.c    | 32 ++++++++++++++++++++++++++++++-
>>   drivers/net/hns3/hns3_ethdev.h    |  3 +++
>>   drivers/net/hns3/hns3_ethdev_vf.c |  3 ++-
>>   drivers/net/hns3/hns3_mbx.c       |  8 +++++---
>>   drivers/net/hns3/hns3_mbx.h       |  1 +
>>   5 files changed, 42 insertions(+), 5 deletions(-)
> 
> 
> New devarg need to be documented in the 'doc/guides/nics/hns3.rst',
> in "Runtime Config Options" section.
> .