Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-17565

migrate racing with another ops can corrupt filesystem

Details

    • Bug
    • Resolution: Unresolved
    • Minor
    • None
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      mdt_reint_migrate() lookups names, then grabs ldlm locks and doesn't recheck names. another operation like unlink can race and modify directories within this lookup-to-locking window. after that mdt_reint_migrate() generates a distributed transaction which fails on a remote node (due to missing already name), but doesn't rollback properly.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-17575 Require specific enable flag for lfs migrate -m (directory migration)

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Activity

            [LU-17565] migrate racing with another ops can corrupt filesystem
            bzzz Alex Zhuravlev added a comment -

            I think we should be grabbing LDLM locks first or recheck the data once LDLM locks are granted.
            rollback can be an option but looks more complicated to me.

            bzzz Alex Zhuravlev added a comment - I think we should be grabbing LDLM locks first or recheck the data once LDLM locks are granted. rollback can be an option but looks more complicated to me.
            gerrit Gerrit Updater added a comment -

            "Alex Zhuravlev <bzzz@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54104
            Subject: LU-17565 tests: migrate vs unlink race
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 4a01bbff1b519dbf5595346d2d0ba8cb40a0259d

            gerrit Gerrit Updater added a comment - "Alex Zhuravlev <bzzz@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54104 Subject: LU-17565 tests: migrate vs unlink race Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 4a01bbff1b519dbf5595346d2d0ba8cb40a0259d

            People

              wc-triage WC Triage
              bzzz Alex Zhuravlev
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: