From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomas Elf Subject: Re: [PATCH 18/53] drm/i915: Moved the for_each_ring loop outside of i915_gem_context_enable() Date: Thu, 05 Mar 2015 17:04:19 +0000 Message-ID: <54F88C93.9000807@intel.com> References: <1423828140-10653-1-git-send-email-John.C.Harrison@Intel.com> <1424366285-29232-1-git-send-email-John.C.Harrison@Intel.com> <1424366285-29232-19-git-send-email-John.C.Harrison@Intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTP id 8B1726E7F1 for ; Thu, 5 Mar 2015 09:05:00 -0800 (PST) In-Reply-To: <1424366285-29232-19-git-send-email-John.C.Harrison@Intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: John.C.Harrison@Intel.com, Intel-GFX@Lists.FreeDesktop.Org List-Id: intel-gfx@lists.freedesktop.org T24gMTkvMDIvMjAxNSAxNzoxNywgSm9obi5DLkhhcnJpc29uQEludGVsLmNvbSB3cm90ZToKPiBG cm9tOiBKb2huIEhhcnJpc29uIDxKb2huLkMuSGFycmlzb25ASW50ZWwuY29tPgo+Cj4gVGhlIHN0 YXJ0IG9mIGRheSBjb250ZXh0IGluaXRpYWxpc2F0aW9uIGNvZGUgaW4gaTkxNV9nZW1fY29udGV4 dF9lbmFibGUoKSBsb29wcwo+IG92ZXIgZWFjaCByaW5nIGFuZCBjYWxscyB0aGUgbGVnYWN5IHN3 aXRjaCBjb250ZXh0IG9yIHRoZSBleGVjbGlzdCBpbml0IGNvbnRleHQKPiBjb2RlIGFzIGFwcHJv cHJpYXRlLgo+Cj4gVGhpcyBwYXRjaCBtb3ZlcyB0aGUgcmluZyBsb29waW5nIG91dCBvZiB0aGF0 IGZ1bmN0aW9uIGluIHRvIHRoZSB0b3AgbGV2ZWwKPiBjYWxsZXIgaTkxNV9nZW1faW5pdF9odygp LiBUaGlzIG1lYW5zIHRoZSBhIHNpbmdsZSBwYXNzIGNhbiBiZSBtYWRlIG92ZXIgYWxsCj4gcmlu Z3MgZG9pbmcgdGhlIFBQR1RULCBMMyByZW1hcCBhbmQgY29udGV4dCBpbml0aWFsaXNhdGlvbiBv ZiBlYWNoIHJpbmcKPiBhbHRvZ2V0aGVyLgo+Cj4gRm9yOiBWSVotNTExNQo+IFNpZ25lZC1vZmYt Ynk6IEpvaG4gSGFycmlzb24gPEpvaG4uQy5IYXJyaXNvbkBJbnRlbC5jb20+Cj4gLS0tCj4gICBk cml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2Rydi5oICAgICAgICAgfCAgICAyICstCj4gICBkcml2 ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbS5jICAgICAgICAgfCAgIDE4ICsrKysrKysrKystLS0t LS0tCj4gICBkcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbV9jb250ZXh0LmMgfCAgIDMyICsr KysrKysrKysrLS0tLS0tLS0tLS0tLS0tLS0tLS0KPiAgIDMgZmlsZXMgY2hhbmdlZCwgMjMgaW5z ZXJ0aW9ucygrKSwgMjkgZGVsZXRpb25zKC0pCj4KPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUv ZHJtL2k5MTUvaTkxNV9kcnYuaCBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2LmgKPiBp bmRleCBiZmQ3YjQ3Li42NTNjODJkIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1 L2k5MTVfZHJ2LmgKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2Rydi5oCj4gQEAg LTI5ODksNyArMjk4OSw3IEBAIGludCBfX211c3RfY2hlY2sgaTkxNV9nZW1fY29udGV4dF9pbml0 KHN0cnVjdCBkcm1fZGV2aWNlICpkZXYpOwo+ICAgdm9pZCBpOTE1X2dlbV9jb250ZXh0X2Zpbmko c3RydWN0IGRybV9kZXZpY2UgKmRldik7Cj4gICB2b2lkIGk5MTVfZ2VtX2NvbnRleHRfcmVzZXQo c3RydWN0IGRybV9kZXZpY2UgKmRldik7Cj4gICBpbnQgaTkxNV9nZW1fY29udGV4dF9vcGVuKHN0 cnVjdCBkcm1fZGV2aWNlICpkZXYsIHN0cnVjdCBkcm1fZmlsZSAqZmlsZSk7Cj4gLWludCBpOTE1 X2dlbV9jb250ZXh0X2VuYWJsZShzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYpOwo+ ICtpbnQgaTkxNV9nZW1fY29udGV4dF9lbmFibGUoc3RydWN0IGludGVsX2VuZ2luZV9jcyAqcmlu Zyk7Cj4gICB2b2lkIGk5MTVfZ2VtX2NvbnRleHRfY2xvc2Uoc3RydWN0IGRybV9kZXZpY2UgKmRl diwgc3RydWN0IGRybV9maWxlICpmaWxlKTsKPiAgIGludCBpOTE1X3N3aXRjaF9jb250ZXh0KHN0 cnVjdCBpbnRlbF9lbmdpbmVfY3MgKnJpbmcsCj4gICAJCQlzdHJ1Y3QgaW50ZWxfY29udGV4dCAq dG8pOwo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbS5jIGIvZHJp dmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW0uYwo+IGluZGV4IDliYzYwZDcuLjU4NTA5OTEgMTAw NjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW0uYwo+ICsrKyBiL2RyaXZl cnMvZ3B1L2RybS9pOTE1L2k5MTVfZ2VtLmMKPiBAQCAtNDg0NCw2ICs0ODQ0LDggQEAgaTkxNV9n ZW1faW5pdF9odyhzdHJ1Y3QgZHJtX2RldmljZSAqZGV2KQo+ICAgCSAqLwo+ICAgCWluaXRfdW51 c2VkX3JpbmdzKGRldik7Cj4KPiArCUJVR19PTighZGV2X3ByaXYtPnJpbmdbUkNTXS5kZWZhdWx0 X2NvbnRleHQpOwo+ICsKPiAgIAlyZXQgPSBpOTE1X3BwZ3R0X2luaXRfaHcoZGV2KTsKPiAgIAlp ZiAocmV0KSB7Cj4gICAJCURSTV9FUlJPUigiUFBHVFQgZW5hYmxlIEhXIGZhaWxlZCAlZFxuIiwg cmV0KTsKPiBAQCAtNDg1OSw2ICs0ODYxLDggQEAgaTkxNV9nZW1faW5pdF9odyhzdHJ1Y3QgZHJt X2RldmljZSAqZGV2KQo+Cj4gICAJLyogTm93IGl0IGlzIHNhZmUgdG8gZ28gYmFjayByb3VuZCBh bmQgZG8gZXZlcnl0aGluZyBlbHNlOiAqLwo+ICAgCWZvcl9lYWNoX3JpbmcocmluZywgZGV2X3By aXYsIGkpIHsKPiArCQlXQVJOX09OKCFyaW5nLT5kZWZhdWx0X2NvbnRleHQpOwo+ICsKPiAgIAkJ aWYgKHJpbmctPmlkID09IFJDUykgewo+ICAgCQkJZm9yIChpID0gMDsgaSA8IE5VTV9MM19TTElD RVMoZGV2KTsgaSsrKQo+ICAgCQkJCWk5MTVfZ2VtX2wzX3JlbWFwKHJpbmcsIGkpOwo+IEBAIC00 ODcwLDE3ICs0ODc0LDE3IEBAIGk5MTVfZ2VtX2luaXRfaHcoc3RydWN0IGRybV9kZXZpY2UgKmRl dikKPiAgIAkJCWk5MTVfZ2VtX2NsZWFudXBfcmluZ2J1ZmZlcihkZXYpOwo+ICAgCQkJcmV0dXJu IHJldDsKPiAgIAkJfQo+IC0JfQo+Cj4gLQlyZXQgPSBpOTE1X2dlbV9jb250ZXh0X2VuYWJsZShk ZXZfcHJpdik7Cj4gLQlpZiAocmV0ICYmIHJldCAhPSAtRUlPKSB7Cj4gLQkJRFJNX0VSUk9SKCJD b250ZXh0IGVuYWJsZSBmYWlsZWQgJWRcbiIsIHJldCk7Cj4gLQkJaTkxNV9nZW1fY2xlYW51cF9y aW5nYnVmZmVyKGRldik7Cj4gKwkJcmV0ID0gaTkxNV9nZW1fY29udGV4dF9lbmFibGUocmluZyk7 Cj4gKwkJaWYgKHJldCAmJiByZXQgIT0gLUVJTykgewo+ICsJCQlEUk1fRVJST1IoIkNvbnRleHQg ZW5hYmxlIHJpbmcgIyVkIGZhaWxlZCAlZFxuIiwgaSwgcmV0KTsKPiArCQkJaTkxNV9nZW1fY2xl YW51cF9yaW5nYnVmZmVyKGRldik7Cj4KPiAtCQlyZXR1cm4gcmV0Owo+ICsJCQlyZXR1cm4gcmV0 Owo+ICsJCX0KPiAgIAl9Cj4KPiAtCXJldHVybiByZXQ7Cj4gKwlyZXR1cm4gMDsKPiAgIH0KPgo+ ICAgaW50IGk5MTVfZ2VtX2luaXQoc3RydWN0IGRybV9kZXZpY2UgKmRldikKPiBkaWZmIC0tZ2l0 IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW1fY29udGV4dC5jIGIvZHJpdmVycy9ncHUv ZHJtL2k5MTUvaTkxNV9nZW1fY29udGV4dC5jCj4gaW5kZXggODYwM2JmNC4uZGQ4M2Q2MSAxMDA2 NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbV9jb250ZXh0LmMKPiArKysg Yi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbV9jb250ZXh0LmMKPiBAQCAtNDAzLDMyICs0 MDMsMjIgQEAgdm9pZCBpOTE1X2dlbV9jb250ZXh0X2Zpbmkoc3RydWN0IGRybV9kZXZpY2UgKmRl dikKPiAgIAlpOTE1X2dlbV9jb250ZXh0X3VucmVmZXJlbmNlKGRjdHgpOwo+ICAgfQo+Cj4gLWlu dCBpOTE1X2dlbV9jb250ZXh0X2VuYWJsZShzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3By aXYpCj4gK2ludCBpOTE1X2dlbV9jb250ZXh0X2VuYWJsZShzdHJ1Y3QgaW50ZWxfZW5naW5lX2Nz ICpyaW5nKQo+ICAgewo+IC0Jc3RydWN0IGludGVsX2VuZ2luZV9jcyAqcmluZzsKPiAtCWludCBy ZXQsIGk7Cj4gLQo+IC0JQlVHX09OKCFkZXZfcHJpdi0+cmluZ1tSQ1NdLmRlZmF1bHRfY29udGV4 dCk7Cj4gKwlpbnQgcmV0Owo+Cj4gICAJaWYgKGk5MTUuZW5hYmxlX2V4ZWNsaXN0cykgewo+IC0J CWZvcl9lYWNoX3JpbmcocmluZywgZGV2X3ByaXYsIGkpIHsKPiAtCQkJaWYgKHJpbmctPmluaXRf Y29udGV4dCkgewo+IC0JCQkJcmV0ID0gcmluZy0+aW5pdF9jb250ZXh0KHJpbmcsCj4gLQkJCQkJ CXJpbmctPmRlZmF1bHRfY29udGV4dCk7Cj4gLQkJCQlpZiAocmV0KSB7Cj4gLQkJCQkJRFJNX0VS Uk9SKCJyaW5nIGluaXQgY29udGV4dDogJWRcbiIsCj4gLQkJCQkJCQlyZXQpOwo+IC0JCQkJCXJl dHVybiByZXQ7Cj4gLQkJCQl9Cj4gLQkJCX0KPiAtCQl9Cj4gKwkJaWYgKHJpbmctPmluaXRfY29u dGV4dCA9PSBOVUxMKQo+ICsJCQlyZXR1cm4gMDsKPgo+ICsJCXJldCA9IHJpbmctPmluaXRfY29u dGV4dChyaW5nLCByaW5nLT5kZWZhdWx0X2NvbnRleHQpOwo+ICAgCX0gZWxzZQo+IC0JCWZvcl9l YWNoX3JpbmcocmluZywgZGV2X3ByaXYsIGkpIHsKPiAtCQkJcmV0ID0gaTkxNV9zd2l0Y2hfY29u dGV4dChyaW5nLCByaW5nLT5kZWZhdWx0X2NvbnRleHQpOwo+IC0JCQlpZiAocmV0KQo+IC0JCQkJ cmV0dXJuIHJldDsKPiAtCQl9Cj4gKwkJcmV0ID0gaTkxNV9zd2l0Y2hfY29udGV4dChyaW5nLCBy aW5nLT5kZWZhdWx0X2NvbnRleHQpOwo+ICsKPiArCWlmIChyZXQpIHsKPiArCQlEUk1fRVJST1Io InJpbmcgaW5pdCBjb250ZXh0OiAlZFxuIiwgcmV0KTsKPiArCQlyZXR1cm4gcmV0Owo+ICsJfQo+ Cj4gICAJcmV0dXJuIDA7Cj4gICB9Cj4KClJldmlld2VkLWJ5OiBUb21hcyBFbGYgPHRvbWFzLmVs ZkBpbnRlbC5jb20+CgpUaGFua3MsClRvbWFzCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0 cy5mcmVlZGVza3RvcC5vcmcKaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xp c3RpbmZvL2ludGVsLWdmeAo=