On Mon, Nov 09, 2015 at 10:53:17AM +1100, Alexey Kardashevskiy wrote: >On 11/08/2015 10:30 AM, Wei Yang wrote: >>This patchset enables EEH on SRIOV VFs. The general idea is to create proper >>VF edev and VF PE and handle them properly. >> >>Different from the Bus PE, VF PE just contain one VF. This introduces the >>difference of EEH error handling on a VF PE. Generally, it has several >>differences. >> >>First, the VF's removal and re-enumerate rely on its PF. VF has a tight >>relationship between its PF. This is not proper to enumerate a VF by usual >>scan procedure. That's why virtfn_add/virtfn_remove are exported in this patch >>set. >> >>Second, the reset/restore of a VF is done in kernel space. FW is not aware of >>the VF, this means the usual reset function done in FW will not work. One of >>the patch will imitate the reset/restore function in kernel space. >> >>Third, the VF may be removed during the PF's error_detected function. In this >>case, the original error_detected->slot_reset->resume sequence is not proper >>to those removed VFs, since they are re-created by PF in a fresh state. A flag >>in eeh_dev is introduce to mark the eeh_dev is in error state. By doing so, we >>track whether this device needs to be reset or not. >> >>This has been tested both on host and in guest on Power8 with latest kernel >>version. > > >This does not apply on top of neither Linus master tree >(git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git sha1 >ce5c2d2) nor Michael's PPC next tree >(git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git sha1 >8bdf2023). What did you base your work on? > I didn't based this on the latest code. This is based on v4.3, commit "6a13feb Linux 4.3". > >-- >Alexey -- Richard Yang Help you, Help me -- 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
