diff options
author | Hyunchul Lee <hyc.lee@gmail.com> | 2021-07-10 09:34:20 +0900 |
---|---|---|
committer | Namjae Jeon <namjae.jeon@samsung.com> | 2021-07-10 16:23:56 +0900 |
commit | 45a64e8b08493b768fa029a5508cec8cf2b89f2d (patch) | |
tree | 037e63043456953d5c0cb5123c894f6ded61cf8f /fs/ksmbd/vfs.h | |
parent | d63528eb0d43c4796c42aad56889dec12cf4e122 (diff) | |
download | linux-stable-45a64e8b08493b768fa029a5508cec8cf2b89f2d.tar.gz linux-stable-45a64e8b08493b768fa029a5508cec8cf2b89f2d.tar.bz2 linux-stable-45a64e8b08493b768fa029a5508cec8cf2b89f2d.zip |
ksmbd: uninterruptible wait for a file being unlocked
the wait can be canceled by SMB2_CANCEL, SMB2_CLOSE,
SMB2_LOGOFF, disconnection or shutdown, we don't have
to use wait_event_interruptible.
And this remove the warning from Coverity:
CID 1502834 (#1 of 1): Unused value (UNUSED_VALUE)
returned_value: Assigning value from ksmbd_vfs_posix_lock_wait(flock)
to err here, but that stored value is overwritten before it can be used.
Signed-off-by: Hyunchul Lee <hyc.lee@gmail.com>
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/ksmbd/vfs.h')
-rw-r--r-- | fs/ksmbd/vfs.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ksmbd/vfs.h b/fs/ksmbd/vfs.h index b255f90acf8f..cb0cba0d5d07 100644 --- a/fs/ksmbd/vfs.h +++ b/fs/ksmbd/vfs.h @@ -168,7 +168,7 @@ int ksmbd_vfs_fill_dentry_attrs(struct ksmbd_work *work, struct user_namespace *user_ns, struct dentry *dentry, struct ksmbd_kstat *ksmbd_kstat); -int ksmbd_vfs_posix_lock_wait(struct file_lock *flock); +void ksmbd_vfs_posix_lock_wait(struct file_lock *flock); int ksmbd_vfs_posix_lock_wait_timeout(struct file_lock *flock, long timeout); void ksmbd_vfs_posix_lock_unblock(struct file_lock *flock); int ksmbd_vfs_remove_acl_xattrs(struct user_namespace *user_ns, |