From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DE7B4C433F5 for ; Fri, 18 Feb 2022 19:49:31 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 033C010E8C8; Fri, 18 Feb 2022 19:49:31 +0000 (UTC) Received: from mail-lf1-x130.google.com (mail-lf1-x130.google.com [IPv6:2a00:1450:4864:20::130]) by gabe.freedesktop.org (Postfix) with ESMTPS id 7F9EB10E3AC for ; Fri, 18 Feb 2022 19:49:29 +0000 (UTC) Received: by mail-lf1-x130.google.com with SMTP id o2so7423768lfd.1 for ; Fri, 18 Feb 2022 11:49:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language :from:to:cc:references:in-reply-to:content-transfer-encoding; bh=oQ46B1wy5KQXyNUJmE67dU9EbEU68XNEpXJnooxvrek=; b=dnjgi3rjlzO1XDUGjuPTQpOOpCdAsIC2peHQd9B/xmOdJlsxNC4PK18VMBWHvmJlZv w6Kak6LDJYdljpMLSza24ktlDgFYNd8TRMI6Df81/y011kJbamUU1WOOE4daNPStulPk 6usS1FaOuOEGItEL3eUF2cJGV1IYr6i633h2yHqQ2kCyI0HRLu0je4VUt6aoUAuhEHnI t8gD2GAqset1QC2a5u0v4nDo1rk7S8hMZi1cY3IOuhTkRqeIPiH2c36ThvqkQUEJq0gV pCAQiI07r9Vq/FCh9AGx9dtP11JS25W9MghZK/dwrW2S0o6Nsf3I6DgS+bQJIPD7cZBx 8blQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:from:to:cc:references:in-reply-to :content-transfer-encoding; bh=oQ46B1wy5KQXyNUJmE67dU9EbEU68XNEpXJnooxvrek=; b=mmptav+m6xeKAfsYKHnwlm7HSREkEtmqF2Xu4HrKDM3onM0E7E/udCk/eSP3j75yvq yjYvyt576tbymheQR0FYO7d3mJ4D/UJUcu4J4Egt9/33DKOtIfb7tUtp56WLZMl3Y6CK 5CJk5ZQU9C82VJxVNkY6ppFLLPi8bHA52QKc9hIElKhRXc6KyC30H+T/Kt8MGNLa0aek jZfP+OhTrA6pB69vuEkVZFOy7gVh+7ZjUlIKa/Zlp0DDi45XOvaAkBSXeAv7JzBt8I5k BE/O6EhV/mKX5ayqtAH8tWhMAr72pPcoAKGFdXTCx+3+TxwTQZieyOm1kWBOIN9GgaRv DJqQ== X-Gm-Message-State: AOAM5305imKHwVc3lqPj6evXElW1Jh0pCPdBRlM4TKiovlwrcLeRv4Xu w4FnhvlaJBYa938IynaBgeM= X-Google-Smtp-Source: ABdhPJx1l282Aib5WypRFj68xwi8R1SwSvsflqzWzrlkK/z/sUyDCIA3Xny5vwKH5pHczVhEjkTnrQ== X-Received: by 2002:a05:6512:128f:b0:43b:fd1a:3db with SMTP id u15-20020a056512128f00b0043bfd1a03dbmr6163528lfs.405.1645213767696; Fri, 18 Feb 2022 11:49:27 -0800 (PST) Received: from [192.168.2.145] (109-252-138-165.dynamic.spd-mgts.ru. [109.252.138.165]) by smtp.googlemail.com with ESMTPSA id p21sm426757lja.32.2022.02.18.11.49.26 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 18 Feb 2022 11:49:26 -0800 (PST) Message-ID: <12bb6ee9-e3e0-46d3-5cc2-75cc57cbd57b@gmail.com> Date: Fri, 18 Feb 2022 22:49:25 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 Subject: Re: [PATCH libdrm v2 00/25] Update Tegra support Content-Language: en-US From: Dmitry Osipenko To: Thierry Reding References: <20220217191625.2534521-1-thierry.reding@gmail.com> <0e936e9a-57a2-3881-1a55-50682b859acc@gmail.com> In-Reply-To: <0e936e9a-57a2-3881-1a55-50682b859acc@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-tegra@vger.kernel.org, dri-devel@lists.freedesktop.org, =?UTF-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" 18.02.2022 17:16, Dmitry Osipenko пишет: > 18.02.2022 00:37, Thierry Reding пишет: >> On Thu, Feb 17, 2022 at 11:02:53PM +0300, Dmitry Osipenko wrote: >>> 17.02.2022 22:16, Thierry Reding пишет: >>>> From: Thierry Reding >>>> >>>> Hi all, >>>> >>>> this is the userspace part of the kernel patches that were recently >>>> merged into drm-next: >>>> >>>> https://patchwork.freedesktop.org/series/92378/ >>>> >>>> The goal is to provide a userspace implementation of the UAPI exposed by >>>> the kernel and show its usage in some test programs that can also be >>>> used for basic sanity testing. More complete userspace implementations >>>> are available here: >>>> >>>> * https://github.com/cyndis/vaapi-tegra-driver >>>> * https://github.com/grate-driver/xf86-video-opentegra >>>> * https://github.com/grate-driver/grate >>>> >>>> Changes in v2: >>>> - implement vic_clear() as a helper using ->fill() (Michał Mirosław) >>>> - rebase and fix a couple of Meson errors/warnings >>>> >>>> Thierry >>>> >>>> Thierry Reding (25): >>>> tegra: Indent according to .editorconfig >>>> tegra: Remove unused IOCTL implementations >>>> tegra: Extract common buffer object allocation code >>>> tegra: Fix mmap() of GEM buffer objects >>>> tegra: Add flink helpers >>>> tegra: Add PRIME support helpers >>>> tegra: Make API more consistent >>>> tegra: Install tegra-openclose test >>>> tegra: Update for new UABI >>>> tegra: Include private.h in list of source files >>>> tegra: Add channel APIs >>>> tegra: Add job and push buffer APIs >>>> tegra: Add syncpoint APIs >>>> tests: tegra: Add helper library for tests >>>> tests: tegra: Add gr2d-fill test >>>> tests: tegra: Add syncpt-wait test >>>> tests: tegra: Add syncpoint timeout test >>>> tests: tegra: Add VIC support >>>> tests: tegra: Add VIC 3.0 support >>>> tests: tegra: Add VIC 4.0 support >>>> tests: tegra: Add VIC 4.1 support >>>> tests: tegra: Add VIC 4.2 support >>>> tests: tegra: Add VIC clear test >>>> tests: tegra: Add VIC blit test >>>> tests: tegra: Add VIC flip test >>>> >>>> include/drm/tegra_drm.h | 429 +++++++++++++++++++++++-- >>>> tegra/channel.c | 195 ++++++++++++ >>>> tegra/job.c | 187 +++++++++++ >>>> tegra/meson.build | 7 +- >>>> tegra/private.h | 85 ++++- >>>> tegra/pushbuf.c | 184 +++++++++++ >>>> tegra/syncpt.c | 101 ++++++ >>>> tegra/tegra-symbols.txt | 27 +- >>>> tegra/tegra.c | 386 +++++++++++----------- >>>> tegra/tegra.h | 95 +++++- >>>> tests/tegra/.gitignore | 3 +- >>>> tests/tegra/drm-test-tegra.c | 147 +++++++++ >>>> tests/tegra/drm-test-tegra.h | 55 ++++ >>>> tests/tegra/drm-test.c | 248 +++++++++++++++ >>>> tests/tegra/drm-test.h | 72 +++++ >>>> tests/tegra/gr2d-fill.c | 146 +++++++++ >>>> tests/tegra/host1x.h | 34 ++ >>>> tests/tegra/meson.build | 88 +++++- >>>> tests/tegra/openclose.c | 52 +-- >>>> tests/tegra/syncpt-timeout.c | 163 ++++++++++ >>>> tests/tegra/syncpt-wait.c | 151 +++++++++ >>>> tests/tegra/vic-blit.c | 333 +++++++++++++++++++ >>>> tests/tegra/vic-clear.c | 173 ++++++++++ >>>> tests/tegra/vic-flip.c | 333 +++++++++++++++++++ >>>> tests/tegra/vic.c | 184 +++++++++++ >>>> tests/tegra/vic.h | 181 +++++++++++ >>>> tests/tegra/vic30.c | 458 +++++++++++++++++++++++++++ >>>> tests/tegra/vic30.h | 439 ++++++++++++++++++++++++++ >>>> tests/tegra/vic40.c | 370 ++++++++++++++++++++++ >>>> tests/tegra/vic40.h | 285 +++++++++++++++++ >>>> tests/tegra/vic41.c | 374 ++++++++++++++++++++++ >>>> tests/tegra/vic41.h | 372 ++++++++++++++++++++++ >>>> tests/tegra/vic42.c | 374 ++++++++++++++++++++++ >>>> tests/tegra/vic42.h | 597 +++++++++++++++++++++++++++++++++++ >>>> 34 files changed, 7068 insertions(+), 260 deletions(-) >>> >>> >>> Why do we need these tests in libdrm? Why not IGT? >> >> Oops, sorry. I had meant to reply to your question in the previous >> version. The idea was to have this minimal set of tests in libdrm as a >> way to demonstrate how to use the various APIs. At the same time, this >> is meant to serve as an easy way to validate that everything works from >> the comparatively simple libdrm package. > > Who will be validating libdrm on Tegra? > >> But yes, adding more tests on top of this to IGT is something that I've >> been pondering and I certainly wouldn't object if anyone else was going >> to attempt to do so. I don't think IGT and libdrm need to be mutually >> exclusive, though. > > I know that IGT tests are regularly run by the KernelCI at least on TK1. Finishing my thought.. It's good to have a demo within libdrm, but in my opinion it's not very practical to put effort into libdrm from the perspective of testing. Much more practical should be to put effort into IGT, which already has a use on Tegra.