On Mon, Feb 24, 2020 at 04:10:59PM -0800, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Make sure that all the metadata we wrote in the process of formatting > the filesystem have been written correctly, or exit with failure. > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > --- Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx> > mkfs/xfs_mkfs.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > > diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c > index 1f5d2105..1038e604 100644 > --- a/mkfs/xfs_mkfs.c > +++ b/mkfs/xfs_mkfs.c > @@ -3940,13 +3940,16 @@ main( > (XFS_BUF_TO_SBP(buf))->sb_inprogress = 0; > libxfs_writebuf(buf, LIBXFS_EXIT_ON_FAILURE); > > - libxfs_umount(mp); > + /* Report failure if anything failed to get written to our new fs. */ > + error = -libxfs_umount(mp); > + if (error) > + exit(1); > + > if (xi.rtdev) > libxfs_device_close(xi.rtdev); > if (xi.logdev && xi.logdev != xi.ddev) > libxfs_device_close(xi.logdev); > libxfs_device_close(xi.ddev); > libxfs_destroy(); > - > return 0; > } >
