on 8/16/2023 11:47 AM, Theodore Ts'o wrote: > On Thu, Jun 29, 2023 at 08:00:43PM +0800, Kemeng Shi wrote: >> Commit 0acdb8876fead ("ext4: don't call update_backups() multiple times >> for the same bg") add check in ext4_flex_group_add to avoid call >> update_backups multiple times for block group descriptors in the same >> group descriptor block. However, we have already call update_backups in >> block step, so the added check is unnecessary. > > I'm having trouble understaind this comit. What do you mean by the > "block step" in the last paragraph? > Sorry for the confusing stuff. I mean we foreach in group descriptor block step instead of foreach in group descriptor step to update backup. So there is no chance to call update_backups for different descriptors in the same bg. > - Ted >