> Hi Gavin, > > On Fri, 24 Feb 2012 17:38:17 +0800 Gavin Shan <sha...@linux.vnet.ibm.com> > wrote: > > > > 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 s...@canb.auug.org.au > http://www.canb.auug.org.au/~sfr/ _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev