Re: HP Elitebook 840 G5 - acpi failure

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

 



On Thu, Nov 18, 2021 at 2:41 PM Ian Kumlien <ian.kumlien@xxxxxxxxx> wrote:
>
> So, hard to capture, crashes faster with minimal output:
> https://photos.app.goo.gl/9p56395EdhNBuikQA
>
> First time i didn't even see the oops at all...

The dynamic debug output is still not present.

Please apply the attached patch (you may need to adjust it for 5.15 -
the idea is to comment out the execution of _ON and _OFF) and boot
with the dyndbg in power.c enabled.

It should not crash this time, so you should be able to collect full
dmesg and send it (as an attachment, please).
---
 drivers/acpi/power.c |   24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

Index: linux-pm/drivers/acpi/power.c
===================================================================
--- linux-pm.orig/drivers/acpi/power.c
+++ linux-pm/drivers/acpi/power.c
@@ -362,13 +362,13 @@ static int __acpi_power_on(struct acpi_p
 {
 	acpi_handle handle = resource->device.handle;
 	struct acpi_power_dependent_device *dep;
-	acpi_status status = AE_OK;
+	//acpi_status status = AE_OK;
 
-	status = acpi_evaluate_object(handle, "_ON", NULL, NULL);
-	if (ACPI_FAILURE(status)) {
-		resource->state = ACPI_POWER_RESOURCE_STATE_UNKNOWN;
-		return -ENODEV;
-	}
+	//status = acpi_evaluate_object(handle, "_ON", NULL, NULL);
+	//if (ACPI_FAILURE(status)) {
+	//	resource->state = ACPI_POWER_RESOURCE_STATE_UNKNOWN;
+	//	return -ENODEV;
+	//}
 
 	resource->state = ACPI_POWER_RESOURCE_STATE_ON;
 
@@ -420,13 +420,13 @@ static int acpi_power_on(struct acpi_pow
 static int __acpi_power_off(struct acpi_power_resource *resource)
 {
 	acpi_handle handle = resource->device.handle;
-	acpi_status status;
+	//acpi_status status;
 
-	status = acpi_evaluate_object(handle, "_OFF", NULL, NULL);
-	if (ACPI_FAILURE(status)) {
-		resource->state = ACPI_POWER_RESOURCE_STATE_UNKNOWN;
-		return -ENODEV;
-	}
+	//status = acpi_evaluate_object(handle, "_OFF", NULL, NULL);
+	//if (ACPI_FAILURE(status)) {
+	//	resource->state = ACPI_POWER_RESOURCE_STATE_UNKNOWN;
+	//	return -ENODEV;
+	//}
 
 	resource->state = ACPI_POWER_RESOURCE_STATE_OFF;
 

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux