[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