Commit 4d413ae9 authored by Alexander Aring's avatar Alexander Aring Committed by David Teigland

fs: dlm: use dlm_plock_info for do_unlock_close

This patch refactors do_unlock_close() by using only struct dlm_plock_info
as a parameter.
Signed-off-by: default avatarAlexander Aring <aahringo@redhat.com>
Signed-off-by: default avatarDavid Teigland <teigland@redhat.com>
parent ea06d4ca
...@@ -79,8 +79,7 @@ static void send_op(struct plock_op *op) ...@@ -79,8 +79,7 @@ static void send_op(struct plock_op *op)
abandoned waiter. So, we have to insert the unlock-close when the abandoned waiter. So, we have to insert the unlock-close when the
lock call is interrupted. */ lock call is interrupted. */
static void do_unlock_close(struct dlm_ls *ls, u64 number, static void do_unlock_close(const struct dlm_plock_info *info)
struct file *file, struct file_lock *fl)
{ {
struct plock_op *op; struct plock_op *op;
...@@ -89,15 +88,12 @@ static void do_unlock_close(struct dlm_ls *ls, u64 number, ...@@ -89,15 +88,12 @@ static void do_unlock_close(struct dlm_ls *ls, u64 number,
return; return;
op->info.optype = DLM_PLOCK_OP_UNLOCK; op->info.optype = DLM_PLOCK_OP_UNLOCK;
op->info.pid = fl->fl_pid; op->info.pid = info->pid;
op->info.fsid = ls->ls_global_id; op->info.fsid = info->fsid;
op->info.number = number; op->info.number = info->number;
op->info.start = 0; op->info.start = 0;
op->info.end = OFFSET_MAX; op->info.end = OFFSET_MAX;
if (fl->fl_lmops && fl->fl_lmops->lm_grant) op->info.owner = info->owner;
op->info.owner = (__u64) fl->fl_pid;
else
op->info.owner = (__u64)(long) fl->fl_owner;
op->info.flags |= DLM_PLOCK_FL_CLOSE; op->info.flags |= DLM_PLOCK_FL_CLOSE;
send_op(op); send_op(op);
...@@ -167,7 +163,7 @@ int dlm_posix_lock(dlm_lockspace_t *lockspace, u64 number, struct file *file, ...@@ -167,7 +163,7 @@ int dlm_posix_lock(dlm_lockspace_t *lockspace, u64 number, struct file *file,
__func__, ls->ls_global_id, __func__, ls->ls_global_id,
(unsigned long long)number, op->info.pid); (unsigned long long)number, op->info.pid);
dlm_release_plock_op(op); dlm_release_plock_op(op);
do_unlock_close(ls, number, file, fl); do_unlock_close(&op->info);
goto out; goto out;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment