[patchlet] current->state=XX -> __set_task_state in mm/highmem.c

• Next message: Alexander Viro: "Re: hfs support for blocksize != 512"
• Previous message: Rasmus Andersen: "[patch] Returning value from void function (kernel/signal.c) (240t8p1)"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

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(rasmus@jaquet.dk)
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 majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

• Next message: Alexander Viro: "Re: hfs support for blocksize != 512"
• Previous message: Rasmus Andersen: "[patch] Returning value from void function (kernel/signal.c) (240t8p1)"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b29 : Thu Aug 31 2000 - 21:00:28 EST