Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.9.0
    • None
    • 3
    • 9223372036854775807

    Description

      Alexey Lyashkov have troubles with accessing this jira and asked my to create this issue. Here is problem description from him:

      RHEL6.5 have a backported a upstream commit 91dd8c114499e9818f2d5919ef0b9eee61810220, which broke a lustre extent handing on OFD. But Intel have a copy-paste unfixed code and use with RHEL 6.5 and up code. It's looks wrong as it have don't fixed a fiemap vs walk race while fiemap used in lustre code now (lustre commit - 144b5a65c16f1bca81ae76010528a592c821308b)
      it may produce an predictable results with memory corruption.

      attached patch backport a new api from kernel upstream already supported in lustre.

      Attachments

        Activity

          [LU-8410] fiemap vs walk race
          pjones Peter Jones added a comment -

          Landed for 2.9

          pjones Peter Jones added a comment - Landed for 2.9

          Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/21603/
          Subject: LU-8410 ldiskfs: new FIEMAP API
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 036912c414a25bfa56d8702533034ca5277aa4b3

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/21603/ Subject: LU-8410 ldiskfs: new FIEMAP API Project: fs/lustre-release Branch: master Current Patch Set: Commit: 036912c414a25bfa56d8702533034ca5277aa4b3

          Artem Blagodarenko (artem.blagodarenko@seagate.com) uploaded a new patch: http://review.whamcloud.com/21603
          Subject: LU-8410 ldiskfs: new FIEMAP API
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 590ec316194a57d32b0d5c2e567111f1bd640c3b

          gerrit Gerrit Updater added a comment - Artem Blagodarenko (artem.blagodarenko@seagate.com) uploaded a new patch: http://review.whamcloud.com/21603 Subject: LU-8410 ldiskfs: new FIEMAP API Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 590ec316194a57d32b0d5c2e567111f1bd640c3b

          Artem, can you please submit the patch to Gerrit and add Yang Sheng and Bruno as reviewers.

          Please fix the patch summary line to be "... new FIEMAP API" to make this more clear what the patch is for, and also capitalize the sentences in the body of the commit message. s/fiemap/FIEMAP/, s/depricated/deprecated/.

          adilger Andreas Dilger added a comment - Artem, can you please submit the patch to Gerrit and add Yang Sheng and Bruno as reviewers. Please fix the patch summary line to be "... new FIEMAP API" to make this more clear what the patch is for, and also capitalize the sentences in the body of the commit message. s/fiemap/FIEMAP/, s/depricated/deprecated/.

          People

            wc-triage WC Triage
            artem_blagodarenko Artem Blagodarenko (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: