All of lore.kernel.org
 help / color / mirror / Atom feed
From: Keith Packard <keithp@keithp.com>
To: Dave Airlie <airlied@gmail.com>,
	Linus Torvalds <torvalds@linux-foundation.org>
Cc: 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: Fri, 16 Mar 2012 16:01:29 -0700	[thread overview]
Message-ID: <864ntoduba.fsf@sumi.keithp.com> (raw)
In-Reply-To: <CAPM=9tzq=X0uqKnPrLXi4Ye5uKW2F03OZuNyUGBo3e6FS6ASkQ@mail.gmail.com>

<#part sign=pgpmime>
On Fri, 16 Mar 2012 16:47:46 +0000, Dave Airlie <airlied@gmail.com> wrote:

> The hibernate issue is known and I've been hoping someone from Intel
> would run with debugging it, they have a big enough team that I don't
> feel I can expend the personal time to look into it.

Yeah, I've been chatting with a couple of intel folks; we should have a
test patch ready shortly, but we haven't been able to reproduce anything
like this...

> Maybe Keith can push someone or maybe I just refuse pull requests
> until one with a fix appears.

>From what I've seen, this is a problem only on Ironlake machines, which
makes me afraid of some weird GTT flushing issue, given the adventures
we had with VT-d on that hardware where we idle the gpu before any GTT
updates. I wonder what would happen if we idled the GPU before any GTT
updates even when VT-d wasn't running...

> The latest thinking on the hibernate issues is kernel one sets up an
> fbcon, hibernate restores the old memory and the GTT still points at
> the pages,
> then something writes to the console and overwrites real memory., just
> a working theory, nobody has proven it yet.

Presumably the resumed kernel will not be able to write to the console
until the i915 driver is running again, at which point it will have
updated all of the GTT entries. And, presumably the booted kernel won't
be writing to the console after it has loaded the resumed kernel memory?

-- 
keith.packard@intel.com

  reply	other threads:[~2012-03-16 23:01 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 [this message]
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
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=864ntoduba.fsf@sumi.keithp.com \
    --to=keithp@keithp.com \
    --cc=airlied@gmail.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=torvalds@linux-foundation.org \
    /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.