IPMI: Convert system interface defines to an enum
authorCorey Minyard <cminyard@mvista.com>
Tue, 29 Apr 2008 08:01:08 +0000 (01:01 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 29 Apr 2008 15:06:15 +0000 (08:06 -0700)
Convert the #defines for statistics into an enum in the IPMI system interface
and remove the unused timeout_restart statistic.  And comment what these
statistics mean.

Signed-off-by: Corey Minyard <cminyard@mvista.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/char/ipmi/ipmi_si_intf.c

index 286c042a0c669349cdbbcb5be7307f47f241ae1d..ba7e75b731c6e92539cd83b41204180785c19069 100644 (file)
@@ -124,22 +124,50 @@ static struct device_driver ipmi_driver =
 /*
  * Indexes into stats[] in smi_info below.
  */
+enum si_stat_indexes {
+       /*
+        * Number of times the driver requested a timer while an operation
+        * was in progress.
+        */
+       SI_STAT_short_timeouts = 0,
+
+       /*
+        * Number of times the driver requested a timer while nothing was in
+        * progress.
+        */
+       SI_STAT_long_timeouts,
+
+       /* Number of times the interface was idle while being polled. */
+       SI_STAT_idles,
+
+       /* Number of interrupts the driver handled. */
+       SI_STAT_interrupts,
+
+       /* Number of time the driver got an ATTN from the hardware. */
+       SI_STAT_attentions,
 
-#define SI_STAT_short_timeouts         0
-#define SI_STAT_long_timeouts          1
-#define SI_STAT_timeout_restarts       2
-#define SI_STAT_idles                  3
-#define SI_STAT_interrupts             4
-#define SI_STAT_attentions             5
-#define SI_STAT_flag_fetches           6
-#define SI_STAT_hosed_count            7
-#define SI_STAT_complete_transactions  8
-#define SI_STAT_events                 9
-#define SI_STAT_watchdog_pretimeouts   10
-#define SI_STAT_incoming_messages      11
-
-/* If you add a stat, you must update this value. */
-#define SI_NUM_STATS                   12
+       /* Number of times the driver requested flags from the hardware. */
+       SI_STAT_flag_fetches,
+
+       /* Number of times the hardware didn't follow the state machine. */
+       SI_STAT_hosed_count,
+
+       /* Number of completed messages. */
+       SI_STAT_complete_transactions,
+
+       /* Number of IPMI events received from the hardware. */
+       SI_STAT_events,
+
+       /* Number of watchdog pretimeouts. */
+       SI_STAT_watchdog_pretimeouts,
+
+       /* Number of asyncronous messages received. */
+       SI_STAT_incoming_messages,
+
+
+       /* This *must* remain last, add new values above this. */
+       SI_NUM_STATS
+};
 
 struct smi_info
 {
@@ -2399,8 +2427,6 @@ static int stat_file_read_proc(char *page, char **start, off_t off,
                       smi_get_stat(smi, short_timeouts));
        out += sprintf(out, "long_timeouts:         %u\n",
                       smi_get_stat(smi, long_timeouts));
-       out += sprintf(out, "timeout_restarts:      %u\n",
-                      smi_get_stat(smi, timeout_restarts));
        out += sprintf(out, "idles:                 %u\n",
                       smi_get_stat(smi, idles));
        out += sprintf(out, "interrupts:            %u\n",