Details

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

    Description

      In the IOC_LIBCFS_DISCOVER and IOC_LIBCFS_PING_PEER cases of LNetCtl() references on the peers returned by lnet_find_peer() are leaked.

      Attachments

        Activity

          [LU-9918] peer leaks in LNetCtl()
          pjones Peter Jones added a comment -

          Landed for 2.11

          pjones Peter Jones added a comment - Landed for 2.11

          Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28722/
          Subject: LU-9918 lnet: decref on peer after use
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 440c1c03fe08df2ea626d8658ab9960ad108e19d

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28722/ Subject: LU-9918 lnet: decref on peer after use Project: fs/lustre-release Branch: master Current Patch Set: Commit: 440c1c03fe08df2ea626d8658ab9960ad108e19d

          Amir Shehata (amir.shehata@intel.com) uploaded a new patch: https://review.whamcloud.com/28722
          Subject: LU-9918 lnet: decref on peer after use
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 968dcfc7dff0b0a14f41a079702df0389e683260

          gerrit Gerrit Updater added a comment - Amir Shehata (amir.shehata@intel.com) uploaded a new patch: https://review.whamcloud.com/28722 Subject: LU-9918 lnet: decref on peer after use Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 968dcfc7dff0b0a14f41a079702df0389e683260

          I'm working on integrating the LNet Unit test framework, that'll bring in quiet a bit of tests that test the lnetconfig interface.

          ashehata Amir Shehata (Inactive) added a comment - I'm working on integrating the LNet Unit test framework, that'll bring in quiet a bit of tests that test the lnetconfig interface.
          jhammond John Hammond added a comment -

          We need tests somewhere that use both ioctls with valid and invalid peers. More generally we need basic coverage of lnetctl in lustre/tests/.

          jhammond John Hammond added a comment - We need tests somewhere that use both ioctls with valid and invalid peers. More generally we need basic coverage of lnetctl in lustre/tests/.

          People

            ashehata Amir Shehata (Inactive)
            jhammond John Hammond
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: