From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH 1/9] dma-buf: start caching of sg_table objects Date: Wed, 8 May 2019 14:00:23 +0200 Message-ID: <829e1109-c211-cb4f-e56e-8be78ae51416@gmail.com> References: <20190507081338.2571-1-christian.koenig@amd.com> <20190508091533.GU17751@phenom.ffwll.local> <20190508101149.GX17751@phenom.ffwll.local> Reply-To: christian.koenig@amd.com Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wm1-x342.google.com (mail-wm1-x342.google.com [IPv6:2a00:1450:4864:20::342]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5AB7089236 for ; Wed, 8 May 2019 12:00:33 +0000 (UTC) Received: by mail-wm1-x342.google.com with SMTP id s18so1775090wmh.0 for ; Wed, 08 May 2019 05:00:33 -0700 (PDT) In-Reply-To: <20190508101149.GX17751@phenom.ffwll.local> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org QW0gMDguMDUuMTkgdW0gMTI6MTEgc2NocmllYiBEYW5pZWwgVmV0dGVyOgo+IE9uIFdlZCwgTWF5 IDA4LCAyMDE5IGF0IDExOjE1OjMzQU0gKzAyMDAsIERhbmllbCBWZXR0ZXIgd3JvdGU6Cj4+IE9u IFR1ZSwgTWF5IDA3LCAyMDE5IGF0IDEwOjEzOjMwQU0gKzAyMDAsIENocmlzdGlhbiBLw7ZuaWcg d3JvdGU6Cj4+PiBUbyBhbGxvdyBhIHNtb290aCB0cmFuc2l0aW9uIGZyb20gcGlubmluZyBidWZm ZXIgb2JqZWN0cyB0byBkeW5hbWljCj4+PiBpbnZhbGlkYXRpb24gd2UgZmlyc3Qgc3RhcnQgdG8g Y2FjaGUgdGhlIHNnX3RhYmxlIGZvciBhbiBhdHRhY2htZW50Lgo+Pj4KPj4+IFNpZ25lZC1vZmYt Ynk6IENocmlzdGlhbiBLw7ZuaWcgPGNocmlzdGlhbi5rb2VuaWdAYW1kLmNvbT4KPj4+IC0tLQo+ Pj4gICBkcml2ZXJzL2RtYS1idWYvZG1hLWJ1Zi5jIHwgMjQgKysrKysrKysrKysrKysrKysrKysr KysrCj4+PiAgIGluY2x1ZGUvbGludXgvZG1hLWJ1Zi5oICAgfCAxNCArKysrKysrKysrKysrKwo+ Pj4gICAyIGZpbGVzIGNoYW5nZWQsIDM4IGluc2VydGlvbnMoKykKPj4+Cj4+PiBkaWZmIC0tZ2l0 IGEvZHJpdmVycy9kbWEtYnVmL2RtYS1idWYuYyBiL2RyaXZlcnMvZG1hLWJ1Zi9kbWEtYnVmLmMK Pj4+IGluZGV4IDdjODU4MDIwZDE0Yi4uNzc1ZTEzZjU0MDgzIDEwMDY0NAo+Pj4gLS0tIGEvZHJp dmVycy9kbWEtYnVmL2RtYS1idWYuYwo+Pj4gKysrIGIvZHJpdmVycy9kbWEtYnVmL2RtYS1idWYu Ywo+Pj4gQEAgLTU3Myw2ICs1NzMsMjAgQEAgc3RydWN0IGRtYV9idWZfYXR0YWNobWVudCAqZG1h X2J1Zl9hdHRhY2goc3RydWN0IGRtYV9idWYgKmRtYWJ1ZiwKPj4+ICAgCWxpc3RfYWRkKCZhdHRh Y2gtPm5vZGUsICZkbWFidWYtPmF0dGFjaG1lbnRzKTsKPj4+ICAgCj4+PiAgIAltdXRleF91bmxv Y2soJmRtYWJ1Zi0+bG9jayk7Cj4+PiArCj4+PiArCWlmICghZG1hX2J1Zl9pc19keW5hbWljKGRt YWJ1ZikpIHsKPj4+ICsJCXN0cnVjdCBzZ190YWJsZSAqc2d0Owo+Pj4gKwo+Pj4gKwkJc2d0ID0g ZG1hYnVmLT5vcHMtPm1hcF9kbWFfYnVmKGF0dGFjaCwgRE1BX0JJRElSRUNUSU9OQUwpOwo+Pj4g KwkJaWYgKCFzZ3QpCj4+PiArCQkJc2d0ID0gRVJSX1BUUigtRU5PTUVNKTsKPj4+ICsJCWlmIChJ U19FUlIoc2d0KSkgewo+Pj4gKwkJCWRtYV9idWZfZGV0YWNoKGRtYWJ1ZiwgYXR0YWNoKTsKPj4+ ICsJCQlyZXR1cm4gRVJSX0NBU1Qoc2d0KTsKPj4+ICsJCX0KPj4+ICsJCWF0dGFjaC0+c2d0ID0g c2d0Owo+Pj4gKwl9Cj4+PiArCj4+PiAgIAlyZXR1cm4gYXR0YWNoOwo+Pj4gICAKPj4+ICAgZXJy X2F0dGFjaDoKPj4+IEBAIC01OTUsNiArNjA5LDEwIEBAIHZvaWQgZG1hX2J1Zl9kZXRhY2goc3Ry dWN0IGRtYV9idWYgKmRtYWJ1Ziwgc3RydWN0IGRtYV9idWZfYXR0YWNobWVudCAqYXR0YWNoKQo+ Pj4gICAJaWYgKFdBUk5fT04oIWRtYWJ1ZiB8fCAhYXR0YWNoKSkKPj4+ICAgCQlyZXR1cm47Cj4+ PiAgIAo+Pj4gKwlpZiAoYXR0YWNoLT5zZ3QpCj4+PiArCQlkbWFidWYtPm9wcy0+dW5tYXBfZG1h X2J1ZihhdHRhY2gsIGF0dGFjaC0+c2d0LAo+Pj4gKwkJCQkJICAgRE1BX0JJRElSRUNUSU9OQUwp Owo+Pj4gKwo+Pj4gICAJbXV0ZXhfbG9jaygmZG1hYnVmLT5sb2NrKTsKPj4+ICAgCWxpc3RfZGVs KCZhdHRhY2gtPm5vZGUpOwo+Pj4gICAJaWYgKGRtYWJ1Zi0+b3BzLT5kZXRhY2gpCj4+PiBAQCAt NjMwLDYgKzY0OCw5IEBAIHN0cnVjdCBzZ190YWJsZSAqZG1hX2J1Zl9tYXBfYXR0YWNobWVudChz dHJ1Y3QgZG1hX2J1Zl9hdHRhY2htZW50ICphdHRhY2gsCj4+PiAgIAlpZiAoV0FSTl9PTighYXR0 YWNoIHx8ICFhdHRhY2gtPmRtYWJ1ZikpCj4+PiAgIAkJcmV0dXJuIEVSUl9QVFIoLUVJTlZBTCk7 Cj4+PiAgIAo+Pj4gKwlpZiAoYXR0YWNoLT5zZ3QpCj4+PiArCQlyZXR1cm4gYXR0YWNoLT5zZ3Q7 Cj4+PiArCj4+PiAgIAlzZ190YWJsZSA9IGF0dGFjaC0+ZG1hYnVmLT5vcHMtPm1hcF9kbWFfYnVm KGF0dGFjaCwgZGlyZWN0aW9uKTsKPj4+ICAgCWlmICghc2dfdGFibGUpCj4+PiAgIAkJc2dfdGFi bGUgPSBFUlJfUFRSKC1FTk9NRU0pOwo+Pj4gQEAgLTY1Nyw2ICs2NzgsOSBAQCB2b2lkIGRtYV9i dWZfdW5tYXBfYXR0YWNobWVudChzdHJ1Y3QgZG1hX2J1Zl9hdHRhY2htZW50ICphdHRhY2gsCj4+ PiAgIAlpZiAoV0FSTl9PTighYXR0YWNoIHx8ICFhdHRhY2gtPmRtYWJ1ZiB8fCAhc2dfdGFibGUp KQo+Pj4gICAJCXJldHVybjsKPj4+ICAgCj4+PiArCWlmIChhdHRhY2gtPnNndCA9PSBzZ190YWJs ZSkKPj4+ICsJCXJldHVybjsKPj4+ICsKPj4+ICAgCWF0dGFjaC0+ZG1hYnVmLT5vcHMtPnVubWFw X2RtYV9idWYoYXR0YWNoLCBzZ190YWJsZSwKPj4+ICAgCQkJCQkJZGlyZWN0aW9uKTsKPj4+ICAg fQo+Pj4gZGlmZiAtLWdpdCBhL2luY2x1ZGUvbGludXgvZG1hLWJ1Zi5oIGIvaW5jbHVkZS9saW51 eC9kbWEtYnVmLmgKPj4+IGluZGV4IDU4NzI1Zjg5MGI1Yi4uNTIwMzFmZGM3NWJiIDEwMDY0NAo+ Pj4gLS0tIGEvaW5jbHVkZS9saW51eC9kbWEtYnVmLmgKPj4+ICsrKyBiL2luY2x1ZGUvbGludXgv ZG1hLWJ1Zi5oCj4+PiBAQCAtMzIyLDYgKzMyMiw3IEBAIHN0cnVjdCBkbWFfYnVmX2F0dGFjaG1l bnQgewo+Pj4gICAJc3RydWN0IGRtYV9idWYgKmRtYWJ1ZjsKPj4+ICAgCXN0cnVjdCBkZXZpY2Ug KmRldjsKPj4+ICAgCXN0cnVjdCBsaXN0X2hlYWQgbm9kZTsKPj4+ICsJc3RydWN0IHNnX3RhYmxl ICpzZ3Q7Cj4+PiAgIAl2b2lkICpwcml2Owo+Pj4gICB9Owo+Pj4gICAKPj4+IEBAIC0zNzMsNiAr Mzc0LDE5IEBAIHN0YXRpYyBpbmxpbmUgdm9pZCBnZXRfZG1hX2J1ZihzdHJ1Y3QgZG1hX2J1ZiAq ZG1hYnVmKQo+Pj4gICAJZ2V0X2ZpbGUoZG1hYnVmLT5maWxlKTsKPj4+ICAgfQo+Pj4gICAKPj4+ ICsvKioKPj4+ICsgKiBkbWFfYnVmX2lzX2R5bmFtaWMgLSBjaGVjayBpZiBhIERNQS1idWYgdXNl cyBkeW5hbWljIG1hcHBpbmdzLgo+Pj4gKyAqIEBkbWFidWY6IHRoZSBETUEtYnVmIHRvIGNoZWNr Cj4+PiArICoKPj4+ICsgKiBSZXR1cm5zIHRydWUgaWYgYSBETUEtYnVmIGV4cG9ydGVyIHdhbnRz IHRvIGNyZWF0ZSBkeW5hbWljIHNnIHRhYmxlIG1hcHBpbmdzCj4+PiArICogZm9yIGVhY2ggYXR0 YWNobWVudC4gRmFsc2UgaWYgb25seSBhIHNpbmdsZSBzdGF0aWMgc2cgdGFibGUgc2hvdWxkIGJl IHVzZWQuCj4+PiArICovCj4+PiArc3RhdGljIGlubGluZSBib29sIGRtYV9idWZfaXNfZHluYW1p YyhzdHJ1Y3QgZG1hX2J1ZiAqZG1hYnVmKQo+Pj4gK3sKPj4+ICsJLyogQWx3YXlzIHVzZSBhIHN0 YXRpYyBtYXBwaW5nIGZvciBub3cgKi8KPj4+ICsJcmV0dXJuIGZhbHNlOwo+PiBIbSBJIHN0aWxs IGV4cGVjdCB0aGF0IGxhdGVyIG9uIHdlJ2xsIHdhbnQgdGhpcyB0byBiZSBkZWNpZGVkIGJ5IHRo ZQo+PiBhdHRhY2htZW50OiBJdCdzIG9ubHkgZHluYW1pYyBpZiBib3RoIHRoZSBleHBvcnRlciBh bmQgdGhlIGltcG9ydGVyCj4+IHN1cHBvcnQgZHluYW1pYyBkbWEtYnVmIG1hbmFnZW1lbnQsIG90 aGVyd2lzZSB3ZSBuZWVkIHRvIHBpbi4KPj4KPj4gQnV0IGFueXdheSwgSSBmZWVsIGxpa2Ugd2Ug bmVlZCB0byBnbyBvdmVyIHRoZSBlbnRpcmUgdGhpbmcgYW55d2F5IG9uY2UKPj4gbW9yZSB3aGVu IHAycCBoYXMgbGFuZGVkLCBvbiB0aGlzOgo+Pgo+PiBSZXZpZXdlZC1ieTogRGFuaWVsIFZldHRl ciA8ZGFuaWVsLnZldHRlckBmZndsbC5jaD4KPiBDb3JyZWN0aW9uIHRoYXQgSSBvbmx5IGp1c3Qg cmVhbGl6ZWQsIGJ1dCBuZWVkIHRvIHJldHJhY3QgdGhhdCByLWIgb24gdGhpcwo+IGFuZCB0aGUg ZHJtIHNndCBjYWNoZSByZW1vdmFsIHBhdGNoOiBZb3Ugbm93IGhhcmRjb2RlIHRoZSBkaXJlY3Rp b24gdG8KPiBETUFfQklESVJFQ1RJT05BTCwgZHJtX3ByaW1lIGRpZCBvbmx5IGtlZXAgdGhlIGNh Y2hlIGZvciBhIGdpdmVuCj4gZGlyZWN0aW9uLgo+Cj4gTm93IHg4NiBkcml2ZXJzIGFsd2F5cyBz ZXQgRE1BX0JJRElSRUNUSU9OQUwsIGJ1dCBhcm0gc29jIGRyaXZlcnMgKGFuZAo+IGFsc28gdjRs IHZpZGVvYnVmIGxheWVyKSB0cnkgdG8gZ3Vlc3Mgd2hldGhlciBpdCBzaG91bGQgYmUgRE1BX1RP X0RFVklDRQo+IG9yIERNQV9GUk9NX0RFVklDRS4gSSBoYXZlIG5vIGlkZWEgd2hhdCB0aGUgaW1w bGljYXRpb25zIGFyZSwgYWxzbyBhbGwgdGhlCj4gY2FjaGUgY29oZXJlbmN5IG9uIGRtYS1idWZz IGlzIGtpbmRhIGlsbC1kZWZpbmVkLgo+Cj4gQW5kIHdlIGNhbid0IHRocm93IHRoZSBzZ3QgY2Fj aGUgYXdheSBhdCBtYXAgdGltZSBpZiBpdCBkb2Vzbid0IGZpdCBsaWtlCj4gZHJtX3ByaW1lIGRv ZXMsIGJlY2F1c2UgdGhhdCByZWludHJvZHVjZXMgdGhlIHJlc2VydmF0aW9uIG9iamVjdCBsb2Nr LAo+IGRlZmVhdGluZyB0aGUgZW50aXJlIHB1cnBvc2Ugb2YgdGhpcy4gQWxzbyB3ZSBjYW4ndCBq dXN0IGFzc3VtZSBkcm1fcHJpbWUKPiB3b3JrcyBmb3IgZXZlcnlvbmUsIHNpbmNlIHRoZSBzcGVj aWFsIGNhc2VzIGFsbCByb2xsIHRoZWlyIG93biBkbWEtYnVmCj4gaW1wb3J0LiBJIGhhdmUgYWxz byBub3QgY2hlY2tlZCB3aGF0IGV4YWN0bHkgZXhwb3J0ZXJzIGRvLiBObyBpZGVhIHdoYXQgdG8K PiBkbyBoZXJlIG5vdy4KPgo+IC9tZSBjcmllcwoKV2VsbCBpdCBpcyBraW5kIG9mIGFidXNpbmcg dG8gdXNlIHRoZSBETUEgZGlyZWN0aW9uIGhlcmUgaW4gdGhlIGZpcnN0IApwbGFjZSBhbmQgSSBh Y3R1YWxseSBjb25zaWRlcmVkIHRvIGNvbXBsZXRlbHkgcmVtb3ZlIGl0LgoKVGhlIGtleSBwcm9i bGVtIGlzIHRoYXQgdGhlIERNQSBkaXJlY3Rpb24gZGVmaW5lcyB0aGluZ3MgZnJvbSB0aGUgdmll dyAKcG9pbnQgb2YgdGhlIENQVSwgZS5nLiBETUFfVE9fREVWSUNFIG1lYW5zIHRoZSBkYXRhIGhh cyBiZWVuIGFjY2Vzc2VkIGJ5IAp0aGUgQ1BVIGFuZCB3ZSBhcmUgbm93IHB1c2hpbmcgaXQgdG8g YSBkZXZpY2UuCgpCdXQgRE1BLWJ1ZiBpcyBlc3NlbnRpYWxseSBhYm91dCBkZXZpY2UgdG8gZGV2 aWNlIHRyYW5zZmVycywgc28gCkRNQV9UT19ERVZJQ0UgYXMgd2VsbCBhcyBETUFfRlJPTV9ERVZJ Q0UgaXMgY29tcGxldGVseSBhbWJpZ3VvdXMgYmVjYXVzZSAKeW91IGRvbid0IGtub3cgdGhlIHBv aW50IG9mIHZpZXcgdG8gc3RhcnQgd2l0aC4KCkFkZGl0aW9uYWwgdG8gdGhhdCB0aGUgY2FjaGlu ZyBpbXBsZW1lbnRhdGlvbiBpbiBkcm1fcHJpbWUuYyBkaWRuJ3QgCmhhbmRsZWQgaXQgZnVsbCBl aXRoZXI6Cj4gLQkvKgo+IC0JICogdHdvIG1hcHBpbmdzIHdpdGggZGlmZmVyZW50IGRpcmVjdGlv bnMgZm9yIHRoZSBzYW1lIGF0dGFjaG1lbnQgYXJlCj4gLQkgKiBub3QgYWxsb3dlZAo+IC0JICov Cj4gLQlpZiAoV0FSTl9PTihwcmltZV9hdHRhY2gtPmRpciAhPSBETUFfTk9ORSkpCj4gLQkJcmV0 dXJuIEVSUl9QVFIoLUVCVVNZKTsKCkRvIHRoZSBBUk0gZ3V5cyBhbmQgVjRMIGd1eXMgYWN0dWFs bHkgZG8gYW55dGhpbmcgd2l0aCB0aGUgZGlyZWN0aW9uIApleGNlcHQgZm9yIHBhc3NpbmcgaXQg b24gZG1hX21hcF9zZ19hdHRycygpID8KClJlZ2FyZHMsCkNocmlzdGlhbi4KCj4gLURhbmllbAo+ Cj4+PiArfQo+Pj4gKwo+Pj4gICBzdHJ1Y3QgZG1hX2J1Zl9hdHRhY2htZW50ICpkbWFfYnVmX2F0 dGFjaChzdHJ1Y3QgZG1hX2J1ZiAqZG1hYnVmLAo+Pj4gICAJCQkJCQkJc3RydWN0IGRldmljZSAq ZGV2KTsKPj4+ICAgdm9pZCBkbWFfYnVmX2RldGFjaChzdHJ1Y3QgZG1hX2J1ZiAqZG1hYnVmLAo+ Pj4gLS0gCj4+PiAyLjE3LjEKPj4+Cj4+PiBfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fXwo+Pj4gZHJpLWRldmVsIG1haWxpbmcgbGlzdAo+Pj4gZHJpLWRldmVs QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwo+Pj4gaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcv bWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwKPj4gLS0gCj4+IERhbmllbCBWZXR0ZXIKPj4gU29m dHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9uCj4+IGh0dHA6Ly9ibG9nLmZmd2xsLmNo CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2 ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9s aXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWw=