From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 98005] VCE dual instance encoding inconsistent since st/va: enable dual instances encode by sync surface Date: Mon, 17 Oct 2016 23:59:55 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0824961370==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 200886E34F for ; Mon, 17 Oct 2016 23:59:55 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0824961370== Content-Type: multipart/alternative; boundary="14767487950.fB55B00a.29314"; charset="UTF-8" --14767487950.fB55B00a.29314 Date: Mon, 17 Oct 2016 23:59:55 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated https://bugs.freedesktop.org/show_bug.cgi?id=3D98005 --- Comment #3 from Andy Furniss --- (In reply to Boyuan Zhang from comment #2) > Hi Andy, >=20 > The corruption/out of order issue has been reproduced locally. Root cause > and fix can be found at > https://lists.freedesktop.org/archives/mesa-dev/2016-October/132268.html >=20 > Please give a try. >=20 > 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 le= ast it's consistent (md5sums do still vary). Will test more tomorrow as it's late here. --=20 You are receiving this mail because: You are the assignee for the bug.= --14767487950.fB55B00a.29314 Date: Mon, 17 Oct 2016 23:59:55 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated

Comment= # 3 on bug 98005<= /a> from Andy Furniss 
(In reply to Boyuan Zhang from comment #2)
> Hi Andy,
>=20
> The corruption/out of order issue has been reproduced locally. Root ca=
use
> and fix can be found at
> https://lists.freedesktop.org/archives/mesa-dev/2016-October=
/132268.html
>=20
> Please give a try.
>=20
> 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 le=
ast
it's consistent (md5sums do still vary).

Will test more tomorrow as it's late here.

You are receiving this mail because:
  • You are the assignee for the bug.
= --14767487950.fB55B00a.29314-- --===============0824961370== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============0824961370==--