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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


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:
>> 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:
> 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>


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux