On 10/8/2021 7:19 AM, Wu, Jingjing wrote:

quoted

quoted

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?

quoted

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?