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

osd_xattr_set() to batch actual EA update

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.10.1, Lustre 2.11.0
    • None
    • None
    • 9223372036854775807

    Description

      moving EAs from nvlist into bonus/spill is quite expensive, we can save on this a bit collecting changes in nvlist (what we do already) and calling sa_update() from osd_trans_stop().

      Attachments

        Issue Links

          Activity

            [LU-7899] osd_xattr_set() to batch actual EA update
            pjones Peter Jones made changes -
            Labels Original: LTS

            John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/28482/
            Subject: LU-7899 osd: batch EA updates
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set:
            Commit: d53a1183058ebfd52bf61b194d151369ba7c5087

            gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/28482/ Subject: LU-7899 osd: batch EA updates Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: d53a1183058ebfd52bf61b194d151369ba7c5087

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/28482
            Subject: LU-7899 osd: batch EA updates
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: 214a6c3d66456ff5f73fce8eb06b0b8e164347f7

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/28482 Subject: LU-7899 osd: batch EA updates Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 214a6c3d66456ff5f73fce8eb06b0b8e164347f7
            pjones Peter Jones made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Reopened [ 4 ] New: Resolved [ 5 ]
            pjones Peter Jones added a comment -

            Landed for 2.11

            pjones Peter Jones added a comment - Landed for 2.11

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/21893/
            Subject: LU-7899 osd: batch EA updates
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 2c9ff6dffdf4320af95c9db9af07a416529275f0

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/21893/ Subject: LU-7899 osd: batch EA updates Project: fs/lustre-release Branch: master Current Patch Set: Commit: 2c9ff6dffdf4320af95c9db9af07a416529275f0

            Tested latest version of the patch, ran 24 hours on soak, so far. No significant errors.

            cliffw Cliff White (Inactive) added a comment - Tested latest version of the patch, ran 24 hours on soak, so far. No significant errors.

            What are you looking for from slab top? No hard crashes yet, restarting this morning.

            cliffw Cliff White (Inactive) added a comment - What are you looking for from slab top? No hard crashes yet, restarting this morning.

            Cliff, any more details?

            bzzz Alex Zhuravlev added a comment - Cliff, any more details?

            With this patch, i am seeing more jobs fail than succeed.

            cliffw Cliff White (Inactive) added a comment - With this patch, i am seeing more jobs fail than succeed.

            People

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

              Dates

                Created:
                Updated:
                Resolved: