* [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert
2016-01-18 16:10 [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert bugzilla-daemon
@ 2016-01-18 16:10 ` bugzilla-daemon
2016-01-18 19:57 ` bugzilla-daemon
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: bugzilla-daemon @ 2016-01-18 16:10 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 291 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=93758
--- Comment #1 from Andy Furniss <adf.lists@gmail.com> ---
Created attachment 121111
--> https://bugs.freedesktop.org/attachment.cgi?id=121111&action=edit
assert
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1197 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert
2016-01-18 16:10 [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert bugzilla-daemon
2016-01-18 16:10 ` bugzilla-daemon
@ 2016-01-18 19:57 ` bugzilla-daemon
2016-01-18 20:48 ` bugzilla-daemon
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: bugzilla-daemon @ 2016-01-18 19:57 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 367 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=93758
--- Comment #2 from Christian König <deathsimple@vodafone.de> ---
Created attachment 121120
--> https://bugs.freedesktop.org/attachment.cgi?id=121120&action=edit
Possible fix
Hi Andy,
stupid typo, does the attached patch fix the issue?
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1377 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert
2016-01-18 16:10 [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert bugzilla-daemon
2016-01-18 16:10 ` bugzilla-daemon
2016-01-18 19:57 ` bugzilla-daemon
@ 2016-01-18 20:48 ` bugzilla-daemon
2016-01-20 22:02 ` bugzilla-daemon
2016-01-21 8:58 ` bugzilla-daemon
4 siblings, 0 replies; 6+ messages in thread
From: bugzilla-daemon @ 2016-01-18 20:48 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 541 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=93758
--- Comment #3 from Andy Furniss <adf.lists@gmail.com> ---
(In reply to Christian König from comment #2)
> Created attachment 121120 [details] [review]
> Possible fix
>
> Hi Andy,
>
> stupid typo, does the attached patch fix the issue?
Yes that fixes it thanks.
I can see there are more artifacts than with vdpau m/a deint - I was expecting
them to use the same underlying code and so have the same artifacts?
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1691 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert
2016-01-18 16:10 [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert bugzilla-daemon
` (2 preceding siblings ...)
2016-01-18 20:48 ` bugzilla-daemon
@ 2016-01-20 22:02 ` bugzilla-daemon
2016-01-21 8:58 ` bugzilla-daemon
4 siblings, 0 replies; 6+ messages in thread
From: bugzilla-daemon @ 2016-01-20 22:02 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 746 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=93758
--- Comment #4 from Andy Furniss <adf.lists@gmail.com> ---
(In reply to Andy Furniss from comment #3)
> I can see there are more artifacts than with vdpau m/a deint - I was
> expecting them to use the same underlying code and so have the same
> artifacts?
After looking at more samples it seems that sometimes parts of one of the
fields ends up with old content being displayed/not updated.
This is more than a few extra artifacts - it's buggy. With vdpau there were
artifacts, but you wouldn't notice them watching normally (would need to
pause/slow mo). Given the right input this is instantly noticeably corrupt.
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1623 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert
2016-01-18 16:10 [Bug 93758] mpv + st/va: add motion adaptive deinterlacing v2 = segfault/assert bugzilla-daemon
` (3 preceding siblings ...)
2016-01-20 22:02 ` bugzilla-daemon
@ 2016-01-21 8:58 ` bugzilla-daemon
4 siblings, 0 replies; 6+ messages in thread
From: bugzilla-daemon @ 2016-01-21 8:58 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 1452 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=93758
Christian König <deathsimple@vodafone.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #5 from Christian König <deathsimple@vodafone.de> ---
(In reply to Andy Furniss from comment #4)
> (In reply to Andy Furniss from comment #3)
>
> > I can see there are more artifacts than with vdpau m/a deint - I was
> > expecting them to use the same underlying code and so have the same
> > artifacts?
>
> After looking at more samples it seems that sometimes parts of one of the
> fields ends up with old content being displayed/not updated.
>
> This is more than a few extra artifacts - it's buggy. With vdpau there were
> artifacts, but you wouldn't notice them watching normally (would need to
> pause/slow mo). Given the right input this is instantly noticeably corrupt.
Really interesting. I can't see of hand how this could happen.
The only possibility is that the input bitstream somehow gets corrupted
sometimes and because of that UVD fails to decode the field in question. But
that would affect normal playback as well.
Anyway, please open up a new bug report for this. Clearly a separate issue.
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 3134 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread