RFC: (partially) endian-annotate e2fsprogs
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest6 patches 4 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/6 | e2fsprogs: define bitwise types and annotate conversion routinese2fsck/ lib/ lib/ +2 more | +56 | 0 replies | |
| 2/6 | e2fsprogs: fix endian handling of ext3_extent_headerlib/ lib/ | +11 | 1 reply | |
| 3/6 | e2fsprogs: Endian-annotate most on-disk structureslib/ lib/ lib/ +3 more | +48 | 0 replies | |
| 4/6 | debugfs: don't swap htree nodes in-placedebugfs/ | +18 | 0 replies | |
| 5/6 | libext2: minor sparse endian checker fixuplib/ | +4 | 0 replies | |
| 6/6 | quotaio: annotate & fix up for sparse endian checkerlib/ lib/ | +9 | 0 replies |
Reviews trailers per patch
1 of 6 patches has a Reviewed-by or Acked-by.
| Patch | Darrick J. Wong |
|---|---|
| 1/6 | |
| 2/6 | R-b |
| 3/6 | |
| 4/6 | |
| 5/6 | |
| 6/6 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/6] RFC: (partially) endian-annotate e2fsprogs
This is really only partial, and in the end didn't spot any actual problems. And things are a bit odd and tricky, because some structures (superblocks, inodes, etc) are swapped in-place in the same structure (so they can't be easily annotated - if we wish to, we should define separate on-disk and in-memory structures).
Further, i_block in the inode is sometimes swapped on read, and sometimes not (!), depending on whether it's indirect blocks, extents, or inline data. So that's still messy too.
So this is really just kind of an RFC; I did it on a whim, and things aren't yet totally sparse-check clean, but figured I'd send it out and see what people think, whether it's worth merging, or working on cleaning up the above issues to make it all tidier.
(sparse is pretty good at looking for casts in and out of blk64_t too, though I haven't looked much at those.)
Thanks,
-Eric