[LU-10041] osd-zfs to choose dnode size Created: 28/Sep/17 Updated: 15/Dec/18 Resolved: 09/Nov/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.10.1, Lustre 2.11.0 |
| Fix Version/s: | Lustre 2.11.0, Lustre 2.10.2 |
| Type: | Improvement | Priority: | Major |
| Reporter: | Alex Zhuravlev | Assignee: | Alex Zhuravlev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | zfs | ||
| Issue Links: |
|
||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
osd-zfs should use dnode size depending on zfs property |
| Comments |
| Comment by Gerrit Updater [ 28/Sep/17 ] |
|
Alex Zhuravlev (alexey.zhuravlev@intel.com) uploaded a new patch: https://review.whamcloud.com/29242 |
| Comment by Gerrit Updater [ 28/Sep/17 ] |
|
Alex Zhuravlev (alexey.zhuravlev@intel.com) uploaded a new patch: https://review.whamcloud.com/29246 |
| Comment by Andreas Dilger [ 29/Sep/17 ] |
|
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. |
| Comment by Alex Zhuravlev [ 29/Sep/17 ] |
|
Andreas, without the patch large dnodes won't be created as we pass DNODE_MIN_SIZE which is 512bytes. |
| Comment by Gerrit Updater [ 25/Oct/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29242/ |
| Comment by Gerrit Updater [ 25/Oct/17 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/29770 |
| Comment by Gerrit Updater [ 26/Oct/17 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/29770/ |
| Comment by Gerrit Updater [ 09/Nov/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29246/ |
| Comment by Minh Diep [ 09/Nov/17 ] |
|
Landed for 2.11 |
| Comment by Gerrit Updater [ 09/Nov/17 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30023 |
| Comment by Gerrit Updater [ 17/Nov/17 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30023/ |