Hi Vijay,
Trying to add AER support for Mellanox NIC in SRIOV environment, while evaluating/testing encountered a problem which led me to your patch accepted as part of kernel 3.8, commit ID "918b4053184c0ca22236e70e299c5343eea35304".

Have some concerns/questions on:
When working in SRIOV environment VFs may be un-attached, having no driver assigned to, or may be attached to Virtual machine to work in some pass-through mode. Once working in KVM setup there is pci-stub driver which is loaded in the HYP/PF for a given attached VF.

I'm using the aer-inject kernel module and its corresponding aer-inject tool to simulate an error in the HYP. In both cases your commit will cause the AER recovery to fail as there is no driver assigned to PF's VFs that supports AER, comparing the code before your change.

How such cases should work ? my expectation was that the PF will get the error detected message then will recognize whether issue is its own or one of its VFs and work accordingly, in current code looks like recovery failed as part of "voting" once there is no AER handler assigned to the VFs.

Yishai
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to