On Fri, Dec 04, 2020 at 12:10:28AM +0800, Gao Xiang wrote: > It's enough to just use return code, and get rid of an argument. > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> > Signed-off-by: Gao Xiang <hsiangkao@xxxxxxxxxx> > --- > fs/xfs/libxfs/xfs_ialloc.c | 24 +++++++++--------------- > 1 file changed, 9 insertions(+), 15 deletions(-) Look fine. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx