Hi.

In mm/highmem.c:map_new_virtual (in a 2.4.0-t8p1 kernel) we set 
current's state directly. I believe that using __set_task_state
is nicer. The following patch acts out this belief :) Please
comment.

--- linux-240test8-pre1/mm/highmem.c    Thu Aug 24 09:43:36 2000
+++ linux/mm/highmem.c  Thu Aug 31 22:39:36 2000
@@ -165,7 +165,7 @@
                {
                        DECLARE_WAITQUEUE(wait, current);
 
-                       current->state = TASK_UNINTERRUPTIBLE;
+                       __set_task_state(current, TASK_UNINTERRUPTIBLE);
                        add_wait_queue(&pkmap_map_wait, &wait);
                        spin_unlock(&kmap_lock);
                        schedule();

-- 
Regards,
        Rasmus([EMAIL PROTECTED])

Outside of the killings, Washington has one of the lowest crime rates in
the country.     -Mayor Marion Barry, Washington, DC
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
Please read the FAQ at http://www.tux.org/lkml/

Reply via email to