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

Improve ptlrpc_register_bulk() behavior

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.14.0
    • None
    • 9223372036854775807

    Description

      The fix for https://review.whamcloud.com/36309 could be improved a bit. It makes more sense to clear the bd_registered flag in ptlrpc_unregister_bulk() since it is set in ptlrpc_register_bulk().

      Also, we can add a test for this bug.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-12816 LBUG: (niobuf.c:350:ptlrpc_register_bulk()) ASSERTION( !(desc->bd_registered && req->rq_send_state != LUSTRE_IMP_REPLAY) || mbits != desc->bd_last_mbits )

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

          Activity

            [LU-13509] Improve ptlrpc_register_bulk() behavior
            adilger Andreas Dilger made changes -
            Summary Original: LU-12816 Follow-on New: Improve ptlrpc_register_bulk() behavior
            gerrit Gerrit Updater added a comment -

            Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/38806
            Subject: LU-13509 ptlrpc: Clear bd_registered in ptlrpc_unregister_bulk
            Project: fs/lustre-release
            Branch: b2_12
            Current Patch Set: 1
            Commit: 4864523dad609906f9f38e738375ae3c382b0d97

            gerrit Gerrit Updater added a comment - Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/38806 Subject: LU-13509 ptlrpc: Clear bd_registered in ptlrpc_unregister_bulk Project: fs/lustre-release Branch: b2_12 Current Patch Set: 1 Commit: 4864523dad609906f9f38e738375ae3c382b0d97
            pjones Peter Jones made changes -
            Link New: This issue is related to JFC-17 [ JFC-17 ]
            pjones Peter Jones made changes -
            Labels New: LTS12
            pjones Peter Jones made changes -
            Fix Version/s New: Lustre 2.14.0 [ 14490 ]
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            pjones Peter Jones added a comment -

            Landed for 2.14

            pjones Peter Jones added a comment - Landed for 2.14
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38457/
            Subject: LU-13509 ptlrpc: Clear bd_registered in ptlrpc_unregister_bulk
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 15057a17ca1e2a0e796cfada6abeb28aa77679cf

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38457/ Subject: LU-13509 ptlrpc: Clear bd_registered in ptlrpc_unregister_bulk Project: fs/lustre-release Branch: master Current Patch Set: Commit: 15057a17ca1e2a0e796cfada6abeb28aa77679cf
            wshilong Wang Shilong (Inactive) made changes -
            Link New: This issue is related to DDN-1174 [ DDN-1174 ]
            gerrit Gerrit Updater added a comment -

            Chris Horn (chris.horn@hpe.com) uploaded a new patch: https://review.whamcloud.com/38457
            Subject: LU-13509 ptlrpc: Add test and tweak fix for LU-12816
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 23c1dcca82b685b8b34e895564f4ea019175de59

            gerrit Gerrit Updater added a comment - Chris Horn (chris.horn@hpe.com) uploaded a new patch: https://review.whamcloud.com/38457 Subject: LU-13509 ptlrpc: Add test and tweak fix for LU-12816 Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 23c1dcca82b685b8b34e895564f4ea019175de59
            hornc Chris Horn made changes -
            Link New: This issue is related to LU-12816 [ LU-12816 ]

            People

              hornc Chris Horn
              hornc Chris Horn
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: