From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 3/4] drm/i915: Sanitize watermarks after hardware state readout (v2) Date: Fri, 20 Nov 2015 15:14:54 +0200 Message-ID: <20151120131454.GI4437@intel.com> References: <87k2pzo0yd.fsf@intel.com> <1446569697-17425-1-git-send-email-matthew.d.roper@intel.com> <871tc5onu6.fsf@intel.com> <20151105004613.GA9715@intel.com> <87a8qsmwqv.fsf@intel.com> <20151120040141.GA10239@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" 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 D6DB96E694 for ; Fri, 20 Nov 2015 05:14:57 -0800 (PST) Content-Disposition: inline In-Reply-To: <20151120040141.GA10239@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Matt Roper Cc: Jani Nikula , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBOb3YgMTksIDIwMTUgYXQgMDg6MDE6NDFQTSAtMDgwMCwgTWF0dCBSb3BlciB3cm90 ZToKPiBPbiBUaHUsIE5vdiAwNSwgMjAxNSBhdCAwMjo1NToyMFBNICswMjAwLCBKYW5pIE5pa3Vs YSB3cm90ZToKPiA+IE9uIFRodSwgMDUgTm92IDIwMTUsIE1hdHQgUm9wZXIgPG1hdHRoZXcuZC5y b3BlckBpbnRlbC5jb20+IHdyb3RlOgo+ID4gPiBPbiBXZWQsIE5vdiAwNCwgMjAxNSBhdCAwNDox MjozM1BNICswMjAwLCBKYW5pIE5pa3VsYSB3cm90ZToKPiA+ID4+IE9uIFR1ZSwgMDMgTm92IDIw MTUsIE1hdHQgUm9wZXIgPG1hdHRoZXcuZC5yb3BlckBpbnRlbC5jb20+IHdyb3RlOgo+ID4gPj4g PiBBbHRob3VnaCB3ZSBjYW4gZG8gYSBnb29kIGpvYiBvZiByZWFkaW5nIG91dCBoYXJkd2FyZSBz dGF0ZSwgdGhlCj4gPiA+PiA+IGdyYXBoaWNzIGZpcm13YXJlIG1heSBoYXZlIHByb2dyYW1tZWQg dGhlIHdhdGVybWFya3MgaW4gYSBjcmVhdGl2ZSB3YXkKPiA+ID4+ID4gdGhhdCBkb2Vzbid0IG1h dGNoIGhvdyBpOTE1IHdvdWxkIGhhdmUgY2hvc2VuIHRvIHByb2dyYW0gdGhlbS4gIFdlCj4gPiA+ PiA+IHNob3VsZG4ndCB0cnVzdCB0aGUgZmlybXdhcmUncyB3YXRlcm1hcmsgcHJvZ3JhbW1pbmcs IGJ1dCBzaG91bGQgcmF0aGVyCj4gPiA+PiA+IHJlLWNhbGN1bGF0ZSBob3cgd2UgdGhpbmsgV00n cyBzaG91bGQgYmUgcHJvZ3JhbW1lZCBhbmQgdGhlbiBzaG92ZSB0aG9zZQo+ID4gPj4gPiB2YWx1 ZXMgaW50byB0aGUgaGFyZHdhcmUuCj4gPiA+PiA+Cj4gPiA+PiA+IFdlIGNhbiBkbyB0aGlzIHBy ZXR0eSBlYXNpbHkgYnkgY3JlYXRpbmcgYSBkdW1teSB0b3AtbGV2ZWwgc3RhdGUsCj4gPiA+PiA+ IHJ1bm5pbmcgaXQgdGhyb3VnaCB0aGUgY2hlY2sgcHJvY2VzcyB0byBjYWxjdWxhdGUgYWxsIHRo ZSB2YWx1ZXMsIGFuZAo+ID4gPj4gPiB0aGVuIGp1c3QgcHJvZ3JhbW1pbmcgdGhlIHdhdGVybWFy a3MgZm9yIGVhY2ggQ1JUQy4KPiA+ID4+ID4KPiA+ID4+ID4gdjI6ICBNb3ZlIHdhdGVybWFyayBz YW5pdGl6YXRpb24gYWZ0ZXIgb3VyIEJJT1MgZmIgcmVjb25zdHJ1Y3Rpb247IHRoZQo+ID4gPj4g PiAgICAgIHdhdGVybWFyayBjYWxjdWxhdGlvbnMgdGhhdCB3ZSBkbyBoZXJlIG5lZWQgdG8gbG9v ayBhdCBwc3RhdGUtPmZiLAo+ID4gPj4gPiAgICAgIHdoaWNoIGlzbid0IHNldHVwIHlldCBpbiBp bnRlbF9tb2Rlc2V0X3NldHVwX2h3X3N0YXRlKCksIGV2ZW4KPiA+ID4+ID4gICAgICB0aG91Z2gg d2UgaGF2ZSBhbiBlbmFibGVkICYgdmlzaWJsZSBwbGFuZS4KPiA+ID4+ID4KPiA+ID4+ID4gQ2M6 IEphbmkgTmlrdWxhIDxqYW5pLm5pa3VsYUBpbnRlbC5jb20+Cj4gPiA+PiA+IENjOiBNYWFydGVu IExhbmtob3JzdCA8bWFhcnRlbi5sYW5raG9yc3RAbGludXguaW50ZWwuY29tPgo+ID4gPj4gPiBT aWduZWQtb2ZmLWJ5OiBNYXR0IFJvcGVyIDxtYXR0aGV3LmQucm9wZXJAaW50ZWwuY29tPgo+ID4g Pj4gPiAtLS0KPiA+ID4+ID4gSmFuaSwgYmFzZWQgb24geW91ciBsb2dzIGl0IGxvb2tzIGxpa2Ug dGhlIGN1bHByaXQgaXMgdGhhdCB3ZSdyZSBjYWxjdWxhdGluZwo+ID4gPj4gPiB3YXRlcm1hcmtz IGF0IHN0YXJ0dXAgdGltZSBhZnRlciB3ZSd2ZSByZWFkIG91dCBwcmVsaW1pbmFyeSBoYXJkd2Fy ZSBzdGF0ZSAoc28KPiA+ID4+ID4gd2Uga25vdyB0aGUgcHJpbWFyeSBwbGFuZSBpcyBlbmFibGVk ICYgdmlzaWJsZSksIGJ1dCBiZWZvcmUgd2UgcmVjb25zdHJ1Y3QgdGhlCj4gPiA+PiA+IEJJT1Mg ZmIgKHNvIHBzdGF0ZS0+ZmIgaXMgTlVMTCkuICBJIHRoaW5rIG1vdmluZyB0aGUgd2F0ZXJtYXJr IHNhbml0aXphdGlvbgo+ID4gPj4gPiBsYXRlciBpbiB0aGUgcHJvY2VzcyBzbyB0aGF0IHdlJ2xs IGhhdmUgYSBwcm9wZXIgZmIgc2V0dXAgc2hvdWxkIGhvcGVmdWxseQo+ID4gPj4gPiBzb2x2ZSB0 aGUgaXNzdWUuICBDYW4geW91IHRlc3QgdGhpcyB2ZXJzaW9uIHdoZW4geW91IGdldCBhIGNoYW5j ZT8gIEknbGwgYWxzbwo+ID4gPj4gPiBzZW5kIGEgcmViYXNlZCBwYXRjaCAjNCBzaW5jZSB0aGUg Y29kZSBtb3ZlbWVudCBoZXJlIG1lYW5zIHRoYXQgdGhlIHByZXZpb3VzCj4gPiA+PiA+IHZlcnNp b24gd29uJ3QgYXBwbHkgY2xlYW5seS4KPiA+ID4+IAo+ID4gPj4gU29ycnkgTWF0dCwgYmxhY2sg c2NyZWVuIHdpdGggbmV3IHZlcnNpb25zIG9mIHBhdGNoZXMgMy00LiBEbWVzZyBhdAo+ID4gPj4g aHR0cDovL3Bhc3RlYmluLmNvbS8zTFhad3ZXdQo+ID4gPgo+ID4gPiBIbW0sIG9rYXksIGxvb2tz IGxpa2Ugd2UncmUgZ2V0dGluZyBjbG9zZXIgKHN1Y2Nlc3NmdWxseSBhdm9pZGVkIHRoZQo+ID4g PiBkaXZpZGUgYnkgemVybyBwcm9ibGVtKSwgYnV0IEkgbmVnbGVjdGVkIHRvIGdyYWIgdGhlIG5l Y2Vzc2FyeSBsb2NrcyBzbwo+ID4gPiB0aGUgc2FuaXRpemF0aW9uIGRvZXNuJ3QgYWN0dWFsbHkg aGFwcGVuLiAgRG9lcyBhcHBseWluZwo+ID4gPiBodHRwOi8vcGFzdGUuZGViaWFuLm5ldC8zMjI5 MzIgb24gdG9wIG9mIHRoZSBzZXJpZXMgd29yayBhbnkgYmV0dGVyIGZvcgo+ID4gPiB5b3U/ICBJ IGhhdmVuJ3QgaGFkIHRpbWUgdG8gcHVsbCBiYWNrIG91dCBhbiBJTEstc3R5bGUgc3lzdGVtLCBz byB0aGF0J3MKPiA+ID4gb25seSBjb21waWxlLXRlc3RlZCBhdCB0aGUgbW9tZW50Lgo+ID4gCj4g PiBTdGlsbCB3YXJucyBodHRwOi8vcGFzdGViaW4uY29tL3lHdGRlNVgyCj4gPiAKPiA+IEJSLAo+ ID4gSmFuaS4KPiAKPiBPa2F5LCB0aGFua3MgdG8gSmFuaSBzZXR0aW5nIHVwIGhpcyBtYWNoaW5l IHNvIHRoYXQgSSBjYW4gZGVidWcKPiByZW1vdGVseSwgSSB0aGluayBJIHVuZGVyc3RhbmQgd2hl cmUgd2UncmUgZ29pbmcgd3Jvbmcgbm93LiAgVGhlIG1pc3NpbmcKPiBwaWVjZSBvZiB0aGUgcHV6 emxlIGlzIHRoYXQgSmFuaSdzIGRpc3BsYXkgaXMgMzg0MHgyMTYwLiAgVGhhdCBtZWFucyB0aGUK PiBCSU9TIEZCIHdlIG5lZWQgdG8gaW5oZXJpdCBpcyBkZWVtZWQgdG9vIGxhcmdlIGJ5IG91ciBp bml0aWFsIGhhcmR3YXJlCj4gcmVhZG91dCBjb2RlIChpLmUuLCBncmVhdGVyIHRoYW4gaGFsZiBv Zgo+IGRldl9wcml2LT5ndHQuc3RvbGVuX3VzYWJsZV9zaXplKSwKCkkgd291bGQganVzdCByZXZl cnQgdGhlIHBhdGNoIHRoYXQgYWRkZWQgdGhhdCByZXN0cmljdGlvbi4gSU1PIGl0IG1ha2VzCm5v IHNlbnNlIGFzIGxvbmcgYXMgRkJDIHJlbWFpbnMgZGlzYWJsZWQgYnkgZGVmYXVsdC4gQW5kIGV2 ZW4gd29yc2UgaXQKZG9lc24ndCBldmVuIGNoZWNrIHRoYXQgRkJDIGlzIGV2ZW4gc3VwcG9ydGVk IGJ5IHRoZSBwbGF0Zm9ybS4gV2UncmUKanVzdCB3YXN0aW5nIG1lbW9yeSBub3cuCgo+IHNvIHdl IGRvbid0IGV2ZW4gYm90aGVyIGluaGVyaXRpbmcKPiB0aGUgRkIgYW5kIHBzdGF0ZS0+ZmIgc3Rh eXMgTlVMTCwgZXZlbiB0aG91Z2ggcHN0YXRlLT52aXNpYmxlID0gdHJ1ZSBhbmQKPiB0aGUgcGxh bmUgaXMgcGFydCBvZiB0aGUgQ1JUQydzIHBsYW5lX21hc2suCj4gCj4gQWRkaW5nIE1hYXJ0ZW4s IFZpbGxlLCBhbmQgQW5kZXIgdG8gQ2Mgc2luY2UgSSB0aGluayB0aGV5IG1heSBoYXZlIHNvbWUK PiBpbnNpZ2h0cyBvbiBob3cgYmVzdCB0byBoYW5kbGUgdGhpcy4gIFNlZW1zIGxpa2Ugb3VyIG9w dGlvbnMgYXJlOgo+ICAqIElmIG91ciB3YXRlcm1hcmsgY29kZSBjb21lcyB1cCB3aXRoIGEgTlVM TCBGQiB3aGlsZSBwc3RhdGUtPnZpc2libGUsCj4gICAganVzdCBwcmV0ZW5kIHdlJ3JlIHdvcmtp bmcgb24gYSAzMi1iaXQgZnJhbWVidWZmZXIgdGhhdCBtYXRjaGVzIHRoZQo+ICAgIENSVEMgbW9k ZSBkaW1lbnNpb25zLiAgV2UnbGwgdGhlbiBjYWxjdWxhdGUgYXBwcm9wcmlhdGUgd2F0ZXJtYXJr Cj4gICAgdmFsdWVzLCBldmVuIHRob3VnaCB3ZSBkb24ndCBoYXZlIGFueSByZWFsIGluZm9ybWF0 aW9uIGZvciB0aGUgRkIKPiAgICB0aGF0IHRoZSBoYXJkd2FyZSBpcyBzY2FubmluZyBvdXQgb2Yu Cj4gICogU2V0IHBzdGF0ZS0+dmlzaWJsZSA9IGZhbHNlIGFuZCBjbGVhciB0aGUgYml0IGZyb20g cGxhbmVfbWFzay4gIEkKPiAgICB0aGluayB3ZSdsbCBydW4gaW50byBzdGF0ZSB2ZXJpZmljYXRp b24gd2FybmluZ3MgdGhlbiAoInBsYW5lIEEKPiAgICBzaG91bGQgYmUgZGlzYWJsZWQgYnV0IGlz IG5vdCIpIHNpbmNlIG91ciBIVyBzdGF0ZSBhbmQgU1cgc3RhdGUgZG9uJ3QKPiAgICBtYXRjaC4K PiAgKiBBY3R1YWxseSB0dXJuIG9mZiB0aGUgcHJpbWFyeSBwbGFuZSAoaGFyZHdhcmUtd2lzZSkg aWYgd2UgZmFpbCB0bwo+ICAgIGluaGVyaXQgdGhlIEJJT1MgRkIuCgpBbmQgeWVzLCBvcHRpb24g MyBpcyB3aGF0IEkgYWxzbyB0aGluayB3ZSBzaG91bGQgZG8uCgotLSAKVmlsbGUgU3lyasOkbMOk CkludGVsIE9UQwotLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KSW50ZWwgRmlubGFuZCBPeQpSZWdpc3RlcmVkIEFkZHJl c3M6IFBMIDI4MSwgMDAxODEgSGVsc2lua2kgCkJ1c2luZXNzIElkZW50aXR5IENvZGU6IDAzNTc2 MDYgLSA0IApEb21pY2lsZWQgaW4gSGVsc2lua2kgCgpUaGlzIGUtbWFpbCBhbmQgYW55IGF0dGFj aG1lbnRzIG1heSBjb250YWluIGNvbmZpZGVudGlhbCBtYXRlcmlhbCBmb3IKdGhlIHNvbGUgdXNl IG9mIHRoZSBpbnRlbmRlZCByZWNpcGllbnQocykuIEFueSByZXZpZXcgb3IgZGlzdHJpYnV0aW9u CmJ5IG90aGVycyBpcyBzdHJpY3RseSBwcm9oaWJpdGVkLiBJZiB5b3UgYXJlIG5vdCB0aGUgaW50 ZW5kZWQKcmVjaXBpZW50LCBwbGVhc2UgY29udGFjdCB0aGUgc2VuZGVyIGFuZCBkZWxldGUgYWxs IGNvcGllcy4KCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f CkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpo dHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==