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

ptlrpc_request_free() after failed ptlrpc_request_pack() leaves an import referenced

Details

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

    Description

      many functions (like osc_setattr()) call ptlrpc_request_free() after failed ptlrpc_request_pack() to release the request. this seem to leave the import referenced and then lead to umount getting stuck.

      Attachments

        Activity

          [LU-7300] ptlrpc_request_free() after failed ptlrpc_request_pack() leaves an import referenced
          adilger Andreas Dilger made changes -
          Assignee Original: Alex Zhuravlev [ bzzz ] New: WC Triage [ wc-triage ]
          adilger Andreas Dilger made changes -
          Labels New: easy
          pjones Peter Jones made changes -
          Fix Version/s Original: Lustre 2.11.0 [ 13091 ]
          pjones Peter Jones made changes -
          Fix Version/s New: Lustre 2.11.0 [ 13091 ]
          Fix Version/s Original: Lustre 2.10.0 [ 12204 ]
          pjones Peter Jones made changes -
          Fix Version/s New: Lustre 2.10.0 [ 12204 ]
          Fix Version/s Original: Lustre 2.9.0 [ 11891 ]
          jgmitter Joseph Gmitter (Inactive) made changes -
          Assignee Original: WC Triage [ wc-triage ] New: Alex Zhuravlev [ bzzz ]
          jgmitter Joseph Gmitter (Inactive) added a comment -

          Alex,
          Assigning to you per our discussion.
          Thanks.
          Joe

          jgmitter Joseph Gmitter (Inactive) added a comment - Alex, Assigning to you per our discussion. Thanks. Joe
          jgmitter Joseph Gmitter (Inactive) made changes -
          Fix Version/s New: Lustre 2.9.0 [ 11891 ]
          jhammond John Hammond added a comment -

          This would be a good place to fix the strange error handling in __ptlrpc_request_bufs_pack() where we drop a reference on the import. See http://review.whamcloud.com/#/c/16765/.

          jhammond John Hammond added a comment - This would be a good place to fix the strange error handling in __ptlrpc_request_bufs_pack() where we drop a reference on the import. See http://review.whamcloud.com/#/c/16765/ .
          bzzz Alex Zhuravlev created issue -

          People

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

            Dates

              Created:
              Updated: