ffmpeg-v4l2-request-fourier: route H.264 chroma intra deblock (4:2:0) through daedalus-fourier (0013) #102

Merged
marfrit merged 1 commits from claude-noether/marfrit-packages:noether/h264-substitute-deblock-chroma-intra into main 2026-05-25 12:48:20 +00:00
Owner

Substitutes c->v_loop_filter_chroma_intra and c->h_loop_filter_chroma_intra with daedalus wrappers in the 4:2:0 / bit_depth=8 branch. 4:2:2 stays on in-tree NEON (daedalus dispatch is 4:2:0-only).

Fourier dispatches were exposed in PR #11. Same NEON-to-NEON via recipe shape as 0010 luma intra.

Re-architects the chroma init: v_loop_filter_chroma_intra is now assigned inside both branches of the chroma_format_idc conditional — 4:2:0 picks daedalus, 4:2:2 keeps NEON. No regression for 4:2:2 streams.

Closes the deblock substitution layer for the 4:2:0 / 8-bit hot path. All 8 deblock variants for the common 4:2:0 case now route through daedalus: luma+chroma × V+H × inter+intra.

Substitutes c->v_loop_filter_chroma_intra and c->h_loop_filter_chroma_intra with daedalus wrappers in the 4:2:0 / bit_depth=8 branch. 4:2:2 stays on in-tree NEON (daedalus dispatch is 4:2:0-only). Fourier dispatches were exposed in PR #11. Same NEON-to-NEON via recipe shape as 0010 luma intra. Re-architects the chroma init: v_loop_filter_chroma_intra is now assigned inside both branches of the chroma_format_idc conditional — 4:2:0 picks daedalus, 4:2:2 keeps NEON. No regression for 4:2:2 streams. **Closes the deblock substitution layer for the 4:2:0 / 8-bit hot path.** All 8 deblock variants for the common 4:2:0 case now route through daedalus: luma+chroma × V+H × inter+intra.
marfrit added 1 commit 2026-05-25 12:22:18 +00:00
Substitutes c->v_loop_filter_chroma_intra and c->h_loop_filter_chroma_intra
with daedalus wrappers in the bit_depth=8 / chroma_format_idc<=1 (4:2:0)
branch.  4:2:2 stays on the in-tree NEON path (the daedalus chroma intra
dispatch is 4:2:0-only).

The fourier dispatches were exposed in PR #11 (DEFINE_INTRA_DISPATCH
macro generates the public daedalus_dispatch_h264_deblock_chroma_*_intra
symbols + recipe wrappers).

Re-architects the chroma init: v_loop_filter_chroma_intra was previously
assigned unconditionally to the NEON variant (which works for both 4:2:0
and 4:2:2).  We now assign it INSIDE both branches of the chroma_format_idc
conditional — 4:2:0 picks daedalus, 4:2:2 keeps NEON.  No regression for
4:2:2 streams.

Same NEON-to-NEON via recipe shape as 0010 luma intra.

Closes the deblock substitution layer for the 4:2:0 / 8-bit hot path:
- 0005 luma_v non-intra ✓
- 0008 luma_h non-intra ✓
- 0009 chroma_v / chroma_h non-intra ✓
- 0010 luma_v / luma_h intra ✓
- 0013 chroma_v / chroma_h intra ✓

All 8 deblock variants for the common 4:2:0 path now route through
daedalus.  4:2:2 chroma + the chroma422 mbaff variants stay on in-tree
NEON.

Verified the patch applies cleanly on top of 0001-0012 against the
pinned upstream commit b57fbbe5 on hertz.
marfrit merged commit 875156782e into main 2026-05-25 12:48:20 +00:00
marfrit deleted branch noether/h264-substitute-deblock-chroma-intra 2026-05-25 12:48:21 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: marfrit/marfrit-packages#102