From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tvrtko Ursulin Subject: Re: [PATCH v2] drm/i915: Apply a cond_resched() to the saturated signaler Date: Tue, 4 Apr 2017 13:38:36 +0100 Message-ID: <5dc89289-d707-2db9-ddee-e0a50a5be120@linux.intel.com> References: <20170404101059.4116-1-chris@chris-wilson.co.uk> <20170404120531.10737-1-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 mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id 90B056E036 for ; Tue, 4 Apr 2017 12:38:45 +0000 (UTC) In-Reply-To: <20170404120531.10737-1-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 Cc: Mika Kuoppala List-Id: intel-gfx@lists.freedesktop.org Ck9uIDA0LzA0LzIwMTcgMTM6MDUsIENocmlzIFdpbHNvbiB3cm90ZToKPiBJZiB0aGUgZW5naW5l IGlzIGNvbnRpbnVhbGx5IGNvbXBsZXRpbmcgbm9wcywgd2UgY2FuIHNhdHVyYXRlIHRoZQo+IHNp Z25hbGVyIGFuZCBrZWVwIGl0IHdvcmtpbmcgaW5kZWZpbml0ZWx5LiBUaGlzIGFuZ2VycyB0aGUg Tk1JIHdhdGNoZG9nIQo+Cj4gQSBnb29kIGV4YW1wbGUgaXMgdG8gZGlzYWJsZSBzZW1hcGhvcmVz IG9uIHNuYiBhbmQgcnVuIGlndC9nZW1fZXhlY19ub3AgLQo+IHRoZSBwYXJhbGxlbCwgbXVsdGkt ZW5naW5lIHdvcmtsb2FkcyBhcmUgbW9yZSB0aGFuIHN1ZmZpY2llbnQgdG8gaG9nIHRoZQo+IENQ VSwgcHJldmVudGluZyB0aGUgc3lzdGVtIGZyb20gZXZlbiBwcm9jZXNzaW5nIElDTVAgZWNobyBy ZXBsaWVzLgo+Cj4gdjI6IFR2cnRrbyBkdWcgaW50byBjb25kX3Jlc2NoZWQoKSBvbiB4ODYgYW5k IGZvdW5kIHRoYXQgaXQgb25seQo+IGRlcGVuZGVkIHVwb24gcHJlZW1wdF9jb3VuZCBhbmQgbm90 IHRpZl9uZWVkX3Jlc2NoZWQoKSAtIHdoaWNoIG1lYW5zCj4gdGhhdCB3ZSB3b3VsZCBhbHdheXMg Y2FsbCBzY2hlZHVsZSgpIGF0IHRoYXQgcG9pbnQuCj4KPiBGaXhlczogYzgxZDQ2MTM4ZGE2ICgi ZHJtL2k5MTU6IENvbnZlcnQgdHJhY2UtaXJxIHRvIHRoZSBicmVhZGNydW1iIHdhaXRlciIpCj4g U2lnbmVkLW9mZi1ieTogQ2hyaXMgV2lsc29uIDxjaHJpc0BjaHJpcy13aWxzb24uY28udWs+Cj4g Q2M6IE1pa2EgS3VvcHBhbGEgPG1pa2Eua3VvcHBhbGFAaW50ZWwuY29tPgo+IENjOiBUdnJ0a28g VXJzdWxpbiA8dHZydGtvLnVyc3VsaW5AaW50ZWwuY29tPgo+IC0tLQo+ICBkcml2ZXJzL2dwdS9k cm0vaTkxNS9pbnRlbF9icmVhZGNydW1icy5jIHwgMTUgKysrKysrKysrKysrKystCj4gIDEgZmls ZSBjaGFuZ2VkLCAxNCBpbnNlcnRpb25zKCspLCAxIGRlbGV0aW9uKC0pCj4KPiBkaWZmIC0tZ2l0 IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfYnJlYWRjcnVtYnMuYyBiL2RyaXZlcnMvZ3B1 L2RybS9pOTE1L2ludGVsX2JyZWFkY3J1bWJzLmMKPiBpbmRleCAzMDhjNTZhMDIxYWIuLjljY2Jm MjYxMjRjNiAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9icmVhZGNy dW1icy5jCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfYnJlYWRjcnVtYnMuYwo+ IEBAIC01ODAsNiArNTgwLDggQEAgc3RhdGljIGludCBpbnRlbF9icmVhZGNydW1ic19zaWduYWxl cih2b2lkICphcmcpCj4gIAlzaWduYWxlcl9zZXRfcnRwcmlvcml0eSgpOwo+Cj4gIAlkbyB7Cj4g KwkJYm9vbCBkb19zY2hlZHVsZSA9IHRydWU7Cj4gKwo+ICAJCXNldF9jdXJyZW50X3N0YXRlKFRB U0tfSU5URVJSVVBUSUJMRSk7Cj4KPiAgCQkvKiBXZSBhcmUgZWl0aGVyIHdva2VuIHVwIGJ5IHRo ZSBpbnRlcnJ1cHQgYm90dG9tLWhhbGYsCj4gQEAgLTYyNiw3ICs2MjgsMTggQEAgc3RhdGljIGlu dCBpbnRlbF9icmVhZGNydW1ic19zaWduYWxlcih2b2lkICphcmcpCj4gIAkJCXNwaW5fdW5sb2Nr X2lycSgmYi0+cmJfbG9jayk7Cj4KPiAgCQkJaTkxNV9nZW1fcmVxdWVzdF9wdXQocmVxdWVzdCk7 Cj4gLQkJfSBlbHNlIHsKPiArCj4gKwkJCS8qIElmIHRoZSBlbmdpbmUgaXMgc2F0dXJhdGVkIHdl IG1heSBiZSBjb250aW51YWxseQo+ICsJCQkgKiBwcm9jZXNzaW5nIGNvbXBsZXRlZCByZXF1ZXN0 cy4gVGhpcyBhbmdlcnMgdGhlCj4gKwkJCSAqIE5NSSB3YXRjaGRvZyBpZiB3ZSBuZXZlciBsZXQg YW55dGhpbmcgZWxzZQo+ICsJCQkgKiBoYXZlIGFjY2VzcyB0byB0aGUgQ1BVLiBMZXQncyBwcmV0 ZW5kIHRvIGJlIG5pY2UKPiArCQkJICogYW5kIHJlbGlucXVpc2ggdGhlIENQVSBpZiB3ZSBidXJu IHRocm91Z2ggdGhlCj4gKwkJCSAqIGVudGlyZSBSVCB0aW1lc2xpY2UhCj4gKwkJCSAqLwo+ICsJ CQlkb19zY2hlZHVsZSA9IG5lZWRfcmVzY2hlZCgpOwo+ICsJCX0KPiArCj4gKwkJaWYgKHVubGlr ZWx5KGRvX3NjaGVkdWxlKSkgewo+ICAJCQlERUZJTkVfV0FJVChleGVjKTsKPgo+ICAJCQlpZiAo a3RocmVhZF9zaG91bGRfcGFyaygpKQo+CgpNeSB0aGlua2luZyB3YXMgdG8gYWRkIGEgY2hlY2sg YmVmb3JlIHRoZSByZXF1ZXN0IGFzc2lnbm1lbnQgbGlrZToKCglyY3VfcmVhZF9sb2NrKCk7Cgly ZXF1ZXN0ID0gLi4uOwoJaWYgKHJlcXVlc3QgJiYgIW5lZWRfcmVzY2hlZCgpKQoJCXJlcXVlc3Qg PSAuLi47CgllbHNlCgkJcmVxdWVzdCA9IE5VTEw7CglyY3VfcmVhZF91bmxvY2soKTsKCkJ1dCB0 aGlzIGxvb2tzIGNvcnJlY3QgYXMgd2VsbCwgbWF5YmUgaXQgaXMganVzdCBteSBwcmVmZXJlbmNl IG9uIHdoYXQgCndvdWxkIGhhdmUgYmVlbiBlYXNpZXIgdG8gdW5kZXJzdGFuZC4KCkkgdHJ1c3Qg dGhhdCB5b3UgaGF2ZSB0ZXN0ZWQgaXQgYm90aCBmb3Igc29sdmluZyB0aGUgTk1JIGxvY2t1cCBk ZXRlY3RvciAKaXNzdWUgYW5kIHRoYXQgaXQgZG9lc24ndCBhZmZlY3QgdGhlIHNpZ25hbGxlciBs YXRlbmN5IGEgbG90LgoKUmV2aWV3ZWQtYnk6IFR2cnRrbyBVcnN1bGluIDx0dnJ0a28udXJzdWxp bkBpbnRlbC5jb20+CgpSZWdhcmRzLAoKVHZydGtvCgpfX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBs aXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1h bi9saXN0aW5mby9pbnRlbC1nZngK