From patchwork Fri Aug 22 03:11:24 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ben Widawsky X-Patchwork-Id: 4760481 Return-Path: X-Original-To: patchwork-intel-gfx@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 855C0C0338 for ; Fri, 22 Aug 2014 03:12:45 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 815862018E for ; Fri, 22 Aug 2014 03:12:44 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 3A4FA20173 for ; Fri, 22 Aug 2014 03:12:43 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id AB19A6E84A; Thu, 21 Aug 2014 20:12:41 -0700 (PDT) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTP id D00CA6E848 for ; Thu, 21 Aug 2014 20:12:40 -0700 (PDT) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga103.fm.intel.com with ESMTP; 21 Aug 2014 20:04:52 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,862,1389772800"; d="scan'208";a="375575640" Received: from unknown (HELO ironside.intel.com) ([10.255.12.192]) by FMSMGA003.fm.intel.com with ESMTP; 21 Aug 2014 20:08:47 -0700 From: Ben Widawsky To: Intel GFX Date: Thu, 21 Aug 2014 20:11:24 -0700 Message-Id: <1408677155-1840-2-git-send-email-benjamin.widawsky@intel.com> X-Mailer: git-send-email 2.0.4 In-Reply-To: <1408677155-1840-1-git-send-email-benjamin.widawsky@intel.com> References: <1408677155-1840-1-git-send-email-benjamin.widawsky@intel.com> Cc: Ben Widawsky , Ben Widawsky Subject: [Intel-gfx] [PATCH 01/68] 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.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, 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. v2: Change the pin assertion to be GGTT only. This is more accurate. 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 3b99390..16aebc6 100644 --- a/drivers/gpu/drm/i915/i915_gem_context.c +++ b/drivers/gpu/drm/i915/i915_gem_context.c @@ -609,31 +609,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 i915_hw_ppgtt *ppgtt = ctx_to_ppgtt(to); + int ret; + + BUG_ON(from && from->legacy_hw_ctx.rcs_state != 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->legacy_hw_ctx.rcs_state == NULL); BUG_ON(!i915_gem_obj_is_pinned(from->legacy_hw_ctx.rcs_state)); } - 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->legacy_hw_ctx.rcs_state, - get_context_alignment(ring->dev), 0); - if (ret) - return ret; - } + ret = i915_gem_obj_ggtt_pin(to->legacy_hw_ctx.rcs_state, + get_context_alignment(ring->dev), 0); + if (ret) + return ret; /* * Pin can switch back to the default context if we end up calling into @@ -648,12 +674,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 @@ -712,15 +732,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->legacy_hw_ctx.rcs_state); - i915_gem_context_unreference(from); } uninitialized = !to->legacy_hw_ctx.initialized && from == NULL; to->legacy_hw_ctx.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); @@ -731,8 +747,7 @@ done: return 0; unpin_out: - if (ring->id == RCS) - i915_gem_object_ggtt_unpin(to->legacy_hw_ctx.rcs_state); + i915_gem_object_ggtt_unpin(to->legacy_hw_ctx.rcs_state); return ret; } @@ -750,6 +765,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)); @@ -763,7 +779,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)