| Summary: | Booting with vga=0x0F04 results in garbage on the screen when drm driver starts | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | DRI | Reporter: | Rick Bramley <richard.bramley> | ||||||||
| Component: | DRM/Intel | Assignee: | Chris Wilson <chris> | ||||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||||
| Severity: | normal | ||||||||||
| Priority: | medium | CC: | jbarnes | ||||||||
| Version: | unspecified | ||||||||||
| Hardware: | Other | ||||||||||
| OS: | All | ||||||||||
| Whiteboard: | |||||||||||
| i915 platform: | i915 features: | ||||||||||
| Attachments: |
|
||||||||||
|
Description
Rick Bramley
2011-03-24 12:22:10 UTC
I think I can guess what the garbage is. Can you take a photograph of it? And it does clear once overdrawn? Created attachment 44993 [details]
Blurred Screen video
Attached is a video showing the corruption (I suspect it's a distorted splash screen). Ok, I think the patches on the tip of drm-intel-staging should clear that up. In particular: commit f8acdf5aa142926961e1f7ddb9e86490c50f8e6a Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Tue Mar 29 10:40:27 2011 +0100 drm/i915: Disable all outputs early, before KMS takeover If the outputs are active and continuing to access the GATT when we teardown the PTEs, then there is a potential for us to hang the GPU. The hang tends to be a PGTBL_ER with either an invalid host access or an invalid display plane fetch. Reported-by: Pekka Enberg <penberg@kernel.org> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> which requires commit ea1167d6601f370f5d7e425eb0b3c7577edd02cd Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Tue Mar 29 13:19:09 2011 +0100 drm/i915: Move the irq wait queue initialisation into the ring init Required so that we don't obliterate the queue if initialising the rings after the global IRQ handler is installed. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Created attachment 44996 [details] [review] Move the irq queue init onto the rings Created attachment 44997 [details] [review] Disable outputs first Thanks, we'll give it a try and let you know. Thanks guys, this fixed the issue. Lets not close this until I have the patches upstream (even if you are happy to patch your systems for the time being). What would be really, really valuable for me is your tested-by. Do I have you permission to use your results from testing? If you mean saying that we tested it and it worked, sure that's fine. Thank you, I've added your tested-by to the patch and with Ben's review I think I have enough for it to go upstream. commit b259f6730c09bc356df932ba9188f291de816808 Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Tue Mar 29 13:19:09 2011 +0100 drm/i915: Move the irq wait queue initialisation into the ring init and commit 2c7111dbaec72b01c804afb8ad77c6c7523986fd Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Tue Mar 29 10:40:27 2011 +0100 drm/i915: Disable all outputs early, before KMS takeover have landed, yay. |
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.