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

qmt_pool_lock leak in qmt_pool_lookup()

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.15.0
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      It seems that qmt_pool_lookup() jumps to out_err label if qti_pools_add() fails, but the pool lock is not released:

      out_err:
              CERROR("%s: cannot add pool %s: err = %d\n",
                      qmt->qmt_svname, pos->qpi_name, rc);
              RETURN(ERR_PTR(rc));
      

      seems like a bug?
      Introduced by 09f9fb3211c "LU-11023 quota: quota pools for OSTs"

      Attachments

        Issue Links

          Activity

            [LU-13756] qmt_pool_lock leak in qmt_pool_lookup()
            pjones Peter Jones added a comment -

            Landed for 2.15

            pjones Peter Jones added a comment - Landed for 2.15

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/45106/
            Subject: LU-13756 quota: up_read leak in qmt_pool_lookup
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: d16b3141119a3b75276914ad3601e0dd27579b2b

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/45106/ Subject: LU-13756 quota: up_read leak in qmt_pool_lookup Project: fs/lustre-release Branch: master Current Patch Set: Commit: d16b3141119a3b75276914ad3601e0dd27579b2b

            "Sergey Cheremencev <sergey.cheremencev@hpe.com>" uploaded a new patch: https://review.whamcloud.com/45106
            Subject: LU-13756 quota: up_read leak in qmt_pool_lookup
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 71fe6dd27278df3ca01e0516da3242cf5c81513a

            gerrit Gerrit Updater added a comment - "Sergey Cheremencev <sergey.cheremencev@hpe.com>" uploaded a new patch: https://review.whamcloud.com/45106 Subject: LU-13756 quota: up_read leak in qmt_pool_lookup Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 71fe6dd27278df3ca01e0516da3242cf5c81513a

            People

              scherementsev Sergey Cheremencev
              green Oleg Drokin
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: