[Linux-ia64] [patch] fix compares of jiffies

From: Tim Schmielau <tim_at_physik3.uni-rostock.de>
Date: 2002-09-22 21:45:51
Dear Linux kernel code maintainer,

on rechecking the current stable kernel code, I found some places where jiffies
were compared in a way that seems to break when they wrap. For these,
I made up patches to use the macros "time_before()" or "time_after()"
that are supposed to handle wraparound correctly.

For a small part of the patches, I believe you to be the relevant kernel code
maintainer. Appended to this email you will find some lines explaining
why I decided to mail or cc: you on this (an extract of the copyright notice
or the MAINTAINERS file), followed by the patch itself. If I mailed you
in error, please drop me a short note.

I kindly ask you to approve the correctness of the patch, and pass it on to
the relevant people for inclusion into the mainline kernel.

Tim Schmielau (tim@physik3.uni-rostock.de)

IA64 (Itanium) PLATFORM
P:	David Mosberger-Tang
M:	davidm@hpl.hp.com
L:	linux-ia64@linuxia64.org
W:	http://www.linuxia64.org/
S:	Maintained

--- linux-2.4.20-pre6/include/asm-ia64/sn/bte_copy.h	Sat Aug  3 02:39:45 2002
+++ linux-2.4.20-pre6-jfix/include/asm-ia64/sn/bte_copy.h	Wed Sep 11 11:59:32 2002
@@ -250,7 +250,7 @@
 			 * status register into the notification area.
 			 * This fakes the shub performing the copy.
-			if (jiffies > bte->idealTransferTimeout) {
+			if (time_after(jiffies, bte->idealTransferTimeout)) {
 				bte->notify = HUB_L(bte->bte_base_addr);
 				bte->idealTransferTimeout = jiffies +
Received on Sun Sep 22 04:46:36 2002

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:10 EST