Bug 57191 - [i915] garbage in console on the second monitor upon resuming from hibernation
Summary: [i915] garbage in console on the second monitor upon resuming from hibernation
Status: RESOLVED PATCH_ALREADY_AVAILABLE
Alias: None
Product: Drivers
Classification: Unclassified
Component: Video(DRI - Intel) (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: Chris Wilson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-27 13:45 UTC by Nikolay Amiantov
Modified: 2013-05-07 20:24 UTC (History)
3 users (show)

See Also:
Kernel Version: 3.9-rc8-24-gd7d7271
Subsystem:
Regression: Yes
Bisected commit-id:


Attachments
lspci -vv (from the older kernel) (11.40 KB, text/plain)
2013-04-27 13:45 UTC, Nikolay Amiantov
Details
drm/i915: clear the stolen fb before resuming (1.70 KB, patch)
2013-05-06 12:12 UTC, Jani Nikula
Details | Diff

Description Nikolay Amiantov 2013-04-27 13:45:51 UTC
Created attachment 100071 [details]
lspci -vv (from the older kernel)

I have Acer 4820TG 14" laptop and 22" external monitor (connected by VGA). Console is rendered with resolution of the smaller monitor, and on the second one it is displayed in the top-left corner with unused bottom and right space on the monitor (is it a bug, too?). These borders are used to be black (on 3.8.8, for example), but on recent git snapshot it is filled with random colored garbage after resuming from hibernation. I can try to bisect this issue later.
Comment 1 Daniel Vetter 2013-04-27 14:41:49 UTC
Looks like we need

commit d8241785c24d4508ecc26fb91b28b39a0dd26070
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Sat Apr 27 12:44:16 2013 +0100

    drm/i915: Only print the info message about incresing stolen size for FBC once

for the resume path, too.
Comment 2 Daniel Vetter 2013-04-27 14:42:21 UTC
Blergh, copy&pasta of the wrong commit. I've meant this one here:

commit 88afe715dd5469bc24ca7a19ac62dd3c241cab48
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Sun Dec 16 12:15:41 2012 +0000

    drm/i915: Clear the stolen fb before enabling
Comment 3 Jani Nikula 2013-05-06 12:12:22 UTC
Created attachment 100801 [details]
drm/i915: clear the stolen fb before resuming

This patch is a cargo culted version for the resume path. Please give it a try.
Comment 4 Nikolay Amiantov 2013-05-06 17:08:57 UTC
Confirmed, this patch works for me --- no more garbage.
Comment 5 Daniel Vetter 2013-05-07 20:24:05 UTC
Patch merged to drm-intel-fixes with cc: stable:

commit 1ffc5289bfcf7f4c4e4213240bb4be68c48ce603
Author: Jani Nikula <jani.nikula@intel.com>
Date:   Tue May 7 18:54:05 2013 +0300

    drm/i915: clear the stolen fb before resuming

Note You need to log in before you can comment on or make changes to this bug.