2008-06-11 00:29:53

by Ralph Campbell

[permalink] [raw]
Subject: bug in jiffy time comparison macros

It may just be me :-) but this looks like a bug:
In the current include/linux/jiffies.h:

#define time_before(a,b) time_after(b,a)

Shouldn't this be:

#define time_before(a,b) time_after_eq(b,a)

There are a number of similar macros which fail
to change before to after_eq, or before_eq to after,
etc.


2008-06-11 02:51:40

by Johannes Weiner

[permalink] [raw]
Subject: Re: bug in jiffy time comparison macros

Hi,

Ralph Campbell <[email protected]> writes:

> It may just be me :-) but this looks like a bug:
> In the current include/linux/jiffies.h:
>
> #define time_before(a,b) time_after(b,a)
>
> Shouldn't this be:
>
> #define time_before(a,b) time_after_eq(b,a)

Then it would be time_before_eq()..?

If a is before b, b must be after a. If they might be equal, than from
both points of view.

Hannes