[Bug 98005] VCE dual instance encoding inconsistent since st/va: enable dual instances encode by sync surface

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=98005

--- Comment #32 from Boyuan Zhang <boyuan.zhang at amd.com> ---
Hi Andy,

To summarise your test results, basically you saw the following 3 issues with
CQP, is it correct?

- Issue #1. For large clip, sometime you can "invalid GstVaapiCodedBuffer size
(0 bytes)" error.
- Issue #2. Random corruption observed when using CQP. Corruption issue is gone
after disable the statement 
"if (context->desc.h264enc.rate_ctrl.rate_ctrl_method
!=PIPE_H264_ENC_RATE_CONTROL_METHOD_DISABLE"
- Issue #3. Encoding speed is twice faster when setting QP<=28

So far I'm lucky enough that my test haven't trigger any of the issue yet. I
will spend more time to try from my side. Again, if you could share the
commands you were using to trigger the above 3 issue, that would be very
appreciated.

In the meantime, I would like to push the 2 patches to upstream first. Because
this two patches don't change the behaviour of CQP, but only fixing VBR/CBR.
Therefore, I believe CQP fix should be apart from these 2 patches.

Regards,
Boyuan

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161128/37665e1d/attachment-0001.html>