Page MenuHomeFreeBSD

zfs: we can boot from dataset with large_dnode enabled
ClosedPublic

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

Details

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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

tsoome created this revision.Fri, Nov 30, 10:48 AM
avg added a comment.Fri, Nov 30, 11:29 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.Mon, Dec 3, 1:17 PM
This revision is now accepted and ready to land.Mon, Dec 3, 1:17 PM
This revision was automatically updated to reflect the committed changes.