diff options
author | David Teigland <teigland@redhat.com> | 2009-01-19 14:13:33 -0500 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2009-01-21 16:28:45 -0500 |
commit | 24179f488092267c9a033d7e25ce7a58af50ff79 (patch) | |
tree | a4f4fc6a038ba8240350a4b47d9a1f405790e78f | |
parent | f3b8436ad9a8ad36b3c9fa1fe030c7f38e5d3d0b (diff) |
dlm: fix plock notify callback to lockd
We should use the original copy of the file_lock, fl, instead
of the copy, flc in the lockd notify callback. The range in flc has
been modified by posix_lock_file(), so it will not match a copy of the
lock in lockd.
Signed-off-by: David Teigland <teigland@redhat.com>
-rw-r--r-- | fs/dlm/plock.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/dlm/plock.c b/fs/dlm/plock.c index eba87ff3177b..502b1ea5ef6b 100644 --- a/fs/dlm/plock.c +++ b/fs/dlm/plock.c | |||
@@ -168,7 +168,7 @@ static int dlm_plock_callback(struct plock_op *op) | |||
168 | notify = xop->callback; | 168 | notify = xop->callback; |
169 | 169 | ||
170 | if (op->info.rv) { | 170 | if (op->info.rv) { |
171 | notify(flc, NULL, op->info.rv); | 171 | notify(fl, NULL, op->info.rv); |
172 | goto out; | 172 | goto out; |
173 | } | 173 | } |
174 | 174 | ||
@@ -187,7 +187,7 @@ static int dlm_plock_callback(struct plock_op *op) | |||
187 | (unsigned long long)op->info.number, file, fl); | 187 | (unsigned long long)op->info.number, file, fl); |
188 | } | 188 | } |
189 | 189 | ||
190 | rv = notify(flc, NULL, 0); | 190 | rv = notify(fl, NULL, 0); |
191 | if (rv) { | 191 | if (rv) { |
192 | /* XXX: We need to cancel the fs lock here: */ | 192 | /* XXX: We need to cancel the fs lock here: */ |
193 | log_print("dlm_plock_callback: lock granted after lock request " | 193 | log_print("dlm_plock_callback: lock granted after lock request " |