<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED - VCE dual instance encoding inconsistent since st/va: enable dual instances encode by sync surface"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98005#c33">Comment # 33</a>
              on <a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED - VCE dual instance encoding inconsistent since st/va: enable dual instances encode by sync surface"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98005">bug 98005</a>
              from <span class="vcard"><a class="email" href="mailto:adf.lists@gmail.com" title="Andy Furniss <adf.lists@gmail.com>"> <span class="fn">Andy Furniss</span></a>
</span></b>
        <pre>(In reply to Boyuan Zhang from <a href="show_bug.cgi?id=98005#c32">comment #32</a>)
<span class="quote">> 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.</span >

Turns out I don't need a large clip.

Just running the transcode test above but with vaapih264enc rate-control=cqp
init-qp=30 may give this error.

<span class="quote">> - Issue #2. Random corruption observed when using CQP. Corruption issue is
> gone after disable the statement</span >

Issue 1 and 2 are gone with that disabled.

<span class="quote">> "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</span >

This one was my fault because I forgot to put a ! queue ! in my command and is
nothing to do with the patches/not a real issue.

<span class="quote">> 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.</span >

CQP works perfectly with vanilla mesa, it is broken by patch 1.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>