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: Tue, 29 Nov 2016 20:02:55 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1637141266==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id A7BD26E5B6 for ; Tue, 29 Nov 2016 20:02: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 --===============1637141266== Content-Type: multipart/alternative; boundary="14804497751.aFd37c.7784"; charset="UTF-8" --14804497751.aFd37c.7784 Date: Tue, 29 Nov 2016 20:02: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 #34 from Boyuan Zhang --- Hi Andy, I re-checked the logic, cqp behaviour is affected patch#1. By re-testing cq= p, I confirmed that force to submit two consecutive single job logic should be applied to cqp as well, otherwise we have a chance to miss a flush or end up with sending last p and next i frame together which may protentially cause corruption. Nice catch! Solution is simple, as you already tried, remove the "if statement" to apply new logic to cqp as well. New patches can be found at: https://lists.freedesktop.org/archives/mesa-dev/2016-November/136860.html https://lists.freedesktop.org/archives/mesa-dev/2016-November/136855.html I will push them to upstream once you confirmed the fix. :-) Regards, Boyuan --=20 You are receiving this mail because: You are the assignee for the bug.= --14804497751.aFd37c.7784 Date: Tue, 29 Nov 2016 20:02: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

Commen= t # 34 on bug 98005<= /a> from Boyuan Zhang 
Hi Andy,

I re-checked the logic, cqp behaviour is affected patch#1. By re-testing cq=
p, I
confirmed that force to submit two consecutive single job logic should be
applied to cqp as well, otherwise we have a chance to miss a flush or end up
with sending last p and next i frame together which may protentially cause
corruption. Nice catch!

Solution is simple, as you already tried, remove the "if statement&quo=
t; to apply
new logic to cqp as well.

New patches can be found at:

https://lists.freedesktop.org/archives/mesa-dev/2016-November/13=
6860.html
https://lists.freedesktop.org/archives/mesa-dev/2016-November/13=
6855.html

I will push them to upstream once you confirmed the fix. :-)

Regards,
Boyuan

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