diff options
| author | Kent Overstreet <kent.overstreet@gmail.com> | 2021-10-19 15:08:00 -0400 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-22 17:09:14 -0400 |
| commit | 9a796fdb06b56a1811f1afdd40b793e2848a990e (patch) | |
| tree | 0aff8322804e52e2eb7f078bb0688a5d2fda4462 /fs/bcachefs/fs-io.c | |
| parent | d355c6f4f73060c8f3eba95b9ae72929669f7516 (diff) | |
| download | lwn-9a796fdb06b56a1811f1afdd40b793e2848a990e.tar.gz lwn-9a796fdb06b56a1811f1afdd40b793e2848a990e.zip | |
bcachefs: bch2_trans_exit() no longer returns errors
Now that peek_node()/next_node() are converted to return errors
directly, we don't need bch2_trans_exit() to return errors - it's
cleaner this way and wasn't used much anymore.
Signed-off-by: Kent Overstreet <kent.overstreet@gmail.com>
Diffstat (limited to 'fs/bcachefs/fs-io.c')
| -rw-r--r-- | fs/bcachefs/fs-io.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/fs/bcachefs/fs-io.c b/fs/bcachefs/fs-io.c index 736dd71419a5..079c20cbf10e 100644 --- a/fs/bcachefs/fs-io.c +++ b/fs/bcachefs/fs-io.c @@ -2223,7 +2223,8 @@ err: if (ret == -EINTR) goto retry; - return bch2_trans_exit(&trans) ?: ret; + bch2_trans_exit(&trans); + return ret; } static int __bch2_truncate_page(struct bch_inode_info *inode, @@ -3125,7 +3126,7 @@ err: if (ret == -EINTR) goto retry; - ret = bch2_trans_exit(&trans) ?: ret; + bch2_trans_exit(&trans); if (ret) return ret; @@ -3240,7 +3241,7 @@ err: if (ret == -EINTR) goto retry; - ret = bch2_trans_exit(&trans) ?: ret; + bch2_trans_exit(&trans); if (ret) return ret; |