[Bug 796984] vaapih265enc: Support hevc encode low latency B frame.

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=796984

--- Comment #5 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
Hi Fei,

(In reply to Fei from comment #3)
> @Victor, update my commit log. Currently all our existed platforms support P
> and low latency B frame. And I cann't provide correct information if the
> next platform will also support P frame. So I will not mention the hardware
> info in commit log. And both vaapi-driver and media-driver support this.

OK. What about the unrelated code-style changes?

> There is a new interface in libva to provide query:
> https://github.com/intel/libva/pull/220, if the patch merged(need driver to
> support it), we can write another patch to query if the hardware support it
> before do the low latency B. But I think the property is needed, now all our
> existed platforms support both P and low latency B, there should be a
> property to let user choose the right path.

I would wait for the patch to get merged. Otherwise I don't want to follow that
path that would overpopulate the properties for particular architectures. But
if we do, I would prefix them, as intel-*, in this case intel-low-latency-b

In my opinion, we need a third opinion on this case.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.