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

ofd_commitrw_read() can be passed non-existing object

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.16.0
    • None
    • 3
    • 9223372036854775807

    Description

      A client can get evicted during OST_READ's bulk, so the corresponding LDLM lock is cancelled and OST_DESTROY can get over the object. in this scenario ofd_commitrw_read() can meet non-existing (just removed) object. this is a valid case at least until LU-16064 is fixed.

      Attachments

        Issue Links

          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. LU-17585 OSS LBUG: LustreError: 46818:0:(ofd_io.c:1027:ofd_commitrw_read()) ASSERTION( ofd_object_exists(fo) ) failed

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

          Bug - A problem which impairs or prevents the functions of the product. LU-16064 RPC from evicted client can corrupt data

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • In Progress

          Activity

            [LU-16345] ofd_commitrw_read() can be passed non-existing object
            gerrit Gerrit Updater added a comment -

            "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55275
            Subject: LU-16345 ofd: ofd_commitrw_read() with non-existing object
            Project: fs/lustre-release
            Branch: b2_15
            Current Patch Set: 1
            Commit: 59ace374363a8886e4459cbfef66774b44de78df

            gerrit Gerrit Updater added a comment - "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55275 Subject: LU-16345 ofd: ofd_commitrw_read() with non-existing object Project: fs/lustre-release Branch: b2_15 Current Patch Set: 1 Commit: 59ace374363a8886e4459cbfef66774b44de78df
            gerrit Gerrit Updater added a comment -

            "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55274
            Subject: LU-16345 ofd: ofd_commitrw_read() with non-existing object
            Project: fs/lustre-release
            Branch: b2_12
            Current Patch Set: 1
            Commit: edb7f9634fc2a8bb32958f98b16fff5fafd316da

            gerrit Gerrit Updater added a comment - "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55274 Subject: LU-16345 ofd: ofd_commitrw_read() with non-existing object Project: fs/lustre-release Branch: b2_12 Current Patch Set: 1 Commit: edb7f9634fc2a8bb32958f98b16fff5fafd316da
            eaujames Etienne Aujames added a comment - - edited

            The CEA hit that one in production with Lustre 2.12.9 after losing most of the Lustre's clients due to power outage.

            The process causing the race between object deletions and read was "ioping", it sent read/write on an unlink file:

            1. open/creat a new file
            2. unlink the file (pending for deletion on the MDT)
            3. write/read on the file
            4. close (on exit)
            eaujames Etienne Aujames added a comment - - edited The CEA hit that one in production with Lustre 2.12.9 after losing most of the Lustre's clients due to power outage. The process causing the race between object deletions and read was "ioping", it sent read/write on an unlink file: open/creat a new file unlink the file (pending for deletion on the MDT) write/read on the file close (on exit)
            pjones Peter Jones added a comment -

            Landed for 2.16

            pjones Peter Jones added a comment - Landed for 2.16
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49255/
            Subject: LU-16345 ofd: ofd_commitrw_read() with non-existing object
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 5efc4c1cb4f2d0680992188d587f583e7a567a09

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49255/ Subject: LU-16345 ofd: ofd_commitrw_read() with non-existing object Project: fs/lustre-release Branch: master Current Patch Set: Commit: 5efc4c1cb4f2d0680992188d587f583e7a567a09
            gerrit Gerrit Updater added a comment -

            "Alex Zhuravlev <bzzz@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49255
            Subject: LU-16345 ofd: ofd_commitrw_read() with non-existing object
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 71e84077a50dfe16b1a48da2973176397736bae0

            gerrit Gerrit Updater added a comment - "Alex Zhuravlev <bzzz@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49255 Subject: LU-16345 ofd: ofd_commitrw_read() with non-existing object Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 71e84077a50dfe16b1a48da2973176397736bae0

            People

              bzzz Alex Zhuravlev
              bzzz Alex Zhuravlev
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: