On Mon, Apr 17, 2023 at 10:51:46PM +0200, Christophe JAILLET wrote: > > - ret2 = filemap_check_errors(NODE_MAPPING(sbi)); > - if (!ret) > - ret = ret2; > + ret = filemap_check_errors(NODE_MAPPING(sbi)); > > return ret; Let's get rid of "ret" as well: return filemap_check_errors(NODE_MAPPING(sbi)); regards, dan carpenter