<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <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#c3">Comment # 3</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 <a class="email" href="mailto:adf.lists@gmail.com" title="Andy Furniss <adf.lists@gmail.com>"> Andy Furniss</a> <pre>(In reply to Boyuan Zhang from <a href="show_bug.cgi?id=98005#c2">comment #2</a>) > Hi Andy, > > The corruption/out of order issue has been reproduced locally. Root cause > and fix can be found at > <a href="https://lists.freedesktop.org/archives/mesa-dev/2016-October/132268.html">https://lists.freedesktop.org/archives/mesa-dev/2016-October/132268.html</a> > > Please give a try. > > Regards, > Boyuan Thanks, that does fix the out of order and corruption. As you explained in the thread above, dual instance bitrate control is a separate issue, but this patch does change behavior a bit. cbr is unaffected by patch and could be seen as OK (unless I find a better test to break it). md5sums may vary but the bitrate/file size is only different by a tiny amount and visually things seem OK. vbr is different, before patch it may be OK, but there was a chance of some files being less that half size/rate than they should. With the patch it seems that they are all now less than half rate. So at least it's consistent (md5sums do still vary). Will test more tomorrow as it's late here.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>