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

            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.

            the OBD_IOC_GETDTNAME is introduced by "http://review.whamcloud.com/#change,1646" in LU-819, did you apply that patch to b2_1?

            this patch needs ll_get_fsname redefined in http://review.whamcloud.com/2025 && http://review.whamcloud.com/#change,3704

            I have created a updated patch containing these missing stuff in b2_1, pls refs the attachment

            hongchao.zhang Hongchao Zhang added a comment - the OBD_IOC_GETDTNAME is introduced by "http://review.whamcloud.com/#change,1646" in LU-819 , did you apply that patch to b2_1? this patch needs ll_get_fsname redefined in http://review.whamcloud.com/2025 && http://review.whamcloud.com/#change,3704 I have created a updated patch containing these missing stuff in b2_1, pls refs the attachment

            I cherry-picked the patch into b2_1 branch. It was done cleanly. However, compilation failed:
            /usr/src/packages/BUILD/lustre-2.1.3/lustre/llite/llite_lib.c:2256: error: 'OBD_IOC_GETDTNAME' undeclared (first use in this function)

            jaylan Jay Lan (Inactive) added a comment - I cherry-picked the patch into b2_1 branch. It was done cleanly. However, compilation failed: /usr/src/packages/BUILD/lustre-2.1.3/lustre/llite/llite_lib.c:2256: error: 'OBD_IOC_GETDTNAME' undeclared (first use in this function)

            Hi Peter, certainly will do. Thanks~

            jaylan Jay Lan (Inactive) added a comment - Hi Peter, certainly will do. Thanks~
            pjones Peter Jones added a comment -

            Jay

            Can you please confirm whether the patch meets your needs?

            Thanks

            Peter

            pjones Peter Jones added a comment - Jay Can you please confirm whether the patch meets your needs? Thanks Peter

            the updated patch has been pushed to Gerrit

            hongchao.zhang Hongchao Zhang added a comment - the updated patch has been pushed to Gerrit
            pjones Peter Jones added a comment -

            Jay

            I will take this up with you offline

            Peter

            pjones Peter Jones added a comment - Jay I will take this up with you offline Peter

            There is no activity on this LU for almost 6 months. This fix is very important to use. Could you upgrade the priority? Thanks!

            jaylan Jay Lan (Inactive) added a comment - There is no activity on this LU for almost 6 months. This fix is very important to use. Could you upgrade the priority? Thanks!
            hongchao.zhang Hongchao Zhang added a comment - the patch is tracked at http://review.whamcloud.com/#change,1908

            status update:
            the patch is under creating and test.

            hongchao.zhang Hongchao Zhang added a comment - status update: the patch is under creating and test.

            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: