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

short read does not set ki_pos correctly

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.16.0
    • Lustre 2.15.5
    • 3
    • 9223372036854775807

    Description

      00000080:00020000:3.0:1706058118.952841:0:7094:0:(file.c:2098:file_read_confine_iter()) event.log.20240123T162623_0: read [198356992, 198361088] over eof, kms 198356992, file_size 198356992.
      00000080:00020000:3.0:1706058118.952904:0:7094:0:(file.c:2098:file_read_confine_iter()) event.log.20240123T162623_0: read [198361088, 198365184] over eof, kms 198362740, file_size 198362740.

      short read does not set ki_pos correctly, and jumps to wrong position at next read as the file being appended by others.

      Attachments

        Issue Links

          Activity

            [LU-17482] short read does not set ki_pos correctly
            eaujames Etienne Aujames added a comment - - edited

            The CEA hit this on a 2.15.5 in production. This is a blocking issue for them.

            eaujames Etienne Aujames added a comment - - edited The CEA hit this on a 2.15.5 in production. This is a blocking issue for them.

            "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55936
            Subject: LU-17482 llite: short read could mess up next read offset
            Project: fs/lustre-release
            Branch: b2_15
            Current Patch Set: 1
            Commit: ad7f1f4111ae0f7d6de54e1bbb2506e008bac0d8

            gerrit Gerrit Updater added a comment - "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55936 Subject: LU-17482 llite: short read could mess up next read offset Project: fs/lustre-release Branch: b2_15 Current Patch Set: 1 Commit: ad7f1f4111ae0f7d6de54e1bbb2506e008bac0d8
            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/+/53827/
            Subject: LU-17482 llite: short read could mess up next read offset
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 35fb413683affe9d41f3521259c260de6caf81d9

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53827/ Subject: LU-17482 llite: short read could mess up next read offset Project: fs/lustre-release Branch: master Current Patch Set: Commit: 35fb413683affe9d41f3521259c260de6caf81d9
            bobijam Zhenyu Xu added a comment -

            https://review.whamcloud.com/c/fs/lustre-release/+/53827/

            LU-17482 llite: short read could mess up next read offset

            When read reaches EOF, it could read data from stale pagecache, but
            we need to restore the iocb->ki_pos so that next read could continue
            from the correct offset.

            bobijam Zhenyu Xu added a comment - https://review.whamcloud.com/c/fs/lustre-release/+/53827/ LU-17482 llite: short read could mess up next read offset When read reaches EOF, it could read data from stale pagecache, but we need to restore the iocb->ki_pos so that next read could continue from the correct offset.

            People

              bobijam Zhenyu Xu
              bobijam Zhenyu Xu
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: