From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tvrtko Ursulin Subject: Re: [PATCH 2/2] drm/i915: Limit the busy wait on requests to 2us not 10ms! Date: Mon, 16 Nov 2015 12:08:08 +0000 Message-ID: <5649C728.5040109@linux.intel.com> References: <1447594364-4206-1-git-send-email-chris@chris-wilson.co.uk> <1447594364-4206-2-git-send-email-chris@chris-wilson.co.uk> <5649AEED.9090807@linux.intel.com> <20151116111208.GQ569@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20151116111208.GQ569@nuc-i3427.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson , Jens Axboe , intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Daniel Vetter , Eero Tamminen , "Rantala, Valtteri" , stable@kernel.vger.org List-Id: intel-gfx@lists.freedesktop.org Ck9uIDE2LzExLzE1IDExOjEyLCBDaHJpcyBXaWxzb24gd3JvdGU6Cj4gT24gTW9uLCBOb3YgMTYs IDIwMTUgYXQgMTA6MjQ6NDVBTSArMDAwMCwgVHZydGtvIFVyc3VsaW4gd3JvdGU6Cj4+Cj4+IEhp LAo+Pgo+PiBPbiAxNS8xMS8xNSAxMzozMiwgQ2hyaXMgV2lsc29uIHdyb3RlOgo+Pj4gV2hlbiB3 YWl0aW5nIGZvciBoaWdoIGZyZXF1ZW5jeSByZXF1ZXN0cywgdGhlIGZpbml0ZSBhbW91bnQgb2Yg dGltZQo+Pj4gcmVxdWlyZWQgdG8gc2V0IHVwIHRoZSBpcnEgYW5kIHdhaXQgdXBvbiBpdCBsaW1p dHMgdGhlIHJlc3BvbnNlIHJhdGUuIEJ5Cj4+PiBidXN5d2FpdGluZyBvbiB0aGUgcmVxdWVzdCBj b21wbGV0aW9uIGZvciBhIHNob3J0IHdoaWxlIHdlIGNhbiBzZXJ2aWNlCj4+PiB0aGUgaGlnaCBm cmVxdWVuY3kgd2FpdHMgYXMgcXVpY2sgYXMgcG9zc2libGUuIEhvd2V2ZXIsIGlmIGl0IGlzIGEg c2xvdwo+Pj4gcmVxdWVzdCwgd2Ugd2FudCB0byBzbGVlcCBhcyBxdWlja2x5IGFzIHBvc3NpYmxl LiBUaGUgdHJhZGVvZmYgYmV0d2Vlbgo+Pj4gd2FpdGluZyBhbmQgc2xlZXBpbmcgaXMgcm91Z2hs eSB0aGUgdGltZSBpdCB0YWtlcyB0byBzbGVlcCBvbiBhIHJlcXVlc3QsCj4+PiBvbiB0aGUgb3Jk ZXIgb2YgYSBtaWNyb3NlY29uZC4gQmFzZWQgb24gbWVhc3VyZW1lbnRzIGZyb20gYmlnIGNvcmUs IEkKPj4+IGhhdmUgc2V0IHRoZSBsaW1pdCBmb3IgYnVzeXdhaXRpbmcgYXMgMiBtaWNyb3NlY29u ZHMuCj4+Cj4+IFNvdW5kcyBsaWtlIHNvbGlkIHJlYXNvbmluZy4gV291bGQgaXQgYWxzbyBiZSB3 b3J0aCBmaW5kaW5nIHRoZQo+PiB0cmFkZSBvZmYgbGltaXQgZm9yIHNtYWxsIGNvcmU/Cj4KPiBU YWtlcyBhIGJpdCBsb25nZXIsIGJ1dCAydXMgc2VlbXMgIm9rIiBvbiBQaW5lVmlldyAoYXMgaW4g aXQgZG9lc24ndAo+IGxvc2UgdGhlIGJvb3N0IGZyb20gc3Bpbm5pbmcgcmF0aGVyIHRoYW4gc2xl ZXBpbmcpLiBIYXZlIHNvbWUgbW9yZQo+IHRlc3RpbmcgdG8gZG8gb24gdmx2L2J5dC4KCkNvb2wu Cgo+Pj4gVGhlIGNvZGUgY3VycmVudGx5IHVzZXMgdGhlIGppZmZpZSBjbG9jaywgYnV0IHRoYXQg aXMgZmFyIHRvbyBjb2Fyc2UgKG9uCj4+PiB0aGUgb3JkZXIgb2YgMTAgbWlsbGlzZWNvbmRzKSBh bmQgcmVzdWx0cyBpbiBwb29yIGludGVyYWN0aXZpdHkgYXMgdGhlCj4+PiBDUFUgZW5kcyB1cCBi ZWluZyBob2dnZWQgYnkgc2xvdyByZXF1ZXN0cy4gVG8gZ2V0IG1pY3Jvc2Vjb25kIHJlc29sdXRp b24KPj4+IHdlIG5lZWQgdG8gdXNlIGEgaGlnaCByZXNvbHV0aW9uIHRpbWVyLiBUaGUgY2hlYXBl c3Qgb2Ygd2hpY2ggaXMgcG9sbGluZwo+Pj4gbG9jYWxfY2xvY2soKSwgYnV0IHRoYXQgaXMgb25s eSB2YWxpZCBvbiB0aGUgc2FtZSBDUFUuIElmIHdlIHN3aXRjaCBDUFVzCj4+PiBiZWNhdXNlIHRo ZSB0YXNrIHdhcyBwcmVlbXB0ZWQsIHdlIGNhbiBhbHNvIHVzZSB0aGF0IGFzIGFuIGluZGljYXRv ciB0aGF0Cj4+PiAgIHRoZSBzeXN0ZW0gaXMgdG9vIGJ1c3kgdG8gd2FzdGUgY3ljbGVzIG9uIHNw aW5uaW5nIGFuZCB3ZSBzaG91bGQgc2xlZXAKPj4+IGluc3RlYWQuCj4+Cj4+IEhtLCBuZWVkX3Jl c2NoZWQgd291bGQgbm90IGNvdmVyIHRoZSBDUFUgc3dpdGNoIGFueXdheT8gT3IgbWF5YmUKPj4g bmVlZF9yZXNjaGVkIG1lYW5zIHNvbWV0aGluZyBvdGhlciB0aGFuIEkgdGhvdWdodCB3aGljaCBp cyAidGhlcmUKPj4gYXJlIG90aGVyIHJ1bm5hYmxlIHRhc2tzIj8KPgo+IEFzIEkgdW5kZXJzdGFu ZCBpdCwgaXQgbWVhbnMgdGhhdCB0aGUgc2NoZWR1bGVyIHRpY2sgZmlyZWQgKG9yIHNvbWV0aGlu Zwo+IGVsc2UgeWllbGRlZCkuIEkgaGF2ZW4ndCBzcG90dGVkIGlmIGl0IGdldHMgc2V0IGFzIHRo ZSBydW5xdWV1ZSBjaGFuZ2VzLgo+IEFzIGl0IHBlcnRhaW5zIHRvIHVzLCBpdCBtZWFucyB0aGF0 IHdlIG5lZWQgdG8gZ2V0IHRvIHNjaGVkdWxlKCkgYXMKPiBxdWljayBhcyBwb3NzaWJsZSB3aGlj aCBhbG9uZyB0aGlzIHBhdGggbWVhbnMgZ29pbmcgdG8gc2xlZXAuCj4KPiBJJ20gbm90IHN1cmUg aWYgbmVlZF9yZXNjaGVkKCkgd291bGQgY2F0Y2ggdGhlIGNwdSBzd2l0Y2gsIGlmIHdlIHdlcmUK PiBwcmVlbXB0ZWQgYXMgdGhlIGZsYWcgd291bGQgYmUgc2V0IG9uIHRoZSBpZGxlIHByb2Nlc3Mg bm90IHVzLgoKQ291bGQgYmUsIEkgd2Fzbid0IHN1cmUgYXQgYWxsLCBqdXN0IGN1cmlvdXMgYW5k IHRyeWluZyB0byB1bmRlcnN0YW5kIGl0IApmdWxseS4gQ3B1IGNoZWNrIGlzIHNvIGNoZWFwIGFz IGltcGxlbWVudGVkIHRoYXQgaXQgaXMgbm90IHVuZGVyIGFueSAKY3JpdGljaXNtLgoKPj4gVGhp cyB3b3VsZCBhbHNvIGhhdmUgaW1wYWN0IG9uIHRoZSBwYXRjaCBzdWJqZWN0IGxpbmUuSSB0aG91 Z2h0IHdlCj4+IHdvdWxkIGJ1cm4gYSBqaWZmaWUgb2YgQ1BVIGN5Y2xlcyBvbmx5IGlmIHRoZXJl IGFyZSBubyBvdGhlcgo+PiBydW5uYWJsZSB0YXNrcyAtIHNvIGhvdyBjb21lIGFuIGltcGFjdCBv biBpbnRlcmFjdGl2aXR5Pwo+Cj4gSSBoYXZlIGEgY291cGxlIG9mIGlkZWFzIGZvciB0aGUgZWZm ZWN0IG9uIGludGVyYWN0aXZ0eToKPgo+IDEuIEJ1cm5pbmcgdGhyb3VnaCB0aGUgdGltZSBzbGlj ZSBpcyBhY3RpbmcgYXMgYSBwZW5hbHR5IGFnYWluc3QgcnVubmluZwo+IHRoYXQgcHJvY2VzcyAo dHlwaWNhbGx5IHRoZSBjb21wb3NpdG9yKSBpbiB0aGUgbmVhciBmdXR1cmUsIHBlcmhhcHMKPiBl bm91Z2ggdG8gbWlzcyBzb21lIGRlYWRsaW5lcy4KPgo+IDIuIFByb2Nlc3NvciBwb3dlciBiYWxh bmNpbmcuCj4KPj4gQWxzbyBhZ2FpbiBJIHRoaW5rIHRoZSBjb21taXQgbWVzc2FnZSBuZWVkcyBz b21lIGRhdGEgb24gaG93IHRoaXMKPj4gd2FzIGZvdW5kIGFuZCB3aGF0IGlzIHRoZSBpbXBhY3Qu Cj4KPiBUaGUgc3lzdGVtIGZlbHQgdW5yZXNwb25zaXZlLiBJdCB3b3VsZCBiZSBpbnRlcmVzdGlu ZyBmb3IgbWUgdG8ga25vdyBhCj4gZmV3IG1vcmUgZGV0YWlscyBhYm91dCB0aGUgdGljayBvbiB0 aGF0IHN5c3RlbSAoSFosIHRpY2tsZXNzPywKPiBwcmVlbXB0aW9uPykgdG8gc2VlIGlmIGNoYW5n aW5nIHRoZSBjb25maWcgb24gbXkgeHBzMTMgYWxzbyBwcm9kdWNlcyB0aGUKPiBsYWcvaml0dGVy L3Bvb3IgaW50ZXJhY3RpdnR5LgoKWWVzIGludGVyZXN0aW5nIGJ1dCBub3QgY3JpdGljYWwgSSB0 aGluay4gU2luY2UgdGhlIG5ldyBzY2hlbWUgbG9va3MgYXMgCmVmZmljaWVudCBhcyB0aGUgb2xk IG9uZSBzbyB0aGVyZSBzaG91bGQgYmUgbm8gZG93bnNpZGUgYW55d2F5LgoKPj4gQnR3IGFzIGl0 IGhhcHBlbnMsIGp1c3QgbGFzdCB3ZWVrIGFzIEkgd2FzIHBsYXlpbmcgd2l0aCBwZXJmLCBJIGRp ZAo+PiBub3RpY2UgYnVzeSBzcGlubmluZyBpcyB0aGUgdG9wIGN5Y2xlIHdhc3RlciBpbiBzb21l IGJlbmNobWFya3MuIEkKPj4gd2FzIGluIHRoZSBwcm9jZXNzIG9mIHRyeWluZyB0byBxdWFudGl6 ZSB0aGUgZGlmZmVyZW5jZSB3aXRoIGl0IG9uCj4+IG9yIG9mZiBidXQgZGlkIG5vdCBjb21wbGV0 ZSBpdC4KPgo+IEkgZm91bmQgdGhhdCBzcGluLXJlcXVlc3QgYXBwZWFyaW5nIGluIHByb2ZpbGVz IG1ha2VzIHRyYWNraW5nIGRvd24gdGhlCj4gY3VscHJpdCBoaWdlciBpbiB0aGUgc3RhY2sgbXVj aCBlYXNpZXIuIE90aGVyd2lzZSB5b3UgaGF2ZSB0byByZW1lbWJlciB0bwo+IGVuYWJsZSBhIHBh c3Mgd2l0aCB0aGUgdHJhY2Vwb2ludCB0byBmaW5kIHRoZSBzdGFsbHMgKG9yIHVzZQo+IGludGVs LWdwdS1vdmVybGF5IHdoaWNoIGRvZXMgaXQgZm9yIHlvdSkuCgpJJ2xsIHB1dCBpdCBvbiBteSBU T0RPIGxpc3Qgb2YgdGhpbmdzIHRvIHBsYXkgd2l0aC4KCj4+PiArc3RhdGljIHU2NCBsb2NhbF9j bG9ja191cyh1bnNpZ25lZCAqY3B1KQo+Pj4gK3sKPj4+ICsJdTY0IHQ7Cj4+PiArCj4+PiArCSpj cHUgPSBnZXRfY3B1KCk7Cj4+PiArCXQgPSBsb2NhbF9jbG9jaygpID4+IDEwOwo+Pgo+PiBOZWVk cyBjb21tZW50IEkgdGhpbmsgdG8gZXhwbGljaXRseSBtZW50aW9uIHRoZSBhcHByb3hpbWF0aW9u LCBvcgo+PiBtYXliZSBkcm9wIHRoZSBfdXMgc3VmZml4Pwo+Cj4gSSBkaWQgY29uc2lkZXIgX2Fw cHJveF91cyBidXQgdGhvdWdodCB0aGF0IHdhcyBvdmVya2lsbC4gQSBjb21tZW50IGFsb25nCj4g dGhlIGxpbmVzIG9mCj4gLyogQXBwcm94aW1hdGVseSBjb252ZXJ0IG5zIHRvIHVzIC0gdGhlIGVy cm9yIGlzIGxlc3MgdGhhbiB0aGUKPiAgICogdHJ1bmNhdGlvbiEKPiAgICovCgpBbmQgdGhlIHJl c3VsdCBpcyBub3QgdXNlZCBpbiBzdWJzZXF1ZW50IGNhbGN1bGF0aW9ucyBhcGFydCBmcm9tIApj b21wYXJpbmcgYWdhaW5zdCBhbiBhcHByb3hpbWF0ZSB0aW1lb3V0PwoKPj4+IEBAIC0xMTYxLDcg KzExODMsNyBAQCBzdGF0aWMgaW50IF9faTkxNV9zcGluX3JlcXVlc3Qoc3RydWN0IGRybV9pOTE1 X2dlbV9yZXF1ZXN0ICpyZXEsIGludCBzdGF0ZSkKPj4+ICAgCQlpZiAoc2lnbmFsX3BlbmRpbmdf c3RhdGUoc3RhdGUsIGN1cnJlbnQpKQo+Pj4gICAJCQlicmVhazsKPj4+Cj4+PiAtCQlpZiAodGlt ZV9hZnRlcl9lcShqaWZmaWVzLCB0aW1lb3V0KSkKPj4+ICsJCWlmIChidXN5d2FpdF9zdG9wKHRp bWVvdXQsIGNwdSkpCj4+PiAgIAkJCWJyZWFrOwo+Pj4KPj4+ICAgCQljcHVfcmVsYXhfbG93bGF0 ZW5jeSgpOwo+Pj4KPj4KPj4gT3RoZXJ3aXNlIGxvb2tzIGdvb2QuIE5vdCBzdXJlIHdoYXQgd291 bGQgeW91IGNvbnZlcnQgdG8gMzItYml0IGZyb20KPj4geW91ciBmb2xsb3cgdXAgcmVwbHkgc2lu Y2UgeW91IG5lZWQgdXMgcmVzb2x1dGlvbj8KPgo+IHMvdTY0L3Vuc2lnbmVkIGxvbmcvIHMvdGlt ZV9hZnRlcjY0L3RpbWVfYWZ0ZXIvCj4KPiAzMmJpdHMgb2YgdXMgcmVzb2x1dGlvbiBnaXZlcyB1 cyAxMDAwcyBiZWZvcmUgd3JhcGFyb3VuZCBiZXR3ZWVuIHRoZSB0d28KPiBzYW1wbGVzLiBBbmQg SSBob3BlIHRoYXQgYSAxMDAwcyBkb2Vzbid0IHBhc3MgYmV0d2VlbiBsb29wcy4gT3IgaWYgaXQg ZG9lcywKPiB0aGUgR1BVIG1hbmFnZWQgdG8gY29tcGxldGUgaXRzIHRhc2suCgpOb3cgSSBzZWUg dGhhdCB5b3UgZGlkIHNheSBsb3cgYml0cy4uIHllcyB0aGF0IHNvdW5kcyBmaW5lLgoKQnR3IHdo aWxlIHlvdSBhcmUgb3B0aW1pemluZyB0aGluZ3MgbWF5YmUgcGljayB1cCB0aGlzIG1pY3JvIApv cHRpbWl6YXRpb246IGh0dHA6Ly9wYXRjaHdvcmsuZnJlZWRlc2t0b3Aub3JnL3BhdGNoLzY0MzM5 LwoKTm90IGluIHNjb3BlIG9mIHRoaXMgdGhyZWFkIGJ1dCB1bmRlciB0aGUgbm9ybWFsIGRldmVs b3BtZW50IHBhdGNoIGZsb3cuCgpCdHcyLCBhbnkgYmVuY2htYXJrIHJlc3VsdCBjaGFuZ2VzIHdp dGggdGhpcz8KClJlZ2FyZHMsCgpUdnJ0a28KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMu ZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0 aW5mby9pbnRlbC1nZngK