Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-21 Thread Gavin Shan
On Tue, Mar 21, 2017 at 02:34:46PM +, Eli Cohen wrote: >> If we want to talk about the ABI, I would suggest drawing from existing >> ABIs. We already have >> drivers_autoprobe as part of the standard sysfs ABI, so if we want a binary >> switch, then >>sriov_drivers_autoprobe might be a logi

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-21 Thread Gavin Shan
On Tue, Mar 21, 2017 at 08:23:29AM -0600, Alex Williamson wrote: >On Tue, 21 Mar 2017 20:25:18 +1100 >Gavin Shan wrote: >> On Tue, Mar 21, 2017 at 12:01:58AM -0600, Alex Williamson wrote: >> >On Tue, 21 Mar 2017 16:43:05 +1100 >> >Gavin Shan wrote: >> >> On Mon, Mar 20, 2017 at 10:57:08PM -0600

RE: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-21 Thread Eli Cohen
> If we want to talk about the ABI, I would suggest drawing from existing ABIs. > We already have > drivers_autoprobe as part of the standard sysfs ABI, so if we want a binary > switch, then >sriov_drivers_autoprobe might be a logical choice. If you're concerned about >this mythical overhead

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-21 Thread Alex Williamson
On Tue, 21 Mar 2017 20:25:18 +1100 Gavin Shan wrote: > On Tue, Mar 21, 2017 at 12:01:58AM -0600, Alex Williamson wrote: > >On Tue, 21 Mar 2017 16:43:05 +1100 > >Gavin Shan wrote: > >> On Mon, Mar 20, 2017 at 10:57:08PM -0600, Alex Williamson wrote: > >> >On Mon, 20 Mar 2017 18:34:23 -0500 >

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-21 Thread Bodong Wang
On 3/21/2017 1:01 AM, Alex Williamson wrote: On Tue, 21 Mar 2017 16:43:05 +1100 Gavin Shan wrote: On Mon, Mar 20, 2017 at 10:57:08PM -0600, Alex Williamson wrote: On Mon, 20 Mar 2017 18:34:23 -0500 Bodong Wang wrote: .../... Bodong, I'm not sure if there is a requirement to load driver fo

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-21 Thread Gavin Shan
On Tue, Mar 21, 2017 at 12:01:58AM -0600, Alex Williamson wrote: >On Tue, 21 Mar 2017 16:43:05 +1100 >Gavin Shan wrote: >> On Mon, Mar 20, 2017 at 10:57:08PM -0600, Alex Williamson wrote: >> >On Mon, 20 Mar 2017 18:34:23 -0500 >> >Bodong Wang wrote: >> >> .../... >> >> >> > Bodong, I'm not su

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-20 Thread Alex Williamson
On Tue, 21 Mar 2017 16:43:05 +1100 Gavin Shan wrote: > On Mon, Mar 20, 2017 at 10:57:08PM -0600, Alex Williamson wrote: > >On Mon, 20 Mar 2017 18:34:23 -0500 > >Bodong Wang wrote: > > .../... > > >> > Bodong, I'm not sure if there is a requirement to load driver for the > >> > specified numb

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-20 Thread Gavin Shan
On Mon, Mar 20, 2017 at 10:57:08PM -0600, Alex Williamson wrote: >On Mon, 20 Mar 2017 18:34:23 -0500 >Bodong Wang wrote: .../... >> > Bodong, I'm not sure if there is a requirement to load driver for the >> > specified number of VFs? That indicates no driver will be loaded for >> > other VFs. If

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-20 Thread Alex Williamson
On Mon, 20 Mar 2017 18:34:23 -0500 Bodong Wang wrote: > On 3/20/2017 6:07 PM, Gavin Shan wrote: > > On Mon, Mar 20, 2017 at 05:14:34PM +0200, bod...@mellanox.com wrote: > >> From: Bodong Wang > >> > >> Sometimes it is not desirable to probe the virtual functions after > >> SRIOV is enabled. Th

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-20 Thread Bodong Wang
On 3/20/2017 7:24 PM, Gavin Shan wrote: On Mon, Mar 20, 2017 at 06:34:23PM -0500, Bodong Wang wrote: On 3/20/2017 6:07 PM, Gavin Shan wrote: On Mon, Mar 20, 2017 at 05:14:34PM +0200, bod...@mellanox.com wrote: From: Bodong Wang Sometimes it is not desirable to probe the virtual functions aft

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-20 Thread Gavin Shan
On Mon, Mar 20, 2017 at 06:34:23PM -0500, Bodong Wang wrote: >On 3/20/2017 6:07 PM, Gavin Shan wrote: >>On Mon, Mar 20, 2017 at 05:14:34PM +0200, bod...@mellanox.com wrote: >>>From: Bodong Wang >>> >>>Sometimes it is not desirable to probe the virtual functions after >>>SRIOV is enabled. This can

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-20 Thread Bodong Wang
On 3/20/2017 6:07 PM, Gavin Shan wrote: On Mon, Mar 20, 2017 at 05:14:34PM +0200, bod...@mellanox.com wrote: From: Bodong Wang Sometimes it is not desirable to probe the virtual functions after SRIOV is enabled. This can save host side resource usage by VF instances which would be eventually p

Re: [PATCH] pci/sriov: Add an option to probe VFs or not before enabling SR-IOV

2017-03-20 Thread Gavin Shan
On Mon, Mar 20, 2017 at 05:14:34PM +0200, bod...@mellanox.com wrote: >From: Bodong Wang > >Sometimes it is not desirable to probe the virtual functions after >SRIOV is enabled. This can save host side resource usage by VF >instances which would be eventually probed to VMs. > >Added a new PCI sysfs