* Venki Pallipadi (venkatesh.pallipadi@xxxxxxxxx) wrote: > Can you check the test patch below (over latest git) and let me know whether it > resolves the issue. > the patch fixes the issue for me, thanks a lot. Eric > Enable C3 without bm control only for CST based C3. > > Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@xxxxxxxxx> > > Index: linux-2.6/drivers/acpi/processor_idle.c > =================================================================== > --- linux-2.6.orig/drivers/acpi/processor_idle.c 2007-07-31 04:29:26.000000000 -0700 > +++ linux-2.6/drivers/acpi/processor_idle.c 2007-07-31 04:52:50.000000000 -0700 > @@ -969,11 +969,17 @@ > } > > if (pr->flags.bm_check) { > - /* bus mastering control is necessary */ > if (!pr->flags.bm_control) { > - /* In this case we enter C3 without bus mastering */ > - ACPI_DEBUG_PRINT((ACPI_DB_INFO, > - "C3 support without bus mastering control\n")); > + if (pr->flags.has_cst != 1) { > + /* bus mastering control is necessary */ > + ACPI_DEBUG_PRINT((ACPI_DB_INFO, > + "C3 support requires BM control\n")); > + return; > + } else { > + /* Here we enter C3 without bus mastering */ > + ACPI_DEBUG_PRINT((ACPI_DB_INFO, > + "C3 support without BM control\n")); > + } > } > } else { > /* - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html