From patchwork Thu Apr 28 01:10:58 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: tom.orourke@intel.com X-Patchwork-Id: 8964681 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.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 001759F1D3 for ; Thu, 28 Apr 2016 01:12:11 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id D484820219 for ; Thu, 28 Apr 2016 01:12:09 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id ACEC42026D for ; Thu, 28 Apr 2016 01:12:08 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id BABE36E38E; Thu, 28 Apr 2016 01:11:53 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTP id DBE716E37F for ; Thu, 28 Apr 2016 01:11:24 +0000 (UTC) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP; 27 Apr 2016 18:11:24 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,544,1455004800"; d="scan'208";a="941742165" Received: from torourke-desk.ra.intel.com (HELO localhost.localdomain) ([10.10.35.157]) by orsmga001.jf.intel.com with ESMTP; 27 Apr 2016 18:11:24 -0700 From: tom.orourke@intel.com To: intel-gfx@lists.freedesktop.org Date: Wed, 27 Apr 2016 18:10:58 -0700 Message-Id: <1461805865-212590-15-git-send-email-tom.orourke@intel.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1461805865-212590-1-git-send-email-tom.orourke@intel.com> References: <1461805865-212590-1-git-send-email-tom.orourke@intel.com> Cc: Tom O'Rourke , radoslaw.szwichtenberg@intel.com, paulo.r.zanoni@intel.com Subject: [Intel-gfx] [PATCH 14/21] drm/i915/slpc: Notification of Display mode change X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.18 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=-5.2 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 From: Sagar Arun Kamble GuC SLPC needs to be sent data related to Active pipes, refresh rates, widi pipes, fullscreen pipes related via host to GuC display mode change event. Based on this, SLPC will track FPS on active pipes. This patch defines the events and implements trigger of the events. v2: Addressed review comments from Paulo and Ville. Changed the way display mode information is collected in intel_atomic_commit. Coupled display mode change event with SLPC enable/reset event. Updated inactive crtc state in display mode data. Updated refresh rate and vsync_ft_usec calculations to get more accurate value. (Paulo) v2(torourke): Updates suggested by Paulo: replace HAS_SLPC with intel_slpc_active. Return void instead of ignored error code. v3: Addressed checkpatch issues. (Sagar) Commit message update and bitmask op changes in display mode events. (Nick) Added check for mode parameters clock, htotal, vtotal. (Jon) Signed-off-by: Sagar Arun Kamble Signed-off-by: Tom O'Rourke --- drivers/gpu/drm/i915/intel_display.c | 2 + drivers/gpu/drm/i915/intel_slpc.c | 174 ++++++++++++++++++++++++++++++++++- drivers/gpu/drm/i915/intel_slpc.h | 3 + 3 files changed, 178 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index eb7cb94..3bc61b7 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -13716,6 +13716,8 @@ static int intel_atomic_commit(struct drm_device *dev, drm_atomic_helper_cleanup_planes(dev, state); mutex_unlock(&dev->struct_mutex); + intel_slpc_update_atomic_commit_info(dev, state); + drm_atomic_state_free(state); /* As one of the primary mmio accessors, KMS has a high likelihood diff --git a/drivers/gpu/drm/i915/intel_slpc.c b/drivers/gpu/drm/i915/intel_slpc.c index 076d07b..7f26284 100644 --- a/drivers/gpu/drm/i915/intel_slpc.c +++ b/drivers/gpu/drm/i915/intel_slpc.c @@ -73,6 +73,24 @@ static void host2guc_slpc_shutdown(struct drm_device *dev) host2guc_slpc(dev_priv, data, 4); } +static void host2guc_slpc_display_mode_change(struct drm_device *dev) +{ + struct drm_i915_private *dev_priv = dev->dev_private; + u32 data[3 + SLPC_MAX_NUM_OF_PIPES]; + int i; + struct intel_slpc_display_mode_event_params *display_mode_params; + + display_mode_params = &dev_priv->guc.slpc.display_mode_params; + data[0] = HOST2GUC_ACTION_SLPC_REQUEST; + data[1] = SLPC_EVENT(SLPC_EVENT_DISPLAY_MODE_CHANGE, + SLPC_MAX_NUM_OF_PIPES + 1); + data[2] = display_mode_params->global_data; + for(i = 0; i < SLPC_MAX_NUM_OF_PIPES; ++i) + data[3+i] = display_mode_params->per_pipe_info[i].data; + + host2guc_slpc(dev_priv, data, 3 + SLPC_MAX_NUM_OF_PIPES); +} + static u8 slpc_get_platform_sku(struct drm_i915_gem_object *obj) { struct drm_device *dev = obj->base.dev; @@ -181,8 +199,10 @@ void intel_slpc_disable(struct drm_device *dev) void intel_slpc_enable(struct drm_device *dev) { - if (intel_slpc_active(dev)) + if (intel_slpc_active(dev)) { host2guc_slpc_reset(dev); + intel_slpc_update_display_mode_info(dev); + } } void intel_slpc_reset(struct drm_device *dev) @@ -192,3 +212,155 @@ void intel_slpc_reset(struct drm_device *dev) host2guc_slpc_reset(dev); } } + +void intel_slpc_update_display_mode_info(struct drm_device *dev) +{ + struct drm_i915_private *dev_priv = dev->dev_private; + struct intel_crtc *intel_crtc; + struct intel_display_pipe_info *per_pipe_info; + struct intel_slpc_display_mode_event_params *cur_params, old_params; + bool notify = false; + + if (!intel_slpc_active(dev)) + return; + + /* Copy display mode parameters for comparison */ + cur_params = &dev_priv->guc.slpc.display_mode_params; + old_params.global_data = cur_params->global_data; + cur_params->global_data = 0; + + intel_runtime_pm_get(dev_priv); + drm_modeset_lock_all(dev); + + for_each_intel_crtc(dev, intel_crtc) { + per_pipe_info = &cur_params->per_pipe_info[intel_crtc->pipe]; + old_params.per_pipe_info[intel_crtc->pipe].data = + per_pipe_info->data; + per_pipe_info->data = 0; + + if (intel_crtc->active) { + struct drm_display_mode *mode = &intel_crtc->base.mode; + + if (mode->clock == 0 || mode->htotal == 0 || + mode->vtotal == 0) { + DRM_DEBUG_DRIVER( + "Display Mode Info not sent to SLPC\n"); + drm_modeset_unlock_all(dev); + intel_runtime_pm_put(dev_priv); + return; + } + /* FIXME: Update is_widi based on encoder */ + per_pipe_info->is_widi = 0; + per_pipe_info->refresh_rate = + (mode->clock * 1000) / + (mode->htotal * mode->vtotal); + per_pipe_info->vsync_ft_usec = + (mode->htotal * mode->vtotal * 1000) / + mode->clock; + cur_params->active_pipes_bitmask |= + (1 << intel_crtc->pipe); + cur_params->vbi_sync_on_pipes |= + (1 << intel_crtc->pipe); + } else { + cur_params->active_pipes_bitmask &= + ~(1 << intel_crtc->pipe); + cur_params->vbi_sync_on_pipes &= + ~(1 << intel_crtc->pipe); + } + + if (old_params.per_pipe_info[intel_crtc->pipe].data != + per_pipe_info->data) + notify = true; + } + + drm_modeset_unlock_all(dev); + + cur_params->num_active_pipes = + hweight32(cur_params->active_pipes_bitmask); + + /* + * Compare old display mode with current mode. + * Notify SLPC if it is changed. + */ + if (cur_params->global_data != old_params.global_data) + notify = true; + + if (notify) + host2guc_slpc_display_mode_change(dev); + + intel_runtime_pm_put(dev_priv); +} + +void intel_slpc_update_atomic_commit_info(struct drm_device *dev, + struct drm_atomic_state *state) +{ + struct drm_i915_private *dev_priv = dev->dev_private; + struct drm_crtc *crtc; + struct drm_crtc_state *crtc_state; + struct intel_display_pipe_info *per_pipe_info; + struct intel_slpc_display_mode_event_params *cur_params, old_params; + bool notify = false; + int i; + + if (!intel_slpc_active(dev)) + return; + + /* Copy display mode parameters for comparison */ + cur_params = &dev_priv->guc.slpc.display_mode_params; + old_params.global_data = cur_params->global_data; + + for_each_crtc_in_state(state, crtc, crtc_state, i) { + struct intel_crtc *intel_crtc = to_intel_crtc(crtc); + + per_pipe_info = &cur_params->per_pipe_info[intel_crtc->pipe]; + old_params.per_pipe_info[intel_crtc->pipe].data = + per_pipe_info->data; + + per_pipe_info->data = 0; + cur_params->active_pipes_bitmask &= + ~(1 << intel_crtc->pipe); + cur_params->vbi_sync_on_pipes &= + ~(1 << intel_crtc->pipe); + + if (crtc_state->active) { + struct drm_display_mode *mode = &crtc->mode; + + if (mode->clock == 0 || mode->htotal == 0 || + mode->vtotal == 0) { + DRM_DEBUG_DRIVER( + "Display Mode Info not sent to SLPC\n"); + return; + } + + /* FIXME: Update is_widi based on encoder */ + per_pipe_info->is_widi = 0; + per_pipe_info->refresh_rate = + (mode->clock * 1000) / + (mode->htotal * mode->vtotal); + per_pipe_info->vsync_ft_usec = + (mode->htotal * mode->vtotal * 1000) / + mode->clock; + cur_params->active_pipes_bitmask |= + (1 << intel_crtc->pipe); + cur_params->vbi_sync_on_pipes |= + (1 << intel_crtc->pipe); + } + + if (old_params.per_pipe_info[intel_crtc->pipe].data != + per_pipe_info->data) + notify = true; + } + + cur_params->num_active_pipes = + hweight32(cur_params->active_pipes_bitmask); + + /* + * Compare old display mode with current mode. + * Notify SLPC if it is changed. + */ + if (cur_params->global_data != old_params.global_data) + notify = true; + + if (notify) + host2guc_slpc_display_mode_change(dev); +} diff --git a/drivers/gpu/drm/i915/intel_slpc.h b/drivers/gpu/drm/i915/intel_slpc.h index b342fa2..39b4657 100644 --- a/drivers/gpu/drm/i915/intel_slpc.h +++ b/drivers/gpu/drm/i915/intel_slpc.h @@ -150,5 +150,8 @@ void intel_slpc_suspend(struct drm_device *dev); void intel_slpc_disable(struct drm_device *dev); void intel_slpc_enable(struct drm_device *dev); void intel_slpc_reset(struct drm_device *dev); +void intel_slpc_update_display_mode_info(struct drm_device *dev); +void intel_slpc_update_atomic_commit_info(struct drm_device *dev, + struct drm_atomic_state *state); #endif