From: Darrick J. Wong <djwong@xxxxxxxxxx>
xfs_repair crashes when it tries to initialize the libxfs library but
the initialization fails because the fs is already mounted:
# xfs_repair /dev/sdd
xfs_repair: /dev/sdd contains a mounted filesystem
xfs_repair: urcu.c:553: urcu_memb_register_thread: Assertion `!URCU_TLS(rcu_reader).registered' failed.
Aborted
This is because libxfs_init() registers the main thread with liburcu,
but doesn't unregister the thread if libxfs library initialization
fails. When repair sets more dangerous options and tries again, the
second initialization attempt causes liburcu to abort. Fix this by
unregistering the thread with liburcu if libxfs initialization fails.
Observed by running xfs/284.
Fixes: e4da1b16 ("xfsprogs: introduce liburcu support")
Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx>
---
libxfs/init.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/libxfs/init.c b/libxfs/init.c
index d0753ce5..14911596 100644
--- a/libxfs/init.c
+++ b/libxfs/init.c
@@ -407,8 +407,10 @@ libxfs_init(libxfs_init_t *a)
unlink(rtpath);
if (fd >= 0)
close(fd);
- if (!rval)
+ if (!rval) {
libxfs_close_devices(a);
+ rcu_unregister_thread();
+ }
return rval;
}
