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

LNet events should generated without resource lock held

Details

    • Improvement
    • Resolution: Fixed
    • Major
    • Lustre 2.14.0
    • None
    • any lustre
    • 9223372036854775807

    Description

      LNet execute an events with resource lock held, but event handler can allocate an memory, so it will block other callers to lockup over hash table or parallel event execution.
      I will send a patch son.

      Attachments

        Issue Links

          Activity

            [LU-10428] LNet events should generated without resource lock held

            Neil's patch has landed. If it hasn't resolved the problem feel free to reopen.

            simmonsja James A Simmons added a comment - Neil's patch has landed. If it hasn't resolved the problem feel free to reopen.

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37068/
            Subject: LU-10428 lnet: call event handlers without res_lock
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: d05427a7856e8f89cf6ec47f2731e12c6fa22901

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37068/ Subject: LU-10428 lnet: call event handlers without res_lock Project: fs/lustre-release Branch: master Current Patch Set: Commit: d05427a7856e8f89cf6ec47f2731e12c6fa22901

            Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37068
            Subject: LU-10428 lnet: call event handlers without res_lock
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 706e000cd325fb47f2e4b77e3f3afd17fc8b62b8

            gerrit Gerrit Updater added a comment - Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37068 Subject: LU-10428 lnet: call event handlers without res_lock Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 706e000cd325fb47f2e4b77e3f3afd17fc8b62b8
            shadow Alexey Lyashkov added a comment - It looks not a improvement now. http://testing.linuxhacker.ru:3333/lustre-reports/1276/testresults/runtests-ssk-ldiskfs-SSK-centos7_x86_64-centos7_x86_64/oleg78-client-console.txt
            spitzcor Cory Spitz added a comment -

            For some reason, a comment wasn't added here when https://review.whamcloud.com/#/c/30657 was uploaded for this issue.

            spitzcor Cory Spitz added a comment - For some reason, a comment wasn't added here when https://review.whamcloud.com/#/c/30657 was uploaded for this issue.

            People

              shadow Alexey Lyashkov
              shadow Alexey Lyashkov
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: