From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eero Tamminen Subject: Re: [PATCH v2 2/2] drm/i915/bxt: Fix inadvertent CPU snooping due to incorrect MOCS config Date: Wed, 27 Apr 2016 16:25:09 +0300 Message-ID: <5720BDB5.2050600@intel.com> References: <1461674662-29353-1-git-send-email-imre.deak@intel.com> <1461674662-29353-2-git-send-email-imre.deak@intel.com> <20160426125726.GJ27856@nuc-i3427.alporthouse.com> <1461676675.18080.18.camel@intel.com> <20160426132321.GL27856@nuc-i3427.alporthouse.com> <571F7AA3.8080509@intel.com> <20160426143005.GT8291@phenom.ffwll.local> <571FA2E9.10307@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTP id D183F6EACA for ; Wed, 27 Apr 2016 13:10:19 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "Frederick, Michael T" , Daniel Vetter Cc: "Rantala, Valtteri" , "intel-gfx@lists.freedesktop.org" List-Id: intel-gfx@lists.freedesktop.org SGksCgpPbiAyNi4wNC4yMDE2IDIwOjI1LCBGcmVkZXJpY2ssIE1pY2hhZWwgVCB3cm90ZToKPiBT b3JyeSBJJ20gbm90IHRyYWNraW5nIGFsbCB0aGUgTU9DcyBkaXNjdXNzaW9ucy4gIEkganVzdCB3 YW50IHRvIGluZGljYXRlIHdoYXQgdGhlIGNvaGVyZW5jeSBtZWFucyBpbiBTb0MgZm9yIEJYVC4K Pgo+IEdUSSBzZXRzIHRoZSBub24taW5jbHVzaXZlIGJpdCBvbiB0aGUgSURJIGludGVyZmFjZSBi YXNlZCBvbiBob3cgaXQgdHJlYXRzIHRoZSBtZW1vcnkuICBJbiBCWFQgY2FzZSB3aGVyZSB0aGVy ZSBpcyBubyB1bmNvcmUgY2FjaGUsICJub24taW5jbHVzaXZlIiBqdXN0IGluZGljYXRlcyBzbm9v cCBvciBub3QuICBCWFQgaGFzIGEgc25vb3AgZmlsdGVyIGluIG9yZGVyIHRvIG1ha2UgdGhlIGxh dGVuY3kgb2Ygc25vb3BpbmcgR1QgZnJvbSBhIGNvcmUgcm91Z2hseSBzaW1pbGFyIHRvIHNub29w aW5nIGFub3RoZXIgY29yZS4KPgo+IEZvciBCWFQ6Cj4gSWYgR1RJIHNldHMgbm9uLWluY2x1c2l2 ZT0wIChpLmUuIGNvaGVyZW50KTogdHJhbnNhY3Rpb24gbG9va3MgdXAgaW4gdGhlIFNGIGFuZCB0 aGUgU0Egc25vb3BzIHRoZSBjb3Jlcy4gIFRoZSBwb3RlbnRpYWwgaW1wYWN0IGhlcmUgaXMgdGhh dCBmb3IgaGlnaCBCVyBjb2hlcmVudCB0cmFmZmljLCB0aGUgU0Ygd2lsbCBiZWNvbWUgdGhlIEJX IGxpbWl0ZXIgb2YgdGhlIHN5c3RlbSBhbmQgY2FwIEJXIGF0IDMzJSAqIDM0R0Jwcy4gRm9yIHdy aXRlcyBsaWtlIFdDSUxGcyBzbm9vcHMgdG8gY29yZXMgbXVzdCBiZSByZXNvbHZlZCBiZWZvcmUg U0EgcmVxdWVzdHMgV1IgZGF0YSBmcm9tIEdULiAgRm9yIHJlYWRzIHRoZSBjb21tb24gY2FzZSBz aG91bGQgaGF2ZSBubyBpbXBhY3QgYmVjYXVzZSBzbm9vcCBsYXRlbmN5IGlzIGdlbmVyYWxseSBt dWNoIGxlc3MgdGhhbiBtZW1vcnkgZGF0YSBsYXRlbmN5LiAgSW4gZ2VuZXJhbCBzbm9vcCBsYXRl bmN5IGZvciBhIGNvcmUgaXMgcmVsYXRpdmVseSBzbWFsbCwgYnV0IHRoZXJlIGlzIGFsc28gdGhl IHByb3NwZWN0IHRoYXQgYSBjb3JlIGNvdWxkIGJlIGRvd24gKGUuZy4gcmF0aW8gY2hhbmdlKSBv ciBsb2FkZWQgdy8gc25vb3BpbmcuCj4gSWYgR1RJIHNldHMgbm9uLWluY2x1c2l2ZT0xIChpLmUu IG5vbi1jb2hlcmVudCk6IHRyYW5zYWN0aW9uIHRha2VzIHRoZSBTRiBieXBhc3MgYW5kIHRoZSBT QSBkb2VzIG5vdCBzbm9vcCB0aGUgY29yZXMuICBUaGlzIGlzIGJlc3QgZm9yIGhpZ2gtQlcgc2lu Y2UgaXQgcmVtb3ZlcyB0aGUgU0YgYm90dGxlbmVjayBhbmQgZG9lc24ndCByZXF1aXJlIGNvcmUg aW50ZXJhY3Rpb24uCgpUaGFua3MgZm9yIHRoZSBleHBsYW5hdGlvbiEKCkFGQUlLOgoKKiBJbiBy ZWdhcmRzIHRvIDNEIGRyaXZlciBvcGVyYXRpb25zLCBDUFUgc2lkZSBkb2Vzbid0IG1vZGlmeSB0 aGUgYnVmZmVyIApjb250ZW50cyB3aGlsZSBHUFUgaXMgd29ya2luZyBvbiB0aGVtLiAgQ1BVIHNp ZGUgc2V0cyB1cCB0aGUgYnVmZmVycyAKKHRleHR1cmVzLCBWQk9zLCBiYXRjaGVzIGV0YyksIGFu ZCB0aGVuIChhZnRlciBhIGZsdXNoKSBHUFUgaXMgYXNrZWQgdG8gCmFjdCBvbiB0aGVtLgoKKiBG b3IgdGhpbmdzIGxpa2UgdGV4dHVyZSBzdHJlYW1pbmcsIHRoZSBkcml2ZXIgZWl0aGVyIGludGVy bmFsbHkgCnN5bmNocm9uaXplcyB0aGUgZGF0YSBvciBjcmVhdGVzIGEgbmV3IGNvcHkgb2YgaXQg d2hlbmV2ZXIgYXBwbGljYXRpb24gCnRlbGxzIHRoYXQgZGF0YSBpcyB1cGRhdGVkLiAgVGhlcmUn cyBhbHdheXMgc29tZSBraW5kIG9mICJ1cGxvYWQiIAppbnZvbHZlZCAoR0wgQVBJIG5lZWRzIGl0 IGFzIG5vbi1pbnRlZ3JhdGVkIEdQVSdzIGRvbid0IHNoYXJlIG1lbW9yeSAKd2l0aCBDUFUpLgoK V2hpbGUgaXQncyBwb3NzaWJsZSB0aGF0IHRoZXJlJ3MgYSBjYXNlIHdoZXJlIHNub29waW5nIHdv dWxkIGJlIG5lZWRlZCwgCkkgY2Fubm90IHRoaW5rIG9mIGFueSBteXNlbGYuCgpEYW5pZWwsIENo cmlzLCBkaWQgeW91IGhhdmUgc29tZSBjb25jcmV0ZSBleGFtcGxlIGluIG1pbmQgd2hlcmUgM0Qg CmRyaXZlciB3b3VsZCByZXF1aXJlIENQVSB0byBzbm9vcCBHUFU/CgoKCS0gRWVybwoKRGlzY2xh aW1lcjogSSdtIG5vdCBhIDNEIGRyaXZlciBkZXZlbG9wZXIgbXlzZWxmLgoKPgo+IFRoYW5rcywg TWlrZQo+Cj4KPiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQo+IEZyb206IFRhbW1pbmVuLCBF ZXJvIFQKPiBTZW50OiBUdWVzZGF5LCBBcHJpbCAyNiwgMjAxNiAxMDoxOSBBTQo+IFRvOiBEYW5p ZWwgVmV0dGVyIDxkYW5pZWxAZmZ3bGwuY2g+Cj4gQ2M6IENocmlzIFdpbHNvbiA8Y2hyaXNAY2hy aXMtd2lsc29uLmNvLnVrPjsgRGVhaywgSW1yZSA8aW1yZS5kZWFrQGludGVsLmNvbT47IGludGVs LWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmc7IFJhbnRhbGEsIFZhbHR0ZXJpIDx2YWx0dGVyaS5y YW50YWxhQGludGVsLmNvbT47IEZyZWRlcmljaywgTWljaGFlbCBUIDxtaWNoYWVsLnQuZnJlZGVy aWNrQGludGVsLmNvbT47IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBsaW51eC5pbnRl bC5jb20+Cj4gU3ViamVjdDogUmU6IFtJbnRlbC1nZnhdIFtQQVRDSCB2MiAyLzJdIGRybS9pOTE1 L2J4dDogRml4IGluYWR2ZXJ0ZW50IENQVSBzbm9vcGluZyBkdWUgdG8gaW5jb3JyZWN0IE1PQ1Mg Y29uZmlnCj4KPiBIaSwKPgo+IE9uIDI2LjA0LjIwMTYgMTc6MzAsIERhbmllbCBWZXR0ZXIgd3Jv dGU6Cj4+IE9uIFR1ZSwgQXByIDI2LCAyMDE2IGF0IDA1OjI2OjQzUE0gKzAzMDAsIEVlcm8gVGFt bWluZW4gd3JvdGU6Cj4gWy4uLl0KPj4+IFdoYXQgdGhpcyBrZXJuZWwgQUJJIChpbmRleCBlbnRy eSAjMikgaGFzIGJlZW4gYWdyZWVkICYgZG9jdW1lbnRlZCB0bwo+Pj4gcHJvdmlkZT8KPj4+Cj4+ PiBJIHRob3VnaHQgdGhpcyBlbnRyeSBpcyBzdXBwb3NlZCB0byByZXBsYWNlIHRoZSB3cml0ZWJh Y2sgTExDL2VMTEMKPj4+IGNhY2hlIE1PQ1Mgc2V0dGluZyBNZXNhIGlzIHVzaW5nIG9uIChlLmcu IEJEVykgdG8gc3BlZWQgdXAgYWNjZXNzZXMKPj4+IHRvIGEgbWVtb3J5IGFyZWEgd2hpY2ggaXQg a25vd3MgYWx3YXlzIHRvIGJlIGFjY2Vzc2VkIHNvIHRoYXQgaXQgY2FuIGJlIGNhY2hlZC4KPj4+ Cj4+PiBJZiBhcHAgcnVucyBvbiBIVyB3aGVyZSBMTEMvZUxMQyBpcyBtaXNzaW5nLCBnaXZpbmcg dGhlIGFwcCBleHRyYQo+Pj4gc2xvd2Rvd24gaW5zdGVhZCBvZiBwb3RlbnRpYWwgc3BlZWR1cCBz b3VuZHMgbGlrZSBmYWlsZWQgSFcKPj4+IGFic3RyYWN0aW9uLiA6LSkKPj4KPj4gV2VsbCBtZXNh IG5lZWRzIHRvIGtub3cgbGxjIHZzLiAhbGxjIGFueXdheSB0byBub3QgdG90YWxseSBzdWNrLAo+ Cj4gV2hhdCBkbyB5b3UgdGhpbmsgaXQgc2hvdWxkIGRvIHdpdGggdGhhdCBpbmZvcm1hdGlvbj8K Pgo+IEkgYXNzdW1lIHlvdSB0byBtZWFuLCB0aGF0IE1lc2EgbmVlZHMgdG8ga25vdyB0aGUgKmFt b3VudCogb2YgTExDIGFuZCBjaGFuZ2UgaXRzIGJlaGF2aW9yIGJhc2VkIG9uIHRoYXQgYW1vdW50 LCBub3QganVzdCB3aGV0aGVyIGl0J3MgcHJlc2VudC4KPgo+IEluIHRoYXQgY2FzZSBNZXNhIGRv ZXMsIGFuZCBoYXMgYWx3YXlzIHRvdGFsbHkgInN1Y2tlZCIuICBNZXNhIG9uIGVhcmxpZXIgR0VO KHMpIGNhY2hlZCBldmVyeXRoaW5nIHRoYXQgY2FuIGJlIGNhY2hlZCwgYW5kIEkgYXNzdW1lIGl0 IHRvIHRyeSB0byBkbyB0aGF0IHdpdGggR0VOOSB0b28uCj4KPgo+IEhvd2V2ZXIsIGJhc2VkIG9u IG91ciBNT0NTIHRlc3Rpbmcgb24gQkRXLCB0aGF0IGFjdHVhbGx5IGdpdmVzIHRoZSBiZXN0IG92 ZXJhbGwgcGVyZiByZXN1bHRzLiAgT24gYXZlcmFnZSBpdCBkb2Vzbid0IGdpdmUgbXVjaCwgYnV0 IGl0IHdhcyBiZXR0ZXIgdGhhbiBhbnkgc3RyYWlnaHRmb3J3YXJkIChidWZmZXIgc2l6ZS90eXBl KSBoZXVyaXN0aWNzIGZvciBtYWtpbmcgc29tZXRoaW5nIG5vdCB0byBiZSBjYWNoZWQgaW4gZWZm b3J0IHRvIHV0aWxpemUgTExDICJiZXR0ZXIiLgo+Cj4gSXQgc2VlbWVkIHRoYXQgTExDIGlzIHRv byBzbWFsbCB0byBoYXZlIG1lYW5pbmdmdWwgZ2VuZXJpYyBoZXVyaXN0aWNzIGZvciBub3JtYWwg M0Qgd29ya2xvYWRzIChvciB0aGV5IG5lZWQgdG8gYmUgdmVyeSBjb21wbGV4LCBzb21ldGhpbmcg bmVlZGluZyBtb250aHMgb2YgdGVzdGluZyAmIGl0ZXJhdGlvbiwgb3IgYmUgcGVyIGFwcGxpY2F0 aW9uLCBub3QgZ2VuZXJpYykuCj4KPiBlTExDIGNvdWxkIGJlIGEgZGlmZmVyZW50IG1hdHRlciBh cyBpdCdzIGxhcmdlIGVub3VnaCB0aGF0IG9uZSBjYW4gcHV0IGUuZy4gY29sb3IvZGVwdGggYnVm ZmVyIHRoZXJlLgo+Cj4gU2tpcCBjYWNoZSBzZXR0aW5nIGZvciBMTEMgbWF5IGFsc28gYmUgdXNl ZnVsLCBpZiBpdCB3b3JrcyAoYXMgaXQgaW4gYSBzZW5zZSBleHRlbmRzIHRoZSBjYWNoZSBzaXpl KSwgYW5kIHJlbmRlciBjb21wcmVzc2lvbiBjYW4gYWxzbyBjaGFuZ2UgdGhpbmdzLiAgUHJvYmxl bSB3aXRoIFJCQyBpcyB0aGF0IGl0IG1ha2VzIGFzc3VtcHRpb25zIGFib3V0IG1lbW9yeSBhcmVh cyB1c2FnZSBldmVuIGxlc3MgcmVsaWFibGUgYXMgeW91IGRvbid0IGtub3cgaG93IHdlbGwgdGhl IGNvbnRlbnQgY29tcHJlc3Nlcy4KPgo+Cj4+IGFuZCBkZWZpbmluZyBlbnRyeSAjMiBhcyAiY29o ZXJlbnQsIGFsd2F5cyIgbWFrZXMgc2Vuc2UuIEkgdGhvdWdodAo+PiBlbnRyeSAwIHdhcyB0aGUg cmVhb25hYmxlIGRlZmF1bHQgYWthIHB0ZSBwYXNzdGhyb3VnaCBhbmQgaGVuY2UgbWFuYWdlZCBi eSBrZXJuZWw/Cj4+Cj4+IElmIG1lc2EgYXNrcyBmb3Igbm9uc2Vuc2UsIHRoZSBrZXJuZWwgaXMg aGFwcHkgdG8gb2JsaWdlLgo+Cj4KPiAJLSBFZXJvCj4KCl9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4 QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWls bWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=