From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tvrtko Ursulin Subject: Re: [PATCH v2 06/11] drm/i915: Move pinning of dev_priv->kernel_context into its creator Date: Mon, 23 May 2016 13:09:02 +0100 Message-ID: <5742F2DE.2040204@linux.intel.com> References: <1464003258-23669-1-git-send-email-chris@chris-wilson.co.uk> <1464003258-23669-7-git-send-email-chris@chris-wilson.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTP id CDCA889DF9 for ; Mon, 23 May 2016 12:09:09 +0000 (UTC) In-Reply-To: <1464003258-23669-7-git-send-email-chris@chris-wilson.co.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org Ck9uIDIzLzA1LzE2IDEyOjM0LCBDaHJpcyBXaWxzb24gd3JvdGU6Cj4gUmF0aGVyIHRoYW4gaGF2 ZSBldmVyeSBjb250ZXh0IGFzayAiYW0gSSBvd25lZCBieSB0aGUga2VybmVsPyBwaW4hIiwKPiBt b3ZlIHRoYXQgbG9naWMgaW50byB0aGUgY3JlYXRvciBvZiB0aGUga2VybmVsIGNvbnRleHQsIGlu IG9yZGVyIHRvCj4gaW1wcm92ZSBjb2RlIGNvbXByZWhlbnNpb24uCj4KPiB2MjogVGhyb3cgYXdh eSB0aGUgdXNlcl9oYW5kbGUgb24gZmFpbHVyZSB0byBhbGxvY2F0ZSB0aGUgcHBndHQuCj4KPiBT aWduZWQtb2ZmLWJ5OiBDaHJpcyBXaWxzb24gPGNocmlzQGNocmlzLXdpbHNvbi5jby51az4KPiBD YzogVHZydGtvIFVyc3VsaW4gPHR2cnRrby51cnN1bGluQGludGVsLmNvbT4KPiBDYzogSm9vbmFz IExhaHRpbmVuIDxqb29uYXMubGFodGluZW5AbGludXguaW50ZWwuY29tPgo+IC0tLQo+ICAgZHJp dmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW1fY29udGV4dC5jIHwgNTMgKysrKysrKysrKysrKysr LS0tLS0tLS0tLS0tLS0tLS0tCj4gICAxIGZpbGUgY2hhbmdlZCwgMjQgaW5zZXJ0aW9ucygrKSwg MjkgZGVsZXRpb25zKC0pCj4KPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkx NV9nZW1fY29udGV4dC5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW1fY29udGV4dC5j Cj4gaW5kZXggYTRjNjM3N2VlYTMyLi40NzQxNzRjMjBiZTIgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVy cy9ncHUvZHJtL2k5MTUvaTkxNV9nZW1fY29udGV4dC5jCj4gKysrIGIvZHJpdmVycy9ncHUvZHJt L2k5MTUvaTkxNV9nZW1fY29udGV4dC5jCj4gQEAgLTMwMyw5ICszMDMsNyBAQCBzdGF0aWMgc3Ry dWN0IGk5MTVfZ2VtX2NvbnRleHQgKgo+ICAgaTkxNV9nZW1fY3JlYXRlX2NvbnRleHQoc3RydWN0 IGRybV9kZXZpY2UgKmRldiwKPiAgIAkJCXN0cnVjdCBkcm1faTkxNV9maWxlX3ByaXZhdGUgKmZp bGVfcHJpdikKPiAgIHsKPiAtCWNvbnN0IGJvb2wgaXNfZ2xvYmFsX2RlZmF1bHRfY3R4ID0gZmls ZV9wcml2ID09IE5VTEw7Cj4gICAJc3RydWN0IGk5MTVfZ2VtX2NvbnRleHQgKmN0eDsKPiAtCWlu dCByZXQgPSAwOwo+Cj4gICAJQlVHX09OKCFtdXRleF9pc19sb2NrZWQoJmRldi0+c3RydWN0X211 dGV4KSk7Cj4KPiBAQCAtMzEzLDMwICszMTEsMTUgQEAgaTkxNV9nZW1fY3JlYXRlX2NvbnRleHQo c3RydWN0IGRybV9kZXZpY2UgKmRldiwKPiAgIAlpZiAoSVNfRVJSKGN0eCkpCj4gICAJCXJldHVy biBjdHg7Cj4KPiAtCWlmIChpc19nbG9iYWxfZGVmYXVsdF9jdHggJiYgY3R4LT5sZWdhY3lfaHdf Y3R4LnJjc19zdGF0ZSkgewo+IC0JCS8qIFdlIG1heSBuZWVkIHRvIGRvIHRoaW5ncyB3aXRoIHRo ZSBzaHJpbmtlciB3aGljaAo+IC0JCSAqIHJlcXVpcmUgdXMgdG8gaW1tZWRpYXRlbHkgc3dpdGNo IGJhY2sgdG8gdGhlIGRlZmF1bHQKPiAtCQkgKiBjb250ZXh0LiBUaGlzIGNhbiBjYXVzZSBhIHBy b2JsZW0gYXMgcGlubmluZyB0aGUKPiAtCQkgKiBkZWZhdWx0IGNvbnRleHQgYWxzbyByZXF1aXJl cyBHVFQgc3BhY2Ugd2hpY2ggbWF5IG5vdAo+IC0JCSAqIGJlIGF2YWlsYWJsZS4gVG8gYXZvaWQg dGhpcyB3ZSBhbHdheXMgcGluIHRoZSBkZWZhdWx0Cj4gLQkJICogY29udGV4dC4KPiAtCQkgKi8K PiAtCQlyZXQgPSBpOTE1X2dlbV9vYmpfZ2d0dF9waW4oY3R4LT5sZWdhY3lfaHdfY3R4LnJjc19z dGF0ZSwKPiAtCQkJCQkgICAgZ2V0X2NvbnRleHRfYWxpZ25tZW50KHRvX2k5MTUoZGV2KSksIDAp Owo+IC0JCWlmIChyZXQpIHsKPiAtCQkJRFJNX0RFQlVHX0RSSVZFUigiQ291bGRuJ3QgcGluICVk XG4iLCByZXQpOwo+IC0JCQlnb3RvIGVycl9kZXN0cm95Owo+IC0JCX0KPiAtCX0KPiAtCj4gICAJ aWYgKFVTRVNfRlVMTF9QUEdUVChkZXYpKSB7Cj4gICAJCXN0cnVjdCBpOTE1X2h3X3BwZ3R0ICpw cGd0dCA9IGk5MTVfcHBndHRfY3JlYXRlKGRldiwgZmlsZV9wcml2KTsKPgo+IC0JCWlmIChJU19F UlJfT1JfTlVMTChwcGd0dCkpIHsKPiArCQlpZiAoSVNfRVJSKHBwZ3R0KSkgewo+ICAgCQkJRFJN X0RFQlVHX0RSSVZFUigiUFBHVFQgc2V0dXAgZmFpbGVkICglbGQpXG4iLAo+ICAgCQkJCQkgUFRS X0VSUihwcGd0dCkpOwo+IC0JCQlyZXQgPSBQVFJfRVJSKHBwZ3R0KTsKPiAtCQkJZ290byBlcnJf dW5waW47Cj4gKwkJCWlkcl9yZW1vdmUoJmZpbGVfcHJpdi0+Y29udGV4dF9pZHIsIGN0eC0+dXNl cl9oYW5kbGUpOwo+ICsJCQlpOTE1X2dlbV9jb250ZXh0X3B1dChjdHgpOwo+ICsJCQlyZXR1cm4g RVJSX0NBU1QocHBndHQpOwo+ICAgCQl9Cj4KPiAgIAkJY3R4LT5wcGd0dCA9IHBwZ3R0Owo+IEBA IC0zNDUsMTQgKzMyOCw2IEBAIGk5MTVfZ2VtX2NyZWF0ZV9jb250ZXh0KHN0cnVjdCBkcm1fZGV2 aWNlICpkZXYsCj4gICAJdHJhY2VfaTkxNV9jb250ZXh0X2NyZWF0ZShjdHgpOwo+Cj4gICAJcmV0 dXJuIGN0eDsKPiAtCj4gLWVycl91bnBpbjoKPiAtCWlmIChpc19nbG9iYWxfZGVmYXVsdF9jdHgg JiYgY3R4LT5sZWdhY3lfaHdfY3R4LnJjc19zdGF0ZSkKPiAtCQlpOTE1X2dlbV9vYmplY3RfZ2d0 dF91bnBpbihjdHgtPmxlZ2FjeV9od19jdHgucmNzX3N0YXRlKTsKPiAtZXJyX2Rlc3Ryb3k6Cj4g LQlpZHJfcmVtb3ZlKCZmaWxlX3ByaXYtPmNvbnRleHRfaWRyLCBjdHgtPnVzZXJfaGFuZGxlKTsK PiAtCWk5MTVfZ2VtX2NvbnRleHRfcHV0KGN0eCk7Cj4gLQlyZXR1cm4gRVJSX1BUUihyZXQpOwo+ ICAgfQo+Cj4gICBzdGF0aWMgdm9pZCBpOTE1X2dlbV9jb250ZXh0X3VucGluKHN0cnVjdCBpOTE1 X2dlbV9jb250ZXh0ICpjdHgsCj4gQEAgLTQyNiw2ICs0MDEsMjYgQEAgaW50IGk5MTVfZ2VtX2Nv bnRleHRfaW5pdChzdHJ1Y3QgZHJtX2RldmljZSAqZGV2KQo+ICAgCQlyZXR1cm4gUFRSX0VSUihj dHgpOwo+ICAgCX0KPgo+ICsJaWYgKGN0eC0+bGVnYWN5X2h3X2N0eC5yY3Nfc3RhdGUpIHsKPiAr CQlpbnQgcmV0Owo+ICsKPiArCQkvKiBXZSBtYXkgbmVlZCB0byBkbyB0aGluZ3Mgd2l0aCB0aGUg c2hyaW5rZXIgd2hpY2gKPiArCQkgKiByZXF1aXJlIHVzIHRvIGltbWVkaWF0ZWx5IHN3aXRjaCBi YWNrIHRvIHRoZSBkZWZhdWx0Cj4gKwkJICogY29udGV4dC4gVGhpcyBjYW4gY2F1c2UgYSBwcm9i bGVtIGFzIHBpbm5pbmcgdGhlCj4gKwkJICogZGVmYXVsdCBjb250ZXh0IGFsc28gcmVxdWlyZXMg R1RUIHNwYWNlIHdoaWNoIG1heSBub3QKPiArCQkgKiBiZSBhdmFpbGFibGUuIFRvIGF2b2lkIHRo aXMgd2UgYWx3YXlzIHBpbiB0aGUgZGVmYXVsdAo+ICsJCSAqIGNvbnRleHQuCj4gKwkJICovCj4g KwkJcmV0ID0gaTkxNV9nZW1fb2JqX2dndHRfcGluKGN0eC0+bGVnYWN5X2h3X2N0eC5yY3Nfc3Rh dGUsCj4gKwkJCQkJICAgIGdldF9jb250ZXh0X2FsaWdubWVudChkZXZfcHJpdiksIDApOwo+ICsJ CWlmIChyZXQpIHsKPiArCQkJRFJNX0VSUk9SKCJGYWlsZWQgdG8gcGlubmVkIGRlZmF1bHQgZ2xv YmFsIGNvbnRleHQgKGVycm9yICVkKVxuIiwKPiArCQkJCSAgcmV0KTsKCmlkcl9yZW1vdmUgaGVy ZSBhcyB3ZWxsPyBNYXliZSB3ZSBzaG91bGQgc3BsaXQgb3V0IHRoZSBpZHIgY29udGV4dCBBUEkg CmZyb20gdGhlIEdFTSBjb250ZXh0IHRvIG1ha2UgaXQgZXhwbGljaXQgLSBjcmVhdGUgY29udGV4 dCwgZXhwb3J0IGl0IHRvIAp1c2Vyc3BhY2UgYW5kIGJhY2s/Cgo+ICsJCQlpOTE1X2dlbV9jb250 ZXh0X3B1dChjdHgpOwo+ICsJCQlyZXR1cm4gcmV0Owo+ICsJCX0KPiArCX0KPiArCj4gICAJZGV2 X3ByaXYtPmtlcm5lbF9jb250ZXh0ID0gY3R4Owo+Cj4gICAJRFJNX0RFQlVHX0RSSVZFUigiJXMg Y29udGV4dCBzdXBwb3J0IGluaXRpYWxpemVkXG4iLAo+CgpSZWdhcmRzLAoKVHZydGtvCl9fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWls aW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZy ZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=