All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ben Widawsky <ben@bwidawsk.net>
To: Chris Wilson <chris@chris-wilson.co.uk>,
	Ben Widawsky <benjamin.widawsky@intel.com>,
	Intel GFX <intel-gfx@lists.freedesktop.org>
Cc: Mika Kuoppala <miku@iki.fi>
Subject: Re: [PATCH 1/2] drm/i915: Do a fuller init after reset
Date: Mon, 14 Oct 2013 15:07:25 -0700	[thread overview]
Message-ID: <20131014220724.GA8949@bwidawsk.net> (raw)
In-Reply-To: <20131014212417.GB29679@nuc-i3427.alporthouse.com>

On Mon, Oct 14, 2013 at 10:24:17PM +0100, Chris Wilson wrote:
> On Mon, Oct 14, 2013 at 10:01:36AM -0700, Ben Widawsky wrote:
> > From: Ben Widawsky <ben@bwidawsk.net>
> > 
> > I had this lying around from he original PPGTT series, and thought we
> > might try to get it in by itself.
> 
> This conflicts with Mika's work to replay pending operations after the
> reset. But I wholeheartedly agree with just being able to call
> init_hw(). I'd suggest we punt this todo task to Mika :)
> -Chris
> 

The patch is 4 months old, and I didn't really do much other than
resolve conflicts and basic tests. Also hopefully there is nothing
fundamentally wrong with this so that he can't do that - it helps me in
the long run with ppgtt a bit.

I'll add Mika to the CC so he doesn't miss it, hopefully we needn't wait
for him.

-- 
Ben Widawsky, Intel Open Source Technology Center

  reply	other threads:[~2013-10-14 22:07 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-14 17:01 [PATCH 1/2] drm/i915: Do a fuller init after reset Ben Widawsky
2013-10-14 17:01 ` [PATCH 2/2] drm/i915: cleanup context fini Ben Widawsky
2013-10-14 21:24 ` [PATCH 1/2] drm/i915: Do a fuller init after reset Chris Wilson
2013-10-14 22:07   ` Ben Widawsky [this message]
2013-10-15  3:46 ` [PATCH] drm/i915: Replace has_bsd/blt/vebox with a mask Ben Widawsky
2013-10-15  8:50   ` Chris Wilson
2013-10-15 15:03     ` Ben Widawsky
2013-10-15 15:08       ` Chris Wilson
2013-10-15 17:02         ` Ben Widawsky
2013-10-16  9:09           ` Daniel Vetter

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=20131014220724.GA8949@bwidawsk.net \
    --to=ben@bwidawsk.net \
    --cc=benjamin.widawsky@intel.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=miku@iki.fi \
    /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.