Page MenuHomeFreeBSD

ext2fs: Simplify location of entries in an HTree index
Needs ReviewPublic

Authored by jhb on Jan 20 2026, 3:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Aug 11, 6:47 PM
Unknown Object (File)
Sun, Aug 9, 3:56 PM
Unknown Object (File)
Fri, Aug 7, 7:37 PM
Unknown Object (File)
Wed, Aug 5, 5:07 AM
Unknown Object (File)
Sun, Aug 2, 5:56 AM
Unknown Object (File)
Sat, Aug 1, 8:40 AM
Unknown Object (File)
Sat, Jul 25, 5:23 PM
Unknown Object (File)
Sat, Jul 25, 8:58 AM
Subscribers

Details

Reviewers
fsu
jrtc27
Group Reviewers
cheri
Summary

Just require a fixed size for h_info_len to permit using a simpler
assignment.

Obtained from: CheriBSD

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 70024
Build 66907: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Jan 20 2026, 3:59 PM

This probably needs more of an explanation for the issue with this pattern on CHERI than we had downstream

Hmm, I guess I could talk about subobject bounds and add 'Effort: CHERI' tag to it. My reasoning was that aside from CHERI, it is just more readable for upstream.

IMO it's worth putting all of that in the commit message -- that this came from CHERI and why, but also it's just generally better.

sys/fs/ext2fs/ext2_htree.c
289

Hmmm, looking again at the Linux code in question, it doesn't check the length when mounting, but only in get_dx_countlimit() if rlen == 12.