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

fsync on client does not cause OST_SYNCs to be issued

Details

    • Bug
    • Resolution: Fixed
    • Blocker
    • Lustre 2.6.0
    • Lustre 2.5.0, Lustre 2.6.0
    • 3
    • 12048

    Description

      Issuing an fsync() only causes MDS_SYNC's to be issued.

      Testing:
      dd with conv=fsync vs. dd with oflag=sync

      Attachments

        Activity

          [LU-4388] fsync on client does not cause OST_SYNCs to be issued

          Bobi Jam (bobijam@hotmail.com) uploaded a new patch: http://review.whamcloud.com/15390
          Subject: LU-4388 test: add sanity test case
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 3112c90215374573c987a800c192300817c62200

          gerrit Gerrit Updater added a comment - Bobi Jam (bobijam@hotmail.com) uploaded a new patch: http://review.whamcloud.com/15390 Subject: LU-4388 test: add sanity test case Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 3112c90215374573c987a800c192300817c62200

          Bobi Jam (bobijam@hotmail.com) uploaded a new patch: http://review.whamcloud.com/14840
          Subject: LU-4388 llite: issue OST_SYNC for fsync()
          Project: fs/lustre-release
          Branch: b2_5
          Current Patch Set: 1
          Commit: 3951a7c7aed155c206043838e4e800c94ac1c692

          gerrit Gerrit Updater added a comment - Bobi Jam (bobijam@hotmail.com) uploaded a new patch: http://review.whamcloud.com/14840 Subject: LU-4388 llite: issue OST_SYNC for fsync() Project: fs/lustre-release Branch: b2_5 Current Patch Set: 1 Commit: 3951a7c7aed155c206043838e4e800c94ac1c692

          Given the severity (data loss, inconsistent databases, etc..), shouldn't it be included in the maintenance release?

          happe Hans Henrik Happe added a comment - Given the severity (data loss, inconsistent databases, etc..), shouldn't it be included in the maintenance release?
          bobijam Zhenyu Xu added a comment -

          2.6.0 has the patches and 2.5.0 does not.

          bobijam Zhenyu Xu added a comment - 2.6.0 has the patches and 2.5.0 does not.

          Should this have been fixed after 2.5.0 or is it only 2.6.0? I see the same problem with b2_5 on zfs (fdatasync okay and slow, fsync not touching the disk). I think it is important that fsync works as expected.

          happe Hans Henrik Happe added a comment - Should this have been fixed after 2.5.0 or is it only 2.6.0? I see the same problem with b2_5 on zfs (fdatasync okay and slow, fsync not touching the disk). I think it is important that fsync works as expected.

          Patch landed to Master.

          jlevi Jodi Levi (Inactive) added a comment - Patch landed to Master.

          The patch http://review.whamcloud.com/8626 has not landed yet.

          adilger Andreas Dilger added a comment - The patch http://review.whamcloud.com/8626 has not landed yet.

          Patch has landed to Master. Please reopen if more work is needed in this ticket.

          jlevi Jodi Levi (Inactive) added a comment - Patch has landed to Master. Please reopen if more work is needed in this ticket.
          bobijam Zhenyu Xu added a comment - patch tracking at http://review.whamcloud.com/8684

          I see. So this is a misunderstanding of @datasync parameter. This bug may exist in all series of lustre client implementation. Thanks for explanation.

          jay Jinshan Xiong (Inactive) added a comment - I see. So this is a misunderstanding of @datasync parameter. This bug may exist in all series of lustre client implementation. Thanks for explanation.

          People

            bobijam Zhenyu Xu
            utopiabound Nathaniel Clark
            Votes:
            0 Vote for this issue
            Watchers:
            13 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: