zfs: we can boot from dataset with large_dnode enabled
ClosedPublic
Actions

Authored by tsoome on Nov 30 2018, 10:48 AM.

Details

Reviewers

avg
allanjude

Commits

rS341438: zfs: we can boot from dataset with large_dnode enabled

Summary

loader has been supporting large_dnode for some time, no need to block the
feature for boot dataset.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tsoome created this revision.Nov 30 2018, 10:48 AM

Herald added subscribers: delphij, imp. · View Herald TranscriptNov 30 2018, 10:48 AM

Harbormaster completed remote builds in B21279: Diff 51429.Nov 30 2018, 10:51 AM

Maybe rather than removing the code it would be better to ifdef it out?
But not sure. What are plans for illumos in this respect?

In D18391#391262, @avg wrote:

Maybe rather than removing the code it would be better to ifdef it out?
But not sure. What are plans for illumos in this respect?

The loader part is ready and waiting, the large_dnode is in process of double checking the commits from zol - once it is ready, I do plan to push it and then there is also no need for such limit.

avg accepted this revision.Dec 3 2018, 1:17 PM

This revision is now accepted and ready to land.Dec 3 2018, 1:17 PM

Closed by commit rS341438: zfs: we can boot from dataset with large_dnode enabled (authored by tsoome). · Explain WhyDec 3 2018, 7:36 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

cddl/

contrib/

opensolaris/

uts/

common/

fs/

zfs/

spa.c

10 lines

zfs_ioctl.c

10 lines

Diff 51545

View Options

head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/spa.c