All of lore.kernel.org
 help / color / mirror / Atom feed
From: Keith Packard <keithp@keithp.com>
To: Hugh Dickins <hughd@google.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Al Viro <viro@zeniv.linux.org.uk>,
	DRI <dri-devel@lists.freedesktop.org>
Subject: Re: i915 modeset memory corruption issues? (Fwd: Oops in ext3_block_to_path.isra.40+0x26/0x11b)
Date: Sat, 17 Mar 2012 20:42:09 -0700	[thread overview]
Message-ID: <863996d17y.fsf@sumi.keithp.com> (raw)
In-Reply-To: <alpine.LSU.2.00.1203171817460.1352@eggly.anvils>

<#part sign=pgpmime>
On Sat, 17 Mar 2012 18:44:18 -0700 (PDT), Hugh Dickins <hughd@google.com> wrote:

> __GFP_MOVABLE is a hint to page allocation that there's a good likelihood
> that this logical page can be migrated elsewhere in physical memory later
> on if mm wants, so it's a good idea to allocate it from a physical area of
> similarly MOVABLE pages;

So, allocating things with __GFP_MOVABLE may just change where in memory
the graphics pages get allocated, moving whatever GPU-inspired damage to
less sensitive bits of the kernel data. Sounds fabulous!

> I keep worrying about the sequence when the machine is powered on again
> after hibernation: can i915 get up to anything before it is resumed from
> the hibernation image?

Well, the frame buffer is presumably still using whatever mapping it had
before suspend occurred; is there any way it could be writing through
that before the graphics driver was resumed?

What I don't understand is the relationship between the boot kernel and
the resumed kernel; when does the boot kernel stop writing to the
console, and how does it hand off control of the frame buffer at that
time.

It would be great if we could separate out the boot kernel access to the
graphics system from the resumed system -- if the boot kernel was run
without the i915 driver loaded at all, and just used VGA text mode, then
any damage as a result of resume wouldn't be caused by the boot kernel
GTT mappings getting used at the wrong time.

-- 
keith.packard@intel.com

  reply	other threads:[~2012-03-18  3:42 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-16 16:11 i915 modeset memory corruption issues? (Fwd: Oops in ext3_block_to_path.isra.40+0x26/0x11b) Linus Torvalds
2012-03-16 16:47 ` Dave Airlie
2012-03-16 23:01   ` Keith Packard
2012-03-17  7:41     ` Dave Airlie
2012-03-17 18:59       ` Keith Packard
2012-03-17 19:20         ` Dave Airlie
2012-03-17 20:23           ` Dave Airlie
2012-03-17 20:27             ` Dave Airlie
2012-03-17 21:14             ` Linus Torvalds
2012-03-17 22:09               ` Hugh Dickins
2012-03-17 22:52                 ` Linus Torvalds
2012-03-18  0:43                   ` Keith Packard
2012-03-18  1:44                     ` Hugh Dickins
2012-03-18  3:42                       ` Keith Packard [this message]
2012-03-18  5:43                         ` Hugh Dickins
2012-03-18 11:55                           ` Rafael J. Wysocki
2012-03-19 17:58                             ` Hugh Dickins
2012-03-19 17:51                   ` Hugh Dickins
2012-03-19 16:16               ` Jerome Glisse
2012-03-16 22:52 ` Keith Packard
2012-03-16 23:24   ` Linus Torvalds
2012-03-17  2:55     ` Keith Packard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=863996d17y.fsf@sumi.keithp.com \
    --to=keithp@keithp.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=hughd@google.com \
    --cc=torvalds@linux-foundation.org \
    --cc=viro@zeniv.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.