From: Gavin Shan <gwshan@linux.vnet.ibm.com> To: Wei Yang <weiyang@linux.vnet.ibm.com> Cc: gwshan@linux.vnet.ibm.com, bhelgaas@google.com, linuxppc-dev@lists.ozlabs.org, linux-pci@vger.kernel.org Subject: Re: [PATCH V3 1/9] pci/iov: rename and export virtfn_add/virtfn_remove Date: Mon, 11 May 2015 12:13:31 +1000 [thread overview] Message-ID: <20150511021331.GC3694@gwshan> (raw) In-Reply-To: <1430723258-21299-2-git-send-email-weiyang@linux.vnet.ibm.com> On Mon, May 04, 2015 at 03:07:30PM +0800, Wei Yang wrote: >During the EEH procedure, when a device's driver is not EEH aware or no ^^^^^^^^^^^^^ EEH recovery, >driver is binded with a device, EEH core would do hotplug on this devices. ^^^^^^ ^^^^^^^^^^^^ >While it isn't feasible for a VF with usual hotplug procedure. During >removal of a VF, virt_bus should be removed if necessary. During the >re-creation, the pci_scan_slot() doesn't work on a VF. > Is "virt_bus" related to those functions you try to export? Even so, you can simply have "virt_bus" in the commit log, please have " virtual bus" or the formal things like that. >This patch exports two functions to handle the hotplug case for VF >properly. They will be invoked when the EEH core do the hotplug case for ^^^^ >VFs. Couple of typos as above. Thanks, Gavin > >Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> >--- > drivers/pci/iov.c | 10 +++++----- > include/linux/pci.h | 2 ++ > 2 files changed, 7 insertions(+), 5 deletions(-) > >diff --git a/drivers/pci/iov.c b/drivers/pci/iov.c >index ee0ebff..cc941dd 100644 >--- a/drivers/pci/iov.c >+++ b/drivers/pci/iov.c >@@ -108,7 +108,7 @@ resource_size_t pci_iov_resource_size(struct pci_dev *dev, int resno) > return dev->sriov->barsz[resno - PCI_IOV_RESOURCES]; > } > >-static int virtfn_add(struct pci_dev *dev, int id, int reset) >+int pci_iov_virtfn_add(struct pci_dev *dev, int id, int reset) > { > int i; > int rc = -ENOMEM; >@@ -183,7 +183,7 @@ failed: > return rc; > } > >-static void virtfn_remove(struct pci_dev *dev, int id, int reset) >+void pci_iov_virtfn_remove(struct pci_dev *dev, int id, int reset) > { > char buf[VIRTFN_ID_LEN]; > struct pci_dev *virtfn; >@@ -320,7 +320,7 @@ static int sriov_enable(struct pci_dev *dev, int nr_virtfn) > } > > for (i = 0; i < initial; i++) { >- rc = virtfn_add(dev, i, 0); >+ rc = pci_iov_virtfn_add(dev, i, 0); > if (rc) > goto failed; > } >@@ -332,7 +332,7 @@ static int sriov_enable(struct pci_dev *dev, int nr_virtfn) > > failed: > for (j = 0; j < i; j++) >- virtfn_remove(dev, j, 0); >+ pci_iov_virtfn_remove(dev, j, 0); > > iov->ctrl &= ~(PCI_SRIOV_CTRL_VFE | PCI_SRIOV_CTRL_MSE); > pci_cfg_access_lock(dev); >@@ -361,7 +361,7 @@ static void sriov_disable(struct pci_dev *dev) > return; > > for (i = 0; i < iov->num_VFs; i++) >- virtfn_remove(dev, i, 0); >+ pci_iov_virtfn_remove(dev, i, 0); > > pcibios_sriov_disable(dev); > >diff --git a/include/linux/pci.h b/include/linux/pci.h >index 353db8d..94bacfa 100644 >--- a/include/linux/pci.h >+++ b/include/linux/pci.h >@@ -1679,6 +1679,8 @@ int pci_iov_virtfn_devfn(struct pci_dev *dev, int id); > > int pci_enable_sriov(struct pci_dev *dev, int nr_virtfn); > void pci_disable_sriov(struct pci_dev *dev); >+int pci_iov_virtfn_add(struct pci_dev *dev, int id, int reset); >+void pci_iov_virtfn_remove(struct pci_dev *dev, int id, int reset); > int pci_num_vf(struct pci_dev *dev); > int pci_vfs_assigned(struct pci_dev *dev); > int pci_sriov_set_totalvfs(struct pci_dev *dev, u16 numvfs); >-- >1.7.9.5 >
WARNING: multiple messages have this Message-ID (diff)
From: Gavin Shan <gwshan@linux.vnet.ibm.com> To: Wei Yang <weiyang@linux.vnet.ibm.com> Cc: bhelgaas@google.com, linux-pci@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, gwshan@linux.vnet.ibm.com Subject: Re: [PATCH V3 1/9] pci/iov: rename and export virtfn_add/virtfn_remove Date: Mon, 11 May 2015 12:13:31 +1000 [thread overview] Message-ID: <20150511021331.GC3694@gwshan> (raw) In-Reply-To: <1430723258-21299-2-git-send-email-weiyang@linux.vnet.ibm.com> On Mon, May 04, 2015 at 03:07:30PM +0800, Wei Yang wrote: >During the EEH procedure, when a device's driver is not EEH aware or no ^^^^^^^^^^^^^ EEH recovery, >driver is binded with a device, EEH core would do hotplug on this devices. ^^^^^^ ^^^^^^^^^^^^ >While it isn't feasible for a VF with usual hotplug procedure. During >removal of a VF, virt_bus should be removed if necessary. During the >re-creation, the pci_scan_slot() doesn't work on a VF. > Is "virt_bus" related to those functions you try to export? Even so, you can simply have "virt_bus" in the commit log, please have " virtual bus" or the formal things like that. >This patch exports two functions to handle the hotplug case for VF >properly. They will be invoked when the EEH core do the hotplug case for ^^^^ >VFs. Couple of typos as above. Thanks, Gavin > >Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> >--- > drivers/pci/iov.c | 10 +++++----- > include/linux/pci.h | 2 ++ > 2 files changed, 7 insertions(+), 5 deletions(-) > >diff --git a/drivers/pci/iov.c b/drivers/pci/iov.c >index ee0ebff..cc941dd 100644 >--- a/drivers/pci/iov.c >+++ b/drivers/pci/iov.c >@@ -108,7 +108,7 @@ resource_size_t pci_iov_resource_size(struct pci_dev *dev, int resno) > return dev->sriov->barsz[resno - PCI_IOV_RESOURCES]; > } > >-static int virtfn_add(struct pci_dev *dev, int id, int reset) >+int pci_iov_virtfn_add(struct pci_dev *dev, int id, int reset) > { > int i; > int rc = -ENOMEM; >@@ -183,7 +183,7 @@ failed: > return rc; > } > >-static void virtfn_remove(struct pci_dev *dev, int id, int reset) >+void pci_iov_virtfn_remove(struct pci_dev *dev, int id, int reset) > { > char buf[VIRTFN_ID_LEN]; > struct pci_dev *virtfn; >@@ -320,7 +320,7 @@ static int sriov_enable(struct pci_dev *dev, int nr_virtfn) > } > > for (i = 0; i < initial; i++) { >- rc = virtfn_add(dev, i, 0); >+ rc = pci_iov_virtfn_add(dev, i, 0); > if (rc) > goto failed; > } >@@ -332,7 +332,7 @@ static int sriov_enable(struct pci_dev *dev, int nr_virtfn) > > failed: > for (j = 0; j < i; j++) >- virtfn_remove(dev, j, 0); >+ pci_iov_virtfn_remove(dev, j, 0); > > iov->ctrl &= ~(PCI_SRIOV_CTRL_VFE | PCI_SRIOV_CTRL_MSE); > pci_cfg_access_lock(dev); >@@ -361,7 +361,7 @@ static void sriov_disable(struct pci_dev *dev) > return; > > for (i = 0; i < iov->num_VFs; i++) >- virtfn_remove(dev, i, 0); >+ pci_iov_virtfn_remove(dev, i, 0); > > pcibios_sriov_disable(dev); > >diff --git a/include/linux/pci.h b/include/linux/pci.h >index 353db8d..94bacfa 100644 >--- a/include/linux/pci.h >+++ b/include/linux/pci.h >@@ -1679,6 +1679,8 @@ int pci_iov_virtfn_devfn(struct pci_dev *dev, int id); > > int pci_enable_sriov(struct pci_dev *dev, int nr_virtfn); > void pci_disable_sriov(struct pci_dev *dev); >+int pci_iov_virtfn_add(struct pci_dev *dev, int id, int reset); >+void pci_iov_virtfn_remove(struct pci_dev *dev, int id, int reset); > int pci_num_vf(struct pci_dev *dev); > int pci_vfs_assigned(struct pci_dev *dev); > int pci_sriov_set_totalvfs(struct pci_dev *dev, u16 numvfs); >-- >1.7.9.5 >
next prev parent reply other threads:[~2015-05-11 2:19 UTC|newest] Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-05-04 7:07 [PATCH V3 0/9] VF EEH on Power8 Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-04 7:07 ` [PATCH V3 1/9] pci/iov: rename and export virtfn_add/virtfn_remove Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-11 2:13 ` Gavin Shan [this message] 2015-05-11 2:13 ` Gavin Shan 2015-05-04 7:07 ` [PATCH V3 2/9] powerpc/pci_dn: cache vf_index in pci_dn Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-11 2:21 ` Gavin Shan 2015-05-11 2:21 ` Gavin Shan 2015-05-11 5:54 ` Wei Yang 2015-05-11 5:54 ` Wei Yang 2015-05-12 6:15 ` Gavin Shan 2015-05-12 6:15 ` Gavin Shan 2015-05-12 7:29 ` Wei Yang 2015-05-12 7:29 ` Wei Yang 2015-05-04 7:07 ` [PATCH V3 3/9] powerpc/pci: remove PCI devices in reverse order Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-04 7:07 ` [PATCH V3 4/9] powerpc/eeh: cache address range just for normal device Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-04 7:07 ` [PATCH V3 5/9] powerpc/eeh: create EEH_PE_VF for VF PE Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-11 2:37 ` Gavin Shan 2015-05-11 2:37 ` Gavin Shan 2015-05-11 6:25 ` Wei Yang 2015-05-11 6:25 ` Wei Yang 2015-05-12 6:28 ` Gavin Shan 2015-05-12 6:28 ` Gavin Shan 2015-05-12 7:52 ` Wei Yang 2015-05-12 7:52 ` Wei Yang 2015-05-04 7:07 ` [PATCH V3 6/9] powerpc/powernv: create/release eeh_dev for VF Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-11 2:48 ` Gavin Shan 2015-05-11 2:48 ` Gavin Shan 2015-05-12 8:06 ` Wei Yang 2015-05-12 8:06 ` Wei Yang 2015-05-12 23:09 ` Gavin Shan 2015-05-12 23:09 ` Gavin Shan 2015-05-04 7:07 ` [PATCH V3 7/9] powerpc/powernv: Support EEH reset for VFs Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-11 3:03 ` Gavin Shan 2015-05-11 3:03 ` Gavin Shan 2015-05-04 7:07 ` [PATCH V3 8/9] powerpc/powernv: Support PCI config restore " Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-11 4:22 ` Gavin Shan 2015-05-11 4:22 ` Gavin Shan 2015-05-12 1:31 ` Wei Yang 2015-05-12 1:31 ` Wei Yang 2015-05-12 6:34 ` Gavin Shan 2015-05-12 6:34 ` Gavin Shan 2015-05-12 8:16 ` Wei Yang 2015-05-12 8:16 ` Wei Yang 2015-05-12 23:16 ` Gavin Shan 2015-05-12 23:16 ` Gavin Shan 2015-05-04 7:07 ` [PATCH V3 9/9] powerpc/eeh: handle VF PE properly Wei Yang 2015-05-04 7:07 ` Wei Yang 2015-05-13 1:16 ` Gavin Shan 2015-05-13 1:16 ` Gavin Shan 2015-05-14 9:35 ` Wei Yang 2015-05-14 9:35 ` Wei Yang 2015-05-14 12:15 ` Gavin Shan 2015-05-14 12:15 ` Gavin Shan 2015-05-14 10:02 ` Wei Yang 2015-05-14 10:02 ` Wei Yang 2015-05-14 12:30 ` Gavin Shan 2015-05-14 12:30 ` Gavin Shan
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20150511021331.GC3694@gwshan \ --to=gwshan@linux.vnet.ibm.com \ --cc=bhelgaas@google.com \ --cc=linux-pci@vger.kernel.org \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=weiyang@linux.vnet.ibm.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.