From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tvrtko Ursulin Subject: Re: [PATCH] drm/i915: Only force GGTT coherency w/a on required chipsets Date: Fri, 20 Jul 2018 16:27:20 +0100 Message-ID: <1bbe5fb9-e8fb-d614-527c-80ee404ac1b9@linux.intel.com> References: <20180720075931.14362-1-chris@chris-wilson.co.uk> <20180720101910.11153-1-chris@chris-wilson.co.uk> <2e0dec84-a05e-4dd0-0c77-9f342e84f8f9@linux.intel.com> <153210007558.13721.1524810019435691848@skylake-alporthouse-com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTPS id CDEBD8991D for ; Fri, 20 Jul 2018 15:27:22 +0000 (UTC) In-Reply-To: <153210007558.13721.1524810019435691848@skylake-alporthouse-com> Content-Language: en-GB 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 Ck9uIDIwLzA3LzIwMTggMTY6MjEsIENocmlzIFdpbHNvbiB3cm90ZToKPiBRdW90aW5nIFR2cnRr byBVcnN1bGluICgyMDE4LTA3LTIwIDE2OjEzOjE0KQo+Pgo+PiBPbiAyMC8wNy8yMDE4IDExOjE5 LCBDaHJpcyBXaWxzb24gd3JvdGU6Cj4+PiBOb3QgYWxsIGNoaXBzZXRzIGhhdmUgYW4gaW50ZXJu YWwgYnVmZmVyIGRlbGF5aW5nIHRoZSB2aXNpYmlsaXR5IG9mCj4+PiB3cml0ZXMgdmlhIHRoZSBH R1RUIGJlaW5nIHZpc2libGUgYnkgb3RoZXIgcGh5c2ljYWwgcGF0aHMsIGJ1dCB3ZSB1c2UgYQo+ Pj4gdmVyeSBoZWF2eSB3b3JrYXJvdW5kIGZvciBhbGwuIFdlIG9ubHkgbmVlZCB0byBhcHBseSB0 aGF0IHdvcmthcm91bmRzIHRvCj4+PiB0aGUgY2hpcHNldHMgd2Uga25vdyBzdWZmZXIgZnJvbSB0 aGUgZGVsYXkgYW5kIHRoZSByZXN1bHRpbmcgY29oZXJlbmN5Cj4+PiBpc3N1ZS4KPj4+Cj4+PiBT aW1pbGFybHksIHRoZSBzYW1lIGluY29uc2lzdGVudCBjb2hlcmVuY3kgZm91bHMgdXAgb3VyIEFC SSBwcm9taXNlIHRoYXQKPj4+IGEgd3JpdGUgaW50byBhIG1tYXBfZ3R0IGlzIGltbWVkaWF0ZWx5 IHZpc2libGUgdG8gb3RoZXJzLiBTaW5jZSB0aGUgSFcKPj4+IGhhcyBtYWRlIHRoYXQgYSBsaWUs IGxldCB1c2Vyc3BhY2Uga25vdyB3aGVuIHRoYXQgY29udHJhY3QgaXMgYnJva2VuLgo+Pj4gKE5v dCB0aGF0IHVzZXJzcGFjZSB3b3VsZCB3YW50IHRvIHVzZSBtbWFwX2d0dCBvbiB0aG9zZSBjaGlw c2V0cyBmb3IKPj4+IG90aGVyIHBlcmZvcm1hbmNlIHJlYXNvbnMuLi4pCj4+Pgo+Pj4gVGVzdGNh c2U6IGlndC9kcnZfc2VsZnRlc3QvbGl2ZV9jb2hlcmVuY3kKPj4+IFRlc3RjYXNlOiBpZ3QvZ2Vt X21tYXBfZ3R0L2NvaGVyZW5jeQo+Pgo+PiBUaGUgbGlzdCBvZiBwbGF0Zm9ybXMgdG8gbWFyayB3 aXRoIGZhbHNlIHdhcyBjb21waWxlZCBmcm9tIHRoZSB0ZXN0IHJlc3VsdHM/Cj4gCj4gWWVzLiBD b3ZlcmFnZSBvZiBvbGRlciBjaGlwc2V0cyBpcyBsZXNzIHRoYW4gaWRlYWwsIGFzIHdlIGhhdmUg ZmV3ZXIgb2YKPiB0aGVtIGJlaW5nIHRlc3RlZCBhbmQgdGhlIGdtY2ggd2Fzbid0IHRpZWQgdG8g YW55IHByb2Nlc3Nvci4gU28gd2hlcmVhcwo+IG15IFBJSUltICsgaTkxNWdtIHBhc3NlcywgQ0kn cyBQNCArIGk5MTVnIGZhaWxzLiBUaGF0IGlzIG9kZC4KCkhvcGUgdGhlIG9kZGl0eSBpcyBub3Qg aGlkaW5nIHNvbWV0aGluZyBidXQgd2UnbGwgc2VlLgoKPj4gQnV0IHRoZW4gYmVmb3JlIHRoaXMg cGF0Y2ggd29ya2Fyb3VuZCB3YXMgYXBwbGllZCBldmVyeXdoZXJlIC0gc28gaWYgdGhlCj4+IHRl c3Qgd2FzIGZhaWxpbmcgZXZlbiB0aGVuIC0gdGhhdCBtZWFucyB3b3JrYXJvdW5kIHdhc24ndCBz dWZmaWNpZW50Pwo+IAo+IFRoZSB0ZXN0IGlzIGJlaW5nIHJ1biBpbiB1c2Vyc3BhY2U7IGJ5cGFz c2luZyBhbnkgZG9tYWluIGNvbnRyb2wgaW4gdGhlCj4ga2VybmVsLCBhc3N1bWluZyB0aGUgY29o ZXJlbmN5IG1vZGVsIGJ1aWx0IGludG8gdGhlIEFCSS4KCkFoIHllcy4KCj4+PiArLyoKPj4+ICsg KiBPbmNlIHVwb24gYSB0aW1lIHdlIHN1cHBvc2VkIHRoYXQgd3JpdGVzIHRocm91Z2ggdGhlIEdH VFQgd291bGQgYmUKPj4+ICsgKiBpbW1lZGlhdGVseSBpbiBwaHlzaWNhbCBtZW1vcnkgKG9uY2Ug Zmx1c2hlZCBvdXQgb2YgdGhlIENQVSBwYXRoKS4gSG93ZXZlciwKPj4+ICsgKiBvbiBhIGZldyBk aWZmZXJlbnQgcHJvY2Vzc29ycyBhbmQgY2hpcHNldHMsIHRoaXMgaXMgbm90IG5lY2Vzc2FyaWx5 IHRoZSBjYXNlCj4+PiArICogYXMgdGhlIHdyaXRlcyBhcHBlYXIgdG8gYmUgYnVmZmVyZWQgaW50 ZXJuYWxseS4gVGh1cyBhIHJlYWQgb2YgdGhlIGJhY2tpbmcKPj4+ICsgKiBzdG9yYWdlIChwaHlz aWNhbCBtZW1vcnkpIHZpYSBhIGRpZmZlcmVudCBwYXRoICh3aXRoIGRpZmZlcmVudCBwaHlzaWNh bCB0YWdzCj4+PiArICogdG8gdGhlIGluZGlyZWN0IHdyaXRlIHZpYSB0aGUgR0dUVCkgd2lsbCBz ZWUgc3RhbGUgdmFsdWVzIGZyb20gYmVmb3JlCj4+PiArICogdGhlIEdHVFQgd3JpdGUuIEluc2lk ZSB0aGUga2VybmVsLCB3ZSBjYW4gZm9yIHRoZSBtb3N0IHBhcnQga2VlcCB0cmFjayBvZgo+Pj4g KyAqIHRoZSBkaWZmZXJlbnQgcmVhZC93cml0ZSBkb21haW5zIGluIHVzZSAoZS5nLiBzZXQtZG9t YWluKSwgYnV0IHRoZSBhc3N1bXB0aW9uCj4+PiArICogb2YgY29oZXJlbmN5IGlzIGJha2VkIGlu dG8gdGhlIEFCSSwgaGVuY2UgcmVwb3J0aW5nIGl0cyB0cnVlIHN0YXRlIGluIHRoaXMKPj4+ICsg KiBwYXJhbWV0ZXIuCj4+PiArICoKPj4+ICsgKiBJZiBzZXQgdG8gdHJ1ZSwgd3JpdGVzIHZpYSBt bWFwX2d0dCBhcmUgaW1tZWRpYXRlbHkgdmlzaWJsZSBmb2xsb3dpbmcgYW4KPj4+ICsgKiBsZmVu Y2UgdG8gZmx1c2ggdGhlIFdDQi4KPj4+ICsgKgo+Pj4gKyAqIElmIHNldCB0byBmYWxzZSwgd3Jp dGVzIHZpYSBtbWFwX2d0dCBhcmUgaW5kZXRlcm1pbmF0bmx5IGRlbGF5ZWQgaW4gYW4gaW4KPj4+ ICsgKiBpbnRlcm5hbCBidWZmZXIgYW5kIGFyZSBfbm90XyBpbW1lZGlhdGVseSB2aXNpYmxlIHRv IHRoaXJkIHBhcnRpZXMgYWNjZXNzaW5nCj4+PiArICogZGlyZWN0bHkgdmlhIG1tYXBfY3B1L21t YXBfd2MuIFVzZSBvZiBtbWFwX2d0dCBhcyBwYXJ0IG9mIGFuIElQQwo+Pj4gKyAqIGNvbW11bmlj YXRpb25zIGNoYW5uZWwgd2hlbiBzZXQgdG8gZmFsc2UgaXMgc3Ryb25nbHkgZGlzYWR2aXNlZC4K Pj4KPj4gSSB3b3VsZCBwZXJoYXBzIGNoYW5nZSB0aGUgbGFuZ3VhZ2UgZnJvbSAic2V0IHRvIHRy dWUvZmFsc2UiIHRvCj4+ICJyZXBvcnRlZCBhcyB0cnVlL2ZhbHNlIi4KPj4KPj4+ICsgKi8KPj4+ ICsjZGVmaW5lIEk5MTVfUEFSQU1fTU1BUF9HVFRfQ09IRVJFTlQgNTIKPj4+ICsKPj4+ICAgIHR5 cGVkZWYgc3RydWN0IGRybV9pOTE1X2dldHBhcmFtIHsKPj4+ICAgICAgICBfX3MzMiBwYXJhbTsK Pj4+ICAgICAgICAvKgo+Pj4KPj4KPj4gTG9va3MgZmluZSB0byBtZSBpbiBwcmluY2lwbGUuIElz IHRoZXJlIHNvbWUgdXNlcnNwYWNlIHJlYWR5IHRvIHN0YXJ0Cj4+IGNvbnN1bWluZyBpdD8KPiAK PiBNeSBvbmx5IHBsYW4gaXMgZm9yIGlndCB0byBrbm93IHdoZW4gdGhlIHRlc3RzIGFyZSBleHBl Y3RlZCB0byBmYWlsLgo+IFJlYWwgdXNlcnNwYWNlIHNob3VsZCBub3QgYmUgdXNpbmcgR1RULCBp dCBpcyBzbG93IChldmVuIHNsb3dlciB0aGFuCj4gaW50ZW5kZWQgOykgYW5kIGNvbnN0cmFpbmVk LCBzbyBzdWJqZWN0IHRvIGFwZXJ0dXJlIHRocmFzaGluZywgZmVuY2luZwo+IGlzIG9ubHkgdXNh YmxlIGZvciB0d28gb3V0IG9mIHRoZSBtYW55IHRpbGluZyBtb2RlcywgZXRjLCBldGMsIGV0Yy4K CkknbGwgdGFrZSB0aGUgdmlldyB0aGF0IGdldF9wYXJhbSBpcyB0aW55IGVub3VnaCBmb3IgdGhp cyB0byBiZSBmaW5lLgoKUmV2aWV3ZWQtYnk6IFR2cnRrbyBVcnN1bGluIDx0dnJ0a28udXJzdWxp bkBpbnRlbC5jb20+CgpSZWdhcmRzLAoKVHZydGtvCl9fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxp c3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFu L2xpc3RpbmZvL2ludGVsLWdmeAo=