From patchwork Thu Dec 22 10:56:22 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chris Wilson X-Patchwork-Id: 9484687 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 D5D05601D3 for ; Thu, 22 Dec 2016 10:57:12 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id D05102818E for ; Thu, 22 Dec 2016 10:57:12 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id C391428324; Thu, 22 Dec 2016 10:57:12 +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=-4.1 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_MED,T_DKIM_INVALID 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 55D042818E for ; Thu, 22 Dec 2016 10:57:12 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 227DF6F219; Thu, 22 Dec 2016 10:57:11 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mail-lf0-x243.google.com (mail-lf0-x243.google.com [IPv6:2a00:1450:4010:c07::243]) by gabe.freedesktop.org (Postfix) with ESMTPS id B61E36E261 for ; Thu, 22 Dec 2016 10:56:34 +0000 (UTC) Received: by mail-lf0-x243.google.com with SMTP id x140so2014246lfa.2 for ; Thu, 22 Dec 2016 02:56:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=5wRASJ+y59IszW7Wb2z6ZSVzikD2aSJXn0XqqaTjp00=; b=bojhq1RAR907cT5fOKgnjAW+Uvfpzz8NODd8VVSD1DCi4CGt8ikXr17fDzzcmZexfi cq1ooNWm9HsPMD7se735LETC95WVDaDvvFsCcIlxbR0ap52j+AukCl2L+4AubaOiAfau woLzKizk7ArpAImFBCQWKyqqUsLhqp2dYoaFX2yhlo54RyyVUiLsGXP7AOsIx400z4yq Ik5LSxZxbbS28UKO3gcxkPlf+5quCX9Hz+7DWgiIQAx3q5sil9mIZuEu9qe8YxNqG6G0 B0dHOwRqtitKrD+nq5HcOvx9NwMSvl+bfDeapXte3rTZkozmH3eUPCmxs2Zd7p7JP0M9 3kAQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references; bh=5wRASJ+y59IszW7Wb2z6ZSVzikD2aSJXn0XqqaTjp00=; b=o29Foir/cAC7HZbg4Fy0gpY3bAPNi36h0ig7hkTsOmW61r4Y45sbyZpASssHemTSAt aSA04D1hWDwbpBdIwPl0ebKrwxz5W485RoWV6zUxA7q8SMfxoK8WJBEzxEJjeaSke9Om HjH+w61sNC+6HqH6C6dhZ5/VaC4+hueu6AEtzjpe3ZXcs7CFU1AbUXh/K7srbIwBoiLo SEmHQFfZ8RH8kythgkyIviwX9VNJHNxKMvrutfcnT7VSm+HiNVZf0qIHHj9cqaDqXvlv fFEWguPMGpD/nKc2tyuTsrbl7ptb+inY4lokpCKEETk8w6NtXTeU8tW774edkMITUY86 EPxg== X-Gm-Message-State: AIkVDXKyvibE9gITnlPimZL7rlMPvjYzDbaZDAWn9qGlHu8OjCvBrrM8VFlAW9hE7/T78g== X-Received: by 10.28.214.133 with SMTP id n127mr11659734wmg.28.1482404191226; Thu, 22 Dec 2016 02:56:31 -0800 (PST) Received: from haswell.alporthouse.com ([78.156.65.138]) by smtp.gmail.com with ESMTPSA id u78sm31553040wma.11.2016.12.22.02.56.30 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 22 Dec 2016 02:56:30 -0800 (PST) From: Chris Wilson To: intel-gfx@lists.freedesktop.org Date: Thu, 22 Dec 2016 10:56:22 +0000 Message-Id: <20161222105625.10832-2-chris@chris-wilson.co.uk> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20161222105625.10832-1-chris@chris-wilson.co.uk> References: <20161222105625.10832-1-chris@chris-wilson.co.uk> Subject: [Intel-gfx] [PATCH 2/5] drm/i915: Convert i915_ggtt_view to use an anonymous union 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-Virus-Scanned: ClamAV using ClamSMTP Save a lot of characters by making the union anonymous, with the side-effect of ignoring unset bits when comparing views. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_gem.c | 8 ++++---- drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++----- drivers/gpu/drm/i915/i915_gem_gtt.h | 2 +- drivers/gpu/drm/i915/i915_vma.c | 9 ++++----- drivers/gpu/drm/i915/i915_vma.h | 15 ++++++++++++--- drivers/gpu/drm/i915/intel_display.c | 2 +- 6 files changed, 26 insertions(+), 19 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 32da5ac29cdb..a0dc8cbc2aa4 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i915/i915_gem.c @@ -1865,10 +1865,10 @@ int i915_gem_fault(struct vm_area_struct *area, struct vm_fault *vmf) memset(&view, 0, sizeof(view)); view.type = I915_GGTT_VIEW_PARTIAL; - view.params.partial.offset = rounddown(page_offset, chunk_size); - view.params.partial.size = + view.partial.offset = rounddown(page_offset, chunk_size); + view.partial.size = min_t(unsigned int, chunk_size, - vma_pages(area) - view.params.partial.offset); + vma_pages(area) - view.partial.offset); /* If the partial covers the entire object, just create a * normal VMA. @@ -1903,7 +1903,7 @@ int i915_gem_fault(struct vm_area_struct *area, struct vm_fault *vmf) /* Finally, remap it using the new GTT offset */ ret = remap_io_mapping(area, - area->vm_start + (vma->ggtt_view.params.partial.offset << PAGE_SHIFT), + area->vm_start + (vma->ggtt_view.partial.offset << PAGE_SHIFT), (ggtt->mappable_base + vma->node.start) >> PAGE_SHIFT, min_t(u64, vma->size, area->vm_end - area->vm_start), &ggtt->mappable); diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c index 5eda33c17abc..d6fa8f740f23 100644 --- a/drivers/gpu/drm/i915/i915_gem_gtt.c +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c @@ -3457,7 +3457,7 @@ intel_partial_pages(const struct i915_ggtt_view *view, { struct sg_table *st; struct scatterlist *sg, *iter; - unsigned int count = view->params.partial.size; + unsigned int count = view->partial.size; unsigned int offset; int ret = -ENOMEM; @@ -3469,9 +3469,7 @@ intel_partial_pages(const struct i915_ggtt_view *view, if (ret) goto err_sg_alloc; - iter = i915_gem_object_get_sg(obj, - view->params.partial.offset, - &offset); + iter = i915_gem_object_get_sg(obj, view->partial.offset, &offset); GEM_BUG_ON(!iter); sg = st->sgl; @@ -3523,7 +3521,8 @@ i915_get_ggtt_vma_pages(struct i915_vma *vma) vma->pages = vma->obj->mm.pages; else if (vma->ggtt_view.type == I915_GGTT_VIEW_ROTATED) vma->pages = - intel_rotate_fb_obj_pages(&vma->ggtt_view.params.rotated, vma->obj); + intel_rotate_fb_obj_pages(&vma->ggtt_view.rotated, + vma->obj); else if (vma->ggtt_view.type == I915_GGTT_VIEW_PARTIAL) vma->pages = intel_partial_pages(&vma->ggtt_view, vma->obj); else diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.h b/drivers/gpu/drm/i915/i915_gem_gtt.h index 4c7bef07e38a..25614feccdba 100644 --- a/drivers/gpu/drm/i915/i915_gem_gtt.h +++ b/drivers/gpu/drm/i915/i915_gem_gtt.h @@ -164,7 +164,7 @@ struct i915_ggtt_view { unsigned int size; } partial; struct intel_rotation_info rotated; - } params; + }; }; extern const struct i915_ggtt_view i915_ggtt_view_normal; diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c index 3229f37eabea..4c8f545d15ff 100644 --- a/drivers/gpu/drm/i915/i915_vma.c +++ b/drivers/gpu/drm/i915/i915_vma.c @@ -96,15 +96,14 @@ __i915_vma_create(struct drm_i915_gem_object *obj, vma->ggtt_view = *view; if (view->type == I915_GGTT_VIEW_PARTIAL) { GEM_BUG_ON(range_overflows_t(u64, - view->params.partial.offset, - view->params.partial.size, + view->partial.offset, + view->partial.size, obj->base.size >> PAGE_SHIFT)); - vma->size = view->params.partial.size; + vma->size = view->partial.size; vma->size <<= PAGE_SHIFT; GEM_BUG_ON(vma->size >= obj->base.size); } else if (view->type == I915_GGTT_VIEW_ROTATED) { - vma->size = - intel_rotation_info_size(&view->params.rotated); + vma->size = intel_rotation_info_size(&view->rotated); vma->size <<= PAGE_SHIFT; } } diff --git a/drivers/gpu/drm/i915/i915_vma.h b/drivers/gpu/drm/i915/i915_vma.h index e3b2b3b1e056..edd8b2647eb7 100644 --- a/drivers/gpu/drm/i915/i915_vma.h +++ b/drivers/gpu/drm/i915/i915_vma.h @@ -202,9 +202,18 @@ i915_vma_compare(struct i915_vma *vma, if (vma->ggtt_view.type != view->type) return vma->ggtt_view.type - view->type; - return memcmp(&vma->ggtt_view.params, - &view->params, - sizeof(view->params)); + switch (view->type) { + case I915_GGTT_VIEW_ROTATED: + return memcmp(&vma->ggtt_view.rotated, + &view->rotated, + sizeof(view->rotated)); + case I915_GGTT_VIEW_PARTIAL: + return memcmp(&vma->ggtt_view.partial, + &view->partial, + sizeof(view->partial)); + default: + return 0; + } } int i915_vma_bind(struct i915_vma *vma, enum i915_cache_level cache_level, diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index ef5dde5ab1cf..3608787730c4 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -2140,7 +2140,7 @@ intel_fill_fb_ggtt_view(struct i915_ggtt_view *view, { if (drm_rotation_90_or_270(rotation)) { *view = i915_ggtt_view_rotated; - view->params.rotated = to_intel_framebuffer(fb)->rot_info; + view->rotated = to_intel_framebuffer(fb)->rot_info; } else { *view = i915_ggtt_view_normal; }