Details

    • Improvement
    • Resolution: Fixed
    • Major
    • Lustre 2.11.0, Lustre 2.10.2
    • Lustre 2.10.1, Lustre 2.11.0
    • 9223372036854775807

    Description

      osd-zfs should use dnode size depending on zfs property

      Attachments

        Issue Links

          Activity

            [LU-10041] osd-zfs to choose dnode size

            John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30023/
            Subject: LU-10041 tests: sanity/51 to cleanup properly
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set:
            Commit: 3c4cb654139842d99eb902e1703b965fc0603ced

            gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30023/ Subject: LU-10041 tests: sanity/51 to cleanup properly Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: 3c4cb654139842d99eb902e1703b965fc0603ced

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30023
            Subject: LU-10041 tests: sanity/51 to cleanup properly
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: 14f47be7897e5f57698113990136cde2d846e049

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30023 Subject: LU-10041 tests: sanity/51 to cleanup properly Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 14f47be7897e5f57698113990136cde2d846e049
            mdiep Minh Diep added a comment -

            Landed for 2.11

            mdiep Minh Diep added a comment - Landed for 2.11

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29246/
            Subject: LU-10041 tests: sanity/51 to cleanup properly
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 0d5ceb891e904cc9bab3a6b5ce0944b8bab06975

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29246/ Subject: LU-10041 tests: sanity/51 to cleanup properly Project: fs/lustre-release Branch: master Current Patch Set: Commit: 0d5ceb891e904cc9bab3a6b5ce0944b8bab06975

            John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/29770/
            Subject: LU-10041 osd: osd-zfs to choose dnode size
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set:
            Commit: db0d3c621d1cb75961a8e8e158cda6daae0fdc3d

            gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/29770/ Subject: LU-10041 osd: osd-zfs to choose dnode size Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: db0d3c621d1cb75961a8e8e158cda6daae0fdc3d

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/29770
            Subject: LU-10041 osd: osd-zfs to choose dnode size
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: 1d7ad92b0ca5cba6d94ebffa269c0e63c3f59441

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/29770 Subject: LU-10041 osd: osd-zfs to choose dnode size Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 1d7ad92b0ca5cba6d94ebffa269c0e63c3f59441

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29242/
            Subject: LU-10041 osd: osd-zfs to choose dnode size
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: d993de7b40cd9625b48e5361ae29f77bfceb207a

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29242/ Subject: LU-10041 osd: osd-zfs to choose dnode size Project: fs/lustre-release Branch: master Current Patch Set: Commit: d993de7b40cd9625b48e5361ae29f77bfceb207a

            Andreas, without the patch large dnodes won't be created as we pass DNODE_MIN_SIZE which is 512bytes.

            bzzz Alex Zhuravlev added a comment - Andreas, without the patch large dnodes won't be created as we pass DNODE_MIN_SIZE which is 512bytes.

            Alex, without https://review.whamcloud.com/29242 applied, if the pool is set with dnodesize=auto, will osd-zfs still use a larger dnode, or is the large dnode feature non functional without this patch? Will the unpatched version of osd-zfs use a larger dnodesize if that property is not even set to auto for the filesystem in use?

            I'm trying to decide if this is only a performance enhancement and could wait a bit, or if it could cause problems for users that upgrade to 0.7.x and may need to downgrade if there is a problem and it is urgent to include this into 2.10.1.

            adilger Andreas Dilger added a comment - Alex, without https://review.whamcloud.com/29242 applied, if the pool is set with dnodesize=auto , will osd-zfs still use a larger dnode, or is the large dnode feature non functional without this patch? Will the unpatched version of osd-zfs use a larger dnodesize if that property is not even set to auto for the filesystem in use? I'm trying to decide if this is only a performance enhancement and could wait a bit, or if it could cause problems for users that upgrade to 0.7.x and may need to downgrade if there is a problem and it is urgent to include this into 2.10.1.

            Alex Zhuravlev (alexey.zhuravlev@intel.com) uploaded a new patch: https://review.whamcloud.com/29246
            Subject: LU-10041 tests: sanity/51b to cleanup properly
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 710cdcb22baa31a0ee8903f754b404e4387e48d5

            gerrit Gerrit Updater added a comment - Alex Zhuravlev (alexey.zhuravlev@intel.com) uploaded a new patch: https://review.whamcloud.com/29246 Subject: LU-10041 tests: sanity/51b to cleanup properly Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 710cdcb22baa31a0ee8903f754b404e4387e48d5

            People

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

              Dates

                Created:
                Updated:
                Resolved: