vfs: remove a redundant might_sleep in wait_on_inode

Message ID 20231104221117.2584708-1-mjguzik@gmail.com
State New
Headers
Series vfs: remove a redundant might_sleep in wait_on_inode |

Commit Message

Mateusz Guzik Nov. 4, 2023, 10:11 p.m. UTC
  wait_on_bit already does it.

Signed-off-by: Mateusz Guzik <mjguzik@gmail.com>
---
 include/linux/writeback.h | 1 -
 1 file changed, 1 deletion(-)
  

Comments

Christian Brauner Nov. 6, 2023, 9:45 a.m. UTC | #1
On Sat, 04 Nov 2023 23:11:17 +0100, Mateusz Guzik wrote:
> wait_on_bit already does it.
> 
> 

Applied to the vfs.misc branch of the vfs/vfs.git tree.
Patches in the vfs.misc branch should appear in linux-next soon.

Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.

It's encouraged to provide Acked-bys and Reviewed-bys even though the
patch has now been applied. If possible patch trailers will be updated.

Note that commit hashes shown below are subject to change due to rebase,
trailer updates or similar. If in doubt, please check the listed branch.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.misc

[1/1] vfs: remove a redundant might_sleep in wait_on_inode
      https://git.kernel.org/vfs/vfs/c/357196038a51
  

Patch

diff --git a/include/linux/writeback.h b/include/linux/writeback.h
index 083387c00f0c..6d0a14f7019d 100644
--- a/include/linux/writeback.h
+++ b/include/linux/writeback.h
@@ -193,7 +193,6 @@  void inode_io_list_del(struct inode *inode);
 /* writeback.h requires fs.h; it, too, is not included from here. */
 static inline void wait_on_inode(struct inode *inode)
 {
-	might_sleep();
 	wait_on_bit(&inode->i_state, __I_NEW, TASK_UNINTERRUPTIBLE);
 }