On Mon, Dec 04, 2023 at 08:38:08PM -0800, Christoph Hellwig wrote: > > +void > > +xfs_destroy_perag( > > + xfs_mount_t *mp, > > + xfs_agnumber_t agstart, > > + xfs_agnumber_t agend) > > Not sure xfs_destroy_perag is the right name, as it frees a range of > AGs, how about xfs_free_unuses_perag_range? Yes, it looks better. But I'm not sure if it's better to use xfs_free_unused_perag_range? > > Also a comment explaining that this must never be used for AGs that > have been visible (that is included in mp->m_sb.sb_agcount) would > probably be useful. > This will be changed in the next version. Thanks, Long Li
