Re: e1000e pci_disable_link_state_locked() issues

2015-05-22 Thread Yijing Wang
On 2015/5/21 3:47, Bjorn Helgaas wrote: > I think we have some issues with the e1000e usage of > pci_disable_link_state_locked(), which Yinghai added with 9f728f53dd70 > ("PCI/e1000e: Add and use pci_disable_link_state_locked()"). > > That fixed an AER deadlock in the following path, where pci_bus

Re: e1000e pci_disable_link_state_locked() issues

2015-05-22 Thread Yijing Wang
On 2015/5/21 3:47, Bjorn Helgaas wrote: > I think we have some issues with the e1000e usage of > pci_disable_link_state_locked(), which Yinghai added with 9f728f53dd70 > ("PCI/e1000e: Add and use pci_disable_link_state_locked()"). > > That fixed an AER deadlock in the following path, where pci_bus

RE: [Intel-wired-lan] e1000e pci_disable_link_state_locked() issues

2015-05-21 Thread Lubetkin, YanirX
> To: Yinghai Lu; Kirsher, Jeffrey T > Cc: linux-...@vger.kernel.org; intel-wired-...@lists.osuosl.org; linux- > ker...@vger.kernel.org; netdev@vger.kernel.org > Subject: [Intel-wired-lan] e1000e pci_disable_link_state_locked() issues > > I think we have s

e1000e pci_disable_link_state_locked() issues

2015-05-20 Thread Bjorn Helgaas
I think we have some issues with the e1000e usage of pci_disable_link_state_locked(), which Yinghai added with 9f728f53dd70 ("PCI/e1000e: Add and use pci_disable_link_state_locked()"). That fixed an AER deadlock in the following path, where pci_bus_sem is held by pci_walk_bus(), and we deadlocked