xfs_repair: reject unwritten shared extents

We don't allow sharing of unwritten extents, which means that repair
should reject an unwritten extent if someone else has already claimed
the space.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>

diff --git a/repair/dinode.c b/repair/dinode.c
index 3b022a06230fe57efe910abc337ad65248edfa6e..bc6007de1fe39d3fcf7349ec6b0db953d1dc5eb1 100644 (file)
--- a/repair/dinode.c
+++ b/repair/dinode.c
@@ -278,7 +278,8 @@ _("bad state in rt extent map %" PRIu64 "\n"),
                        break;
                case XR_E_INUSE:
                case XR_E_MULT:
-                       if (xfs_has_rtreflink(mp))
+                       if (xfs_has_rtreflink(mp) &&
+                           irec->br_state == XFS_EXT_NORM)
                                break;
                        set_rtbmap(ext, XR_E_MULT);
                        break;
@@ -354,8 +355,14 @@ _("data fork in rt inode %" PRIu64 " found rt metadata extent %" PRIu64 " in rt
                        return 1;
                case XR_E_INUSE:
                case XR_E_MULT:
-                       if (xfs_has_rtreflink(mp))
-                               break;
+                       if (xfs_has_rtreflink(mp)) {
+                               if (irec->br_state == XFS_EXT_NORM)
+                                       break;
+                               do_warn(
+_("data fork in rt inode %" PRIu64 " claims shared unwritten rt extent %" PRIu64 "\n"),
+                                       ino, b);
+                               return 1;
+                       }
                        do_warn(
 _("data fork in rt inode %" PRIu64 " claims used rt extent %" PRIu64 "\n"),
                                ino, b);
@@ -672,8 +679,14 @@ _("%s fork in inode %" PRIu64 " claims metadata block %" PRIu64 "\n"),
                        case XR_E_INUSE:
                        case XR_E_MULT:
                                if (type == XR_INO_DATA &&
-                                   xfs_has_reflink(mp))
-                                       break;
+                                   xfs_has_reflink(mp)) {
+                                       if (irec.br_state == XFS_EXT_NORM)
+                                               break;
+                                       do_warn(
+_("%s fork in %s inode %" PRIu64 " claims shared unwritten block %" PRIu64 "\n"),
+                                               forkname, ftype, ino, b);
+                                       goto done;
+                               }
                                do_warn(
 _("%s fork in %s inode %" PRIu64 " claims used block %" PRIu64 "\n"),
                                        forkname, ftype, ino, b);