Details

    • Technical task
    • Resolution: Fixed
    • Minor
    • Lustre 2.12.0
    • None
    • 9223372036854775807

    Description

      Call mdt_dom_check_and_discard() after mdt_object_unlock() to avoid possible deadlock if some third lock is conflicting with both like in the scenario below:

      • thread1: mdt_object_lock() with some bits
      • thread2: take conflicting lock and wait
      • thread1: mdt_dom_check_and_discard() with bits conflicting with thread2 causes deadlock.

      Attachments

        Issue Links

          Activity

            [LU-11121] DoM: mdt take discard lock at cleanup stage

            Mike Pershin (mpershin@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33026
            Subject: LU-11121 mdt: discard data after all object are put
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: b9e8e2b076537c120778f2523d286df8040d04e7

            gerrit Gerrit Updater added a comment - Mike Pershin (mpershin@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33026 Subject: LU-11121 mdt: discard data after all object are put Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: b9e8e2b076537c120778f2523d286df8040d04e7
            pjones Peter Jones added a comment -

            Landed for 2.12

            pjones Peter Jones added a comment - Landed for 2.12

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/29930/
            Subject: LU-11121 mdt: take discard lock at cleanup stage
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 2506fa2a42b7ae542fae9346a9cc8a6ae271f5de

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/29930/ Subject: LU-11121 mdt: take discard lock at cleanup stage Project: fs/lustre-release Branch: master Current Patch Set: Commit: 2506fa2a42b7ae542fae9346a9cc8a6ae271f5de
            jgmitter Joseph Gmitter (Inactive) added a comment - Patch is at https://review.whamcloud.com/#/c/29930/

            People

              tappro Mikhail Pershin
              jgmitter Joseph Gmitter (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: