ext2fs updates from ffs
The attached patch is meant for FreeBSD but the affected code doesn't
seem to have changed in DragonFly (other that relocation that is). It
may require some tweaking to apply correctly though.
It basically adopts some changes that we made with the Lite2 commit and
a afterwards in ffs. A one liner fix from Bruce Evans is also included.
This should also help future adoption of the BSD-licensed ext2fs port
that has been progressing as part of FreeBSD's Google Summer of Code.
#2 Updated by dillon over 7 years ago
:Here is an updated patch.
Thanks Pedro. I will apply the brelse fixes, the i_next_alloc_block
fix, and a few others to DragonFly. We will have to forego the
application of the ASYNC enhancements and most of the syntax changes
(though I'll definitely keep the 'error =' syntax adjustments),
mainly because we are too close to the release to test them and I
don't want to create any accidental bugs. ext2fs support is really
just there for accessibility, we don't expect people to use ext2fs for
a main filesystem in DragonFly.
#3 Updated by pfgshield-freebsd over 7 years ago
Matthew Dillon wrote:
> :Hi again;
> :Here is an updated patch.
> Thanks Pedro. I will apply the brelse fixes, the i_next_alloc_block
> fix, and a few others to DragonFly. We will have to forego the
> application of the ASYNC enhancements and most of the syntax changes
> (though I'll definitely keep the 'error =' syntax adjustments),
> mainly because we are too close to the release to test them and I
> don't want to create any accidental bugs. ext2fs support is really
> just there for accessibility, we don't expect people to use ext2fs for
> a main filesystem in DragonFly.
The ASYNC enhancements are actually harmless but I agree with the idea
of minimizing unnecessary risks before a release.
Ext2fs is rather uninteresting for default use but with Aditya's port of
NetBSD's allocation code, the new BSD-licensed ext2fs is a copy cat of
UFS1. This means we could start using some enhancements from FFS
(softupdates sounds crazy but is doable) and we could check some of
linux's developments (preallocation, Htree) to see if they make sense in
Not as fun as HAMMER, but nevertheless fun ;-).
#5 Updated by pfgshield-freebsd about 7 years ago
Alex Hornung (via DragonFly issue tracker) wrote:
> Alex Hornung <email@example.com> added the comment:
> Has this made it in yet? If not, do we want it in for after the release?
> Alex Hornung
Yes, Some parts were left out (async enhancements) but Matt Dillon
committed the critical parts.
BTW, the locking would require redoing but it would be nice to see the
BSD licensed ext2fs from FreeBSD merged someday.