| Message ID | 20250224154538.548028-1-john.g.garry@oracle.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | iomap: Minor code simplification in iomap_dio_bio_iter() | expand |
On Mon, Feb 24, 2025 at 03:45:38PM +0000, John Garry wrote: > Combine 'else' and 'if' conditional statements onto a single line and drop > unrequired braces, as is standard coding style. > > The code had been like this since commit c3b0e880bbfa ("iomap: support > REQ_OP_ZONE_APPEND"). > > Signed-off-by: John Garry <john.g.garry@oracle.com> Looks fine to me, Reviewed-by: "Darrick J. Wong" <djwong@kernel.org> --D > > diff --git a/fs/iomap/direct-io.c b/fs/iomap/direct-io.c > index 138d246ec29d..cdcd5ff399c1 100644 > --- a/fs/iomap/direct-io.c > +++ b/fs/iomap/direct-io.c > @@ -473,12 +473,10 @@ static loff_t iomap_dio_bio_iter(const struct iomap_iter *iter, > bio_put(bio); > goto zero_tail; > } > - if (dio->flags & IOMAP_DIO_WRITE) { > + if (dio->flags & IOMAP_DIO_WRITE) > task_io_account_write(n); > - } else { > - if (dio->flags & IOMAP_DIO_DIRTY) > - bio_set_pages_dirty(bio); > - } > + else if (dio->flags & IOMAP_DIO_DIRTY) > + bio_set_pages_dirty(bio); > > dio->size += n; > copied += n; > -- > 2.31.1 > >
On Mon, 24 Feb 2025 15:45:38 +0000, John Garry wrote: > Combine 'else' and 'if' conditional statements onto a single line and drop > unrequired braces, as is standard coding style. > > The code had been like this since commit c3b0e880bbfa ("iomap: support > REQ_OP_ZONE_APPEND"). > > > [...] Applied to the vfs.fixes branch of the vfs/vfs.git tree. Patches in the vfs.fixes 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.fixes [1/1] iomap: Minor code simplification in iomap_dio_bio_iter() https://git.kernel.org/vfs/vfs/c/b5799106b44e
diff --git a/fs/iomap/direct-io.c b/fs/iomap/direct-io.c index 138d246ec29d..cdcd5ff399c1 100644 --- a/fs/iomap/direct-io.c +++ b/fs/iomap/direct-io.c @@ -473,12 +473,10 @@ static loff_t iomap_dio_bio_iter(const struct iomap_iter *iter, bio_put(bio); goto zero_tail; } - if (dio->flags & IOMAP_DIO_WRITE) { + if (dio->flags & IOMAP_DIO_WRITE) task_io_account_write(n); - } else { - if (dio->flags & IOMAP_DIO_DIRTY) - bio_set_pages_dirty(bio); - } + else if (dio->flags & IOMAP_DIO_DIRTY) + bio_set_pages_dirty(bio); dio->size += n; copied += n;
Combine 'else' and 'if' conditional statements onto a single line and drop unrequired braces, as is standard coding style. The code had been like this since commit c3b0e880bbfa ("iomap: support REQ_OP_ZONE_APPEND"). Signed-off-by: John Garry <john.g.garry@oracle.com>