Thanks Rafael, the code looks a lot more natural IMHO. this bit: > +#ifdef DEBUG > +static void dbg_get_time(struct timeval *start) > +{ > + do_gettimeofday(start); > +} is a plain wrapper over gettimeofday so you might want to inline it. (although GCC will do it too most of the time) Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html