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

osd-zfs should allow more concurrent reads

Details

    • Improvement
    • Resolution: Won't Fix
    • Critical
    • None
    • Lustre 2.6.0
    • 15644

    Description

      In the OBD layer, ofd_preprw_read() calls dt_bufs_get(dbo_bufs_get==osd_bufs_get_read) multiple times to prepare local buffers and then calls dt_read_prep(dbo_read_prep==osd_read_prep) once for all local buffers, in the hope that the osd driver would issue all reads concurrently in one go.

      But the osd-zfs driver currently issues reads and waits for IO to complete in osd_bufs_get_read(), and does NOOP in osd_read_prep (see also LU-4820). So in ofd_preprw_read() the reads are essentially done sequentially, i.e. no concurrent IO.

      The ZFS IO scheduler services reads from osd-zfs in FIFO order, i.e. no reordering or merging. It might be possible to reduce seeks by having more concurrent reads and using the Linux deadline IO scheduler under the pool - see HP-5 for an example of poor read throughput due to excessive seeks.

      Attachments

        Activity

          [LU-5598] osd-zfs should allow more concurrent reads
          pjones Peter Jones made changes -
          Fix Version/s Original: Lustre 2.8.0 [ 11113 ]
          Resolution New: Won't Fix [ 2 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          jlevi Jodi Levi (Inactive) made changes -
          Fix Version/s New: Lustre 2.8.0 [ 11113 ]
          Fix Version/s Original: Lustre 2.7.0 [ 10631 ]
          pjones Peter Jones made changes -
          Labels Original: 22i zfs New: zfs
          jlevi Jodi Levi (Inactive) made changes -
          Fix Version/s New: Lustre 2.7.0 [ 10631 ]
          adilger Andreas Dilger made changes -
          Priority Original: Major [ 3 ] New: Critical [ 2 ]
          adilger Andreas Dilger made changes -
          Affects Version/s New: Lustre 2.6.0 [ 10595 ]
          adilger Andreas Dilger made changes -
          Priority Original: Minor [ 4 ] New: Major [ 3 ]
          pjones Peter Jones made changes -
          Labels Original: zfs New: 22i zfs
          isaac Isaac Huang (Inactive) created issue -

          People

            isaac Isaac Huang (Inactive)
            isaac Isaac Huang (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: