Re: 2.6.21-rc1: known regressions (part 2)

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ingo Molnar
Date: Monday, March 5, 2007 - 9:41 am

* Michael S. Tsirkin <mst@mellanox.co.il> wrote:


you could probably get quite a bit further in bisecting the other 
breakage, by using the following method:

manully apply the patch below to 81450b73dde and retest. It will most 
likely work. Then FIRST unapply the patch and mark the tree via 
'git-bisect good' and continue the bisection. Then try to apply the 
patch again. If it's already included - ignore the rejected patch. 
Whenever git-bisect offers you a new commit, just try to apply the 
patch. Ok? This way you'll be able to avoid the known ACPI breakage, and 
zoom in on the unknown breakage.

	Ingo

---------------->
commit f3ccb06f3b8e0cf42b579db21f3ca7f17fcc3f38
Author: Alexey Starikovskiy <alexey.y.starikovskiy@linux.intel.com>
Date:   Tue Feb 13 02:35:50 2007 -0500

    ACPI: Disable wake GPEs only once.
    
    fixes Suspend/Resume regressions due to recent ACPICA update.
    
    Signed-off-by: Alexey Starikovskiy <alexey.y.starikovskiy@linux.intel.com>
    Signed-off-by: Len Brown <len.brown@intel.com>

diff --git a/drivers/acpi/events/evgpe.c b/drivers/acpi/events/evgpe.c
index dfac3ec..635ba44 100644
--- a/drivers/acpi/events/evgpe.c
+++ b/drivers/acpi/events/evgpe.c
@@ -636,17 +636,6 @@ acpi_ev_gpe_dispatch(struct acpi_gpe_event_info *gpe_event_info, u32 gpe_number)
 		}
 	}
 
-	if (!acpi_gbl_system_awake_and_running) {
-		/*
-		 * We just woke up because of a wake GPE. Disable any further GPEs
-		 * until we are fully up and running (Only wake GPEs should be enabled
-		 * at this time, but we just brute-force disable them all.)
-		 * 1) We must disable this particular wake GPE so it won't fire again
-		 * 2) We want to disable all wake GPEs, since we are now awake
-		 */
-		(void)acpi_hw_disable_all_gpes();
-	}
-
 	/*
 	 * Dispatch the GPE to either an installed handler, or the control method
 	 * associated with this GPE (_Lxx or _Exx). If a handler exists, we invoke
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: 2.6.21-rc1: known regressions (part 2), Ingo Molnar, (Fri Mar 2, 1:04 am)
Re: 2.6.21-rc1: known regressions (part 2), Ingo Molnar, (Fri Mar 2, 3:20 am)
[patch] KVM: T60 resume fix, Ingo Molnar, (Fri Mar 2, 3:22 am)
Re: [patch] KVM: T60 resume fix, Michael S. Tsirkin, (Fri Mar 2, 4:39 am)
Re: 2.6.21-rc1: known regressions (part 2), Linus Torvalds, (Fri Mar 2, 9:36 am)
Re: [patch] KVM: T60 resume fix, Avi Kivity, (Sat Mar 3, 1:21 am)
Re: [patch] KVM: T60 resume fix, Avi Kivity, (Sat Mar 3, 1:22 am)
Re: [patch] KVM: T60 resume fix, Andrew Morton, (Sat Mar 3, 4:57 am)
Re: [patch] KVM: T60 resume fix, Ingo Molnar, (Mon Mar 5, 1:22 am)
Re: [patch] KVM: T60 resume fix, Michael S. Tsirkin, (Mon Mar 5, 3:23 am)
Re: [patch] KVM: T60 resume fix, Ingo Molnar, (Mon Mar 5, 3:29 am)
Re: 2.6.21-rc1: known regressions (part 2), Ingo Molnar, (Mon Mar 5, 7:04 am)
Re: 2.6.21-rc1: known regressions (part 2), Michael S. Tsirkin, (Mon Mar 5, 8:44 am)
Re: 2.6.21-rc1: known regressions (part 2), Michael S. Tsirkin, (Mon Mar 5, 9:14 am)
Re: 2.6.21-rc1: known regressions (part 2), Ingo Molnar, (Mon Mar 5, 9:41 am)
Re: 2.6.21-rc1: known regressions (part 2), Jens Axboe, (Mon Mar 5, 11:16 am)