Re: [PATCH] PCI: Restrict device disabled status check to DT

From: Vitaly Kuznetsov
Date: Thu Apr 20 2023 - 05:31:05 EST


Bjorn Helgaas <helgaas@xxxxxxxxxx> writes:

> [+cc Vitaly, Jesse, Tony, Andy, regressions, regarding reports of
> hang or crash during boot in igb driver, some with AWS Xen]
>
> On Wed, Apr 19, 2023 at 02:35:13PM -0500, Rob Herring wrote:
>> Commit 6fffbc7ae137 ("PCI: Honor firmware's device disabled status")
>> checked the firmware device status for both DT and ACPI devices. That
>> caused a regression in some ACPI systems. The exact reason isn't clear.
>> It's possibly a firmware bug. For now, at least, refactor the check to
>> be for DT based systems only.
>>
>> Note that the original implementation leaked a refcount which is now
>> correctly handled.
>>
>> Fixes: 6fffbc7ae137 ("PCI: Honor firmware's device disabled status")
>> Link: https://lore.kernel.org/all/m2fs9lgndw.fsf@xxxxxxxxx/
>> Reported-by: Donald Hunter <donald.hunter@xxxxxxxxx>
>> Cc: Binbin Zhou <zhoubinbin@xxxxxxxxxxx>
>> Cc: Liu Peibao <liupeibao@xxxxxxxxxxx>
>> Cc: Huacai Chen <chenhuacai@xxxxxxxxxxx>
>> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
>
> Applied to for-linus for (hopefully) v6.3. I added:
>
> [bhelgaas: Per ACPI r6.5, sec 6.3.7, for devices on an enumerable
> bus, _STA must return with bit[0] ("device is present") set]
>
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=217317
>
> It would be really great if anybody who has seen this issue could test
> and report whether this patch solves it.
>

[Cc: Myron]

I can confirm the patch fixes the issue I've reported with AWS Xen
instances, so:

Tested-by: Vitaly Kuznetsov <vkuznets@xxxxxxxxxx>

--
Vitaly