Details

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

    Description

      When doing readahead, we see an amazing amount of time (~5%) just looking up the page index from the lov layer.

       a
      -   84.42%     0.34%  ior              [kernel.kallsyms]     [k] ll_readpage                a
         - 84.08% ll_readpage                                                                     a
            - 82.68% ll_io_read_page                                                              a
               + 68.53% ll_readahead.isra.27                                                      a
               - 14.14% cl_io_submit_rw                                                           a
                  - 13.40% lov_io_submit                                                          a
                     - 8.24% lov_page_index                                                       a
                          5.56% cl_page_at  

      This never changes once a page is created, so we can just store it at the top layer.

      Attachments

        Activity

          [LU-12535] lov_page_index surprisingly slow
          adilger Andreas Dilger made changes -
          Link New: This issue is related to DDN-867 [ DDN-867 ]
          pjones Peter Jones made changes -
          Fix Version/s New: Lustre 2.13.0 [ 14290 ]
          Resolution New: Fixed [ 1 ]
          Status Original: Reopened [ 4 ] New: Resolved [ 5 ]
          mdiep Minh Diep made changes -
          Comment [ Patrick Farrell (pfarrell@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36060
          Subject: LU-12535 lov: Move page index to top level
          Project: fs/lustre-wc
          Branch: io500
          Current Patch Set: 1
          Commit: 3c13394ed4e6014bb0b5ac238f42e8ed1f8c242a ]
          pfarrell Patrick Farrell (Inactive) made changes -
          Resolution Original: Won't Do [ 10200 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
          pfarrell Patrick Farrell (Inactive) made changes -
          Comment [ Aw, crap.  This is due to a bug.  Nevermind... ]
          pfarrell Patrick Farrell (Inactive) made changes -
          Comment [ Bug in another, not landed, patch. ]
          pfarrell Patrick Farrell (Inactive) made changes -
          Resolution New: Won't Do [ 10200 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          pfarrell Patrick Farrell (Inactive) created issue -

          People

            pfarrell Patrick Farrell (Inactive)
            pfarrell Patrick Farrell (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: