<div dir="ltr"><div dir="ltr">Hi Ferruh,</div><div dir="ltr"><br></div><div>Sorry for the late reply. Please see inline.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Sep 27, 2021 at 9:49 PM Ferruh Yigit <<a href="mailto:ferruh.yigit@intel.com" target="_blank">ferruh.yigit@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 7/26/2021 7:43 AM, Chengwen Feng wrote:<br>
> According to the definition of rte_eth_dev_reset(), the<br>
> RTE_ETH_EVENT_INTR_RESET event could also use when PF resets.<br>
> <br>
<br>
Can you please highlight the part in the 'rte_eth_dev_reset()' definition<br>
related to the RESET event usage for PF?<br>
<br>
> This patch modifies the comment of RTE_ETH_EVENT_INTR_RESET event, so<br>
> that it could use in all resets.<br>
> <br>
<br>
The original intention seems as the comment mentions, please check related<br>
commits [1].<br>
<br>
As far as I can get from below comments, usecase is,<br>
- PF sends reset command to VFs (driver internal command)<br>
- VF sends RESET event to application, to request reset to be performed by<br>
application.<br>
<br>
So event is more like a reset request from driver to application.<br>
<br>
Overall it is OK to extend the usage of the RESET event to PF, if there is a<br>
usecase for it. What is your usecase?<br>
And should we extend comment (API documentation) a little more to clarify when<br>
this even should be sent and what application should do when event received,<br>
what do you think?<br>
<br>
btw, cc'ed Ajit & Kales, as far as I remember in the past they suggest a recover<br>
event, maybe relevant with this discussion.<br></blockquote><div>[Kalesh]: The recovery event we proposed was for a different purpose and looks different from this.</div><div><br></div><div>In that case, PMD itself recovers from the FW error/reset conditions using a handshaking protocol between PMD and the device FW without needing application intervention.</div><div><br></div><div>I will send a new version of the patch set incorporating the comments I received on the last version I had sent.</div><div><br></div><div>Regards,</div><div>Kalesh</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
<br>
[1]<br>
Commit ae19955e7c86 ("i40evf: support reporting PF reset")<br>
Commit 514302ff6e00 ("ethdev: add NIC reset operation")<br>
<br>
> Signed-off-by: Chengwen Feng <<a href="mailto:fengchengwen@huawei.com" target="_blank">fengchengwen@huawei.com</a>><br>
> ---<br>
>  lib/ethdev/rte_ethdev.h | 3 +--<br>
>  1 file changed, 1 insertion(+), 2 deletions(-)<br>
> <br>
> diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h<br>
> index d2b27c3..e6646a6 100644<br>
> --- a/lib/ethdev/rte_ethdev.h<br>
> +++ b/lib/ethdev/rte_ethdev.h<br>
> @@ -3499,8 +3499,7 @@ enum rte_eth_event_type {<br>
>       RTE_ETH_EVENT_INTR_LSC, /**< lsc interrupt event */<br>
>       RTE_ETH_EVENT_QUEUE_STATE,<br>
>                               /**< queue state event (enabled/disabled) */<br>
> -     RTE_ETH_EVENT_INTR_RESET,<br>
> -                     /**< reset interrupt event, sent to VF on PF reset */<br>
> +     RTE_ETH_EVENT_INTR_RESET, /**< reset interrupt event */<br>
>       RTE_ETH_EVENT_VF_MBOX,  /**< message from the VF received by PF */<br>
>       RTE_ETH_EVENT_MACSEC,   /**< MACsec offload related event */<br>
>       RTE_ETH_EVENT_INTR_RMV, /**< device removal event */<br>
> <br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr">Regards,<div>Kalesh A P</div></div></div></div>