diff options
| author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2023-11-08 23:46:04 +0300 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2023-12-11 03:51:37 +0300 |
| commit | a9540e35624d1475f47dbf6353eed8b99936d36e (patch) | |
| tree | 59f00fabbadea1cbad9de8e0143f43b34b9139b7 | |
| parent | 8525d5984b7b061ba02469cb58c17d1a1b98eb12 (diff) | |
| download | linux-a9540e35624d1475f47dbf6353eed8b99936d36e.tar.xz | |
smb: do not test the return value of folio_start_writeback()
In preparation for removing the return value entirely, stop testing it
in smb.
Link: https://lkml.kernel.org/r/20231108204605.745109-4-willy@infradead.org
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Cc: David Howells <dhowells@redhat.com>
Cc: Steve French <sfrench@samba.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
| -rw-r--r-- | fs/smb/client/file.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/smb/client/file.c b/fs/smb/client/file.c index cf17e3dd703e..45ca492c141c 100644 --- a/fs/smb/client/file.c +++ b/fs/smb/client/file.c @@ -2706,8 +2706,7 @@ static void cifs_extend_writeback(struct address_space *mapping, */ if (!folio_clear_dirty_for_io(folio)) WARN_ON(1); - if (folio_start_writeback(folio)) - WARN_ON(1); + folio_start_writeback(folio); *_count -= folio_nr_pages(folio); folio_unlock(folio); @@ -2742,8 +2741,7 @@ static ssize_t cifs_write_back_from_locked_folio(struct address_space *mapping, int rc; /* The folio should be locked, dirty and not undergoing writeback. */ - if (folio_start_writeback(folio)) - WARN_ON(1); + folio_start_writeback(folio); count -= folio_nr_pages(folio); len = folio_size(folio); |