On Mon, 2008-10-27 at 14:35 -0700, Alexey Starikovskiy wrote: > With the better solution for EC interrupt storm issue, > there is no need to use msleep over udelay. > Why is msleep replaced by udelay again? When udelay is used, CPU will execute the loop while doing EC transaction.Although this only affects some laptops, this still is not very reasonable. For example: EC GPE interrupt storms; there is no EC interrupt while doing some EC transaction; force_poll is 1. IMO it is necessary to get the root cause of bug. We had better not revert something for some bugs and when another bug appears, we revert it again. Thanks. > References: > http://bugzilla.kernel.org/show_bug.cgi?id=11810 > http://bugzilla.kernel.org/show_bug.cgi?id=10724 > > > Signed-off-by: Alexey Starikovskiy <astarikovskiy@xxxxxxx> > --- > > drivers/acpi/ec.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > > diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c > index ef42316..3ef5b79 100644 > --- a/drivers/acpi/ec.c > +++ b/drivers/acpi/ec.c > @@ -239,10 +239,10 @@ static int ec_check_sci(struct acpi_ec *ec, u8 state) > static int ec_poll(struct acpi_ec *ec) > { > unsigned long delay = jiffies + msecs_to_jiffies(ACPI_EC_DELAY); > - msleep(1); > + udelay(ACPI_EC_UDELAY); > while (time_before(jiffies, delay)) { > gpe_transaction(ec, acpi_ec_read_status(ec)); > - msleep(1); > + udelay(ACPI_EC_UDELAY); > if (ec_transaction_done(ec)) > return 0; > } > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
