On Tue, Jan 24, 2017 at 12:42:57PM -0800, Darrick J. Wong wrote:
> > + if (error) {
> > + /*
> > + * Because we didn't have per-AG reservations when the
> > + * finobt feature was added we might not ne able to
> > + * reservere all needed blocks. Warn and fall back to
>
> "...might not be able to reserve all..."
Fixed.
> So if __xfs_ag_resv_init() returns ENOSPC, we don't actually halt the
> mount attempt, which means that we can start operating with
> mp->m_inotbt_nores = false if there's not even enough space for the
> refcountbt-only reservation. In theory this is impossible, but I think
> we'd be in a better defensive position by moving this to before the
> __xfs_ag_resv_init call.
Ok.
> > xfs_agnumber_t agno, struct xfs_buf **bpp);
> >
> > -
> > #endif /* __XFS_IALLOC_H__ */
>
> Unnecessary change?
Fixed.
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
