diff options
| author |   Rohith Surabattula <rohiths@microsoft.com> | 2021-05-05 10:56:47 +0000 |
|---|---|---|
| committer |   Steve French <stfrench@microsoft.com> | 2021-05-19 21:11:26 -0500 |
| commit | 860b69a9d77160d21ca00357fd6c5217f9d41fb1 (patch) | |
| tree | 5668410148272ff6aaacdc8418320f79b14f503f /fs/cifs/cifsglob.h | |
| parent | cifs: Fix inconsistent indenting (diff) | |
| download | linux-dev-860b69a9d77160d21ca00357fd6c5217f9d41fb1.tar.xz linux-dev-860b69a9d77160d21ca00357fd6c5217f9d41fb1.zip | |
Fix kernel oops when CONFIG_DEBUG_ATOMIC_SLEEP is enabled.
Removed oplock_break_received flag which was added to achieve
synchronization between oplock handler and open handler by earlier commit.
It is not needed because there is an existing lock open_file_lock to achieve
the same. find_readable_file takes open_file_lock and then traverses the
openFileList. Similarly, cifs_oplock_break while closing the deferred
handle (i.e cifsFileInfo_put) takes open_file_lock and then sends close
to the server.
Added comments for better readability.
Signed-off-by: Rohith Surabattula <rohiths@microsoft.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs/cifsglob.h')
| -rw-r--r-- | fs/cifs/cifsglob.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/cifs/cifsglob.h b/fs/cifs/cifsglob.h index d88b4b523dcc..ea90c53386b8 100644 --- a/fs/cifs/cifsglob.h +++ b/fs/cifs/cifsglob.h @@ -1257,8 +1257,7 @@ struct cifsFileInfo { struct work_struct oplock_break; /* work for oplock breaks */ struct work_struct put; /* work for the final part of _put */ struct delayed_work deferred; - bool oplock_break_received; /* Flag to indicate oplock break */ - bool deferred_scheduled; + bool deferred_close_scheduled; /* Flag to indicate close is scheduled */ }; struct cifs_io_parms { |