Re: [PATCH 20/21] Introduce struct eeh_stats for EEH

Hi Gavin,

On Fri, 24 Feb 2012 17:38:17 +0800 Gavin Shan [off-list ref] wrote:

quoted

diff --git a/arch/powerpc/include/asm/eeh.h b/arch/powerpc/include/asm/eeh.h
index 1310971..226c9a5 100644
--- a/arch/powerpc/include/asm/eeh.h
+++ b/arch/powerpc/include/asm/eeh.h

@@ -98,6 +98,21 @@ struct eeh_ops {
 	int (*configure_bridge)(struct device_node *dn);
 };
 
+/*
+ * The struct is used to maintain the EEH global statistic
+ * information. Besides, the EEH global statistics will be
+ * exported to user space through procfs
+ */
+struct eeh_stats {
+	unsigned long no_device;	/* PCI device not found		*/
+	unsigned long no_dn;		/* OF node not found		*/
+	unsigned long no_cfg_addr;	/* Config address not found	*/
+	unsigned long ignored_check;	/* EEH check skipped		*/
+	unsigned long total_mmio_ffs;	/* Total EEH checks		*/
+	unsigned long false_positives;	/* Unnecessary EEH checks	*/
+	unsigned long slot_resets;	/* PE reset			*/
+};

If this is used in only one place, there is not much point in putting it
in a header file.

Thanks, Stephen. I'll move it to eeh.c in next revision.

Thanks,
Gavin

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/