We can roll this into the base code. Then, I would like to see a new interface to get these things, rather than have upper code poking around in ACPICA globals. Like: AcpiGetStatistics We could return event counts, gpe counts, total SCI count, # methods executed, etc., etc. Bob >-----Original Message----- >From: Len Brown [mailto:lenb@xxxxxxxxxx] >Sent: Friday, November 02, 2007 10:05 AM >To: Moore, Robert >Cc: Linux-acpi@xxxxxxxxxxxxxxx >Subject: [ACPICA PATCH] fixed event counters > >Bob, >I'm planning to add the following counters to the ACPICA code in Linux. > >thanks, >-Len > >diff --git a/drivers/acpi/events/evevent.c b/drivers/acpi/events/evevent.c >index a1f87b5..8bf4dbe 100644 >--- a/drivers/acpi/events/evevent.c >+++ b/drivers/acpi/events/evevent.c >@@ -261,7 +261,7 @@ u32 acpi_ev_fixed_event_detect(void) > enable_bit_mask)) { > > /* Found an active (signalled) event */ >- >+ acpi_fixed_event_count[i]++; > int_status |= acpi_ev_fixed_event_dispatch((u32) i); > } > } >diff --git a/drivers/acpi/utilities/utglobal.c >b/drivers/acpi/utilities/utglobal.c >index af33358..fbb9bec 100644 >--- a/drivers/acpi/utilities/utglobal.c >+++ b/drivers/acpi/utilities/utglobal.c >@@ -673,6 +673,8 @@ void acpi_ut_init_globals(void) > /* GPE support */ > > acpi_gpe_count = 0; >+ for (i = 0; i < ACPI_NUM_FIXED_EVENTS; i++) >+ acpi_fixed_event_count[i] = 0; > acpi_gbl_gpe_xrupt_list_head = NULL; > acpi_gbl_gpe_fadt_blocks[0] = NULL; > acpi_gbl_gpe_fadt_blocks[1] = NULL; >diff --git a/include/acpi/acglobal.h b/include/acpi/acglobal.h >index 24c3f05..a27ba06 100644 >--- a/include/acpi/acglobal.h >+++ b/include/acpi/acglobal.h >@@ -120,6 +120,7 @@ extern u32 acpi_gbl_nesting_level; > /* Event counters */ > > ACPI_EXTERN u32 acpi_gpe_count; >+ACPI_EXTERN u32 acpi_fixed_event_count[ACPI_NUM_FIXED_EVENTS]; > > /* Support for dynamic control method tracing mechanism */ > - 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
