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

Minor DIO test improvements

Details

    • Improvement
    • Resolution: Unresolved
    • Minor
    • None
    • None
    • None
    • 9223372036854775807

    Description

      This ticket serves to cover a few test improvements to tests added under LU-13799, so that ticket can be closed.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-13799 DIO/AIO efficiency improvements

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            [LU-15483] Minor DIO test improvements
            paf0186 Patrick Farrell added a comment -

            This specific patch was just merged, so I don't think so.  This patch might make those timeouts more frequent - There's a partial rewrite of this code in flight to fix several bugs in this area:
            https://review.whamcloud.com/47187/

            (I can't hit the 398b hangs locally so I don't know for sure if it solves them.)

            paf0186 Patrick Farrell added a comment - This specific patch was just merged, so I don't think so.  This patch might make those timeouts more frequent - There's a partial rewrite of this code in flight to fix several bugs in this area: https://review.whamcloud.com/47187/ (I can't hit the 398b hangs locally so I don't know for sure if it solves them.)
            bzzz Alex Zhuravlev added a comment -

            can this patch be related to recent timeouts (stats over last 4 weeks):

            bzzz Alex Zhuravlev added a comment - can this patch be related to recent timeouts (stats over last 4 weeks):
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44321/
            Subject: LU-15483 tests: Improve test 398b
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: b4880f37582a1a27350b4a76c08e5a7860039b06

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44321/ Subject: LU-15483 tests: Improve test 398b Project: fs/lustre-release Branch: master Current Patch Set: Commit: b4880f37582a1a27350b4a76c08e5a7860039b06
            paf0186 Patrick Farrell added a comment - - edited

            Ah, thanks for catching both of those.  46140 is just an internal branch copy of one of these, didn't intend to note it here.  I'll remove it to reduce confusion.

            paf0186 Patrick Farrell added a comment - - edited Ah, thanks for catching both of those.  46140 is just an internal branch copy of one of these, didn't intend to note it here.  I'll remove it to reduce confusion.
            spitzcor Cory Spitz added a comment -

            https://review.whamcloud.com/44321 too.

            FYI, https://review.whamcloud.com/#/c/46140/ appears to be private.

            spitzcor Cory Spitz added a comment - https://review.whamcloud.com/44321 too. FYI, https://review.whamcloud.com/#/c/46140/ appears to be private.
            paf0186 Patrick Farrell added a comment - - edited

            https://review.whamcloud.com/#/c/46139/

            https://review.whamcloud.com/#/c/46135/

            paf0186 Patrick Farrell added a comment - - edited https://review.whamcloud.com/#/c/46139/ https://review.whamcloud.com/#/c/46135/

            People

              paf0186 Patrick Farrell
              paf0186 Patrick Farrell
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated: