From patchwork Tue Apr 24 11:36:32 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Maarten Lankhorst X-Patchwork-Id: 10359339 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id EE60B6038F for ; Tue, 24 Apr 2018 11:48:38 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id DCE8A28C59 for ; Tue, 24 Apr 2018 11:48:38 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id DB57428D9B; Tue, 24 Apr 2018 11:48:38 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00, MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 4508028C59 for ; Tue, 24 Apr 2018 11:48:37 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 0E2E46E3D4; Tue, 24 Apr 2018 11:48:37 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mblankhorst.nl (mblankhorst.nl [141.105.120.124]) by gabe.freedesktop.org (Postfix) with ESMTPS id 11E616E3D6; Tue, 24 Apr 2018 11:48:36 +0000 (UTC) From: Maarten Lankhorst To: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Subject: [PATCH 1/2] drm/rect: Add midpoint to scale calculation functions Date: Tue, 24 Apr 2018 13:36:32 +0200 Message-Id: <20180424113633.16140-1-maarten.lankhorst@linux.intel.com> X-Mailer: git-send-email 2.17.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Vidya Srinivas MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP When calculating limits we want to be as pessimistic as possible, so we have to explicitly say whether we want to round up or down to accurately calculate whether we are below min_scale or above max_scale. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_atomic_helper.c | 4 +-- drivers/gpu/drm/drm_rect.c | 39 +++++++++++++++++++++-------- drivers/gpu/drm/i915/intel_sprite.c | 8 +++--- include/drm/drm_rect.h | 8 +++--- 4 files changed, 39 insertions(+), 20 deletions(-) diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c index 9cb2209f6fc8..7643202bfcf7 100644 --- a/drivers/gpu/drm/drm_atomic_helper.c +++ b/drivers/gpu/drm/drm_atomic_helper.c @@ -754,8 +754,8 @@ int drm_atomic_helper_check_plane_state(struct drm_plane_state *plane_state, drm_rect_rotate(src, fb->width << 16, fb->height << 16, rotation); /* Check scaling */ - hscale = drm_rect_calc_hscale(src, dst, min_scale, max_scale); - vscale = drm_rect_calc_vscale(src, dst, min_scale, max_scale); + hscale = drm_rect_calc_hscale(src, dst, min_scale, 1<<16, max_scale); + vscale = drm_rect_calc_vscale(src, dst, min_scale, 1<<16, max_scale); if (hscale < 0 || vscale < 0) { DRM_DEBUG_KMS("Invalid scaling of plane\n"); drm_rect_debug_print("src: ", &plane_state->src, true); diff --git a/drivers/gpu/drm/drm_rect.c b/drivers/gpu/drm/drm_rect.c index 9817c1445ba9..2093474c268c 100644 --- a/drivers/gpu/drm/drm_rect.c +++ b/drivers/gpu/drm/drm_rect.c @@ -96,7 +96,7 @@ bool drm_rect_clip_scaled(struct drm_rect *src, struct drm_rect *dst, } EXPORT_SYMBOL(drm_rect_clip_scaled); -static int drm_calc_scale(int src, int dst) +static int drm_calc_scale(int src, int dst, int scale_center) { int scale = 0; @@ -106,7 +106,10 @@ static int drm_calc_scale(int src, int dst) if (dst == 0) return 0; - scale = src / dst; + if (DIV_ROUND_UP(dst, scale_center) > src) + return DIV_ROUND_UP(src, dst); + else + scale = src / dst; return scale; } @@ -116,21 +119,25 @@ static int drm_calc_scale(int src, int dst) * @src: source window rectangle * @dst: destination window rectangle * @min_hscale: minimum allowed horizontal scaling factor + * @mid_hscale: mid point, below this point round down scaling, above round up. * @max_hscale: maximum allowed horizontal scaling factor * * Calculate the horizontal scaling factor as * (@src width) / (@dst width). * + * If the scale is below @mid_hscale we round down, if above up. This will + * calculate the hscale with the most pessimistic limit calculation. + * * RETURNS: * The horizontal scaling factor, or errno of out of limits. */ int drm_rect_calc_hscale(const struct drm_rect *src, const struct drm_rect *dst, - int min_hscale, int max_hscale) + int min_hscale, int mid_hscale, int max_hscale) { int src_w = drm_rect_width(src); int dst_w = drm_rect_width(dst); - int hscale = drm_calc_scale(src_w, dst_w); + int hscale = drm_calc_scale(src_w, dst_w, mid_hscale); if (hscale < 0 || dst_w == 0) return hscale; @@ -147,21 +154,25 @@ EXPORT_SYMBOL(drm_rect_calc_hscale); * @src: source window rectangle * @dst: destination window rectangle * @min_vscale: minimum allowed vertical scaling factor + * @mid_vscale: mid point, below this point round down scaling, above round up. * @max_vscale: maximum allowed vertical scaling factor * * Calculate the vertical scaling factor as * (@src height) / (@dst height). * + * If the scale is below @mid_vscale we round down, if above up. This will + * calculate the vscale with the most pessimistic limit calculation. + * * RETURNS: * The vertical scaling factor, or errno of out of limits. */ int drm_rect_calc_vscale(const struct drm_rect *src, const struct drm_rect *dst, - int min_vscale, int max_vscale) + int min_vscale, int mid_vscale, int max_vscale) { int src_h = drm_rect_height(src); int dst_h = drm_rect_height(dst); - int vscale = drm_calc_scale(src_h, dst_h); + int vscale = drm_calc_scale(src_h, dst_h, mid_vscale); if (vscale < 0 || dst_h == 0) return vscale; @@ -178,6 +189,7 @@ EXPORT_SYMBOL(drm_rect_calc_vscale); * @src: source window rectangle * @dst: destination window rectangle * @min_hscale: minimum allowed horizontal scaling factor + * @mid_hscale: mid point, below this point round down scaling, above round up. * @max_hscale: maximum allowed horizontal scaling factor * * Calculate the horizontal scaling factor as @@ -189,16 +201,19 @@ EXPORT_SYMBOL(drm_rect_calc_vscale); * If the calculated scaling factor is above @max_vscale, * decrease the height of rectangle @src to compensate. * + * If the scale is below @mid_hscale we round down, if above up. This will + * calculate the hscale with the most pessimistic limit calculation. + * * RETURNS: * The horizontal scaling factor. */ int drm_rect_calc_hscale_relaxed(struct drm_rect *src, struct drm_rect *dst, - int min_hscale, int max_hscale) + int min_hscale, int mid_hscale, int max_hscale) { int src_w = drm_rect_width(src); int dst_w = drm_rect_width(dst); - int hscale = drm_calc_scale(src_w, dst_w); + int hscale = drm_calc_scale(src_w, dst_w, mid_hscale); if (hscale < 0 || dst_w == 0) return hscale; @@ -228,6 +243,7 @@ EXPORT_SYMBOL(drm_rect_calc_hscale_relaxed); * @src: source window rectangle * @dst: destination window rectangle * @min_vscale: minimum allowed vertical scaling factor + * @mid_vscale: mid point, below this point round down scaling, above round up. * @max_vscale: maximum allowed vertical scaling factor * * Calculate the vertical scaling factor as @@ -239,16 +255,19 @@ EXPORT_SYMBOL(drm_rect_calc_hscale_relaxed); * If the calculated scaling factor is above @max_vscale, * decrease the height of rectangle @src to compensate. * + * If the scale is below @mid_vscale we round down, if above up. This will + * calculate the vscale with the most pessimistic limit calculation. + * * RETURNS: * The vertical scaling factor. */ int drm_rect_calc_vscale_relaxed(struct drm_rect *src, struct drm_rect *dst, - int min_vscale, int max_vscale) + int min_vscale, int mid_vscale, int max_vscale) { int src_h = drm_rect_height(src); int dst_h = drm_rect_height(dst); - int vscale = drm_calc_scale(src_h, dst_h); + int vscale = drm_calc_scale(src_h, mid_vscale, dst_h); if (vscale < 0 || dst_h == 0) return vscale; diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/drm/i915/intel_sprite.c index aa1dfaa692b9..f8f7c66fc3ac 100644 --- a/drivers/gpu/drm/i915/intel_sprite.c +++ b/drivers/gpu/drm/i915/intel_sprite.c @@ -998,10 +998,10 @@ intel_check_sprite_plane(struct intel_plane *plane, drm_rect_rotate(src, fb->width << 16, fb->height << 16, state->base.rotation); - hscale = drm_rect_calc_hscale_relaxed(src, dst, min_scale, max_scale); + hscale = drm_rect_calc_hscale_relaxed(src, dst, min_scale, 1 << 16, max_scale); BUG_ON(hscale < 0); - vscale = drm_rect_calc_vscale_relaxed(src, dst, min_scale, max_scale); + vscale = drm_rect_calc_vscale_relaxed(src, dst, min_scale, 1 << 16, max_scale); BUG_ON(vscale < 0); if (crtc_state->base.enable) @@ -1017,7 +1017,7 @@ intel_check_sprite_plane(struct intel_plane *plane, if (state->base.visible) { /* check again in case clipping clamped the results */ - hscale = drm_rect_calc_hscale(src, dst, min_scale, max_scale); + hscale = drm_rect_calc_hscale(src, dst, min_scale, 1 << 16, max_scale); if (hscale < 0) { DRM_DEBUG_KMS("Horizontal scaling factor out of limits\n"); drm_rect_debug_print("src: ", src, true); @@ -1026,7 +1026,7 @@ intel_check_sprite_plane(struct intel_plane *plane, return hscale; } - vscale = drm_rect_calc_vscale(src, dst, min_scale, max_scale); + vscale = drm_rect_calc_vscale(src, dst, min_scale, 1 << 16, max_scale); if (vscale < 0) { DRM_DEBUG_KMS("Vertical scaling factor out of limits\n"); drm_rect_debug_print("src: ", src, true); diff --git a/include/drm/drm_rect.h b/include/drm/drm_rect.h index 44bc122b9ee0..dbb8631a9288 100644 --- a/include/drm/drm_rect.h +++ b/include/drm/drm_rect.h @@ -179,16 +179,16 @@ bool drm_rect_clip_scaled(struct drm_rect *src, struct drm_rect *dst, int hscale, int vscale); int drm_rect_calc_hscale(const struct drm_rect *src, const struct drm_rect *dst, - int min_hscale, int max_hscale); + int min_hscale, int mid_hscale, int max_hscale); int drm_rect_calc_vscale(const struct drm_rect *src, const struct drm_rect *dst, - int min_vscale, int max_vscale); + int min_vscale, int mid_vscale, int max_vscale); int drm_rect_calc_hscale_relaxed(struct drm_rect *src, struct drm_rect *dst, - int min_hscale, int max_hscale); + int min_hscale, int mid_hscale, int max_hscale); int drm_rect_calc_vscale_relaxed(struct drm_rect *src, struct drm_rect *dst, - int min_vscale, int max_vscale); + int min_vscale, int mid_vscale, int max_vscale); void drm_rect_debug_print(const char *prefix, const struct drm_rect *r, bool fixed_point); void drm_rect_rotate(struct drm_rect *r,