diff options
author | Jeff Layton <jlayton@redhat.com> | 2012-09-19 09:22:46 -0400 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2012-09-24 22:46:34 -0400 |
commit | 3d6d854a13844223b603fd7a16a4a4a4afd62c72 (patch) | |
tree | aa50ad285af91b227aba7a733643c678547e0f19 /fs/cifs | |
parent | ecdb69e2cc80cca77d6afcc0aca244b72cc5ee68 (diff) |
cifs: add FL_CLOSE to fl_flags mask in cifs_read_flock
FL_CLOSE is quite common when you close a file on which you hold a
lock. The spurious "Unknown lock flags" message in cFYI is
confusing in this case.
Reported-by: Alexander Bokovoy <abokovoy@redhat.com>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Reviewed-by: Pavel Shilovsky <piastry@etersoft.ru>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs')
-rw-r--r-- | fs/cifs/file.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/cifs/file.c b/fs/cifs/file.c index 88e9c74e2ca..075f7cfd1da 100644 --- a/fs/cifs/file.c +++ b/fs/cifs/file.c | |||
@@ -1173,7 +1173,8 @@ cifs_read_flock(struct file_lock *flock, __u32 *type, int *lock, int *unlock, | |||
1173 | if (flock->fl_flags & FL_LEASE) | 1173 | if (flock->fl_flags & FL_LEASE) |
1174 | cFYI(1, "Lease on file - not implemented yet"); | 1174 | cFYI(1, "Lease on file - not implemented yet"); |
1175 | if (flock->fl_flags & | 1175 | if (flock->fl_flags & |
1176 | (~(FL_POSIX | FL_FLOCK | FL_SLEEP | FL_ACCESS | FL_LEASE))) | 1176 | (~(FL_POSIX | FL_FLOCK | FL_SLEEP | |
1177 | FL_ACCESS | FL_LEASE | FL_CLOSE))) | ||
1177 | cFYI(1, "Unknown lock flags 0x%x", flock->fl_flags); | 1178 | cFYI(1, "Unknown lock flags 0x%x", flock->fl_flags); |
1178 | 1179 | ||
1179 | *type = server->vals->large_lock_type; | 1180 | *type = server->vals->large_lock_type; |