moonlight-stream/moonlight-qt
1
0
Fork 0
mirror of https://github.com/moonlight-stream/moonlight-qt.git synced 2026-02-16 02:30:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Handle missing bitstream restrictions element in H.264 SPS fixup

Browse Source 
NVENC on the 591.59 driver seems to not include the bitstream
restrictions element, which renders our attempt to set the
max_dec_frame_buffering value ineffective. Handle this by
providing inserting default bitstream restrictions if one
is not present.
This commit is contained in:
Cameron Gutman
2025-12-30 17:12:37 -06:00
parent b6d10a34c8
commit b108684edc
1 changed files with 15 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
app/streaming/video/ffmpeg.cpp
View File

@@ -1751,9 +1751,24 @@ void FFmpegVideoDecoder::writeBuffer(PLENTRY entry, int& offset)
SDL_assert(nalEnd == entry->length);
// Fixup the SPS to what OS X needs to use hardware acceleration
// This is also critical for decoding latency on the Pi 2.
stream->sps->num_ref_frames = 1;
stream->sps->vui.max_dec_frame_buffering = 1;
// NVENC doesn't seem to add bitstream restrictions anymore (591.59),
// so we need to add them ourselves if not present to ensure that
// the max_dec_frame_buffering option actually takes effect.
// We use the defaults for everything except max_dec_frame_buffering.
if (!stream->sps->vui.bitstream_restriction_flag) {
stream->sps->vui.bitstream_restriction_flag = 1;
stream->sps->vui.motion_vectors_over_pic_boundaries_flag = 1;
stream->sps->vui.max_bytes_per_pic_denom = 2;
stream->sps->vui.max_bits_per_mb_denom = 1;
stream->sps->vui.log2_max_mv_length_horizontal = 16;
stream->sps->vui.log2_max_mv_length_vertical = 16;
stream->sps->vui.num_reorder_frames = 0;
}
int initialOffset = offset;
// Copy the modified NALU data. This clobbers byte 0 and starts NALU data at byte 1.