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

OFD object isn't released on error

Details

    • Bug
    • Resolution: Duplicate
    • Major
    • None
    • Lustre 2.9.0
    • 3
    • 9223372036854775807

    Attachments

      Issue Links

        is related to

        Bug - A problem which impairs or prevents the functions of the product. LU-6434 Object reference is not zero when umounting OST

        • Major - Major loss of function.
        • Resolved

        Activity

          [LU-8302] OFD object isn't released on error
          pjones Peter Jones added a comment -

          This is believed to be a duplicate of LU-6434

          pjones Peter Jones added a comment - This is believed to be a duplicate of LU-6434
          pjones Peter Jones added a comment -

          Alex/Mike

          Could you please review this patch?

          Thanks

          Peter

          pjones Peter Jones added a comment - Alex/Mike Could you please review this patch? Thanks Peter
          gerrit Gerrit Updater added a comment -

          Alexander Boyko (alexander.boyko@seagate.com) uploaded a new patch: http://review.whamcloud.com/20873
          Subject: LU-8302 ofd: ofd: correctly release object reference
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: d227cb5c17215273feea3293332517cc888edb95

          gerrit Gerrit Updater added a comment - Alexander Boyko (alexander.boyko@seagate.com) uploaded a new patch: http://review.whamcloud.com/20873 Subject: LU-8302 ofd: ofd: correctly release object reference Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: d227cb5c17215273feea3293332517cc888edb95
          aboyko Alexander Boyko added a comment -

          commitrw_read

                   fo = ofd_object_find(env, ofd, fid);
         if (IS_ERR(fo))
               RETURN(PTR_ERR(fo));

          This part forgets to release object reference, and actually it cannot do that. Small refactoring is introduced.

          aboyko Alexander Boyko added a comment - commitrw_read fo = ofd_object_find(env, ofd, fid); if (IS_ERR(fo)) RETURN(PTR_ERR(fo)); This part forgets to release object reference, and actually it cannot do that. Small refactoring is introduced.

          People

            tappro Mikhail Pershin
            aboyko Alexander Boyko
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: