On Thu, Aug 13, 2015 at 2:15 PM, Tony Luck <tony.l...@gmail.com> wrote:
>
> could you at least care enough to write that as
>
> if (time_after(now, last + HZ)) {
Absolutely not. That would be wrong.
"time_after()" covers half the "unsigned long" space (very much on
purpose). We want the cached case to trigger _only_ for the much
tighter case of exactly 1000 jiffies.
Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
