From patchwork Tue Jun 24 01:48:43 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ben Widawsky X-Patchwork-Id: 4406091 Return-Path: X-Original-To: patchwork-intel-gfx@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id A624D9F26E for ; Tue, 24 Jun 2014 01:49:19 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id C435C20303 for ; Tue, 24 Jun 2014 01:49:18 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 561022034B for ; Tue, 24 Jun 2014 01:49:17 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id D974E6E4BE; Mon, 23 Jun 2014 18:49:14 -0700 (PDT) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id 512D76E4A2 for ; Mon, 23 Jun 2014 18:49:13 -0700 (PDT) Received: from azsmga001.ch.intel.com ([10.2.17.19]) by azsmga101.ch.intel.com with ESMTP; 23 Jun 2014 18:48:53 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.01,534,1400050800"; d="scan'208";a="449094755" Received: from unknown (HELO ironside.intel.com) ([10.255.13.30]) by azsmga001.ch.intel.com with ESMTP; 23 Jun 2014 18:48:52 -0700 From: Ben Widawsky To: Intel GFX Date: Mon, 23 Jun 2014 18:48:43 -0700 Message-Id: <1403574526-15585-3-git-send-email-benjamin.widawsky@intel.com> X-Mailer: git-send-email 2.0.0 In-Reply-To: <1403574526-15585-1-git-send-email-benjamin.widawsky@intel.com> References: <1403574526-15585-1-git-send-email-benjamin.widawsky@intel.com> Cc: Ben Widawsky , Ben Widawsky Subject: [Intel-gfx] [PATCH 3/6] drm/i915: Split up do_switch X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP There are two important reasons for this patch. It should make the existing code a lot more readable. It also makes the next patch much easier to understand in my opinion. There are 2 main variables that effect this function, leaving 4 permutations: ring: RCS vs !RCS PPGTT: full or not I didn't find extracting the full PPGTT usage to be very beneficial at this point, but it may be in the future. This was originally recommended by Daniel Vetter, and in this case, I agree. There was no intentional behavioral change. Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_gem_context.c | 76 +++++++++++++++++++++------------ 1 file changed, 49 insertions(+), 27 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915/i915_gem_context.c index 0d2c75b..35985ad 100644 --- a/drivers/gpu/drm/i915/i915_gem_context.c +++ b/drivers/gpu/drm/i915/i915_gem_context.c @@ -591,31 +591,57 @@ mi_set_context(struct intel_engine_cs *ring, return ret; } -static int do_switch(struct intel_engine_cs *ring, - struct intel_context *to) +static void do_switch_fini_common(struct intel_engine_cs *ring, + struct intel_context *from, + struct intel_context *to) +{ + if (likely(from)) + i915_gem_context_unreference(from); + i915_gem_context_reference(to); + ring->last_context = to; +} + +static int do_switch_xcs(struct intel_engine_cs *ring, + struct intel_context *from, + struct intel_context *to) +{ + struct drm_device *dev = ring->dev; + struct i915_hw_ppgtt *ppgtt = ctx_to_ppgtt(to); + int ret; + + BUG_ON(from && from->obj != NULL); + + if (USES_FULL_PPGTT(dev)) { + ret = ppgtt->switch_mm(ppgtt, ring, false); + if (ret) + return ret; + } + + if (from) + do_switch_fini_common(ring, from, to); + + return 0; +} + +static int do_switch_rcs(struct intel_engine_cs *ring, + struct intel_context *from, + struct intel_context *to) { struct drm_i915_private *dev_priv = ring->dev->dev_private; - struct intel_context *from = ring->last_context; struct i915_hw_ppgtt *ppgtt = ctx_to_ppgtt(to); u32 hw_flags = 0; bool uninitialized = false; int ret, i; - if (from != NULL && ring == &dev_priv->ring[RCS]) { + if (from != NULL) { BUG_ON(from->obj == NULL); BUG_ON(!i915_gem_obj_is_pinned(from->obj)); } - if (from == to && !to->remap_slice) - return 0; - /* Trying to pin first makes error handling easier. */ - if (ring == &dev_priv->ring[RCS]) { - ret = i915_gem_obj_ggtt_pin(to->obj, - get_context_alignment(ring->dev), 0); - if (ret) - return ret; - } + ret = i915_gem_obj_ggtt_pin(to->obj, get_context_alignment(ring->dev), 0); + if (ret) + return ret; /* * Pin can switch back to the default context if we end up calling into @@ -630,12 +656,6 @@ static int do_switch(struct intel_engine_cs *ring, goto unpin_out; } - if (ring != &dev_priv->ring[RCS]) { - if (from) - i915_gem_context_unreference(from); - goto done; - } - /* * Clear this page out of any CPU caches for coherent swap-in/out. Note * that thanks to write = false in this call and us not setting any gpu @@ -694,15 +714,11 @@ static int do_switch(struct intel_engine_cs *ring, /* obj is kept alive until the next request by its active ref */ i915_gem_object_ggtt_unpin(from->obj); - i915_gem_context_unreference(from); } uninitialized = !to->is_initialized && from == NULL; to->is_initialized = true; - -done: - i915_gem_context_reference(to); - ring->last_context = to; + do_switch_fini_common(ring, from, to); if (uninitialized) { ret = i915_gem_render_state_init(ring); @@ -713,8 +729,7 @@ done: return 0; unpin_out: - if (ring->id == RCS) - i915_gem_object_ggtt_unpin(to->obj); + i915_gem_object_ggtt_unpin(to->obj); return ret; } @@ -732,6 +747,7 @@ int i915_switch_context(struct intel_engine_cs *ring, struct intel_context *to) { struct drm_i915_private *dev_priv = ring->dev->dev_private; + struct intel_context *from = ring->last_context; WARN_ON(!mutex_is_locked(&dev_priv->dev->struct_mutex)); @@ -745,7 +761,13 @@ int i915_switch_context(struct intel_engine_cs *ring, return 0; } - return do_switch(ring, to); + if (from == to && !to->remap_slice) + return 0; + + if (ring->id == RCS) + return do_switch_rcs(ring, from, to); + else + return do_switch_xcs(ring, from, to); } static bool hw_context_enabled(struct drm_device *dev)