Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.5.0
    • Lustre 2.5.0
    • 8789

    Description

      Binary heap in libcfs enables NRS policies to sort requests efficiently. But the API of it is not sufficient enough for some use cases.

      For example, if we want to change the values of the node in the heap, we have to remove the node from the heap, change the values and then insert the node into the heap again. However, the process of inserting into the heap might fails because of memory insufficiency. Sometimes, it is not convienient and efficient for a caller to handle such kind of failure which can be easily avioded by adding new APIs

      What's more, there is no API for a caller to judge whether a node is in the heap or not, which is not convienient too.

      Attachments

        Activity

          [LU-3494] Libcfs binary heap improvements

          Patch cherry-picked for master as 8beee9d0ff74f78fc5b072d1cf98f55d927dcdbe

          bobbielind Bobbie Lind (Inactive) added a comment - Patch cherry-picked for master as 8beee9d0ff74f78fc5b072d1cf98f55d927dcdbe
          pjones Peter Jones added a comment -

          Bobbie

          Could you please take care of this one?

          Thanks

          Peter

          pjones Peter Jones added a comment - Bobbie Could you please take care of this one? Thanks Peter
          pjones Peter Jones added a comment -

          Thanks for the patch Li Xi

          pjones Peter Jones added a comment - Thanks for the patch Li Xi

          This patch adds a funtion which relocates the changed node in a heap.
          http://review.whamcloud.com/#/c/6743/

          lixi Li Xi (Inactive) added a comment - This patch adds a funtion which relocates the changed node in a heap. http://review.whamcloud.com/#/c/6743/

          People

            bobbielind Bobbie Lind (Inactive)
            lixi Li Xi (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: