fs/buffer: remove superfluous statements

Davidlohr Bueso · brauner · commit d11a24999605 · 2025-05-21T09:34:29.000+02:00
Get rid of those unnecessary return statements. Signed-off-by: Davidlohr Bueso <dave@stgolabs.net> Link: https://lore.kernel.org/20250515173925.147823-4-dave@stgolabs.net Reviewed-by: Jan Kara <jack@suse.cz> Signed-off-by: Christian Brauner <brauner@kernel.org>
diff --git a/fs/buffer.c b/fs/buffer.c
@@ -297,7 +297,6 @@ static void end_buffer_async_read(struct buffer_head *bh, int uptodate)
 
 still_busy:
 	spin_unlock_irqrestore(&first->b_uptodate_lock, flags);
-	return;
 }
 
 struct postprocess_bh_ctx {
@@ -422,7 +421,6 @@ static void end_buffer_async_write(struct buffer_head *bh, int uptodate)
 
 still_busy:
 	spin_unlock_irqrestore(&first->b_uptodate_lock, flags);
-	return;
 }
 
 /*
@@ -1682,7 +1680,6 @@ void block_invalidate_folio(struct folio *folio, size_t offset, size_t length)
 		filemap_release_folio(folio, 0);
 out:
 	folio_clear_mappedtodisk(folio);
-	return;
 }
 EXPORT_SYMBOL(block_invalidate_folio);
 

Original file line number	Diff line number	Diff line change
`@@ -297,7 +297,6 @@ static void end_buffer_async_read(struct buffer_head *bh, int uptodate)`
`297`	`297`
`298`	`298`	`still_busy:`
`299`	`299`	`spin_unlock_irqrestore(&first->b_uptodate_lock, flags);`
`300`		`- return;`
`301`	`300`	`}`
`302`	`301`
`303`	`302`	`struct postprocess_bh_ctx {`
`@@ -422,7 +421,6 @@ static void end_buffer_async_write(struct buffer_head *bh, int uptodate)`
`422`	`421`
`423`	`422`	`still_busy:`
`424`	`423`	`spin_unlock_irqrestore(&first->b_uptodate_lock, flags);`
`425`		`- return;`
`426`	`424`	`}`
`427`	`425`
`428`	`426`	`/*`
`@@ -1682,7 +1680,6 @@ void block_invalidate_folio(struct folio *folio, size_t offset, size_t length)`
`1682`	`1680`	`filemap_release_folio(folio, 0);`
`1683`	`1681`	`out:`
`1684`	`1682`	`folio_clear_mappedtodisk(folio);`
`1685`		`- return;`
`1686`	`1683`	`}`
`1687`	`1684`	`EXPORT_SYMBOL(block_invalidate_folio);`
`1688`	`1685`