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

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?

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.