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

Define negative PFL extent start/end as invalid

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.16.0
    • Lustre 2.12.8, Lustre 2.15.1
    • None
    • Servers: CentOS 7.9 (3.10.0-1160.49.1.el7.x86_64), Lustre 2.12.8, ZFS 0.8.5
      2.12 Clients: CentOS 7.9 (3.10.0-1160.53.1.el7.x86_64), Lustre 2.12.8
      2.15 Clients: CentOS 7.9 (3.10.0-1160.76.1.el7.x86_64), Lustre 2.15.1
    • 3
    • 9223372036854775807

    Description

      Even though lu_extent.e_start and lu_extent.e_end are u64 type, we convert them to s64 and check whether they are negative values on MDS. All negative values except -1 (EOF) are regarded as invalid.

      Attachments

        Issue Links

          Activity

            [LU-16194] Define negative PFL extent start/end as invalid
            pjones Peter Jones added a comment -

            All patches merged for 2.16

            pjones Peter Jones added a comment - All patches merged for 2.16

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53771/
            Subject: LU-16194 tests: set minversion of MDS for sanity/65q
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 7e88ae40b8a7489b20680c780baa733473a490f9

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53771/ Subject: LU-16194 tests: set minversion of MDS for sanity/65q Project: fs/lustre-release Branch: master Current Patch Set: Commit: 7e88ae40b8a7489b20680c780baa733473a490f9

            "Feng Lei <flei@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53771
            Subject: LU-16194 tests: set minversion of MDS for sanity/65p
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 204ac072e9bf33e40522803d6ec6d9fbe2957705

            gerrit Gerrit Updater added a comment - "Feng Lei <flei@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53771 Subject: LU-16194 tests: set minversion of MDS for sanity/65p Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 204ac072e9bf33e40522803d6ec6d9fbe2957705

            This test is failing interop between master (2.15.60.20) and 2.15.4. Please review failure and push a patch. Either skip because it is not expected to work with old servers, or fix as needed:
            https://testing.whamcloud.com/test_sets/dc77145c-b7d3-4010-a7a2-f8435f9353ff

            adilger Andreas Dilger added a comment - This test is failing interop between master (2.15.60.20) and 2.15.4. Please review failure and push a patch. Either skip because it is not expected to work with old servers, or fix as needed: https://testing.whamcloud.com/test_sets/dc77145c-b7d3-4010-a7a2-f8435f9353ff
            pjones Peter Jones added a comment -

            Landed for 2.16

            pjones Peter Jones added a comment - Landed for 2.16

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48684/
            Subject: LU-16194 lod: define negative extent offset as invalid
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 3d6dbc72e161927ada762d2f0a68cf2d7acef4cb

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48684/ Subject: LU-16194 lod: define negative extent offset as invalid Project: fs/lustre-release Branch: master Current Patch Set: Commit: 3d6dbc72e161927ada762d2f0a68cf2d7acef4cb

            "Feng Lei <flei@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/48684
            Subject: LU-16194 lod: define negative extent offset as invalid
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 76c2cde4123e7a1bee1dbd1dfc49f9fbf19842b5

            gerrit Gerrit Updater added a comment - "Feng Lei <flei@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/48684 Subject: LU-16194 lod: define negative extent offset as invalid Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 76c2cde4123e7a1bee1dbd1dfc49f9fbf19842b5

            People

              flei Feng Lei
              flei Feng Lei
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: