Message ID | 20191101185245.GC15222@magnolia (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | xfsprogs: random fixes | expand |
On 11/1/19 1:52 PM, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@oracle.com> > > If we encounter a failure while fixing the freelist during mkfs, we > shouldn't print a misleading message about space reservation. Fix it so > that we print something about what we were trying to do when the error > happened. > > Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> 8 going once going twice... sold, to the man in the front row, 8/5 is our final winning bid for this series. ;) Reviewed-by: Eric Sandeen <sandeen@redhat.com>
diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c index 10a40cd4..18338a61 100644 --- a/mkfs/xfs_mkfs.c +++ b/mkfs/xfs_mkfs.c @@ -3481,8 +3481,11 @@ initialise_ag_freespace( libxfs_alloc_fix_freelist(&args, 0); libxfs_perag_put(args.pag); c = -libxfs_trans_commit(tp); - if (c) - res_failed(c); + if (c) { + errno = c; + perror(_("initializing AG free space list")); + exit(1); + } } /*