Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.10.0
    • None
    • None
    • 9223372036854775807

    Description

      CL_PAGE_INVOKE, CL_PAGE_INVOID and CL_PAGE_INVOID_REVERSE all simply call list_for_each_entry (or list_for_each_entry_reverse) and call one of the functions from cl_page_operations.

      It's much simpler and easy to read to unroll these #defines, along with CL_PAGE_OP and use the correct code.

      Attachments

        Activity

          [LU-9523] unravel #DEFINEs in cl_page.c
          pjones Peter Jones added a comment -

          Landed for 2.10

          pjones Peter Jones added a comment - Landed for 2.10

          Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/27168/
          Subject: LU-9523 obdclass: remove CL_PAGE_INVOKE et. al.
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 603557c13111f188d51a1e7983343471e4a5cf71

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/27168/ Subject: LU-9523 obdclass: remove CL_PAGE_INVOKE et. al. Project: fs/lustre-release Branch: master Current Patch Set: Commit: 603557c13111f188d51a1e7983343471e4a5cf71

          Ben Evans (bevans@cray.com) uploaded a new patch: https://review.whamcloud.com/27168
          Subject: LU-9523 obdclass: remove CL_PAGE_INVOKE et. al.
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 2fc985c2f20cfa59a5db9041576cd36453a05eed

          gerrit Gerrit Updater added a comment - Ben Evans (bevans@cray.com) uploaded a new patch: https://review.whamcloud.com/27168 Subject: LU-9523 obdclass: remove CL_PAGE_INVOKE et. al. Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 2fc985c2f20cfa59a5db9041576cd36453a05eed

          People

            bevans Ben Evans (Inactive)
            bevans Ben Evans (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: