Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Wed, 18 Dec 2002 18:54:08 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Wed, 18 Dec 2002 18:54:08 -0500 Received: from fmr06.intel.com ([134.134.136.7]:32733 "EHLO caduceus.jf.intel.com") by vger.kernel.org with ESMTP id ; Wed, 18 Dec 2002 18:54:06 -0500 Subject: [PATCH 2.5.52] Use __set_current_state() instead of current->state = (take 1) To: torvalds@transmeta.org, linux-kernel@vger.kernel.org Message-Id: From: Inaky Perez-Gonzalez Date: Wed, 18 Dec 2002 15:56:58 -0800 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3560 Lines: 131 Hi all In fs/*.c, many functions manually set the task state directly accessing current->state, or with a macro, kind of inconsistently. This patch changes all of them to use [__]set_current_state(). Changelog: - Ported forward to 2.5.52 diff -u fs/dquot.c:1.1.1.4 fs/dquot.c:1.1.1.1.6.2 --- fs/dquot.c:1.1.1.4 Wed Dec 11 11:13:35 2002 +++ fs/dquot.c Wed Dec 18 13:20:24 2002 @@ -264,7 +264,7 @@ goto repeat; } remove_wait_queue(&dquot->dq_wait_lock, &wait); - current->state = TASK_RUNNING; + __set_current_state(TASK_RUNNING); } static inline void wait_on_dquot(struct dquot *dquot) @@ -298,7 +298,7 @@ goto repeat; } remove_wait_queue(&dquot->dq_wait_free, &wait); - current->state = TASK_RUNNING; + __set_current_state(TASK_RUNNING); } /* Wait for all duplicated dquot references to be dropped */ @@ -314,7 +314,7 @@ goto repeat; } remove_wait_queue(&dquot->dq_wait_free, &wait); - current->state = TASK_RUNNING; + __set_current_state(TASK_RUNNING); } static int read_dqblk(struct dquot *dquot) diff -u fs/exec.c:1.1.1.8 fs/exec.c:1.1.1.1.6.2 --- fs/exec.c:1.1.1.8 Mon Dec 16 18:44:31 2002 +++ fs/exec.c Wed Dec 18 13:20:24 2002 @@ -587,7 +587,7 @@ count = 1; while (atomic_read(&oldsig->count) > count) { oldsig->group_exit_task = current; - current->state = TASK_UNINTERRUPTIBLE; + __set_current_state(TASK_UNINTERRUPTIBLE); spin_unlock_irq(&oldsig->siglock); schedule(); spin_lock_irq(&oldsig->siglock); diff -u fs/inode.c:1.1.1.6 fs/inode.c:1.1.1.1.6.2 --- fs/inode.c:1.1.1.6 Mon Dec 16 18:44:31 2002 +++ fs/inode.c Wed Dec 18 13:20:24 2002 @@ -1195,7 +1195,7 @@ goto repeat; } remove_wait_queue(wq, &wait); - current->state = TASK_RUNNING; + __set_current_state(TASK_RUNNING); } void wake_up_inode(struct inode *inode) diff -u fs/locks.c:1.1.1.6 fs/locks.c:1.1.1.1.6.2 --- fs/locks.c:1.1.1.6 Wed Dec 11 11:13:35 2002 +++ fs/locks.c Wed Dec 18 13:20:24 2002 @@ -571,7 +571,7 @@ int result = 0; DECLARE_WAITQUEUE(wait, current); - current->state = TASK_INTERRUPTIBLE; + __set_current_state (TASK_INTERRUPTIBLE); add_wait_queue(fl_wait, &wait); if (timeout == 0) schedule(); @@ -580,7 +580,7 @@ if (signal_pending(current)) result = -ERESTARTSYS; remove_wait_queue(fl_wait, &wait); - current->state = TASK_RUNNING; + __set_current_state (TASK_RUNNING); return result; } diff -u fs/namei.c:1.1.1.6 fs/namei.c:1.1.1.1.6.2 --- fs/namei.c:1.1.1.6 Wed Dec 11 11:13:35 2002 +++ fs/namei.c Wed Dec 18 13:20:24 2002 @@ -410,7 +410,7 @@ if (current->total_link_count >= 40) goto loop; if (need_resched()) { - current->state = TASK_RUNNING; + __set_current_state(TASK_RUNNING); schedule(); } err = security_inode_follow_link(dentry, nd); diff -u fs/select.c:1.1.1.3 fs/select.c:1.1.1.1.6.2 --- fs/select.c:1.1.1.3 Wed Dec 11 11:10:14 2002 +++ fs/select.c Wed Dec 18 13:20:24 2002 @@ -235,7 +235,7 @@ } __timeout = schedule_timeout(__timeout); } - current->state = TASK_RUNNING; + __set_current_state (TASK_RUNNING); poll_freewait(&table); @@ -417,7 +417,7 @@ break; timeout = schedule_timeout(timeout); } - current->state = TASK_RUNNING; + __set_current_state (TASK_RUNNING); return count; } -- Inaky Perez-Gonzalez -- Not speaking for Intel - opinions are my own [or my fault] - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/