On Wednesday, March 04, 2015 06:48:35 PM Lee, Chun-Yi wrote: > From: Nan Li <nli@xxxxxxxx> > > The count field is an unsigned 32bit value, and the > counter_show() function should also treat it as a unsigned > value. > > Otherwise the counter may show negative number as we found on a > machine: > ... > gpe23: 0 invalid > gpe24: -2071733 enabled > gpe25: 0 invalid > ... > gpe_all: -2070980 > sci: -2070949 > > Cc: "Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx> > Cc: Len Brown <lenb@xxxxxxxxxx> > Cc: Thomas Renninger <trenn@xxxxxxx> > Signed-off-by: Nan Li <nli@xxxxxxxx> > Signed-off-by: Lee, Chun-Yi <jlee@xxxxxxxx> Queued up for 4.1, thanks! > --- > drivers/acpi/sysfs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/acpi/sysfs.c b/drivers/acpi/sysfs.c > index 13e577c..0876d77 100644 > --- a/drivers/acpi/sysfs.c > +++ b/drivers/acpi/sysfs.c > @@ -527,7 +527,7 @@ static ssize_t counter_show(struct kobject *kobj, > acpi_irq_not_handled; > all_counters[num_gpes + ACPI_NUM_FIXED_EVENTS + COUNT_GPE].count = > acpi_gpe_count; > - size = sprintf(buf, "%8d", all_counters[index].count); > + size = sprintf(buf, "%8u", all_counters[index].count); > > /* "gpe_all" or "sci" */ > if (index >= num_gpes + ACPI_NUM_FIXED_EVENTS) > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- 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
