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

generate warnings in case of discarding dirty pages

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.4.0, Lustre 2.1.4
    • Lustre 1.8.x (1.8.0 - 1.8.5)
    • None
    • sles10sp3, sles11sp1, centos5.6, centos6
    • 3
    • 21,812
    • 5412

    Description

      "NASA/AMES is asking for the ability to be able to tell users which files may be suspect after an
      adverse cluster event; panics, hangs, client evictions, etc.

      One example that may be easier than others is when a client is evicted and the client is forced to
      toss dirty pages of open files. The site has been experimenting with a way to list the inode
      numbers on the mds with the associated pages tossed on the clients. I'll leave it to them to
      discuss here.
      " – from Oracle BZ 21812.

      Oracle has released a patch (attachment 33114.) It affected client side. We like Whamcloud to include this patch for 1.8.x and 2.1.x.

      Attachments

        Issue Links

          Activity

            [LU-921] generate warnings in case of discarding dirty pages
            pjones Peter Jones added a comment -

            Landed for 2.1.4 and 2.4

            pjones Peter Jones added a comment - Landed for 2.1.4 and 2.4
            emoly.liu Emoly Liu added a comment -

            Port for b2_1 is here http://review.whamcloud.com/#change,4716. I combine the two related patches into a single one.

            emoly.liu Emoly Liu added a comment - Port for b2_1 is here http://review.whamcloud.com/#change,4716 . I combine the two related patches into a single one.
            hongchao.zhang Hongchao Zhang added a comment - the patch is tracked at http://review.whamcloud.com/#change,4443

            Yes, this test is put into ALWAYS_EXCEPT in bz5494, and I tested it under master and b2_1, both passed locally,
            I'll try to create a patch to remove it from A_E, and check whether it pass or not in Toro.

            hongchao.zhang Hongchao Zhang added a comment - Yes, this test is put into ALWAYS_EXCEPT in bz5494, and I tested it under master and b2_1, both passed locally, I'll try to create a patch to remove it from A_E, and check whether it pass or not in Toro.
            pjones Peter Jones added a comment -

            Ah sorry - I did not realize that. In that case I will defer to Hongchao to comment.

            pjones Peter Jones added a comment - Ah sorry - I did not realize that. In that case I will defer to Hongchao to comment.

            Peter, a new test case recovery-small 24b was created to test this issue as part of the patch. That was how I discovered that test_24[ab] were skipped.

            Let me know if you want to fix that in this case, since recovery-small.sh is modified anyway, or you want me to open a new case. I am fine either way.

            jaylan Jay Lan (Inactive) added a comment - Peter, a new test case recovery-small 24b was created to test this issue as part of the patch. That was how I discovered that test_24 [ab] were skipped. Let me know if you want to fix that in this case, since recovery-small.sh is modified anyway, or you want me to open a new case. I am fine either way.
            pjones Peter Jones added a comment -

            Jay that is really a separate issue and should be tracked as such. Could you please open a separate ticket to track the issue about test 24? Thanks!

            pjones Peter Jones added a comment - Jay that is really a separate issue and should be tracked as such. Could you please open a separate ticket to track the issue about test 24? Thanks!

            The test result looks good!

            Can you make changes to remove test "24" from ALWAYS_EXCEPT as part of the patch? Thanks!

            jaylan Jay Lan (Inactive) added a comment - The test result looks good! Can you make changes to remove test "24" from ALWAYS_EXCEPT as part of the patch? Thanks!

            Why is recovery-small test 24 ALWAYS_EXCEPT?

            I think we can remove it from the ALWAYS_EXCEPT.

            jaylan Jay Lan (Inactive) added a comment - Why is recovery-small test 24 ALWAYS_EXCEPT? I think we can remove it from the ALWAYS_EXCEPT.

            I provide our Lustre group the new rpms and ask for feedback.

            jaylan Jay Lan (Inactive) added a comment - I provide our Lustre group the new rpms and ask for feedback.

            People

              hongchao.zhang Hongchao Zhang
              jaylan Jay Lan (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: