diff options
author | NeilBrown <neilb@suse.com> | 2018-11-29 18:04:08 -0500 |
---|---|---|
committer | Jeff Layton <jlayton@kernel.org> | 2018-11-30 11:26:12 -0500 |
commit | 4d62d3f70b9b24497e489af79b0d730be792380f (patch) | |
tree | 90e540026a40993de7eed999e370d2307c1c54c9 /fs/gfs2 | |
parent | 7b587e1a5a6cf6035ab1fbd6a1d00ce604eaf488 (diff) |
gfs2: properly initial file_lock used for unlock.
Rather than assuming all-zeros is sufficient, use the available API to
initialize the file_lock structure use for unlock. VFS-level changes
will soon make it important that the list_heads in file_lock are
always properly initialized.
Signed-off-by: NeilBrown <neilb@suse.com>
Reviewed-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Jeff Layton <jlayton@kernel.org>
Diffstat (limited to 'fs/gfs2')
-rw-r--r-- | fs/gfs2/file.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/fs/gfs2/file.c b/fs/gfs2/file.c index 45a17b770d97..a2dea5bc0427 100644 --- a/fs/gfs2/file.c +++ b/fs/gfs2/file.c | |||
@@ -1199,13 +1199,13 @@ static int do_flock(struct file *file, int cmd, struct file_lock *fl) | |||
1199 | mutex_lock(&fp->f_fl_mutex); | 1199 | mutex_lock(&fp->f_fl_mutex); |
1200 | 1200 | ||
1201 | if (gfs2_holder_initialized(fl_gh)) { | 1201 | if (gfs2_holder_initialized(fl_gh)) { |
1202 | struct file_lock request; | ||
1202 | if (fl_gh->gh_state == state) | 1203 | if (fl_gh->gh_state == state) |
1203 | goto out; | 1204 | goto out; |
1204 | locks_lock_file_wait(file, | 1205 | locks_init_lock(&request); |
1205 | &(struct file_lock) { | 1206 | request.fl_type = F_UNLCK; |
1206 | .fl_type = F_UNLCK, | 1207 | request.fl_flags = FL_FLOCK; |
1207 | .fl_flags = FL_FLOCK | 1208 | locks_lock_file_wait(file, &request); |
1208 | }); | ||
1209 | gfs2_glock_dq(fl_gh); | 1209 | gfs2_glock_dq(fl_gh); |
1210 | gfs2_holder_reinit(state, flags, fl_gh); | 1210 | gfs2_holder_reinit(state, flags, fl_gh); |
1211 | } else { | 1211 | } else { |