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

Wrong read length over isize

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • Lustre 2.8.0
    • Lustre 2.8.0
    • 3
    • 9223372036854775807

    Description

      When running on osd-ldiskfs, if the isize is equal to 4095, a read length of 4096 will be returned because a wrong calculation of EOF.

      Attachments

        Issue Links

          is related to

          New Feature - A new feature of the product, which has yet to be developed. LU-12275 Client-side file data encryption

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Activity

            [LU-7371] Wrong read length over isize
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-12275 [ LU-12275 ]
            pjones Peter Jones made changes -
            Link New: This issue is related to LDEV-367 [ LDEV-367 ]
            pjones Peter Jones made changes -
            Link New: This issue is related to JFC-17 [ JFC-17 ]
            jgmitter Joseph Gmitter (Inactive) made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            jgmitter Joseph Gmitter (Inactive) added a comment -

            Both patches have landed for 2.8.0

            jgmitter Joseph Gmitter (Inactive) added a comment - Both patches have landed for 2.8.0
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/17060/
            Subject: LU-7371 test: wrong read length over isize
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 7023698133970372031a16beac276e5e3e64cfbe

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/17060/ Subject: LU-7371 test: wrong read length over isize Project: fs/lustre-release Branch: master Current Patch Set: Commit: 7023698133970372031a16beac276e5e3e64cfbe
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/17020/
            Subject: LU-7371 osd-ldiskfs: fix wrong read length over isize
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 97c4c162be77d2ee9bad5d800c9b5803f252caa0

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/17020/ Subject: LU-7371 osd-ldiskfs: fix wrong read length over isize Project: fs/lustre-release Branch: master Current Patch Set: Commit: 97c4c162be77d2ee9bad5d800c9b5803f252caa0
            jfc John Fuchs-Chesney (Inactive) made changes -
            Labels New: patch
            gerrit Gerrit Updater added a comment -

            Li Xi (lixi@ddn.com) uploaded a new patch: http://review.whamcloud.com/17060
            Subject: LU-7371 test: wrong read length over isize
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: bc9893da37ae60918b61e2d5fd84b4cb87ad3b82

            gerrit Gerrit Updater added a comment - Li Xi (lixi@ddn.com) uploaded a new patch: http://review.whamcloud.com/17060 Subject: LU-7371 test: wrong read length over isize Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: bc9893da37ae60918b61e2d5fd84b4cb87ad3b82
            adilger Andreas Dilger added a comment - - edited

            I had originally thought dd if=/dev/zero of=$DIR/$tfile bs=4095 count=1 conv=sync would be enough to create the file at 4095 bytes, and then the with bs=4096 bytes would trigger the bug. If that doesn't work, then another option is to add an OBD_FAIL_OST_* check in the code to reproduce the original symptom.

            adilger Andreas Dilger added a comment - - edited I had originally thought dd if=/dev/zero of=$DIR/$tfile bs=4095 count=1 conv=sync would be enough to create the file at 4095 bytes, and then the with bs=4096 bytes would trigger the bug. If that doesn't work, then another option is to add an OBD_FAIL_OST_* check in the code to reproduce the original symptom.

            People

              bzzz Alex Zhuravlev
              lixi Li Xi (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: