From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 2/2] GPU-DRM-Savage: Less function calls in savage_bci_cmdbuf() after error detection Date: Fri, 19 Aug 2016 09:50:21 +0200 Message-ID: <20160819075021.GU6232@phenom.ffwll.local> References: <566ABCD9.1060404@users.sourceforge.net> <77052ac1-1933-49af-3ad8-6e0eecc1aafe@users.sourceforge.net> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wm0-x244.google.com (mail-wm0-x244.google.com [IPv6:2a00:1450:400c:c09::244]) by gabe.freedesktop.org (Postfix) with ESMTPS id 4ACBA6EB93 for ; Fri, 19 Aug 2016 07:50:26 +0000 (UTC) Received: by mail-wm0-x244.google.com with SMTP id i5so2374845wmg.2 for ; Fri, 19 Aug 2016 00:50:26 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: SF Markus Elfring Cc: kernel-janitors@vger.kernel.org, LKML , "dri-devel@lists.freedesktop.org" , Julia Lawall List-Id: dri-devel@lists.freedesktop.org T24gVGh1LCBBdWcgMTgsIDIwMTYgYXQgMDk6NDg6MDRQTSArMDIwMCwgU0YgTWFya3VzIEVsZnJp bmcgd3JvdGU6Cj4gRnJvbTogTWFya3VzIEVsZnJpbmcgPGVsZnJpbmdAdXNlcnMuc291cmNlZm9y Z2UubmV0Pgo+IERhdGU6IFRodSwgMTggQXVnIDIwMTYgMjE6Mjg6NTggKzAyMDAKPiAKPiBUaGUg a2ZyZWUoKSBmdW5jdGlvbiB3YXMgY2FsbGVkIGluIGEgZmV3IGNhc2VzIGJ5IHRoZQo+IHNhdmFn ZV9iY2lfY21kYnVmKCkgZnVuY3Rpb24gZHVyaW5nIGVycm9yIGhhbmRsaW5nCj4gZXZlbiBpZiBh IHBhc3NlZCB2YXJpYWJsZSBjb250YWluZWQgYSBudWxsIHBvaW50ZXIuCj4gCj4gQWRqdXN0IGp1 bXAgdGFyZ2V0cyBhY2NvcmRpbmcgdG8gdGhlIExpbnV4IGNvZGluZyBzdHlsZSBjb252ZW50aW9u Lgo+IAo+IFNpZ25lZC1vZmYtYnk6IE1hcmt1cyBFbGZyaW5nIDxlbGZyaW5nQHVzZXJzLnNvdXJj ZWZvcmdlLm5ldD4KCk5vdCBzdXJlIHRoaXMgaXMgd29ydGggaXQsIEknbGwgcGFzcy4gUGF0Y2gg MSBtZXJnZWQuIEJ0dyBJIGNvbnNpZGVyCmNvY2NpIHBhdGNoZXMgYSBnb29kIHdheSB0byBnZXQg c3RhcnRlZCBzb21ld2hlcmUsIGJ1dCB0aGVuIGl0J3MgbXVjaCBtb3JlCnVzZWZ1bCB0byBkbyBh IGJpdCBtb3JlIGludm9sdmVkIHRoaW5ncy4gV2Uga2VlcCBhIGxpc3Qgb2Ygc21hbGwmYmlnCmph bml0b3IgdGFza3M6CgpodHRwczovL3d3dy54Lm9yZy93aWtpL0RSTUphbml0b3JzLwoKQ2xlYW5p bmcgdXAgYWxsIHRoZSBjb2NjaSBlcnJvcnMgaW4gZHJtIGlzbid0IGdvb2Qgc2luY2UgdGhlbiB0 aGUgbmV4dApwZXJzb24gd29uJ3QgaGF2ZSBzb21ldGhpbmcgZWFzeSB0byBnZXQgc3RhcnRlZCwg aS5lLiBjb25zaWRlciB5b3UncmUKYnVkZ2V0IHVzZWQgdXAgOy0pCi1EYW5pZWwKCj4gLS0tCj4g IGRyaXZlcnMvZ3B1L2RybS9zYXZhZ2Uvc2F2YWdlX3N0YXRlLmMgfCAzMCArKysrKysrKysrKysr KystLS0tLS0tLS0tLS0tLS0KPiAgMSBmaWxlIGNoYW5nZWQsIDE1IGluc2VydGlvbnMoKyksIDE1 IGRlbGV0aW9ucygtKQo+IAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vc2F2YWdlL3Nh dmFnZV9zdGF0ZS5jIGIvZHJpdmVycy9ncHUvZHJtL3NhdmFnZS9zYXZhZ2Vfc3RhdGUuYwo+IGlu ZGV4IDNkYzBkOGYuLjViNDg0YWEgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL3NhdmFn ZS9zYXZhZ2Vfc3RhdGUuYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9zYXZhZ2Uvc2F2YWdlX3N0 YXRlLmMKPiBAQCAtMTAwNCw3ICsxMDA0LDcgQEAgaW50IHNhdmFnZV9iY2lfY21kYnVmKHN0cnVj dCBkcm1fZGV2aWNlICpkZXYsIHZvaWQgKmRhdGEsIHN0cnVjdCBkcm1fZmlsZSAqZmlsZV8KPiAg CQlrdmJfYWRkciA9IG1lbWR1cF91c2VyKGNtZGJ1Zi0+dmJfYWRkciwgY21kYnVmLT52Yl9zaXpl KTsKPiAgCQlpZiAoSVNfRVJSKGt2Yl9hZGRyKSkgewo+ICAJCQlyZXQgPSBQVFJfRVJSKGt2Yl9h ZGRyKTsKPiAtCQkJZ290byBkb25lOwo+ICsJCQlnb3RvIGZyZWVfY21kOwo+ICAJCX0KPiAgCQlj bWRidWYtPnZiX2FkZHIgPSBrdmJfYWRkcjsKPiAgCX0KPiBAQCAtMTAxMywxMyArMTAxMywxMyBA QCBpbnQgc2F2YWdlX2JjaV9jbWRidWYoc3RydWN0IGRybV9kZXZpY2UgKmRldiwgdm9pZCAqZGF0 YSwgc3RydWN0IGRybV9maWxlICpmaWxlXwo+ICAJCQkJCSAgR0ZQX0tFUk5FTCk7Cj4gIAkJaWYg KGtib3hfYWRkciA9PSBOVUxMKSB7Cj4gIAkJCXJldCA9IC1FTk9NRU07Cj4gLQkJCWdvdG8gZG9u ZTsKPiArCQkJZ290byBmcmVlX3ZiOwo+ICAJCX0KPiAgCj4gIAkJaWYgKGNvcHlfZnJvbV91c2Vy KGtib3hfYWRkciwgY21kYnVmLT5ib3hfYWRkciwKPiAgCQkJCSAgICAgICBjbWRidWYtPm5ib3gg KiBzaXplb2Yoc3RydWN0IGRybV9jbGlwX3JlY3QpKSkgewo+ICAJCQlyZXQgPSAtRUZBVUxUOwo+ IC0JCQlnb3RvIGRvbmU7Cj4gKwkJCWdvdG8gZnJlZV92YjsKPiAgCQl9Cj4gIAljbWRidWYtPmJv eF9hZGRyID0ga2JveF9hZGRyOwo+ICAJfQo+IEBAIC0xMDUyLDcgKzEwNTIsNyBAQCBpbnQgc2F2 YWdlX2JjaV9jbWRidWYoc3RydWN0IGRybV9kZXZpY2UgKmRldiwgdm9pZCAqZGF0YSwgc3RydWN0 IGRybV9maWxlICpmaWxlXwo+ICAJCQkJCSAgImJleW9uZCBlbmQgb2YgY29tbWFuZCBidWZmZXJc biIpOwo+ICAJCQkJRE1BX0ZMVVNIKCk7Cj4gIAkJCQlyZXQgPSAtRUlOVkFMOwo+IC0JCQkJZ290 byBkb25lOwo+ICsJCQkJZ290byBmcmVlX2JveDsKPiAgCQkJfQo+ICAJCQkvKiBmYWxsIHRocm91 Z2ggKi8KPiAgCQljYXNlIFNBVkFHRV9DTURfRE1BX1BSSU06Cj4gQEAgLTEwNzEsNyArMTA3MSw3 IEBAIGludCBzYXZhZ2VfYmNpX2NtZGJ1ZihzdHJ1Y3QgZHJtX2RldmljZSAqZGV2LCB2b2lkICpk YXRhLCBzdHJ1Y3QgZHJtX2ZpbGUgKmZpbGVfCj4gIAkJCQkgICAgICBjbWRidWYtPnZiX3N0cmlk ZSwKPiAgCQkJCSAgICAgIGNtZGJ1Zi0+bmJveCwgY21kYnVmLT5ib3hfYWRkcik7Cj4gIAkJCQlp ZiAocmV0ICE9IDApCj4gLQkJCQkJZ290byBkb25lOwo+ICsJCQkJCWdvdG8gZnJlZV9ib3g7Cj4g IAkJCQlmaXJzdF9kcmF3X2NtZCA9IE5VTEw7Cj4gIAkJCX0KPiAgCQl9Cj4gQEAgLTEwODYsNyAr MTA4Niw3IEBAIGludCBzYXZhZ2VfYmNpX2NtZGJ1ZihzdHJ1Y3QgZHJtX2RldmljZSAqZGV2LCB2 b2lkICpkYXRhLCBzdHJ1Y3QgZHJtX2ZpbGUgKmZpbGVfCj4gIAkJCQkJICAiYmV5b25kIGVuZCBv ZiBjb21tYW5kIGJ1ZmZlclxuIik7Cj4gIAkJCQlETUFfRkxVU0goKTsKPiAgCQkJCXJldCA9IC1F SU5WQUw7Cj4gLQkJCQlnb3RvIGRvbmU7Cj4gKwkJCQlnb3RvIGZyZWVfYm94Owo+ICAJCQl9Cj4g IAkJCXJldCA9IHNhdmFnZV9kaXNwYXRjaF9zdGF0ZShkZXZfcHJpdiwgJmNtZF9oZWFkZXIsCj4g IAkJCQkoY29uc3QgdWludDMyX3QgKiljbWRidWYtPmNtZF9hZGRyKTsKPiBAQCAtMTA5OSw3ICsx MDk5LDcgQEAgaW50IHNhdmFnZV9iY2lfY21kYnVmKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsIHZv aWQgKmRhdGEsIHN0cnVjdCBkcm1fZmlsZSAqZmlsZV8KPiAgCQkJCQkgICJiZXlvbmQgZW5kIG9m IGNvbW1hbmQgYnVmZmVyXG4iKTsKPiAgCQkJCURNQV9GTFVTSCgpOwo+ICAJCQkJcmV0ID0gLUVJ TlZBTDsKPiAtCQkJCWdvdG8gZG9uZTsKPiArCQkJCWdvdG8gZnJlZV9ib3g7Cj4gIAkJCX0KPiAg CQkJcmV0ID0gc2F2YWdlX2Rpc3BhdGNoX2NsZWFyKGRldl9wcml2LCAmY21kX2hlYWRlciwKPiAg CQkJCQkJICAgIGNtZGJ1Zi0+Y21kX2FkZHIsCj4gQEAgLTExMTcsMTIgKzExMTcsMTIgQEAgaW50 IHNhdmFnZV9iY2lfY21kYnVmKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsIHZvaWQgKmRhdGEsIHN0 cnVjdCBkcm1fZmlsZSAqZmlsZV8KPiAgCQkJCSAgY21kX2hlYWRlci5jbWQuY21kKTsKPiAgCQkJ RE1BX0ZMVVNIKCk7Cj4gIAkJCXJldCA9IC1FSU5WQUw7Cj4gLQkJCWdvdG8gZG9uZTsKPiArCQkJ Z290byBmcmVlX2JveDsKPiAgCQl9Cj4gIAo+ICAJCWlmIChyZXQgIT0gMCkgewo+ICAJCQlETUFf RkxVU0goKTsKPiAtCQkJZ290byBkb25lOwo+ICsJCQlnb3RvIGZyZWVfYm94Owo+ICAJCX0KPiAg CX0KPiAgCj4gQEAgLTExMzMsNyArMTEzMyw3IEBAIGludCBzYXZhZ2VfYmNpX2NtZGJ1ZihzdHJ1 Y3QgZHJtX2RldmljZSAqZGV2LCB2b2lkICpkYXRhLCBzdHJ1Y3QgZHJtX2ZpbGUgKmZpbGVfCj4g IAkJCWNtZGJ1Zi0+bmJveCwgY21kYnVmLT5ib3hfYWRkcik7Cj4gIAkJaWYgKHJldCAhPSAwKSB7 Cj4gIAkJCURNQV9GTFVTSCgpOwo+IC0JCQlnb3RvIGRvbmU7Cj4gKwkJCWdvdG8gZnJlZV9ib3g7 Cj4gIAkJfQo+ICAJfQo+ICAKPiBAQCAtMTE0NywxMSArMTE0NywxMSBAQCBpbnQgc2F2YWdlX2Jj aV9jbWRidWYoc3RydWN0IGRybV9kZXZpY2UgKmRldiwgdm9pZCAqZGF0YSwgc3RydWN0IGRybV9m aWxlICpmaWxlXwo+ICAJCXNhdmFnZV9mcmVlbGlzdF9wdXQoZGV2LCBkbWFidWYpOwo+ICAJfQo+ ICAKPiAtZG9uZToKPiAtCS8qIElmIHdlIGRpZG4ndCBuZWVkIHRvIGFsbG9jYXRlIHRoZW0sIHRo ZXNlJ2xsIGJlIE5VTEwgKi8KPiAtCWtmcmVlKGtjbWRfYWRkcik7Cj4gLQlrZnJlZShrdmJfYWRk cik7Cj4gK2ZyZWVfYm94Ogo+ICAJa2ZyZWUoa2JveF9hZGRyKTsKPiAtCj4gK2ZyZWVfdmI6Cj4g KwlrZnJlZShrdmJfYWRkcik7Cj4gK2ZyZWVfY21kOgo+ICsJa2ZyZWUoa2NtZF9hZGRyKTsKPiAg CXJldHVybiByZXQ7Cj4gIH0KPiAtLSAKPiAyLjkuMwo+IAoKLS0gCkRhbmllbCBWZXR0ZXIKU29m dHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9uCmh0dHA6Ly9ibG9nLmZmd2xsLmNoCl9f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBt YWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3Rz LmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=