> 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

Reply via email to