From patchwork Fri Jan 25 11:02:04 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Thomas Zimmermann X-Patchwork-Id: 10781151 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 8D181922 for ; Fri, 25 Jan 2019 11:02:24 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7B1592F7F9 for ; Fri, 25 Jan 2019 11:02:24 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 6D6672F800; Fri, 25 Jan 2019 11:02:24 +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 1170C2F7F9 for ; Fri, 25 Jan 2019 11:02:23 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 7BCC86E42A; Fri, 25 Jan 2019 11:02:20 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mx1.suse.de (mx2.suse.de [195.135.220.15]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5E86C6E42A for ; Fri, 25 Jan 2019 11:02:19 +0000 (UTC) X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id C74ACB050; Fri, 25 Jan 2019 11:02:17 +0000 (UTC) From: Thomas Zimmermann To: christian.koenig@amd.com, ray.huang@amd.com, Jerry.Zhang@amd.com, airlied@redhat.com, bskeggs@redhat.com, linux-graphics-maintainer@vmware.com, thellstrom@vmware.com, dri-devel@lists.freedesktop.org Subject: [PATCH v2 0/7] Replace ttm_bo_{unref, reference} with ttm_bo_{get|put} Date: Fri, 25 Jan 2019 12:02:04 +0100 Message-Id: <20190125110211.5877-1-tzimmermann@suse.de> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.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: Thomas Zimmermann Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP This patchset cleans up the last remaining callers of ttm_bo_reference and ttm_bo_unref. Calls are replaced with ttm_bo_get and ttm_bo_put, which follow Linux' get/put semantics more closely. The most notable difference is that ttm_bo_get does not clear the supplied pointer's value. This behaviour is now located in the calling code for cases where it might be required. v2: * rebased onto drm-tip Thomas Zimmermann (7): drm/ast: Replace ttm_bo_unref with ttm_bo_put drm/nouveau: Replace ttm_bo_reference with ttm_bo_get drm/nouveau: Replace ttm_bo_unref with ttm_bo_put drm/vmwgfx: Replace ttm_bo_reference with ttm_bo_get drm/vmwgfx: Replace ttm_bo_unref with ttm_bo_put drm/mgag200: Replace ttm_bo_unref with ttm_bo_put drm/ttm: Remove ttm_bo_reference and ttm_bo_unref drivers/gpu/drm/ast/ast_main.c | 6 +---- drivers/gpu/drm/mgag200/mgag200_main.c | 8 ++----- drivers/gpu/drm/nouveau/nouveau_bo.h | 12 ++++++---- drivers/gpu/drm/nouveau/nouveau_gem.c | 3 +-- drivers/gpu/drm/ttm/ttm_bo.c | 9 ------- drivers/gpu/drm/vmwgfx/vmwgfx_bo.c | 11 ++++----- drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 11 +++++---- drivers/gpu/drm/vmwgfx/vmwgfx_drv.h | 9 +++---- drivers/gpu/drm/vmwgfx/vmwgfx_mob.c | 21 ++++++++++------ drivers/gpu/drm/vmwgfx/vmwgfx_resource.c | 9 ++++--- drivers/gpu/drm/vmwgfx/vmwgfx_validation.c | 6 +++-- include/drm/ttm/ttm_bo_api.h | 28 ---------------------- 12 files changed, 49 insertions(+), 84 deletions(-) Reviewed-by: Christian König Reviewed-by: Huang Rui