From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 7/7] drm/i915: GEM operations need to be done under the big lock Date: Mon, 11 Jan 2016 17:36:46 +0200 Message-ID: <20160111153646.GC23290@intel.com> References: <1452521321-4032-1-git-send-email-tvrtko.ursulin@linux.intel.com> <1452521321-4032-8-git-send-email-tvrtko.ursulin@linux.intel.com> <20160111143800.GF652@nuc-i3427.alporthouse.com> <5693C075.5070006@linux.intel.com> <20160111150020.GH652@nuc-i3427.alporthouse.com> <20160111150458.GI652@nuc-i3427.alporthouse.com> <5693C740.3040801@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" 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 15BDD6E37D for ; Mon, 11 Jan 2016 07:36:51 -0800 (PST) Content-Disposition: inline In-Reply-To: <5693C740.3040801@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Tvrtko Ursulin Cc: Intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gTW9uLCBKYW4gMTEsIDIwMTYgYXQgMDM6MTY6MTZQTSArMDAwMCwgVHZydGtvIFVyc3VsaW4g d3JvdGU6Cj4gCj4gCj4gT24gMTEvMDEvMTYgMTU6MDQsIENocmlzIFdpbHNvbiB3cm90ZToKPiA+ IE9uIE1vbiwgSmFuIDExLCAyMDE2IGF0IDAzOjAwOjIwUE0gKzAwMDAsIENocmlzIFdpbHNvbiB3 cm90ZToKPiA+PiBPbiBNb24sIEphbiAxMSwgMjAxNiBhdCAwMjo0NzoxN1BNICswMDAwLCBUdnJ0 a28gVXJzdWxpbiB3cm90ZToKPiA+Pj4KPiA+Pj4gT24gMTEvMDEvMTYgMTQ6MzgsIENocmlzIFdp bHNvbiB3cm90ZToKPiA+Pj4+IE9uIE1vbiwgSmFuIDExLCAyMDE2IGF0IDAyOjA4OjQxUE0gKzAw MDAsIFR2cnRrbyBVcnN1bGluIHdyb3RlOgo+ID4+Pj4+IEZyb206IFR2cnRrbyBVcnN1bGluIDx0 dnJ0a28udXJzdWxpbkBpbnRlbC5jb20+Cj4gPj4+Pj4KPiA+Pj4+PiBWTUEgY3JlYXRpb24gYW5k IEdFTSBsaXN0IG1hbmFnZW1lbnQgbmVlZCB0aGUgYmlnIGxvY2suCj4gPj4+Pj4KPiA+Pj4+PiB2 MjoKPiA+Pj4+Pgo+ID4+Pj4+IE11dGV4IHVubG9jayBlbmRlZCBvbiB0aGUgd3JvbmcgcGF0aCBz b21laG93LiAoMC1kYXksIEp1bGlhIExhd2FsbCkKPiA+Pj4+Pgo+ID4+Pj4+IE5vdCB0byBtZW50 aW9uIGRybV9nZW1fb2JqZWN0X3VucmVmZXJlbmNlIHdhcyB0aGVyZSBpbiBleGlzdGluZwo+ID4+ Pj4+IGNvZGUgd2l0aCBubyBtdXRleCBoZWxkLgo+ID4+Pj4+Cj4gPj4+Pj4gdjM6Cj4gPj4+Pj4K PiA+Pj4+PiBTb21lIGNhbGxlcnMgb2YgaTkxNV9nZW1fb2JqZWN0X2NyZWF0ZV9zdG9sZW5fZm9y X3ByZWFsbG9jYXRlZAo+ID4+Pj4+IGFscmVhZHkgaG9sZCB0aGUgbG9jayBzbyBtb3ZlIHRoZSBt dXRleCBpbnRvIHRoZSBvdGhlciBjYWxsZXIKPiA+Pj4+PiBhcyB3ZWxsLgo+ID4+Pj4KPiA+Pj4+ IGludGVsX3BtLmMgdmFsbGV5dmlld19zZXR1cF9wY3R4Pwo+ID4+Pgo+ID4+PiBBbHJlYWR5IGhv bGRzIGl0IHRyYWNlZCBieSB0aGUgV0FSTl9PTiBhdCBpdHMgdG9wLgo+ID4+Cj4gPj4gV2hpY2gg aXMgYSBuaWNlIGxpdHRsZSBtdXRleCBpbnZlcnNpb24gb2YgaXRzIG93bi4gOnwKPiA+PiBpLmUu IHJwbSB2cyBzdHJ1Y3RfbXV0ZXggYnVnCj4gPgo+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1 L2RybS9pOTE1L2ludGVsX2Rpc3BsYXkuYyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rp c3BsYXkuYwo+ID4gaW5kZXggYjFmYjQzZmNmZWVhLi5jOGY2ODRmODc5OWMgMTAwNjQ0Cj4gPiAt LS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPiA+ICsrKyBiL2RyaXZl cnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rpc3BsYXkuYwo+ID4gQEAgLTE2MDE4LDkgKzE2MDE4LDcg QEAgdm9pZCBpbnRlbF9tb2Rlc2V0X2dlbV9pbml0KHN0cnVjdCBkcm1fZGV2aWNlICpkZXYpCj4g PiAgICAgICAgICBzdHJ1Y3QgZHJtX2NydGMgKmM7Cj4gPiAgICAgICAgICBzdHJ1Y3QgZHJtX2k5 MTVfZ2VtX29iamVjdCAqb2JqOwo+ID4KPiA+IC0gICAgICAgbXV0ZXhfbG9jaygmZGV2LT5zdHJ1 Y3RfbXV0ZXgpOwo+ID4gICAgICAgICAgaW50ZWxfaW5pdF9ndF9wb3dlcnNhdmUoZGV2KTsKPiA+ IC0gICAgICAgbXV0ZXhfdW5sb2NrKCZkZXYtPnN0cnVjdF9tdXRleCk7Cj4gPgo+ID4gICAgICAg ICAgaW50ZWxfbW9kZXNldF9pbml0X2h3KGRldik7Cj4gPgo+ID4gZGlmZiAtLWdpdCBhL2RyaXZl cnMvZ3B1L2RybS9pOTE1L2ludGVsX3BtLmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9w bS5jCj4gPiBpbmRleCBhMDgyYjQ1Nzc1OTkuLjkwZTViZjdhMjQwMiAxMDA2NDQKPiA+IC0tLSBh L2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3BtLmMKPiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2Ry bS9pOTE1L2ludGVsX3BtLmMKPiA+IEBAIC01MjI0LDggKzUyMjQsNiBAQCBzdGF0aWMgdm9pZCBj aGVycnl2aWV3X3NldHVwX3BjdHgoc3RydWN0IGRybV9kZXZpY2UgKmRldikKPiA+ICAgICAgICAg IHUzMiBwY2JyOwo+ID4gICAgICAgICAgaW50IHBjdHhfc2l6ZSA9IDMyKjEwMjQ7Cj4gPgo+ID4g LSAgICAgICBXQVJOX09OKCFtdXRleF9pc19sb2NrZWQoJmRldi0+c3RydWN0X211dGV4KSk7Cj4g PiAtCj4gPiAgICAgICAgICBwY2JyID0gSTkxNV9SRUFEKFZMVl9QQ0JSKTsKPiA+ICAgICAgICAg IGlmICgocGNiciA+PiBWTFZfUENCUl9BRERSX1NISUZUKSA9PSAwKSB7Cj4gPiAgICAgICAgICAg ICAgICAgIERSTV9ERUJVR19EUklWRVIoIkJJT1MgZGlkbid0IHNldCB1cCBQQ0JSLCBmaXhpbmcg dXBcbiIpOwo+ID4gQEAgLTUyNDcsOCArNTI0NSw3IEBAIHN0YXRpYyB2b2lkIHZhbGxleXZpZXdf c2V0dXBfcGN0eChzdHJ1Y3QgZHJtX2RldmljZSAqZGV2KQo+ID4gICAgICAgICAgdTMyIHBjYnI7 Cj4gPiAgICAgICAgICBpbnQgcGN0eF9zaXplID0gMjQqMTAyNDsKPiA+Cj4gPiAtICAgICAgIFdB Uk5fT04oIW11dGV4X2lzX2xvY2tlZCgmZGV2LT5zdHJ1Y3RfbXV0ZXgpKTsKPiA+IC0KPiA+ICsg ICAgICAgbXV0ZXhfbG9jaygmZGV2LT5zdHJ1Y3RfbXV0ZXgpOwo+ID4gICAgICAgICAgcGNiciA9 IEk5MTVfUkVBRChWTFZfUENCUik7Cj4gPiAgICAgICAgICBpZiAocGNicikgewo+ID4gICAgICAg ICAgICAgICAgICAvKiBCSU9TIHNldCBpdCB1cCBhbHJlYWR5LCBncmFiIHRoZSBwcmUtYWxsb2Mn ZCBzcGFjZSAqLwo+ID4gQEAgLTUyNzUsNyArNTI3Miw3IEBAIHN0YXRpYyB2b2lkIHZhbGxleXZp ZXdfc2V0dXBfcGN0eChzdHJ1Y3QgZHJtX2RldmljZSAqZGV2KQo+ID4gICAgICAgICAgcGN0eCA9 IGk5MTVfZ2VtX29iamVjdF9jcmVhdGVfc3RvbGVuKGRldiwgcGN0eF9zaXplKTsKPiA+ICAgICAg ICAgIGlmICghcGN0eCkgewo+ID4gICAgICAgICAgICAgICAgICBEUk1fREVCVUcoIm5vdCBlbm91 Z2ggc3RvbGVuIHNwYWNlIGZvciBQQ1RYLCBkaXNhYmxpbmdcbiIpOwo+ID4gLSAgICAgICAgICAg ICAgIHJldHVybjsKPiA+ICsgICAgICAgICAgICAgICBnb3RvIG91dDsKPiA+ICAgICAgICAgIH0K PiA+Cj4gPiAgICAgICAgICBwY3R4X3BhZGRyID0gZGV2X3ByaXYtPm1tLnN0b2xlbl9iYXNlICsg cGN0eC0+c3RvbGVuLT5zdGFydDsKPiA+IEBAIC01Mjg0LDYgKzUyODEsNyBAQCBzdGF0aWMgdm9p ZCB2YWxsZXl2aWV3X3NldHVwX3BjdHgoc3RydWN0IGRybV9kZXZpY2UgKmRldikKPiA+ICAgb3V0 Ogo+ID4gICAgICAgICAgRFJNX0RFQlVHX0RSSVZFUigiUENCUjogMHglMDh4XG4iLCBJOTE1X1JF QUQoVkxWX1BDQlIpKTsKPiA+ICAgICAgICAgIGRldl9wcml2LT52bHZfcGN0eCA9IHBjdHg7Cj4g PiArICAgICAgIG11dGV4X3VubG9jaygmZGV2LT5zdHJ1Y3RfbXV0ZXgpOwo+ID4gICB9Cj4gPgo+ ID4gICBzdGF0aWMgdm9pZCB2YWxsZXl2aWV3X2NsZWFudXBfcGN0eChzdHJ1Y3QgZHJtX2Rldmlj ZSAqZGV2KQo+IAo+IERvbid0IGtub3csIEkgbGVhdmUgdGhpcyBvbmUgdG8gd2hvZXZlciBncmFi YmVkIHRoZSBsb2NrIGFyb3VuZCAKPiBpbnRlbF9pbml0X2d0X3Bvd2Vyc2F2ZSBpbiB0aGUgZmly c3QgcGxhY2UuIE1heWJlIHRoZXJlIHdhcyBhIHNwZWNpYWwgCj4gcmVhc29uLi4gYWZ0ZXIgZ2l0 IGJsYW1lIG9kIGludGVsX2Rpc3BsYXkuYyBldmVudHVhbGx5IGNvbXBsZXRlZCwgYWRkaW5nIAo+ IEltcmUgYW5kIFZpbGxlIHRvIGNjLgoKSG1tLiBJIGRvbid0IHJlY2FsbCB0aGUgZGV0YWlscyBh bnltb3JlLCBidXQgbG9va2luZyBhdCB0aGUgY29kZSBwdXNoaW5nCnRoZSBsb2NraW5nIGRvd24g dG8gdmFsbGV5dmlld19zZXR1cF9wY3R4KCkgbG9va3MgZW50aXJlbHkgcmVhc29uYWJsZSB0bwpt ZS4gQW5kIHllYWgsIGRvZXNuJ3QgbG9vayBsaWtlIGl0J3MgcmVhbGx5IHByb3RlY3RpbmcgYW55 dGhpbmcgaW4gdGhlCmNodiBmdW5jdGlvbiwgc28gY2FuIGp1c3QgYmUgZHJvcHBlZCB0aGVyZS4K ClRoZSBjbGVhbnVwIHBhdGggY291bGQgdXNlIHRoZSBzYW1lIHRyZWF0ZW1udCBJIHRoaW5rLgoK LS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlz dHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9s aXN0aW5mby9pbnRlbC1nZngK