From patchwork Wed Aug 22 08:49:28 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chunming Zhou X-Patchwork-Id: 10572737 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 3F83114E1 for ; Wed, 22 Aug 2018 08:49:45 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 314B32AFCE for ; Wed, 22 Aug 2018 08:49:45 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 245E32AFDC; Wed, 22 Aug 2018 08:49:45 +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=BAD_ENC_HEADER,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 04D042AFCE for ; Wed, 22 Aug 2018 08:49:43 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 6DE236E1AA; Wed, 22 Aug 2018 08:49:42 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0041.outbound.protection.outlook.com [104.47.42.41]) by gabe.freedesktop.org (Postfix) with ESMTPS id A450D6E18D; Wed, 22 Aug 2018 08:49:40 +0000 (UTC) Received: from CY1PR12CA0043.namprd12.prod.outlook.com (2a01:111:e400:c42b::11) by DM5PR1201MB2504.namprd12.prod.outlook.com (2603:10b6:3:e3::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1080.14; Wed, 22 Aug 2018 08:49:38 +0000 Received: from DM3NAM03FT015.eop-NAM03.prod.protection.outlook.com (2a01:111:f400:7e49::207) by CY1PR12CA0043.outlook.office365.com (2a01:111:e400:c42b::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1080.13 via Frontend Transport; Wed, 22 Aug 2018 08:49:38 +0000 Received-SPF: None (protection.outlook.com: amd.com does not designate permitted sender hosts) Received: from SATLEXCHOV01.amd.com (165.204.84.17) by DM3NAM03FT015.mail.protection.outlook.com (10.152.82.195) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.20.1080.9 via Frontend Transport; Wed, 22 Aug 2018 08:49:38 +0000 Received: from zhoucm1.amd.com (10.34.1.3) by SATLEXCHOV01.amd.com (10.181.40.71) with Microsoft SMTP Server id 14.3.389.1; Wed, 22 Aug 2018 03:49:36 -0500 From: Chunming Zhou To: Subject: [PATCH 2/2] [RFC]drm: add syncobj timeline support Date: Wed, 22 Aug 2018 16:49:28 +0800 Message-ID: <20180822084928.19353-1-david1.zhou@amd.com> X-Mailer: git-send-email 2.14.1 MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:165.204.84.17; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(346002)(376002)(136003)(39860400002)(396003)(2980300002)(428003)(189003)(199004)(7696005)(51416003)(97736004)(86362001)(575784001)(6916009)(8936002)(356003)(47776003)(104016004)(81166006)(81156014)(8676002)(2906002)(305945005)(316002)(68736007)(54906003)(6666003)(16586007)(50226002)(1076002)(4326008)(36756003)(48376002)(50466002)(5660300001)(478600001)(336012)(53936002)(72206003)(486006)(77096007)(476003)(126002)(2616005)(426003)(14444005)(26005)(186003)(106466001)(105586002)(53416004)(2351001); DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR1201MB2504; H:SATLEXCHOV01.amd.com; FPR:; SPF:None; LANG:en; PTR:InfoDomainNonexistent; MX:1; A:1; X-Microsoft-Exchange-Diagnostics: 1; DM3NAM03FT015; 1:O9b7/rKHGns9Xn9VFqDiOxGk37jOHAEKYr/+qdMA/9N18Vp2gF9JVNl3Id0dPQmjU+SfYMCd0pBQuKmd+LzvdblOnOU8s5Pfdg5RaVUograjrdpq81kaQudRQPQJi5jP X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: cf488551-0295-4cc6-0cdc-08d6080c3198 X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989137)(4534165)(4627221)(201703031133081)(201702281549075)(8990107)(5600074)(711020)(2017052603328)(7153060); SRVR:DM5PR1201MB2504; X-Microsoft-Exchange-Diagnostics: 1; DM5PR1201MB2504; 3:uU8qV3V65njjVCaP2pIQS/AFVtK26vf5uzmtPm9w2gCBEeI5p1kqy0Q0EDG49vFrA6a5THnFZclNf9CMBAo9i3W/xA90V5R5QIfFYRFlhMdWGJYd6e7oSTRehHPdXnQOawdIcoOtrmgsGFre+8blTp2JXyl5C7RYPLire/2tg4Y+bgM0GUfGKHP3P5vfmMLUft6fiefnqipj9R4ApDUWTAJae4Oh7JjIqIi0KSKldumPkLcpzIMFzlA573v0HSWvV+PP5Dj5OAnbmtpD1DfpFC2Jp7c05wbirQ+sOKd7/L76iQnt2zxNJYT7C4EQEilD8jMo/foyRAULhYLu69/+CCIAscXodJ7GJsfEHaeNLg4=; 25:7dS58y+3QnCNICXnzAtcsWL0C6Z4C5XTiGk8U5d+fE4cRjerXMQN8JunFcgkINqQTb4qjdjHHQYjcfBKcWoocUOnNTYuFX2OR2MCiJYzuKvYOgUqLuLsuyabvOLvTKqC9UPcfpB9fyrmqyQ39FjxA/TKovr+rNoOWghXhntb4bovk7LsW/PM7zLY1R/SeYcktqFEqahKqBznbtXX1dF4y9mtP2F4XJxnY2vcfb8TKU2fVZrA/33uW8V5/MeZ7hBtXW3T8RZkTd4g85DqrT8S7GOSxoeBLTXZhNtSl+4XtQc89pEiQYbpNmDZcM4w1YSOCJv+g6+yBSqL/SDKBIDsgA== X-MS-TrafficTypeDiagnostic: DM5PR1201MB2504: X-Microsoft-Exchange-Diagnostics: 1; DM5PR1201MB2504; 31:19JMdBVEsmd8mokthtovGKvtVMUaGJfay5aVF83Y6w1Qvubmoz2lugjOyEy3378BriFURV/0nV6vmkv1w8vsicK24vX64H+PxU88F/xB+00K3LvYQUMA/lwh4s86sLAfyQrjhFcPjzkGr5YClVH31Hy5fD891q+VF4WuDVveIRDfZk+fiGGKVYVxgcJ2OfGJj2ezXOy9Wg8CW/xcRKDRr0ZF6QmPR65A3vF+gYaC/lw=; 20:GIQc8i37jP20zAWOlKD+eAK+UbQGUfzXComYTO7SbYIV92Vg8ZGQurykJZLLl24oRhcQcjWBKY0KrMrloe5sTpk7yFi6QytP1ULjCbRDDuSA33EF7fG6T21fCqPYd6LFOtsPiP43GAzpkloH0nHF5FsVCYvSFA29glnncFBT+zDZciEq3RhETQkPZbDV1DYHaPF+XlsM526hg8omI+xR910qCDOqO1PdKstRsGb0B5WreuQPxUAH7IDdWCFw8FYpkRf0UoJXJQBHWF+8IQjHnWrPFy8/LRpD56Q225n+++fzU99KkxL3ejJOx0Ze4uZdBpnSXBk5WQe5R5uL4ty38gUPE+X+zdO2yXvid0e0LWrHFyp8dvhomggLXhc6t+PQ10ykb738tV0iui6l4fry/D72f+aVgWthcsxUP39+nKaeNP+qx3dvKtCRW22MGNbKjrukmHl+dcv7L3seaQXkdFX/wbykc5In3LuuacRZpOYDVgknWecv4p5XKcnKzrq4 X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(767451399110)(788757137089); X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(3231311)(944501410)(52105095)(10201501046)(3002001)(93006095)(93003095)(6055026)(149027)(150027)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123562045)(20161123558120)(20161123564045)(20161123560045)(201708071742011)(7699016); SRVR:DM5PR1201MB2504; BCL:0; PCL:0; RULEID:; SRVR:DM5PR1201MB2504; X-Microsoft-Exchange-Diagnostics: 1; DM5PR1201MB2504; 4:+g0koaomF+ncbj7pph7q7BHXVzTqVN4wEtmsw2/eHLi8+pxEG3vRj2L9BoYgA8YxRuuFi8hsiIqh7DFwSbbYt6w0/Sflbm5GzejTobza+RjgjMP5pMlGtTMuFmDX4GvoPtIT6dRG+4fWzq76Rlbv+vx6GPMvX19dsCA+kowPmpCzAV1/TD2oOTlrDjvRvUbQT/3q8MVQMOTb5D0BLIqJ8afhb7TM60sEt3bKTtsbnMzBWJJMyALBK5r5Bj3OODAeWGrWrvC6j8jUi/qx7SiPo/kVouq4qXsQsdkDVLJljyDcr3RU+KM5blmLhZpN8jpeg1e1sG1RX/dF7l788VU9KA== X-Forefront-PRVS: 0772E5DAD5 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; DM5PR1201MB2504; 23:/ecJgk1HyIi1CcqnFL0le2BYt2ocP3zib46Nw/x?= ze9fDCmGgte2l6OH+7c5PaUj+pPa/Ua7MmQJioU2W8J2rPWKYxBUk6dR0H4TQYSjDzsjJf9f5yUaiNTWlNUWkxVK7jfV9KeWtUcPwUajYkzxLF4mWZw7yYCddmTr//BlhP6JSI0x6nXHHpGODNP8wtyMTX7dfChAY6j2s1UrzJxuK0dxjbubQ+fqCrBV7Je9L/aVpjydMFy66vgZEgedtppe/mVyCHTT01MQaJi8ho188wNd+25LBO7VxaPwXSo0encec8kq+7jx30xxXi8f7+1ZI7+ea/xQjuweWi0atdgtAJjb5W8yNqAWPlCFBEAC/vk/nUdQMrfoAO8rYFYh4VpZ6t5g3q2BbWopf+aRRgPpH1GllNiTdthoHS9rYhD8ZKZrZNdaKyDjmt9MW1Gs5GHdeU0nganaWq2B3hteE2OenYOhF3DfljqzoKUmZzaidSrJw+6K/H4HfEh9MCIbTUqMFMJuGM41wlABRdXME4q0/KJzaFExe+/pVoI/6OzyHZCWawaFDUOwKWQSJIVn/7x2sAngve+6YFNvY0u0O/mSmgKYFS1ixIZ1a37iHU443FUUG5LNzoJBU6A/FnaxaYwPk5Sp34+4UYmjG9eAa2GIac9IC9KbxflTMqbHniaP5nwS8pFWAmhdnyy72ssSbbHKgde7WhqqTCOWWlO+S04kZQcGwKuD5s5PfRkFkQ60GD6sQjNVLlhgQ3J+B1wRB/JmaJ65A2vUsSUs5iaT2fxwjnu/Ls00HqVG40aBbvkk3CiP8MSrL/+Cnraa7joi/1Y1Yvq3mRmr6frINQ0RHWAjWaej+71pYjwyzFsgcYkypNYK75achuh3pwbwkqqvS/SUD9uRC0q9H++a3SiKrHx5TKYF8wYeI+R7WdOD4NOTjBD5ZNUKmnhgFbhHygYMC5sd3ynuRNO5zkzbeAtxGKt1YRz5d4vQPJVJacw8peR/FPN98BWRQMJHU6GHQg98IM29I+2gn5/vHn7Vgx7GxPVvFRoVWckekgpLQUtw8JeWXn0xPqgiDItlAY7J7e8T8wL4cYqhtXSfZWdbrWMDbxP96SuOWqaA9dk9dZA/T6d19vmMQIe670K7LrKK9/KMlQBc5mGLPi1jKw4/MX9Glq6Y7maeLXBbO3vsT2nTwIZPQYR8= X-Microsoft-Antispam-Message-Info: kmxcw9S7hBrUkJYREEpty6S0PlfZyKB453EM+KAa7vm6OYxoPkWVwyfSfM2EAJXrS+UHDl7Fkl2mrhthVOb1SjGSuRNtfagcsv1iGaJAfIxwlLiGcEhMJw+gLlsuhYlsrK5HewLCueoG61lI0QewV5SdPhGhS1owyl+831Ezc4F09/mWFYBRbKClMPmSnsBUScHaJ01bWg6Yk6yE/X0ahrM/w30kBJGrH1GNVcHQ3Y7Nh2c/tTHSKQQ1onv/CqwoJdTU5cOIBPDTee1gVR1V7k6KFir0jHVj3LZnhx6Sn/mbnlyrqc5VK5NfzCv48rQl7RX72lYfxdoo+hk6LDH7dSNhQFK+7C76MFywa1QnJJU= X-Microsoft-Exchange-Diagnostics: 1; DM5PR1201MB2504; 6:0a11JwCvrFiQeTSAPOEhfls3hjzG60rKKIB77QitnNn5ybwFyAG88/7snpApW0UK6yWjYPKv4qlVEM1UZQlPN61FtOVfuutYiw6XGvLOsfykNEild4MKTY0hYG/qHAwNYi6DZ8u3vxIig3eHtLeerFq5a3NARNCTxOWIvR3k7UiKjY5i5VU8sNqyF/LNv3L27t0UIWpQgXgDQ9WHoWYd9/cXO6ZsT+kQq7SKqqDMA5Gm0+ivzZZtS28Mc2rGQv13zysI9bNk84HnEpZH5WRBJG7+BDsHBugzKvnU/k3YLXZnVIxnWDN0wguxxRQjHmbuUtbVjaJ7/uQZCcqY5UKVOFyA88Dd5/K/FzubcND7ckvMcfDtkEcLs5jTFYmx6NYB2g3npTIoia//Wk329U1Ox5UtdGoOWDwTV6SZCURBCatlOlux97bRxAdRSKPA4YQPDPYw2NUKOZd8o9Wk+Fm0Wg==; 5:/bTI/pSANzNnG0U3X4NTNn9Nc4id0Taf3jRGQlII9ZjasNYfm8ZC+au3MOk5xsSw9u702H20nnudzD9x69HdB68uKbmoMZt0Lp1xyj7eZDnXPA5Vxmf1LN2MrUKEc09rbt+c2nKzyw7H26LD1ulPAcjquA9iFvrbFw1z0+axrCg=; 7:rzDxT5c+hovOCChHekhfBNZMU/R3VUiVJtqtYQOjK6fzbMBCyg5Fmi4FD8ZwxU7TwRbGhIQOnxAUIHDuSiqx1IRb/7pef1RuM8ceqsfpFgsEhbcKgV4BUDSZHKERTWbbS2NP8iQvKJnmTCft0oJKzoEZjWWeksjYGi3HfvOsoVZAL1eFKqqhZ6b6CUGmYamgClHUc23rVZuQQVZq9dcPgkB6KUB/m87UWkNDPzduYTGn6f3RIXysEvq9f75HDjit SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DM5PR1201MB2504; 20:O8Onjb9N4G79/7hjfowMNrnKgwN64YNS4DaeOkSjMTNLnpP5TQyECo6ExjLdCxKZVvSBicyma4BRAQV66hGIBno/zLdm5opjFdsJdGp1NYsBT0RWO7a7IOIqCfp9zCdbXrFUaBKrF4Qz9E3dEidQUjVLCsru80TlmpjsWS0mhKNEHNQ+WhwFW98IK04Hg3HMpnNMvC1MxjUYnTsgG4H5KoF+MiBmM/KUAw6ZRFgOGyvHHPgVQ1587DGBp/2LHkcn X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Aug 2018 08:49:38.2150 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: cf488551-0295-4cc6-0cdc-08d6080c3198 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[SATLEXCHOV01.amd.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR1201MB2504 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: Daniel Rakos , Christian Konig , amd-gfx@lists.freedesktop.org, Dave Airlie Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP VK_KHR_timeline_semaphore: This extension introduces a new type of semaphore that has an integer payload identifying a point in a timeline. Such timeline semaphores support the following operations: * Host query - A host operation that allows querying the payload of the timeline semaphore. * Host wait - A host operation that allows a blocking wait for a timeline semaphore to reach a specified value. * Device wait - A device operation that allows waiting for a timeline semaphore to reach a specified value. * Device signal - A device operation that allows advancing the timeline semaphore to a specified value. Since it's a timeline, that means the front time point(PT) always is signaled before the late PT. a. signal PT design: Signal PT fence N depends on PT[N-1] fence and signal opertion fence, when PT[N] fence is signaled, the timeline will increase to value of PT[N]. b. wait PT design: Wait PT fence is signaled by reaching timeline point value, when timeline is increasing, will compare wait PTs value with new timeline value, if PT value is lower than timeline value, then wait PT will be signaled, otherwise keep in list. semaphore wait operation can wait on any point of timeline, so need a RB tree to order them. And wait PT could ahead of signal PT, we need a sumission fence to perform that. TODO: CPU query and wait on timeline semaphore. Change-Id: I9f09aae225e268442c30451badac40406f24262c Signed-off-by: Chunming Zhou Cc: Christian Konig Cc: Dave Airlie Cc: Daniel Rakos --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 7 +- drivers/gpu/drm/drm_syncobj.c | 385 ++++++++++++++++++++++++++++++++- drivers/gpu/drm/v3d/v3d_gem.c | 4 +- drivers/gpu/drm/vc4/vc4_gem.c | 2 +- include/drm/drm_syncobj.h | 45 +++- include/uapi/drm/drm.h | 3 +- 6 files changed, 435 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c index d42d1c8f78f6..463cc8960723 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c @@ -1105,7 +1105,7 @@ static int amdgpu_syncobj_lookup_and_add_to_sync(struct amdgpu_cs_parser *p, { int r; struct dma_fence *fence; - r = drm_syncobj_find_fence(p->filp, handle, &fence); + r = drm_syncobj_find_fence(p->filp, handle, &fence, 0); if (r) return r; @@ -1193,8 +1193,9 @@ static void amdgpu_cs_post_dependencies(struct amdgpu_cs_parser *p) { int i; - for (i = 0; i < p->num_post_dep_syncobjs; ++i) - drm_syncobj_replace_fence(p->post_dep_syncobjs[i], p->fence); + for (i = 0; i < p->num_post_dep_syncobjs; ++i) { + drm_syncobj_signal_fence(p->post_dep_syncobjs[i], p->fence, 0); + } } static int amdgpu_cs_submit(struct amdgpu_cs_parser *p, diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers/gpu/drm/drm_syncobj.c index 70af32d0def1..3709f36c901e 100644 --- a/drivers/gpu/drm/drm_syncobj.c +++ b/drivers/gpu/drm/drm_syncobj.c @@ -187,6 +187,191 @@ void drm_syncobj_replace_fence(struct drm_syncobj *syncobj, } EXPORT_SYMBOL(drm_syncobj_replace_fence); +static void drm_syncobj_timeline_signal_submission_fences(struct drm_syncobj *syncobj) +{ + struct rb_node *node = NULL; + struct drm_syncobj_wait_pt *wait_pt = NULL; + + spin_lock(&syncobj->lock); + for(node = rb_first(&syncobj->syncobj_timeline.wait_pt_tree); + node != NULL; node = rb_next(node)) { + wait_pt = rb_entry(node, struct drm_syncobj_wait_pt, node); + if (wait_pt->value <= syncobj->syncobj_timeline.signal_point) { + if (wait_pt->submission_fence) + dma_fence_signal(&wait_pt->submission_fence->base); + } else { + /* the loop is from left to right, the later entry value is + * bigger, so don't need to check any more */ + break; + } + } + spin_unlock(&syncobj->lock); +} + +static void drm_syncobj_timeline_signal_wait_pts(struct drm_syncobj *syncobj) +{ + struct rb_node *node = NULL; + struct drm_syncobj_wait_pt *wait_pt = NULL; + + spin_lock(&syncobj->lock); + for(node = rb_first(&syncobj->syncobj_timeline.wait_pt_tree); + node != NULL; ) { + wait_pt = rb_entry(node, struct drm_syncobj_wait_pt, node); + node = rb_next(node); + if (wait_pt->value <= syncobj->syncobj_timeline.timeline) { + dma_fence_signal(&wait_pt->base.base); + rb_erase(&wait_pt->node, + &syncobj->syncobj_timeline.wait_pt_tree); + RB_CLEAR_NODE(&wait_pt->node); + if (wait_pt->submission_fence) + dma_fence_put(&wait_pt->submission_fence->base); + /* kfree(wait_pt) is excuted by fence put */ + dma_fence_put(&wait_pt->base.base); + } else { + /* the loop is from left to right, the later entry value is + * bigger, so don't need to check any more */ + break; + } + } + spin_unlock(&syncobj->lock); +} + + +static void pt_fence_cb(struct drm_syncobj_signal_pt *signal_pt) +{ + struct dma_fence *fence = NULL; + struct drm_syncobj *syncobj; + + fence = signal_pt->signal_fence; + signal_pt->signal_fence = NULL; + dma_fence_put(fence); + fence = signal_pt->pre_pt_base; + signal_pt->pre_pt_base = NULL; + dma_fence_put(fence); + + syncobj = signal_pt->syncobj; + spin_lock(&syncobj->lock); + list_del(&signal_pt->list); + syncobj->syncobj_timeline.timeline = signal_pt->value; + spin_unlock(&syncobj->lock); + /* kfree(signal_pt) will be executed by below fence put */ + dma_fence_put(&signal_pt->base.base); + drm_syncobj_timeline_signal_wait_pts(syncobj); +} +static void pt_signal_fence_func(struct dma_fence *fence, + struct dma_fence_cb *cb) +{ + struct drm_syncobj_signal_pt *signal_pt = + container_of(cb, struct drm_syncobj_signal_pt, signal_cb); + + if (signal_pt->pre_pt_base && + !dma_fence_is_signaled(signal_pt->pre_pt_base)) + return; + + pt_fence_cb(signal_pt); +} +static void pt_pre_fence_func(struct dma_fence *fence, + struct dma_fence_cb *cb) +{ + struct drm_syncobj_signal_pt *signal_pt = + container_of(cb, struct drm_syncobj_signal_pt, pre_pt_cb); + + if (signal_pt->signal_fence && + !dma_fence_is_signaled(signal_pt->pre_pt_base)) + return; + + pt_fence_cb(signal_pt); +} + +static int drm_syncobj_timeline_signal_fence(struct drm_syncobj *syncobj, + struct dma_fence *fence, + u64 point) +{ + struct drm_syncobj_signal_pt *signal_pt = + kzalloc(sizeof(struct drm_syncobj_signal_pt), GFP_KERNEL); + struct drm_syncobj_signal_pt *tail_pt; + struct dma_fence *tail_pt_fence = NULL; + int ret = 0; + + if (!signal_pt) + return -ENOMEM; + if (syncobj->syncobj_timeline.signal_point >= point) { + DRM_WARN("A later signal is ready!"); + goto out; + } + if (fence) + dma_fence_get(fence); + spin_lock(&syncobj->lock); + spin_lock_init(&signal_pt->base.lock); + dma_fence_init(&signal_pt->base.base, + &drm_syncobj_stub_fence_ops, + &signal_pt->base.lock, + syncobj->syncobj_timeline.timeline_context, point); + signal_pt->signal_fence = + rcu_dereference_protected(fence, + lockdep_is_held(&fence->lock)); + if (!list_empty(&syncobj->syncobj_timeline.signal_pt_list)) { + tail_pt = list_last_entry(&syncobj->syncobj_timeline.signal_pt_list, + struct drm_syncobj_signal_pt, list); + tail_pt_fence = &tail_pt->base.base; + if (dma_fence_is_signaled(tail_pt_fence)) + tail_pt_fence = NULL; + } + if (tail_pt_fence) + signal_pt->pre_pt_base = + dma_fence_get(rcu_dereference_protected(tail_pt_fence, + lockdep_is_held(&tail_pt_fence->lock))); + + signal_pt->value = point; + syncobj->syncobj_timeline.signal_point = point; + signal_pt->syncobj = syncobj; + INIT_LIST_HEAD(&signal_pt->list); + list_add_tail(&signal_pt->list, &syncobj->syncobj_timeline.signal_pt_list); + spin_unlock(&syncobj->lock); + drm_syncobj_timeline_signal_submission_fences(syncobj); + /** + * Every pt is depending on signal fence and previous pt fence, add + * callbacks to them + */ + if (!dma_fence_is_signaled(signal_pt->signal_fence)) + dma_fence_add_callback(signal_pt->signal_fence, + &signal_pt->signal_cb, + pt_signal_fence_func); + else + pt_signal_fence_func(signal_pt->signal_fence, + &signal_pt->signal_cb); + if (signal_pt->pre_pt_base && !dma_fence_is_signaled(signal_pt->pre_pt_base)) + dma_fence_add_callback(signal_pt->pre_pt_base, + &signal_pt->pre_pt_cb, + pt_pre_fence_func); + else + pt_pre_fence_func(signal_pt->pre_pt_base, &signal_pt->pre_pt_cb); + + + return 0; +out: + kfree(signal_pt); + return ret; +} + +/** + * drm_syncobj_signal_fence - place fence into a sync object. + * @syncobj: Sync object to replace fence in + * @fence: fence to install in sync file. + * + * This places the fence into normal or timeline sync object + */ +void drm_syncobj_signal_fence(struct drm_syncobj *syncobj, + struct dma_fence *fence, + u64 point) +{ + if (syncobj->type == DRM_SYNCOBJ_TYPE_NORMAL) + drm_syncobj_replace_fence(syncobj, fence); + else if (syncobj->type == DRM_SYNCOBJ_TYPE_TIMELINE) + drm_syncobj_timeline_signal_fence(syncobj, fence, point); +} +EXPORT_SYMBOL(drm_syncobj_signal_fence); + static int drm_syncobj_assign_null_handle(struct drm_syncobj *syncobj) { struct drm_syncobj_stub_fence *fence; @@ -205,12 +390,150 @@ static int drm_syncobj_assign_null_handle(struct drm_syncobj *syncobj) return 0; } +static struct drm_syncobj_wait_pt * +drm_syncobj_timeline_lookup_wait_pt(struct drm_syncobj *syncobj, u64 point) +{ + struct rb_node *node = syncobj->syncobj_timeline.wait_pt_tree.rb_node; + struct drm_syncobj_wait_pt *wait_pt = NULL; + + + spin_lock(&syncobj->lock); + while(node) { + int result = point - wait_pt->value; + + wait_pt = rb_entry(node, struct drm_syncobj_wait_pt, node); + if (result < 0) + node = node->rb_left; + else if (result > 0) + node = node->rb_right; + else + break; + } + spin_unlock(&syncobj->lock); + + return wait_pt; +} + +static struct drm_syncobj_wait_pt * +drm_syncobj_timeline_create_wait_pt(struct drm_syncobj *syncobj, u64 point) +{ + struct drm_syncobj_wait_pt *wait_pt; + struct rb_node **new = &(syncobj->syncobj_timeline.wait_pt_tree.rb_node), *parent = NULL; + + wait_pt = kzalloc(sizeof(*wait_pt), GFP_KERNEL); + if (!wait_pt) + return NULL; + spin_lock_init(&wait_pt->base.lock); + dma_fence_init(&wait_pt->base.base, + &drm_syncobj_stub_fence_ops, + &wait_pt->base.lock, + syncobj->syncobj_timeline.timeline_context, point); + wait_pt->submission_fence = NULL; + /* check if the pt is already sumbitted, if yes, then don't need to + * create submission fence, otherwise create it */ + if (point > syncobj->syncobj_timeline.signal_point) { + wait_pt->submission_fence = + kzalloc(sizeof(struct drm_syncobj_stub_fence), + GFP_KERNEL); + if (!wait_pt->submission_fence) { + dma_fence_put(&wait_pt->base.base); + return NULL; + } + spin_lock_init(&wait_pt->submission_fence->lock); + dma_fence_init(&wait_pt->submission_fence->base, + &drm_syncobj_stub_fence_ops, + &wait_pt->submission_fence->lock, + syncobj->syncobj_timeline.submission_context, + point); + } + wait_pt->value = point; + + /* wait pt must be in an order, so that we can easily lookup and signal + * it */ + spin_lock(&syncobj->lock); + if (point <= syncobj->syncobj_timeline.timeline) + dma_fence_signal(&wait_pt->base.base); + if (wait_pt->submission_fence && + point <= syncobj->syncobj_timeline.signal_point) + dma_fence_signal(&wait_pt->submission_fence->base); + while(*new) { + struct drm_syncobj_wait_pt *this = + rb_entry(*new, struct drm_syncobj_wait_pt, node); + int result = wait_pt->value - this->value; + + parent = *new; + if (result < 0) + new = &((*new)->rb_left); + else if (result > 0) + new = &((*new)->rb_right); + else + goto exist; + } + + rb_link_node(&wait_pt->node, parent, new); + rb_insert_color(&wait_pt->node, &syncobj->syncobj_timeline.wait_pt_tree); + spin_unlock(&syncobj->lock); + return wait_pt; +exist: + spin_unlock(&syncobj->lock); + if (wait_pt->submission_fence) + dma_fence_put(&wait_pt->submission_fence->base); + dma_fence_put(&wait_pt->base.base); + wait_pt = drm_syncobj_timeline_lookup_wait_pt(syncobj, point); + return wait_pt; +} + +static struct dma_fence * +drm_syncobj_timeline_point_get(struct drm_syncobj *syncobj, u64 point, u64 flag) +{ + struct drm_syncobj_wait_pt *wait_pt; + + /* already signaled, simply return a signaled stub fence */ + if (point <= syncobj->syncobj_timeline.timeline) { + struct drm_syncobj_stub_fence *fence; + + fence = kzalloc(sizeof(*fence), GFP_KERNEL); + if (fence == NULL) + return NULL; + + spin_lock_init(&fence->lock); + dma_fence_init(&fence->base, &drm_syncobj_stub_fence_ops, + &fence->lock, 0, 0); + dma_fence_signal(&fence->base); + return &fence->base; + } + + /* check if the wait pt exists */ + wait_pt = drm_syncobj_timeline_lookup_wait_pt(syncobj, point); + if (!wait_pt) { + /* This is a new wait pt, so create it */ + wait_pt = drm_syncobj_timeline_create_wait_pt(syncobj, point); + if (!wait_pt) + return NULL; + } + if (wait_pt) { + struct dma_fence *fence; + if (wait_pt->submission_fence) { + if (flag & DRM_SYNCOBJ_WAIT_FLAGS_WAIT_FOR_SUBMIT) + dma_fence_wait(&wait_pt->submission_fence->base, true); + else if (!dma_fence_is_signaled(&wait_pt->submission_fence->base)) + return NULL; + } + rcu_read_lock(); + fence = dma_fence_get_rcu(&wait_pt->base.base); + rcu_read_unlock(); + return fence; + } + return NULL; +} + /** * drm_syncobj_find_fence - lookup and reference the fence in a sync object * @file_private: drm file private pointer * @handle: sync object handle to lookup. * @fence: out parameter for the fence + * @point: timeline point * * This is just a convenience function that combines drm_syncobj_find() and * drm_syncobj_fence_get(). @@ -221,7 +544,8 @@ static int drm_syncobj_assign_null_handle(struct drm_syncobj *syncobj) */ int drm_syncobj_find_fence(struct drm_file *file_private, u32 handle, - struct dma_fence **fence) + struct dma_fence **fence, + u64 point) { struct drm_syncobj *syncobj = drm_syncobj_find(file_private, handle); int ret = 0; @@ -229,7 +553,15 @@ int drm_syncobj_find_fence(struct drm_file *file_private, if (!syncobj) return -ENOENT; - *fence = drm_syncobj_fence_get(syncobj); + if (syncobj->type == DRM_SYNCOBJ_TYPE_NORMAL) { + *fence = drm_syncobj_fence_get(syncobj); + }else if (syncobj->type == DRM_SYNCOBJ_TYPE_TIMELINE) { + *fence = drm_syncobj_timeline_point_get(syncobj, point, + DRM_SYNCOBJ_WAIT_FLAGS_WAIT_FOR_SUBMIT); + } else { + DRM_ERROR("invalid syncobj type\n"); + return -EINVAL; + } if (!*fence) { ret = -EINVAL; } @@ -238,6 +570,35 @@ int drm_syncobj_find_fence(struct drm_file *file_private, } EXPORT_SYMBOL(drm_syncobj_find_fence); +static void drm_syncobj_timeline_fini(struct drm_syncobj *syncobj, + struct drm_syncobj_timeline *syncobj_timeline) +{ + struct rb_node *node = NULL; + struct drm_syncobj_wait_pt *wait_pt = NULL; + struct drm_syncobj_signal_pt *signal_pt = NULL, *tmp; + + spin_lock(&syncobj->lock); + for(node = rb_first(&syncobj_timeline->wait_pt_tree); + node != NULL; ) { + wait_pt = rb_entry(node, struct drm_syncobj_wait_pt, node); + node = rb_next(node); + rb_erase(&wait_pt->node, + &syncobj_timeline->wait_pt_tree); + RB_CLEAR_NODE(&wait_pt->node); + if (wait_pt->submission_fence) + dma_fence_put(&wait_pt->submission_fence->base); + /* kfree(wait_pt) is excuted by fence put */ + dma_fence_put(&wait_pt->base.base); + } + list_for_each_entry_safe(signal_pt, tmp, + &syncobj_timeline->signal_pt_list, list) { + list_del(&signal_pt->list); + dma_fence_put(signal_pt->signal_fence); + dma_fence_put(signal_pt->pre_pt_base); + dma_fence_put(&signal_pt->base.base); + } + spin_unlock(&syncobj->lock); +} /** * drm_syncobj_free - free a sync object. * @kref: kref to free. @@ -250,10 +611,22 @@ void drm_syncobj_free(struct kref *kref) struct drm_syncobj, refcount); drm_syncobj_replace_fence(syncobj, NULL); + drm_syncobj_timeline_fini(syncobj, &syncobj->syncobj_timeline); kfree(syncobj); } EXPORT_SYMBOL(drm_syncobj_free); +static void drm_syncobj_timeline_init(struct drm_syncobj_timeline + *syncobj_timeline) +{ + syncobj_timeline->timeline_context = dma_fence_context_alloc(1); + syncobj_timeline->submission_context = dma_fence_context_alloc(1); + syncobj_timeline->timeline = 0; + syncobj_timeline->signal_point = 0; + + syncobj_timeline->wait_pt_tree = RB_ROOT; + INIT_LIST_HEAD(&syncobj_timeline->signal_pt_list); +} /** * drm_syncobj_create - create a new syncobj * @out_syncobj: returned syncobj @@ -279,6 +652,12 @@ int drm_syncobj_create(struct drm_syncobj **out_syncobj, uint32_t flags, kref_init(&syncobj->refcount); INIT_LIST_HEAD(&syncobj->cb_list); spin_lock_init(&syncobj->lock); + if (flags & DRM_SYNCOBJ_CREATE_TYPE_TIMELINE) { + syncobj->type = DRM_SYNCOBJ_TYPE_TIMELINE; + drm_syncobj_timeline_init(&syncobj->syncobj_timeline); + } else { + syncobj->type = DRM_SYNCOBJ_TYPE_NORMAL; + } if (flags & DRM_SYNCOBJ_CREATE_SIGNALED) { ret = drm_syncobj_assign_null_handle(syncobj); @@ -491,7 +870,7 @@ static int drm_syncobj_export_sync_file(struct drm_file *file_private, if (fd < 0) return fd; - ret = drm_syncobj_find_fence(file_private, handle, &fence); + ret = drm_syncobj_find_fence(file_private, handle, &fence, 0); if (ret) goto err_put_fd; diff --git a/drivers/gpu/drm/v3d/v3d_gem.c b/drivers/gpu/drm/v3d/v3d_gem.c index 9029590267aa..f24c3eccb4d5 100644 --- a/drivers/gpu/drm/v3d/v3d_gem.c +++ b/drivers/gpu/drm/v3d/v3d_gem.c @@ -521,12 +521,12 @@ v3d_submit_cl_ioctl(struct drm_device *dev, void *data, kref_init(&exec->refcount); ret = drm_syncobj_find_fence(file_priv, args->in_sync_bcl, - &exec->bin.in_fence); + &exec->bin.in_fence, 0); if (ret == -EINVAL) goto fail; ret = drm_syncobj_find_fence(file_priv, args->in_sync_rcl, - &exec->render.in_fence); + &exec->render.in_fence, 0); if (ret == -EINVAL) goto fail; diff --git a/drivers/gpu/drm/vc4/vc4_gem.c b/drivers/gpu/drm/vc4/vc4_gem.c index 7910b9acedd6..f7b4971342e8 100644 --- a/drivers/gpu/drm/vc4/vc4_gem.c +++ b/drivers/gpu/drm/vc4/vc4_gem.c @@ -1173,7 +1173,7 @@ vc4_submit_cl_ioctl(struct drm_device *dev, void *data, if (args->in_sync) { ret = drm_syncobj_find_fence(file_priv, args->in_sync, - &in_fence); + &in_fence, 0); if (ret) goto fail; diff --git a/include/drm/drm_syncobj.h b/include/drm/drm_syncobj.h index b04c492ddbb5..b6e193c6cc9a 100644 --- a/include/drm/drm_syncobj.h +++ b/include/drm/drm_syncobj.h @@ -54,6 +54,41 @@ const struct dma_fence_ops drm_syncobj_stub_fence_ops = { .release = NULL, }; +enum drm_syncobj_type { + DRM_SYNCOBJ_TYPE_NORMAL, + DRM_SYNCOBJ_TYPE_TIMELINE +}; + +struct drm_syncobj; +struct drm_syncobj_wait_pt { + struct drm_syncobj_stub_fence base; + struct drm_syncobj_stub_fence *submission_fence; + u64 value; + struct rb_node node; +}; +struct drm_syncobj_signal_pt { + struct drm_syncobj_stub_fence base; + struct dma_fence *signal_fence; + struct dma_fence *pre_pt_base; + struct dma_fence_cb signal_cb; + struct dma_fence_cb pre_pt_cb; + struct drm_syncobj *syncobj; + u64 value; + struct list_head list; +}; +struct drm_syncobj_timeline { + u64 timeline_context; + u64 submission_context; + /** + * @timeline: syncobj timeline + */ + u64 timeline; + u64 signal_point; + + + struct rb_root wait_pt_tree; + struct list_head signal_pt_list; +}; /** * struct drm_syncobj - sync object. * @@ -64,6 +99,11 @@ struct drm_syncobj { * @refcount: Reference count of this object. */ struct kref refcount; + /** + * @type: indicate syncobj type + */ + enum drm_syncobj_type type; + struct drm_syncobj_timeline syncobj_timeline; /** * @fence: * NULL or a pointer to the fence bound to this object. @@ -162,9 +202,12 @@ void drm_syncobj_remove_callback(struct drm_syncobj *syncobj, struct drm_syncobj_cb *cb); void drm_syncobj_replace_fence(struct drm_syncobj *syncobj, struct dma_fence *fence); +void drm_syncobj_signal_fence(struct drm_syncobj *syncobj, + struct dma_fence *fence, + u64 point); int drm_syncobj_find_fence(struct drm_file *file_private, u32 handle, - struct dma_fence **fence); + struct dma_fence **fence, u64 point); void drm_syncobj_free(struct kref *kref); int drm_syncobj_create(struct drm_syncobj **out_syncobj, uint32_t flags, struct dma_fence *fence); diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h index 300f336633f2..71e6cd1c88f8 100644 --- a/include/uapi/drm/drm.h +++ b/include/uapi/drm/drm.h @@ -717,6 +717,8 @@ struct drm_prime_handle { struct drm_syncobj_create { __u32 handle; #define DRM_SYNCOBJ_CREATE_SIGNALED (1 << 0) +#define DRM_SYNCOBJ_CREATE_TYPE_NORMAL (1 << 1) +#define DRM_SYNCOBJ_CREATE_TYPE_TIMELINE (1 << 2) __u32 flags; }; @@ -734,7 +736,6 @@ struct drm_syncobj_handle { __s32 fd; __u32 pad; }; - #define DRM_SYNCOBJ_WAIT_FLAGS_WAIT_ALL (1 << 0) #define DRM_SYNCOBJ_WAIT_FLAGS_WAIT_FOR_SUBMIT (1 << 1) struct drm_syncobj_wait {