Bug 44324 - [Intel HD3000 Graphics XVideo] XVideo playback does not work after commit 706d3a97b
Summary: [Intel HD3000 Graphics XVideo] XVideo playback does not work after commit 70...
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/intel (show other bugs)
Version: git
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Chris Wilson
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-30 10:38 UTC by Alexey Shumitsky
Modified: 2011-12-31 09:23 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
dmesg output after trying to play video. (216.77 KB, application/octet-stream)
2011-12-30 10:38 UTC, Alexey Shumitsky
no flags Details

Description Alexey Shumitsky 2011-12-30 10:38:25 UTC
Created attachment 54981 [details]
dmesg output after trying to play video.

On git master video playback is broken. Bisect found this commit to be the cause:

Author: Chris Wilson <chris@chris-wilson.co.uk>  2011-12-08 18:55:20
Committer: Chris Wilson <chris@chris-wilson.co.uk>  2011-12-09 02:33:12
Parent: 706d3a97bd3049e60c08d121a3d4d81c03029a87 (sna/trapezoids: Fix detection of rectilinearity after projection)
Branches: master, remotes/origin/master
Follows: 2.17.0
Precedes: 

    uxa/video: Use the common bo allocations and upload
    
    In order to avoid inconsistent usage of coherency domains and to avoid
    completely unnecessary clflushing during video playback, use the same
    buffer allocation and upload functions as the rest of the driver.
    
    Reported-by: Christophe Roland <roll68@gmail.com>
    Bugzilla: http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=60;bug=651316
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

When trying to play video, after this commit video does not shows up and player video field remains black (player is working as audio can be heard). When driver is updated to git master tip, X server hangs when I try to play video (applications continue working, audio is working etc) so machine needs to be rebooted.
Comment 1 Chris Wilson 2011-12-31 09:23:03 UTC
commit 770a953ff03bb8328c3f29e274d225528840f30c
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Sat Dec 31 17:19:08 2011 +0000

    uxa/video: Clear all state structures before uploading
    
    A regression from eb859f644633e left some of the state uninitialised
    before uploading to the GPU leading to undefined behaviour.
    
    Reported-by: Alexey Shumitsky <alexey.shumitsky@gmail.com>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=44338
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=44252
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.