diff options
author | Patrick Caulfield <pcaulfie@redhat.com> | 2007-01-02 12:01:05 -0500 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2007-02-05 13:36:18 -0500 |
commit | 3fb4a251febe70e4c65ea8250545b391fd414d5a (patch) | |
tree | 28de58dc7a76c3e8c00a3cc4d1005196e55fe475 /fs/dlm/lowcomms-tcp.c | |
parent | 5509826f1e548d14bb888c1cb6e3bbf23f855770 (diff) |
[DLM] Fix schedule() calls
I was a little over-enthusiastic turning schedule() calls int cond_sched() when fixing the DLM for Andrew Morton.
These four should really be calls to schedule() or the dlm can busy-wait.
Signed-Off-By: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/dlm/lowcomms-tcp.c')
-rw-r--r-- | fs/dlm/lowcomms-tcp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/dlm/lowcomms-tcp.c b/fs/dlm/lowcomms-tcp.c index 3b224733f18b..18b91c65bdb3 100644 --- a/fs/dlm/lowcomms-tcp.c +++ b/fs/dlm/lowcomms-tcp.c | |||
@@ -996,7 +996,7 @@ static int dlm_recvd(void *data) | |||
996 | while (!kthread_should_stop()) { | 996 | while (!kthread_should_stop()) { |
997 | set_current_state(TASK_INTERRUPTIBLE); | 997 | set_current_state(TASK_INTERRUPTIBLE); |
998 | if (read_list_empty()) | 998 | if (read_list_empty()) |
999 | cond_resched(); | 999 | schedule(); |
1000 | set_current_state(TASK_RUNNING); | 1000 | set_current_state(TASK_RUNNING); |
1001 | 1001 | ||
1002 | process_sockets(); | 1002 | process_sockets(); |
@@ -1030,7 +1030,7 @@ static int dlm_sendd(void *data) | |||
1030 | while (!kthread_should_stop()) { | 1030 | while (!kthread_should_stop()) { |
1031 | set_current_state(TASK_INTERRUPTIBLE); | 1031 | set_current_state(TASK_INTERRUPTIBLE); |
1032 | if (write_and_state_lists_empty()) | 1032 | if (write_and_state_lists_empty()) |
1033 | cond_resched(); | 1033 | schedule(); |
1034 | set_current_state(TASK_RUNNING); | 1034 | set_current_state(TASK_RUNNING); |
1035 | 1035 | ||
1036 | process_state_queue(); | 1036 | process_state_queue(); |