Messages in this thread | | | Date | Fri, 17 Dec 2021 18:14:25 +0800 | From | Qu Wenruo <> | Subject | Re: [PATCH] btrfs: remove unneeded variable |
| |
On 2021/12/17 16:45, cgel.zte@gmail.com wrote: > From: Changcheng Deng <deng.changcheng@zte.com.cn> > > Remove unneeded variable used to store return value. > > Reported-by: Zeal Robot <zealci@zte.com.cn> > Signed-off-by: Changcheng Deng <deng.changcheng@zte.com.cn> > --- > fs/btrfs/disk-io.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c > index d94a1ca856aa..d33575e56da2 100644 > --- a/fs/btrfs/disk-io.c > +++ b/fs/btrfs/disk-io.c > @@ -4658,7 +4658,6 @@ static int btrfs_destroy_delayed_refs(struct btrfs_transaction *trans, > struct rb_node *node; > struct btrfs_delayed_ref_root *delayed_refs; > struct btrfs_delayed_ref_node *ref; > - int ret = 0;
If you're removing @ret, it's better to also change the return value to void.
Normally I would suggest you to checker the caller and return proper error number instead.
But in this particular case, this function is only called in transaction cleanup code, which means we have already aborted a transaction, thus there is not much meaning to further error out, and the cleanup is OK.
So it would be fine for you to delete @ret, change the function to return void. And even better, to remove the "btrfs_" prefix of the function, as the function is not really exported.
And a final suggest for your future patches, there is no need to bother btrfs maintainers at all, just sending the mail to btrfs mailing list is enough.
Thanks, Qu > > delayed_refs = &trans->delayed_refs; > > @@ -4666,7 +4665,7 @@ static int btrfs_destroy_delayed_refs(struct btrfs_transaction *trans, > if (atomic_read(&delayed_refs->num_entries) == 0) { > spin_unlock(&delayed_refs->lock); > btrfs_debug(fs_info, "delayed_refs has NO entry"); > - return ret; > + return 0; > } > > while ((node = rb_first_cached(&delayed_refs->href_root)) != NULL) { > @@ -4729,7 +4728,7 @@ static int btrfs_destroy_delayed_refs(struct btrfs_transaction *trans, > > spin_unlock(&delayed_refs->lock); > > - return ret; > + return 0; > } > > static void btrfs_destroy_delalloc_inodes(struct btrfs_root *root)
| |