On Sunday 06 September 2009, Ingo Molnar wrote:
>
> 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)
I have a rule of not using 'inline' except for in headers. In this case,
however, it's almost like a header, so I'll follow your suggestion.
Thanks,
Rafael
--
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
