[Libva] [PATCH 3/3] VP9 encode: support fractional framerate
Mark Thompson
sw at jkqxz.net
Thu Dec 8 22:54:53 UTC 2016
On 08/12/16 01:13, Zhao Yakui wrote:
> On 12/07/2016 01:48 PM, Xiang, Haihao wrote:
>> Now we have the same way for frame rate for each codec in i965_encoder.c, it would be better to use the result directly
>
> Hi, Mark
>
> Can you refresh your patch based on Haihao's suggestion?
> Now the frame rate is parsed in the function of intel_encoder_check_framerate_parameter. (It is defined in i965_encoder.c) and it is stored as framerate_per_100s.
> So the vp9 can use the framerate_per_100s for the frame_rate_m. The frame_rate_d can be 100.
Right, I went a bit further than that. The following removes all use of any of the RC parameter buffers (RC, HRD, framerate) from the specific encoders, moving top-level RC code into i965_encoder.c.
Tested with VP8, VP9, H.264 and H.265 on Kaby Lake. I haven't tested the low-power H.264 encoder (gen9_vdenc.c) because it is only enabled for Skylake using CQP at the moment, though there are some edits there.
Thanks,
- Mark
More information about the Libva
mailing list