> +struct eeh_stats { > + unsigned int no_device; /* PCI device not found */ ... > + "no device =%d\n" ... Use %u (for all the stats), you really don't want negative values printed. I've NFI how long wrapping these counters might take! If it is feasable (maybe much above 100Hz) then you need 64bit counters. David