Messages in this thread Patch in this message |  | | Date | Thu, 31 Aug 2000 22:53:58 +0200 | From | Rasmus Andersen <> | Subject | [patchlet] current->state=XX -> __set_task_state in mm/highmem.c |
| |
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/
|  |