On Fri, Apr 03, 2020 at 04:45:47PM +1100, Oliver O'Halloran wrote: > On Mon, 2020-03-30 at 15:56 +1100, Sam Bobroff wrote: > > Because the bus notifier calls eeh_rmv_from_parent_pe() (via > > eeh_remove_device()) when a VF is removed, the call in > > remove_sriov_vf_pdns() is redundant. > > eeh_rmv_from_parent_pe() won't actually remove the device if the > recovering flag is set on the PE. Are you sure we're not introducing a > race here? >
Ah, I assume you're referring to the difference between calling eeh_remove_device() and directly calling eeh_rmv_from_parent_pe(), where the behaviour for PE's with EEH_PE_KEEP set is subtly different. I'll take a closer look at it and make sure to explain it better in v2.
signature.asc
Description: PGP signature
