From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joonas Lahtinen Subject: Re: [PATCH 50/55] drm/i915: Move request list retirement to i915_gem_request.c Date: Tue, 26 Jul 2016 15:48:44 +0300 Message-ID: <1469537324.4681.87.camel@linux.intel.com> References: <1469467954-3920-1-git-send-email-chris@chris-wilson.co.uk> <1469467954-3920-51-git-send-email-chris@chris-wilson.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTP id C0EF66E4EA for ; Tue, 26 Jul 2016 12:48:46 +0000 (UTC) In-Reply-To: <1469467954-3920-51-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 T24gbWEsIDIwMTYtMDctMjUgYXQgMTg6MzIgKzAxMDAsIENocmlzIFdpbHNvbiB3cm90ZToKPiBB cyB0aGUgbGlzdCByZXRpcmVtZW50IGlzIG5vdyBjbGVhbiBvZiBpbXBsZW1lbnRhdGlvbiBkZXRh aWxzLCB3ZSBjYW4KPiBtb3ZlIGl0IGNsb3NlciB0byB0aGUgcmVxdWVzdCBtYW5hZ2VtZW50Lgo+ IAoKVGhpcyBpcyBhZ2FpbiBhIGNvbWJpbmVkIG1vdGlvbiArIHNpbXBsaWZpY2F0aW9uLCB3aGlj aCBzaG91bGQgYmUgc3BsaXQKbW90aW9uIGZpcnN0IHRoZW4gY2hhbmdlcy4uLgoKTm90IHNwbGl0 dGluZyBtYWtlcyBldmVuIHNpbXBsZSBwYXRjaGVzIHNsb3cgdG8gcmV2aWV3LgoKUmV2aWV3ZWQt Ynk6IEpvb25hcyBMYWh0aW5lbiA8am9vbmFzLmxhaHRpbmVuQGxpbnV4LmludGVsLmNvbT4KCj4g U2lnbmVkLW9mZi1ieTogQ2hyaXMgV2lsc29uIDxjaHJpc0BjaHJpcy13aWxzb24uY28udWs+Cj4g LS0tCj4gwqBkcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbS5jwqDCoMKgwqDCoMKgwqDCoMKg fCA0NCAtLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KPiDCoGRyaXZlcnMvZ3B1L2Ry bS9pOTE1L2k5MTVfZ2VtX3JlcXVlc3QuYyB8IDM1ICsrKysrKysrKysrKysrKysrKysrKysrKysr Cj4gwqAyIGZpbGVzIGNoYW5nZWQsIDM1IGluc2VydGlvbnMoKyksIDQ0IGRlbGV0aW9ucygtKQo+ IAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbS5jIGIvZHJpdmVy cy9ncHUvZHJtL2k5MTUvaTkxNV9nZW0uYwo+IGluZGV4IGNjODRmMDBjZjg4My4uYzU3MmM4MGE2 NjA0IDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZ2VtLmMKPiArKysg Yi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbS5jCj4gQEAgLTI1NDEsNTAgKzI1NDEsNiBA QCB2b2lkIGk5MTVfZ2VtX3Jlc2V0KHN0cnVjdCBkcm1fZGV2aWNlICpkZXYpCj4gwqAJaTkxNV9n ZW1fcmVzdG9yZV9mZW5jZXMoZGV2KTsKPiDCoH0KPiDCoAo+IC0vKioKPiAtICogVGhpcyBmdW5j dGlvbiBjbGVhcnMgdGhlIHJlcXVlc3QgbGlzdCBhcyBzZXF1ZW5jZSBudW1iZXJzIGFyZSBwYXNz ZWQuCj4gLSAqIEBlbmdpbmU6IGVuZ2luZSB0byByZXRpcmUgcmVxdWVzdHMgb24KPiAtICovCj4g LXZvaWQKPiAtaTkxNV9nZW1fcmV0aXJlX3JlcXVlc3RzX3Jpbmcoc3RydWN0IGludGVsX2VuZ2lu ZV9jcyAqZW5naW5lKQo+IC17Cj4gLQl3aGlsZSAoIWxpc3RfZW1wdHkoJmVuZ2luZS0+cmVxdWVz dF9saXN0KSkgewo+IC0JCXN0cnVjdCBkcm1faTkxNV9nZW1fcmVxdWVzdCAqcmVxdWVzdDsKPiAt Cj4gLQkJcmVxdWVzdCA9IGxpc3RfZmlyc3RfZW50cnkoJmVuZ2luZS0+cmVxdWVzdF9saXN0LAo+ IC0JCQkJCcKgwqDCoHN0cnVjdCBkcm1faTkxNV9nZW1fcmVxdWVzdCwKPiAtCQkJCQnCoMKgwqBs aW5rKTsKPiAtCj4gLQkJaWYgKCFpOTE1X2dlbV9yZXF1ZXN0X2NvbXBsZXRlZChyZXF1ZXN0KSkK PiAtCQkJYnJlYWs7Cj4gLQo+IC0JCWk5MTVfZ2VtX3JlcXVlc3RfcmV0aXJlX3VwdG8ocmVxdWVz dCk7Cj4gLQl9Cj4gLX0KPiAtCj4gLXZvaWQgaTkxNV9nZW1fcmV0aXJlX3JlcXVlc3RzKHN0cnVj dCBkcm1faTkxNV9wcml2YXRlICpkZXZfcHJpdikKPiAtewo+IC0Jc3RydWN0IGludGVsX2VuZ2lu ZV9jcyAqZW5naW5lOwo+IC0KPiAtCWxvY2tkZXBfYXNzZXJ0X2hlbGQoJmRldl9wcml2LT5kcm0u c3RydWN0X211dGV4KTsKPiAtCj4gLQlpZiAoZGV2X3ByaXYtPmd0LmFjdGl2ZV9lbmdpbmVzID09 IDApCj4gLQkJcmV0dXJuOwo+IC0KPiAtCUdFTV9CVUdfT04oIWRldl9wcml2LT5ndC5hd2FrZSk7 Cj4gLQo+IC0JZm9yX2VhY2hfZW5naW5lKGVuZ2luZSwgZGV2X3ByaXYpIHsKPiAtCQlpOTE1X2dl bV9yZXRpcmVfcmVxdWVzdHNfcmluZyhlbmdpbmUpOwo+IC0JCWlmIChsaXN0X2VtcHR5KCZlbmdp bmUtPnJlcXVlc3RfbGlzdCkpCj4gLQkJCWRldl9wcml2LT5ndC5hY3RpdmVfZW5naW5lcyAmPSB+ aW50ZWxfZW5naW5lX2ZsYWcoZW5naW5lKTsKPiAtCX0KPiAtCj4gLQlpZiAoZGV2X3ByaXYtPmd0 LmFjdGl2ZV9lbmdpbmVzID09IDApCj4gLQkJcXVldWVfZGVsYXllZF93b3JrKGRldl9wcml2LT53 cSwKPiAtCQkJCcKgwqDCoCZkZXZfcHJpdi0+Z3QuaWRsZV93b3JrLAo+IC0JCQkJwqDCoMKgbXNl Y3NfdG9famlmZmllcygxMDApKTsKPiAtfQo+IC0KPiDCoHN0YXRpYyB2b2lkCj4gwqBpOTE1X2dl bV9yZXRpcmVfd29ya19oYW5kbGVyKHN0cnVjdCB3b3JrX3N0cnVjdCAqd29yaykKPiDCoHsKPiBk aWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW1fcmVxdWVzdC5jIGIvZHJp dmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW1fcmVxdWVzdC5jCj4gaW5kZXggMTkzNTU5MWE5OGMx Li5mNDE1NzJhYTUzZDQgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9n ZW1fcmVxdWVzdC5jCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW1fcmVxdWVz dC5jCj4gQEAgLTcxNiwzICs3MTYsMzggQEAgY29tcGxldGU6Cj4gwqAKPiDCoAlyZXR1cm4gcmV0 Owo+IMKgfQo+ICsKPiArdm9pZCBpOTE1X2dlbV9yZXRpcmVfcmVxdWVzdHNfcmluZyhzdHJ1Y3Qg aW50ZWxfZW5naW5lX2NzICplbmdpbmUpCgpfZW5naW5lPwoKPiArewo+ICsJc3RydWN0IGRybV9p OTE1X2dlbV9yZXF1ZXN0ICpyZXF1ZXN0LCAqbmV4dDsKPiArCj4gKwlsaXN0X2Zvcl9lYWNoX2Vu dHJ5X3NhZmUocmVxdWVzdCwgbmV4dCwgJmVuZ2luZS0+cmVxdWVzdF9saXN0LCBsaW5rKSB7Cj4g KwkJaWYgKCFpOTE1X2dlbV9yZXF1ZXN0X2NvbXBsZXRlZChyZXF1ZXN0KSkKPiArCQkJYnJlYWs7 Cj4gKwo+ICsJCWk5MTVfZ2VtX3JlcXVlc3RfcmV0aXJlKHJlcXVlc3QpOwo+ICsJfQo+ICt9Cj4g Kwo+ICt2b2lkIGk5MTVfZ2VtX3JldGlyZV9yZXF1ZXN0cyhzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0 ZSAqZGV2X3ByaXYpCj4gK3sKPiArCXN0cnVjdCBpbnRlbF9lbmdpbmVfY3MgKmVuZ2luZTsKPiAr Cj4gKwlsb2NrZGVwX2Fzc2VydF9oZWxkKCZkZXZfcHJpdi0+ZHJtLnN0cnVjdF9tdXRleCk7Cj4g Kwo+ICsJaWYgKGRldl9wcml2LT5ndC5hY3RpdmVfZW5naW5lcyA9PSAwKQo+ICsJCXJldHVybjsK PiArCj4gKwlHRU1fQlVHX09OKCFkZXZfcHJpdi0+Z3QuYXdha2UpOwo+ICsKPiArCWZvcl9lYWNo X2VuZ2luZShlbmdpbmUsIGRldl9wcml2KSB7Cj4gKwkJaTkxNV9nZW1fcmV0aXJlX3JlcXVlc3Rz X3JpbmcoZW5naW5lKTsKPiArCQlpZiAobGlzdF9lbXB0eSgmZW5naW5lLT5yZXF1ZXN0X2xpc3Qp KQo+ICsJCQlkZXZfcHJpdi0+Z3QuYWN0aXZlX2VuZ2luZXMgJj0gfmludGVsX2VuZ2luZV9mbGFn KGVuZ2luZSk7Cj4gKwl9Cj4gKwo+ICsJaWYgKGRldl9wcml2LT5ndC5hY3RpdmVfZW5naW5lcyA9 PSAwKQo+ICsJCXF1ZXVlX2RlbGF5ZWRfd29yayhkZXZfcHJpdi0+d3EsCj4gKwkJCQnCoMKgwqAm ZGV2X3ByaXYtPmd0LmlkbGVfd29yaywKPiArCQkJCcKgwqDCoG1zZWNzX3RvX2ppZmZpZXMoMTAw KSk7Cj4gK30KLS0gCkpvb25hcyBMYWh0aW5lbgpPcGVuIFNvdXJjZSBUZWNobm9sb2d5IENlbnRl cgpJbnRlbCBDb3Jwb3JhdGlvbgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVz a3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9p bnRlbC1nZngK