[dpdk-dev] [PATCH v4 6/6] net/iavf: add watchdog for VFLR

Nicolau, Radu radu.nicolau at intel.com
Fri Oct 8 12:09:42 CEST 2021


On 10/8/2021 7:19 AM, Wu, Jingjing wrote:
>>> Besides checking VFGEN_RSTAT, there is a process to handle
>> VIRTCHNL_OP_EVENT  from PF. What is the change for? Any scenario which
>> VIRTCHNL_OP_EVENT  doesn't cover?
>>> And how is the 500us been determined?
>> Hi Jingjing, thanks for reviewing, I think this can be handled with the
>> VIRTCHNL_OP_EVENT  with no need for a watchdog alarm, I will rework the
>> patch.
>>
> Hi, Radu, I saw the patch is reworked, but looks like watchdog is still there. So what is the scenario
> VIRTCHNL_OP_EVENT  doesn't cover?

Hi Jingjing I went over this with Declan, the reason it was added is 
that we can actually have a hardware initiated reset that may not 
trigger an event; and also the kernel driver is implementing a similar 
mechanism.

Would it be more acceptable to have it disabled by default?




More information about the dev mailing list