On Tue, Feb 10, 2015 at 11:26:19AM +1100, Benjamin Herrenschmidt wrote:

On Thu, 2015-01-15 at 10:27 +0800, Wei Yang wrote:

quoted

+       if ((retval = pcibios_sriov_enable(dev, initial))) {
+               dev_err(&dev->dev, "Failure %d from
pcibios_sriov_setup()\n",
+                       retval);
+               return retval;
+       }
+

Don't we want pcibios_sriov_enable() to be able to crop the number
of VFs or do we think any resource limits have been applied
already ?

The second parameter "initial" is the number of VFs will be enabled. Arch
dependent function will check the resources for these number of VFs.

Do I catch your question correctly?

Ben.

-- 
Richard Yang
Help you, Help me