On Wed, May 15, 2013 at 7:12 PM, Greg Rose <gregory.v.rose@xxxxxxxxx> wrote: > I'm really not a fan of this. Seems to me the tail is wagging the dog > here. Fix the driver to work without a PF driver being present. Greg, Alex, As I wrote over the V1 thread, currently we can't go and patch mlx4 to use the sysfs API nor defer the call from within our probe function to enable sriov since this requires some firmware change to allow enabling SRIOV after some resources are initialized/provisioned. Hence the patch suggested here or any other patch we can agree on which will make sure that VF probing is done only once the PF is ready is preferred, I think. I wasn't sure to totally follow on the argument that things need to work when the PF is absent in the sense there's no driver instance around over which the PF is probed, if you can explain little better, that would help. Or. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html