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 X-Spam-Level: X-Spam-Status: No, score=-3.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 80640C47082 for ; Wed, 26 May 2021 13:52:07 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 45616610A6 for ; Wed, 26 May 2021 13:52:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 45616610A6 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ffwll.ch Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=intel-gfx-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id CCB5B6ECE9; Wed, 26 May 2021 13:51:59 +0000 (UTC) Received: from mail-ot1-x32f.google.com (mail-ot1-x32f.google.com [IPv6:2607:f8b0:4864:20::32f]) by gabe.freedesktop.org (Postfix) with ESMTPS id E9C876E500 for ; Wed, 26 May 2021 13:51:57 +0000 (UTC) Received: by mail-ot1-x32f.google.com with SMTP id u25-20020a0568302319b02902ac3d54c25eso1074316ote.1 for ; Wed, 26 May 2021 06:51:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=NQQ2QlKG8USraZgLg7Z7CLDTbkvH3Ja6tLYSMEaPvoE=; b=cCraq7xvMjGG//smEDF3crqdU4HWsYQchD8qCi2BqMEfFuW1sGYQI0dgAxXIx/Ie4M qFLXn09zsV4A7bWzmhslzvMwlVm++gaBpU5jmtv6lBphC3GrunX5x3rUQttT4iWQdbEN q0NTxpxz8XnUY41hdMRL+Dnb0npyp1q8FAiTg= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=NQQ2QlKG8USraZgLg7Z7CLDTbkvH3Ja6tLYSMEaPvoE=; b=EkYsZ9HNc00s02hIGgaEsqGp5Uxim9mB+UHzEnwJdC+IW6ssaWgX0Z97tt3azmY/p/ tFwYC4twkq5SAnI+HgnYdq5jsb+G0FrsC8AeV1bmEJ6OP1Fz7mKFVDbrLi5oqTjsj1Dg vTsVPVDbY5njZuSRMvVCJ6CBbbsZHmp1yiPkOPRiB9SAShMcFScvL6WlNfObHrEaFy/Q WeTYFlHzeuyvJA63cULlqKuaJ7zjtDh3zz2W+wgbfwVl465ge8hEGFylZWWfZwhsbqxV Q9dZxCx3wbDksyyu0kgEtyTKTMjb0Iz0MxSvsTcGup7p2Y382iKnclAQdcePwkcOPxvL lOjA== X-Gm-Message-State: AOAM533CPMQnIv3q+sM16Qy2X1Mq+Ko2jJ4BThMmueZIILxWxNtWVpdt KT4qzJ/ypUSwRwoZEjpCJo8Jg+quTj/YQGL+/Ta3nQ== X-Google-Smtp-Source: ABdhPJw/zQ1FWrTEAmLzxtyN95lNd7ehO2fL/e4d75AeL/xc+dw8P3f31yEV6YEbV6cEvXkyTvF1e7adV89JJh1vh7g= X-Received: by 2002:a05:6830:1155:: with SMTP id x21mr2405346otq.303.1622037116909; Wed, 26 May 2021 06:51:56 -0700 (PDT) MIME-Version: 1.0 References: <20210521090959.1663703-1-daniel.vetter@ffwll.ch> <31467363-e936-879b-8b0d-5a2a92644221@gmail.com> <312b183f-8eb2-d9e2-24ad-b429b74c886c@gmail.com> In-Reply-To: From: Daniel Vetter Date: Wed, 26 May 2021 15:51:45 +0200 Message-ID: To: =?UTF-8?Q?Christian_K=C3=B6nig?= Subject: Re: [Intel-gfx] [Mesa-dev] [PATCH 01/11] drm/amdgpu: Comply with implicit fencing rules X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Rob Clark , Daniel Stone , =?UTF-8?Q?Michel_D=C3=A4nzer?= , Intel Graphics Development , Kevin Wang , DRI Development , Sumit Semwal , "moderated list:DMA BUFFER SHARING FRAMEWORK" , Luben Tuikov , "Kristian H . Kristensen" , Chen Li , Bas Nieuwenhuizen , Alex Deucher , Daniel Vetter , mesa-dev , =?UTF-8?Q?Christian_K=C3=B6nig?= , Dennis Li , Deepak R Varma Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" T24gV2VkLCBNYXkgMjYsIDIwMjEgYXQgMzozMiBQTSBDaHJpc3RpYW4gS8O2bmlnCjxja29lbmln LmxlaWNodHp1bWVya2VuQGdtYWlsLmNvbT4gd3JvdGU6Cj4KPiBBbSAyNS4wNS4yMSB1bSAxNzoy MyBzY2hyaWViIERhbmllbCBWZXR0ZXI6Cj4gPiBPbiBUdWUsIE1heSAyNSwgMjAyMSBhdCA1OjA1 IFBNIENocmlzdGlhbiBLw7ZuaWcKPiA+IDxja29lbmlnLmxlaWNodHp1bWVya2VuQGdtYWlsLmNv bT4gd3JvdGU6Cj4gPj4gSGkgRGFuaWVsLAo+ID4+Cj4gPj4gQW0gMjUuMDUuMjEgdW0gMTU6MDUg c2NocmllYiBEYW5pZWwgVmV0dGVyOgo+ID4+PiBIaSBDaHJpc3RpYW4sCj4gPj4+Cj4gPj4+IE9u IFNhdCwgTWF5IDIyLCAyMDIxIGF0IDEwOjMwOjE5QU0gKzAyMDAsIENocmlzdGlhbiBLw7ZuaWcg d3JvdGU6Cj4gPj4+PiBBbSAyMS4wNS4yMSB1bSAyMDozMSBzY2hyaWViIERhbmllbCBWZXR0ZXI6 Cj4gPj4+PiBUaGlzIHdvcmtzIGJ5IGFkZGluZyB0aGUgZmVuY2Ugb2YgdGhlIGxhc3QgZXZpY3Rp b24gRE1BIG9wZXJhdGlvbiB0byBCT3MKPiA+Pj4+IHdoZW4gdGhlaXIgYmFja2luZyBzdG9yZSBp cyBuZXdseSBhbGxvY2F0ZWQuIFRoYXQncyB3aGF0IHRoZQo+ID4+Pj4gdHRtX2JvX2FkZF9tb3Zl X2ZlbmNlKCkgZnVuY3Rpb24geW91IHN0dW1ibGVkIG92ZXIgaXMgZ29vZCBmb3I6IGh0dHBzOi8v ZWxpeGlyLmJvb3RsaW4uY29tL2xpbnV4L3Y1LjEzLXJjMi9zb3VyY2UvZHJpdmVycy9ncHUvZHJt L3R0bS90dG1fYm8uYyNMNjkyCj4gPj4+Pgo+ID4+Pj4gTm93IHRoZSBwcm9ibGVtIGlzIGl0IGlz IHBvc3NpYmxlIHRoYXQgdGhlIGFwcGxpY2F0aW9uIGlzIHRlcm1pbmF0ZWQgYmVmb3JlCj4gPj4+ PiBpdCBjYW4gY29tcGxldGUgaXQncyBjb21tYW5kIHN1Ym1pc3Npb24uIEJ1dCBzaW5jZSByZXNv dXJjZSBtYW5hZ2VtZW50IG9ubHkKPiA+Pj4+IHdhaXRzIGZvciB0aGUgc2hhcmVkIGZlbmNlcyB3 aGVuIHRoZXJlIGFyZSBzb21lIHRoZXJlIGlzIGEgY2hhbmNlIHRoYXQgd2UKPiA+Pj4+IGZyZWUg dXAgbWVtb3J5IHdoaWxlIGl0IGlzIHN0aWxsIGluIHVzZS4KPiA+Pj4gSG0gd2hlcmUgaXMgdGhp cyBjb2RlPyBXb3VsZCBoZWxwIGluIG15IGF1ZGl0IHRoYXQgSSB3YW50ZWQgdG8gZG8gdGhpcwo+ ID4+PiB3ZWVrPyBJZiB5b3UgbG9vayBhdCBtb3N0IG90aGVyIHBsYWNlcyBsaWtlCj4gPj4+IGRy bV9nZW1fZmVuY2VfYXJyYXlfYWRkX2ltcGxpY2l0KCkgSSBtZW50aW9uZWQgZWFybGllciwgdGhl biB3ZSBkb24ndAo+ID4+PiB0cmVhdCB0aGUgc2hhcmVkIGZlbmNlcyBzcGVjaWFsIGFuZCBhbHdh eXMgYWxzbyBpbmNsdWRlIHRoZSBleGNsdXNpdmUgb25lLgo+ID4+IFNlZSBhbWRncHVfZ2VtX29i amVjdF9jbG9zZSgpOgo+ID4+Cj4gPj4gLi4uCj4gPj4gICAgICAgICAgIGZlbmNlID0gZG1hX3Jl c3ZfZ2V0X2V4Y2woYm8tPnRiby5iYXNlLnJlc3YpOwo+ID4+ICAgICAgICAgICBpZiAoZmVuY2Up IHsKPiA+PiAgICAgICAgICAgICAgICAgICBhbWRncHVfYm9fZmVuY2UoYm8sIGZlbmNlLCB0cnVl KTsKPiA+PiAgICAgICAgICAgICAgICAgICBmZW5jZSA9IE5VTEw7Cj4gPj4gICAgICAgICAgIH0K PiA+PiAuLi4KPiA+Pgo+ID4+IFdlIGV4cGxpY2l0bHkgYWRkZWQgdGhhdCBiZWNhdXNlIHJlc291 cmNlIG1hbmFnZW1lbnQgb2Ygc29tZSBvdGhlcgo+ID4+IGRyaXZlciB3YXMgZ29pbmcgdG90YWxs eSBiYW5hbmFzIHdpdGhvdXQgdGhhdC4KPiA+Pgo+ID4+IEJ1dCBJJ20gbm90IHN1cmUgd2hpY2gg b25lIHRoYXQgd2FzLiBNYXliZSBkaWcgYSBiaXQgaW4gdGhlIGdpdCBhbmQKPiA+PiBtYWlsaW5n IGhpc3Rvcnkgb2YgdGhhdC4KPiA+IEhtIEkgbG9va2VkIGFuZCBpdCdzCj4gPgo+ID4gY29tbWl0 IDgyYzQxNmIxM2NiN2QyMmI5NmVjMDg4OGIyOTZhNDhkZmY4YTA5ZWIKPiA+IEF1dGhvcjogQ2hy aXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFuLmtvZW5pZ0BhbWQuY29tPgo+ID4gRGF0ZTogICBUaHUg TWFyIDEyIDEyOjAzOjM0IDIwMjAgKzAxMDAKPiA+Cj4gPiAgICAgZHJtL2FtZGdwdTogZml4IGFu ZCBjbGVhbnVwIGFtZGdwdV9nZW1fb2JqZWN0X2Nsb3NlIHY0Cj4gPgo+ID4gVGhhdCBzb3VuZGVk IG1vcmUgbGlrZSBhbWRncHUgaXRzZWxmIG5lZWRpbmcgdGhpcywgbm90IGFub3RoZXIgZHJpdmVy Pwo+Cj4gTm8sIHRoYXQgcGF0Y2ggd2FzIGp1c3QgYSBmb2xsb3cgdXAgbW92aW5nIHRoZSBmdW5j dGlvbmFsaXR5IGFyb3VuZC4KClRoYXQgcGF0Y2ggYWRkZWQgdGhlICJhZGQgZXhjbHVzaXZlIGZl bmNlIHRvIHNoYXJlZCBzbG90cyBiZWZvcmUKYW1kZ3B1X3ZtX2NsZWFyX2ZyZWVkKCkgY2FsbCIs IHdoaWNoIEkgdGhvdWdodCB3YXMgYXQgbGVhc3QgcGFydCBvZgp5b3VyIGZpeC4KCj4gPiBCdXQg bG9va2luZyBhdCBhbWRncHVfdm1fYm9fdXBkYXRlX21hcHBpbmcoKSBpdCBzZWVtcyB0byBwaWNr IHRoZQo+ID4gcmlnaHQgZmVuY2luZyBtb2RlIGZvciBncHUgcHRlIGNsZWFyaW5nLCBzbyBJJ20g cmVhbGx5IG5vdCBzdXJlIHdoYXQKPiA+IHRoZSBidWcgd2FzIHRoYXQgeW91IHdvcmtlZCBhcm91 bmQgaGVyZT9UaGUgaW1wbGVtZW50YXRpb24gYm9pbHMgZG93bgo+ID4gdG8gYW1kZ3B1X3N5bmNf cmVzdigpIHdoaWNoIHN5bmNzIGZvciB0aGUgZXhjbHVzaXZlIGZlbmNlLCBhbHdheXMuIEFuZAo+ ID4gdGhlcmUncyBub3RoaW5nIGVsc2UgdGhhdCBJIGNvdWxkIGZpbmQgaW4gcHVibGljIGhpc3Rv cnkgYXQgbGVhc3QsIG5vCj4gPiByZWZlcmVuY2VzIHRvIGJ1ZyByZXBvcnRzIG9yIGFueXRoaW5n LiBJIHRoaW5rIHlvdSBuZWVkIHRvIGRpZwo+ID4gaW50ZXJuYWxseSwgYmVjYXVzZSBhcy1pcyBJ J20gbm90IHNlZWluZyB0aGUgcHJvYmxlbSBoZXJlLgo+ID4KPiA+IE9yIGFtIEkgbWlzc2luZyBz b21ldGhpbmcgaGVyZT8KPgo+IFNlZSB0aGUgY29kZSBoZXJlIGZvciBleGFtcGxlOgo+IGh0dHBz Oi8vZWxpeGlyLmJvb3RsaW4uY29tL2xpbnV4L2xhdGVzdC9zb3VyY2UvZHJpdmVycy9ncHUvZHJt L25vdXZlYXUvbm91dmVhdV9mZW5jZS5jI0wzNjEKPgo+IE5vdXZlYXUgYXNzdW1lcyB0aGF0IHdo ZW4gYSBzaGFyZWQgZmVuY2UgaXMgcHJlc2VudCBpdCBkb2Vzbid0IG5lZWQgdG8KPiB3YWl0IGZv ciB0aGUgZXhjbHVzaXZlIG9uZSBiZWNhdXNlIHRoZSBzaGFyZWQgYXJlIGFsd2F5cyBzdXBwb3Nl ZCB0bwo+IGZpbmlzaCBhZnRlciB0aGUgZXhjbHVzaXZlIG9uZS4KPgo+IEJ1dCBmb3IgcGFnZSB0 YWJsZSB1bm1hcCBmZW5jZXMgdGhhdCBpc24ndCB0cnVlIGFuZCB3ZSByYW4gaW50byBhIHJlYWxs eQo+IG5hc3R5IGFuZCBoYXJkIHRvIHJlcHJvZHVjZSBidWcgYmVjYXVzZSBvZiB0aGlzLgo+Cj4g SSB0aGluayBpdCB3b3VsZCBiZSBtdWNoIG1vcmUgZGVmZW5zaXZlIGlmIHdlIGNvdWxkIHNheSB0 aGF0IHdlIGFsd2F5cwo+IHdhaXQgZm9yIHRoZSBleGNsdXNpdmUgZmVuY2UgYW5kIGZpeCB0aGUg dXNlIGNhc2UgaW4gbm91dmVhdSBhbmQgZG91YmxlCj4gY2hlY2sgaWYgc29tZWJvZHkgZWxzZSBk b2VzIHN0dWZmIGxpa2UgdGhhdCBhcyB3ZWxsLgoKWWVhaCBtb3N0IG90aGVyIGRyaXZlcnMgZG8g dGhlIGRlZmVuc2l2ZSB0aGluZyBoZXJlLiBJIHRoaW5rIGl0IHdvdWxkCmJlIGdvb2QgdG8gc3Rh bmRhcmRpemUgb24gdGhhdC4gSSdsbCBhZGQgdGhhdCB0byBteSBsaXN0IGFuZCBkbyBtb3JlCmF1 ZGl0aW5nLgotRGFuaWVsCi0tIApEYW5pZWwgVmV0dGVyClNvZnR3YXJlIEVuZ2luZWVyLCBJbnRl bCBDb3Jwb3JhdGlvbgpodHRwOi8vYmxvZy5mZndsbC5jaApfX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdm eEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFp bG1hbi9saXN0aW5mby9pbnRlbC1nZngK 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 X-Spam-Level: X-Spam-Status: No, score=-3.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 626ABC47082 for ; Wed, 26 May 2021 13:52:00 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 297D3611C9 for ; Wed, 26 May 2021 13:52:00 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 297D3611C9 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ffwll.ch Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 4CBE16E500; Wed, 26 May 2021 13:51:59 +0000 (UTC) Received: from mail-ot1-x330.google.com (mail-ot1-x330.google.com [IPv6:2607:f8b0:4864:20::330]) by gabe.freedesktop.org (Postfix) with ESMTPS id EB3C86ECD2 for ; Wed, 26 May 2021 13:51:57 +0000 (UTC) Received: by mail-ot1-x330.google.com with SMTP id 66-20020a9d02c80000b02903615edf7c1aso1030187otl.13 for ; Wed, 26 May 2021 06:51:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=NQQ2QlKG8USraZgLg7Z7CLDTbkvH3Ja6tLYSMEaPvoE=; b=cCraq7xvMjGG//smEDF3crqdU4HWsYQchD8qCi2BqMEfFuW1sGYQI0dgAxXIx/Ie4M qFLXn09zsV4A7bWzmhslzvMwlVm++gaBpU5jmtv6lBphC3GrunX5x3rUQttT4iWQdbEN q0NTxpxz8XnUY41hdMRL+Dnb0npyp1q8FAiTg= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=NQQ2QlKG8USraZgLg7Z7CLDTbkvH3Ja6tLYSMEaPvoE=; b=g/l2QPx93DE2I+U2W3I5pUF+LLyy5BmRwMwmQ4S+F8WojicyRwPH4D7vfK6xr6pzN9 YIkZxl8FS5jj9UmxlICMn6/237wUIFxxjtep8fD0eHFQ60X39i1l3HvMrg+KEGYf0wyS pI0qbV1bI7aFrHwJ7I9HIkMuLr3VTFMZdyxy0m5I7jwweRFzJ2RjE4QETP0cqFNPvAk8 SXHYWDHtIQ53ZMW+l9Xir0MoWwck3Ts4/o0KzLqRV8aGail0JOFFJrH2MuDozpZRmeSA hgywF+/YozzYnaWu296O41SnlLOl4PnA9wAS/YuifrEo+eRQjg3xBV5EpYBO/VFi6gE5 lZYA== X-Gm-Message-State: AOAM531uOgL8XxUgTPUjKHrdC6rsDIHkfEmBiU8ODRib2xIMqyZGxlnn MMZhKhUoqVgpQE5EKNHFj6IrFU6HSiiDdztDNxNU0A== X-Google-Smtp-Source: ABdhPJw/zQ1FWrTEAmLzxtyN95lNd7ehO2fL/e4d75AeL/xc+dw8P3f31yEV6YEbV6cEvXkyTvF1e7adV89JJh1vh7g= X-Received: by 2002:a05:6830:1155:: with SMTP id x21mr2405346otq.303.1622037116909; Wed, 26 May 2021 06:51:56 -0700 (PDT) MIME-Version: 1.0 References: <20210521090959.1663703-1-daniel.vetter@ffwll.ch> <31467363-e936-879b-8b0d-5a2a92644221@gmail.com> <312b183f-8eb2-d9e2-24ad-b429b74c886c@gmail.com> In-Reply-To: From: Daniel Vetter Date: Wed, 26 May 2021 15:51:45 +0200 Message-ID: Subject: Re: [Mesa-dev] [PATCH 01/11] drm/amdgpu: Comply with implicit fencing rules To: =?UTF-8?Q?Christian_K=C3=B6nig?= Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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: Rob Clark , Daniel Stone , =?UTF-8?Q?Michel_D=C3=A4nzer?= , Intel Graphics Development , Kevin Wang , DRI Development , "moderated list:DMA BUFFER SHARING FRAMEWORK" , Luben Tuikov , "Kristian H . Kristensen" , Chen Li , Alex Deucher , Daniel Vetter , mesa-dev , =?UTF-8?Q?Christian_K=C3=B6nig?= , Dennis Li , Deepak R Varma Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Wed, May 26, 2021 at 3:32 PM Christian K=C3=B6nig wrote: > > Am 25.05.21 um 17:23 schrieb Daniel Vetter: > > On Tue, May 25, 2021 at 5:05 PM Christian K=C3=B6nig > > wrote: > >> Hi Daniel, > >> > >> Am 25.05.21 um 15:05 schrieb Daniel Vetter: > >>> Hi Christian, > >>> > >>> On Sat, May 22, 2021 at 10:30:19AM +0200, Christian K=C3=B6nig wrote: > >>>> Am 21.05.21 um 20:31 schrieb Daniel Vetter: > >>>> This works by adding the fence of the last eviction DMA operation to= BOs > >>>> when their backing store is newly allocated. That's what the > >>>> ttm_bo_add_move_fence() function you stumbled over is good for: http= s://elixir.bootlin.com/linux/v5.13-rc2/source/drivers/gpu/drm/ttm/ttm_bo.c#= L692 > >>>> > >>>> Now the problem is it is possible that the application is terminated= before > >>>> it can complete it's command submission. But since resource manageme= nt only > >>>> waits for the shared fences when there are some there is a chance th= at we > >>>> free up memory while it is still in use. > >>> Hm where is this code? Would help in my audit that I wanted to do thi= s > >>> week? If you look at most other places like > >>> drm_gem_fence_array_add_implicit() I mentioned earlier, then we don't > >>> treat the shared fences special and always also include the exclusive= one. > >> See amdgpu_gem_object_close(): > >> > >> ... > >> fence =3D dma_resv_get_excl(bo->tbo.base.resv); > >> if (fence) { > >> amdgpu_bo_fence(bo, fence, true); > >> fence =3D NULL; > >> } > >> ... > >> > >> We explicitly added that because resource management of some other > >> driver was going totally bananas without that. > >> > >> But I'm not sure which one that was. Maybe dig a bit in the git and > >> mailing history of that. > > Hm I looked and it's > > > > commit 82c416b13cb7d22b96ec0888b296a48dff8a09eb > > Author: Christian K=C3=B6nig > > Date: Thu Mar 12 12:03:34 2020 +0100 > > > > drm/amdgpu: fix and cleanup amdgpu_gem_object_close v4 > > > > That sounded more like amdgpu itself needing this, not another driver? > > No, that patch was just a follow up moving the functionality around. That patch added the "add exclusive fence to shared slots before amdgpu_vm_clear_freed() call", which I thought was at least part of your fix. > > But looking at amdgpu_vm_bo_update_mapping() it seems to pick the > > right fencing mode for gpu pte clearing, so I'm really not sure what > > the bug was that you worked around here?The implementation boils down > > to amdgpu_sync_resv() which syncs for the exclusive fence, always. And > > there's nothing else that I could find in public history at least, no > > references to bug reports or anything. I think you need to dig > > internally, because as-is I'm not seeing the problem here. > > > > Or am I missing something here? > > See the code here for example: > https://elixir.bootlin.com/linux/latest/source/drivers/gpu/drm/nouveau/no= uveau_fence.c#L361 > > Nouveau assumes that when a shared fence is present it doesn't need to > wait for the exclusive one because the shared are always supposed to > finish after the exclusive one. > > But for page table unmap fences that isn't true and we ran into a really > nasty and hard to reproduce bug because of this. > > I think it would be much more defensive if we could say that we always > wait for the exclusive fence and fix the use case in nouveau and double > check if somebody else does stuff like that as well. Yeah most other drivers do the defensive thing here. I think it would be good to standardize on that. I'll add that to my list and do more auditing. -Daniel --=20 Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch