Use of the time_after() and time_after_eq() macros, defined at linux/jiffies.h, which deal with wrapping correctly and are nicer to read. Signed-off-by: Marcelo Feitoza Parisi --- linux/drivers/net/e1000/e1000_ethtool.c 2005-07-13 17:52:19.000000000 -0300 +++ linux-kj/drivers/net/e1000/e1000_ethtool.c 2005-07-15 23:51:00.719718560 -0300 @@ -30,6 +30,7 @@ #include "e1000.h" +#include #include extern char e1000_driver_name[]; @@ -1389,12 +1390,12 @@ * enough time to complete the receives, if it's * exceeded, break and error off */ - } while (good_cnt < 64 && jiffies < (time + 20)); + } while (good_cnt < 64 && time_after(jiffies, time + 20)); if(good_cnt != 64) { ret_val = 13; /* ret_val is the same as mis-compare */ break; } - if(jiffies >= (time + 2)) { + if(time_after_eq(jiffies, time + 2)) { ret_val = 14; /* error code for time out error */ break; }