* Venki Pallipadi ([EMAIL PROTECTED]) 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 <[EMAIL PROTECTED]> > > 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-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/