diff options
author | J. Bruce Fields <bfields@redhat.com> | 2011-07-20 20:21:59 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2011-07-20 20:23:19 -0400 |
commit | 8fb47a4fbf858a164e973b8ea8ef5e83e61f2e50 (patch) | |
tree | d3b55fbce0e7ba5708a55b91b1ab4079ad192db6 /fs/dlm | |
parent | c46556c6be057da79f51b1a8325ec4c27938bd49 (diff) |
locks: rename lock-manager ops
Both the filesystem and the lock manager can associate operations with a
lock. Confusingly, one of them (fl_release_private) actually has the
same name in both operation structures.
It would save some confusion to give the lock-manager ops different
names.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/dlm')
-rw-r--r-- | fs/dlm/plock.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/fs/dlm/plock.c b/fs/dlm/plock.c index e2b878004364..01fd5c11a7fb 100644 --- a/fs/dlm/plock.c +++ b/fs/dlm/plock.c | |||
@@ -92,7 +92,7 @@ static void do_unlock_close(struct dlm_ls *ls, u64 number, | |||
92 | op->info.number = number; | 92 | op->info.number = number; |
93 | op->info.start = 0; | 93 | op->info.start = 0; |
94 | op->info.end = OFFSET_MAX; | 94 | op->info.end = OFFSET_MAX; |
95 | if (fl->fl_lmops && fl->fl_lmops->fl_grant) | 95 | if (fl->fl_lmops && fl->fl_lmops->lm_grant) |
96 | op->info.owner = (__u64) fl->fl_pid; | 96 | op->info.owner = (__u64) fl->fl_pid; |
97 | else | 97 | else |
98 | op->info.owner = (__u64)(long) fl->fl_owner; | 98 | op->info.owner = (__u64)(long) fl->fl_owner; |
@@ -128,11 +128,11 @@ int dlm_posix_lock(dlm_lockspace_t *lockspace, u64 number, struct file *file, | |||
128 | op->info.number = number; | 128 | op->info.number = number; |
129 | op->info.start = fl->fl_start; | 129 | op->info.start = fl->fl_start; |
130 | op->info.end = fl->fl_end; | 130 | op->info.end = fl->fl_end; |
131 | if (fl->fl_lmops && fl->fl_lmops->fl_grant) { | 131 | if (fl->fl_lmops && fl->fl_lmops->lm_grant) { |
132 | /* fl_owner is lockd which doesn't distinguish | 132 | /* fl_owner is lockd which doesn't distinguish |
133 | processes on the nfs client */ | 133 | processes on the nfs client */ |
134 | op->info.owner = (__u64) fl->fl_pid; | 134 | op->info.owner = (__u64) fl->fl_pid; |
135 | xop->callback = fl->fl_lmops->fl_grant; | 135 | xop->callback = fl->fl_lmops->lm_grant; |
136 | locks_init_lock(&xop->flc); | 136 | locks_init_lock(&xop->flc); |
137 | locks_copy_lock(&xop->flc, fl); | 137 | locks_copy_lock(&xop->flc, fl); |
138 | xop->fl = fl; | 138 | xop->fl = fl; |
@@ -268,7 +268,7 @@ int dlm_posix_unlock(dlm_lockspace_t *lockspace, u64 number, struct file *file, | |||
268 | op->info.number = number; | 268 | op->info.number = number; |
269 | op->info.start = fl->fl_start; | 269 | op->info.start = fl->fl_start; |
270 | op->info.end = fl->fl_end; | 270 | op->info.end = fl->fl_end; |
271 | if (fl->fl_lmops && fl->fl_lmops->fl_grant) | 271 | if (fl->fl_lmops && fl->fl_lmops->lm_grant) |
272 | op->info.owner = (__u64) fl->fl_pid; | 272 | op->info.owner = (__u64) fl->fl_pid; |
273 | else | 273 | else |
274 | op->info.owner = (__u64)(long) fl->fl_owner; | 274 | op->info.owner = (__u64)(long) fl->fl_owner; |
@@ -327,7 +327,7 @@ int dlm_posix_get(dlm_lockspace_t *lockspace, u64 number, struct file *file, | |||
327 | op->info.number = number; | 327 | op->info.number = number; |
328 | op->info.start = fl->fl_start; | 328 | op->info.start = fl->fl_start; |
329 | op->info.end = fl->fl_end; | 329 | op->info.end = fl->fl_end; |
330 | if (fl->fl_lmops && fl->fl_lmops->fl_grant) | 330 | if (fl->fl_lmops && fl->fl_lmops->lm_grant) |
331 | op->info.owner = (__u64) fl->fl_pid; | 331 | op->info.owner = (__u64) fl->fl_pid; |
332 | else | 332 | else |
333 | op->info.owner = (__u64)(long) fl->fl_owner; | 333 | op->info.owner = (__u64)(long) fl->fl_owner; |