public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tomi Sarvela <tomi.p.sarvela@intel.com>,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	Intel graphics driver community testing & development
	<intel-gfx@lists.freedesktop.org>
Subject: Re: [PATCH] Revert "drm/i915: Fix races on fbdev"
Date: Fri, 01 Apr 2016 15:08:42 +0300	[thread overview]
Message-ID: <1459512522.4199.4.camel@linux.intel.com> (raw)
In-Reply-To: <20160401120256.GE5981@nuc-i3427.alporthouse.com>

On pe, 2016-04-01 at 13:02 +0100, Chris Wilson wrote:
> On Fri, Apr 01, 2016 at 02:41:01PM +0300, Joonas Lahtinen wrote:
> > 
> > This reverts commit a7442b93cf32c1e1ddb721a26cd1f92302e2a222.
> > 
> > With the patch applied SNB, IVB and ILK are experiencing hard machine
> > hangs. Original patch was to fix "just" kernel panics so it's not a good
> > trade-off.
> > 
> > Proper fix for the panic is on the way, lets revert until then.
> > 
> > Fixes: a7442b93cf32 ("drm/i915: Fix races on fbdev")
> > Cc: Lukas Wunner <lukas@wunner.de>
> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> > Cc: Tomi Sarvela <tomi.p.sarvela@intel.com>
> > Cc: stable@vger.kernel.org
> > Acked-by: Lukas Wunner <lukas@wunner.de>
> > Tested-by: Tomi Sarvela <tomi.p.sarvela@intel.com>
> > Suggested-by: Chris Wilson <chris@chris-wilson.co.uk>
> > Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Protocol dictates
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>

Thanks for the review. Merged (although due to delay in proxy or
somewhere, did not end up in the tree). But that does not mean we did
not go by the protocol.

Regards, Joonas

> -Chris
> 
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2016-04-01 12:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-01 11:41 [PATCH] Revert "drm/i915: Fix races on fbdev" Joonas Lahtinen
2016-04-01 12:02 ` Chris Wilson
2016-04-01 12:08   ` Joonas Lahtinen [this message]
2016-04-01 12:06 ` ✓ Fi.CI.BAT: success for " Patchwork

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=1459512522.4199.4.camel@linux.intel.com \
    --to=joonas.lahtinen@linux.intel.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=daniel.vetter@ffwll.ch \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=tomi.p.sarvela@intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox