mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-31 08:15:07 -04:00
xfs: check split_sectors validity before bio_split call
Change the split_sectors check from !split_sectors to split_sectors <= 0 to make the error handling explicit. While bio_split_rw_at() cannot return a negative error code for the current GC I/O path (GC I/O doesn't use REQ_ATOMIC/REQ_NOWAIT flags and has proper alignment), making the check explicit improves code clarity and makes the intent clear. This also makes the code more robust for future maintenance if different I/O patterns are introduced. Signed-off-by: Hongling Zeng <zenghongling@kylinos.cn> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Carlos Maiolino <cem@kernel.org>
This commit is contained in:
committed by
Carlos Maiolino
parent
a94e648f03
commit
6e6a314014
@@ -802,7 +802,7 @@ xfs_zone_gc_split_write(
|
||||
|
||||
split_sectors = bio_split_rw_at(&chunk->bio, lim, &nsegs,
|
||||
lim->max_zone_append_sectors << SECTOR_SHIFT);
|
||||
if (!split_sectors)
|
||||
if (split_sectors <= 0)
|
||||
return NULL;
|
||||
|
||||
/* ensure the split chunk is still block size aligned */
|
||||
|
||||
Reference in New Issue
Block a user