From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 84140] mplayer crashes playing some files using vdpau output Date: Tue, 07 Oct 2014 14:14:57 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0419030288==" Return-path: Received: from culpepper.freedesktop.org (unknown [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 964966E020 for ; Tue, 7 Oct 2014 07:14:57 -0700 (PDT) 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 --===============0419030288== Content-Type: multipart/alternative; boundary="1412691297.ff2c3a61.29751"; charset="UTF-8" --1412691297.ff2c3a61.29751 Date: Tue, 7 Oct 2014 14:14:57 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable https://bugs.freedesktop.org/show_bug.cgi?id=3D84140 Christian K=C3=B6nig changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |deathsimple@vodafone.de --- Comment #24 from Christian K=C3=B6nig --- Created attachment 107497 --> https://bugs.freedesktop.org/attachment.cgi?id=3D107497&action=3Dedit Possible fix (In reply to Marek Ol=C5=A1=C3=A1k from comment #21) > The context should be fully initialized to do anything no matter what the > user API is, is that right, Christian? Correct, yes. The context is the same we do CC conversion with and there it works perfectly fine. We also hold a lock to make sure that the context isn= 't used by more than one theread at the same time. > The auxiliary context is for cases when you don't have any context around > and need to submit some commands. It saves the context_create/destroy > overhead but not much else. If the context created by VDPAU isn't > initialized properly, the aux context can be used (if it works). Well, using the auxiliary context migth work around the problem but won't really solve it. Anyway a patch going this approach is attached, please tes= t. --=20 You are receiving this mail because: You are the assignee for the bug. --1412691297.ff2c3a61.29751 Date: Tue, 7 Oct 2014 14:14:57 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Christian K=C3=B6nig changed bug 84140<= /a>
What Removed Added
CC   deathsimple@vodafone.de

Commen= t # 24 on bug 84140<= /a> from Christian K=C3=B6nig
Created attachment 107497 =
[details] [review]
Possible fix

(In reply to Marek Ol=C5=A1=C3=A1k from comment #21)
> The context should be fully initialized to do an=
ything no matter what the
> user API is, is that right, Christian?

Correct, yes. The context is the same we do CC conversion with and there it
works perfectly fine. We also hold a lock to make sure that the context isn=
't
used by more than one theread at the same time.

> The auxiliary context is for cases when you don'=
t have any context around
> and need to submit some commands. It saves the context_create/destroy
> overhead but not much else. If the context created by VDPAU isn't
> initialized properly, the aux context can be used (if it works).

Well, using the auxiliary context migth work around the problem but won't
really solve it. Anyway a patch going this approach is attached, please tes=
t.


You are receiving this mail because: =20=20=20=20=20=20
  • You are the assignee for the bug.
--1412691297.ff2c3a61.29751-- --===============0419030288== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel --===============0419030288==--