Re: [PATCH v2 2/2] PCI: pciehp: Prevent deadlock on disconnect
From: Sinan Kaya
Date: Sun Aug 18 2019 - 22:28:42 EST
On 8/12/2019 10:31 AM, Mika Westerberg wrote:
> +int pciehp_card_present_or_link_active(struct controller *ctrl)
> - return pciehp_card_present(ctrl) || pciehp_check_link_active(ctrl);
> + int ret;
> + ret = pciehp_card_present(ctrl);
> + if (ret)
> + return ret;
> + return pciehp_check_link_active(ctrl);
The semantics of this function changed here. Before it was checking for
either presence detect bit or link active bit. Now, it is looking to
have both set.
There are PCI controllers that won't report presence detect correctly,
but still report link active.
I think you want
if (ret < 0)
here to match the previous behavior while still handling device removal.