mbox series

[v7,0/3] fix PF reset causes VF memory request failure

Message ID 20210524012346.496560-1-haiyue.wang@intel.com (mailing list archive)
Headers
Series fix PF reset causes VF memory request failure |

Message

Wang, Haiyue May 24, 2021, 1:23 a.m. UTC 
  Trigger the VF reset from PF reset,

        echo 1 > /sys/bus/pci/devices/PF-BDF/reset

the PCI bus master bit will cleared on VF, so the VF needs to enable
this bit before restart.

This patch set adds the API to enable PCI bus master.

v7: fix the commit message typo, and update some description.
v6: update the annotate symbol version, and add some comments in source code
v5: error handling if bus master enable failed
v4: change the API to set type, so can enable or disable
v3: added the missed annotate symbol add time
v2: rebase to new librte directory path

Haiyue Wang (3):
  bus/pci: set PCI master in command register
  net/iavf: enable PCI bus master after reset
  net/i40e: enable PCI bus master after reset

 drivers/bus/pci/pci_common.c      | 28 ++++++++++++++++++++++++++++
 drivers/bus/pci/rte_bus_pci.h     | 14 ++++++++++++++
 drivers/bus/pci/version.map       |  3 +++
 drivers/net/i40e/i40e_ethdev_vf.c | 13 ++++++++++++-
 drivers/net/iavf/iavf_ethdev.c    | 10 +++++++++-
 lib/pci/rte_pci.h                 |  4 ++++
 6 files changed, 70 insertions(+), 2 deletions(-)

Comments

David Marchand June 8, 2021, 8:31 a.m. UTC | #1 
On Mon, May 24, 2021 at 3:44 AM Haiyue Wang <haiyue.wang@intel.com> wrote:
>
> Trigger the VF reset from PF reset,
>
>         echo 1 > /sys/bus/pci/devices/PF-BDF/reset
>
> the PCI bus master bit will cleared on VF, so the VF needs to enable
> this bit before restart.
>
> This patch set adds the API to enable PCI bus master.
>
> v7: fix the commit message typo, and update some description.
> v6: update the annotate symbol version, and add some comments in source code
> v5: error handling if bus master enable failed
> v4: change the API to set type, so can enable or disable
> v3: added the missed annotate symbol add time
> v2: rebase to new librte directory path
>
> Haiyue Wang (3):
>   bus/pci: set PCI master in command register
>   net/iavf: enable PCI bus master after reset
>   net/i40e: enable PCI bus master after reset

Series applied, thanks Haiyue.
I'll post a followup patch on using this API in other parts of dpdk.