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 | |
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')
-rw-r--r-- | fs/dlm/lowcomms-sctp.c | 4 | ||||
-rw-r--r-- | fs/dlm/lowcomms-tcp.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/fs/dlm/lowcomms-sctp.c b/fs/dlm/lowcomms-sctp.c index fe158d7a9285..0940a805b3d7 100644 --- a/fs/dlm/lowcomms-sctp.c +++ b/fs/dlm/lowcomms-sctp.c | |||
@@ -1109,7 +1109,7 @@ static int dlm_recvd(void *data) | |||
1109 | set_current_state(TASK_INTERRUPTIBLE); | 1109 | set_current_state(TASK_INTERRUPTIBLE); |
1110 | add_wait_queue(&lowcomms_recv_wait, &wait); | 1110 | add_wait_queue(&lowcomms_recv_wait, &wait); |
1111 | if (!test_bit(CF_READ_PENDING, &sctp_con.flags)) | 1111 | if (!test_bit(CF_READ_PENDING, &sctp_con.flags)) |
1112 | cond_resched(); | 1112 | schedule(); |
1113 | remove_wait_queue(&lowcomms_recv_wait, &wait); | 1113 | remove_wait_queue(&lowcomms_recv_wait, &wait); |
1114 | set_current_state(TASK_RUNNING); | 1114 | set_current_state(TASK_RUNNING); |
1115 | 1115 | ||
@@ -1141,7 +1141,7 @@ static int dlm_sendd(void *data) | |||
1141 | while (!kthread_should_stop()) { | 1141 | while (!kthread_should_stop()) { |
1142 | set_current_state(TASK_INTERRUPTIBLE); | 1142 | set_current_state(TASK_INTERRUPTIBLE); |
1143 | if (write_list_empty()) | 1143 | if (write_list_empty()) |
1144 | cond_resched(); | 1144 | schedule(); |
1145 | set_current_state(TASK_RUNNING); | 1145 | set_current_state(TASK_RUNNING); |
1146 | 1146 | ||
1147 | if (sctp_con.eagain_flag) { | 1147 | if (sctp_con.eagain_flag) { |
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(); |