From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> Don't flag directories with the RTINHERIT flag set when the filesystem doesn't have a realtime volume configured. The kernel has let us set RTINHERIT without a rt volume for ages, so it's not an invalid state. Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> --- repair/dinode.c | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/repair/dinode.c b/repair/dinode.c index 48b75f7883c5..8aee9f9e8343 100644 --- a/repair/dinode.c +++ b/repair/dinode.c @@ -2393,17 +2393,14 @@ _("bad (negative) size %" PRId64 " on inode %" PRIu64 "\n"), flags &= XFS_DIFLAG_ANY; } - if (flags & (XFS_DIFLAG_REALTIME | XFS_DIFLAG_RTINHERIT)) { - /* need an rt-dev! */ - if (!rt_name) { - if (!uncertain) { - do_warn( + /* need an rt-dev for the realtime flag! */ + if ((flags & XFS_DIFLAG_REALTIME) && !rt_name) { + if (!uncertain) { + do_warn( _("inode %" PRIu64 " has RT flag set but there is no RT device\n"), - lino); - } - flags &= ~(XFS_DIFLAG_REALTIME | - XFS_DIFLAG_RTINHERIT); + lino); } + flags &= ~XFS_DIFLAG_REALTIME; } if (flags & XFS_DIFLAG_NEWRTBM) { /* must be a rt bitmap inode */