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

reuse same cl_dio_aio for one IO

Details

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

    Description

      IO might be restarted if layout changed, this might cause ki_complete() called several times for one IO.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-13846 move iov iter forward by ourself

          • Major - Major loss of function.
          • Resolved

          Activity

            [LU-13835] reuse same cl_dio_aio for one IO
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/39542/
            Subject: LU-13835 llite: reuse same cl_dio_aio for one IO
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: db6f203965d9173c3128c8d1c6188d2f63bc1b3e

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/39542/ Subject: LU-13835 llite: reuse same cl_dio_aio for one IO Project: fs/lustre-release Branch: master Current Patch Set: Commit: db6f203965d9173c3128c8d1c6188d2f63bc1b3e
            wshilong Wang Shilong (Inactive) added a comment -

            I think they are different problems, actually this ticket fixed leftover from LU-13697 and unfortunately we did not make it once in LU-13697

            wshilong Wang Shilong (Inactive) added a comment - I think they are different problems, actually this ticket fixed leftover from LU-13697 and unfortunately we did not make it once in LU-13697
            adilger Andreas Dilger added a comment -

            Yes, sorry for the confusion. I'm wondering if patch https://review.whamcloud.com/39565 "LU-13846 llite: move iov iter forward by ourself" is also needed for correct operations, and whether it makes sense to merge them together.

            adilger Andreas Dilger added a comment - Yes, sorry for the confusion. I'm wondering if patch https://review.whamcloud.com/39565 " LU-13846 llite: move iov iter forward by ourself " is also needed for correct operations, and whether it makes sense to merge them together.
            wshilong Wang Shilong (Inactive) added a comment -

            adilger you mean LU-13846?

            wshilong Wang Shilong (Inactive) added a comment - adilger you mean LU-13846 ?
            adilger Andreas Dilger added a comment -

            Shilong, do we also need the LU-13845 patch to make this work properly?

            adilger Andreas Dilger added a comment - Shilong, do we also need the LU-13845 patch to make this work properly?
            gerrit Gerrit Updater added a comment -

            Wang Shilong (wshilong@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/39542
            Subject: LU-13835 llite: reuse same cl_dio_aio for one IO
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: f6ee60c8f180c1543c08ff7cb616973fb51d360b

            gerrit Gerrit Updater added a comment - Wang Shilong (wshilong@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/39542 Subject: LU-13835 llite: reuse same cl_dio_aio for one IO Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: f6ee60c8f180c1543c08ff7cb616973fb51d360b

            People

              wshilong Wang Shilong (Inactive)
              wshilong Wang Shilong (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: