| Summary: | xorg display corruption after suspend/resume with drm-next | ||
|---|---|---|---|
| Product: | xorg | Reporter: | Pierre Ossman <pierre-bugzilla> |
| Component: | Driver/Radeon | Assignee: | xf86-video-ati maintainers <xorg-driver-ati> |
| Status: | RESOLVED FIXED | QA Contact: | Xorg Project Team <xorg-team> |
| Severity: | normal | ||
| Priority: | medium | ||
| Version: | git | ||
| Hardware: | x86-64 (AMD64) | ||
| OS: | Linux (All) | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
|
Description
Pierre Ossman
2010-07-23 05:18:08 UTC
I did some more testing and the bug exists in plain 2.6.35-rc5 (on which drm-next is currently based), but not in rc4. I'll see if I can find something more precise... Sorry, disregard that last comment. drm-next is based on -rc4, not -rc5. And it seems I tested the wrong kernel. Actual results: 2.6.35-rc4: works fine drm-next: breaks on resume Result from some bisecting: 5cffee1be7a2b62385fe1e80deecf491042a6f06 is the first bad commit commit 5cffee1be7a2b62385fe1e80deecf491042a6f06 Author: Alex Deucher <alexdeucher@gmail.com> Date: Tue Jun 22 01:54:48 2010 -0400 drm/radeon: optimize default 3D state for r6xx/r7xx blits Re-organize emit order to reduce the number of dwords in the default 3D state. This change reduces the default state size from 514 dwords to 282. Signed-off-by: Alex Deucher <alexdeucher@gmail.com> Signed-off-by: Dave Airlie <airlied@redhat.com> Can you revert that commit and try this set of patches instead? http://people.freedesktop.org/~agd5f/reduce_emit/ And see which if any of those cause a problem? Patch 0002 breaks things. And patch 0003 also makes the suspend hang somewhere (same behaviour as RH bug 617561). Applying all 6 patches makes the hang go away, but the corruption remain. Rafał tracked it down to the missing sampler emit. New patch set: http://people.freedesktop.org/~agd5f/reduce_emit/ New patch set seems to work fine. No hangs and no corruption from what I can see. Final version: http://people.freedesktop.org/~agd5f/reduce_emit/ (Thumb dislodged from backside) Latest set of patches work fine as well. Already upstream for 2.6.36. |
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.