From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rodrigo Vivi Subject: Re: [PATCH] drm/i915/psr: Set idle frame count based on sink synchronization latency Date: Tue, 29 May 2018 12:55:51 -0700 Message-ID: <20180529195551.GC21995@intel.com> References: <20180525033047.7596-1-dhinakaran.pandiyan@intel.com> <20180529195123.GB21995@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTPS id 8D0036E58A for ; Tue, 29 May 2018 19:55:50 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20180529195123.GB21995@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Dhinakaran Pandiyan Cc: Jani Nikula , intel-gfx@lists.freedesktop.org, Dhinakaran Pandiyan List-Id: intel-gfx@lists.freedesktop.org T24gVHVlLCBNYXkgMjksIDIwMTggYXQgMTI6NTE6MjNQTSAtMDcwMCwgUm9kcmlnbyBWaXZpIHdy b3RlOgo+IE9uIFRodSwgTWF5IDI0LCAyMDE4IGF0IDA4OjMwOjQ3UE0gLTA3MDAsIERoaW5ha2Fy YW4gUGFuZGl5YW4gd3JvdGU6Cj4gPiBEUENEIDIwMDloICJTeW5jaHJvbml6YXRpb24gbGF0ZW5j eSBpbiBzaW5rIiBoYXMgYml0cyB0aGF0IHRlbGwgdXMgdGhlCj4gPiBtYXhpbXVtIG51bWJlciBv ZiBmcmFtZXMgc2luayBjYW4gdGFrZSB0byByZXN5bmNocm9uaXplIHRvIHNvdXJjZSB0aW1pbmcK PiA+IHdoZW4gZXhpdGluZyBQU1IuIE1vcmUgaW1wb3J0YW50bHksIGFzIHBlciBlRFAgMS40Yiwg dGhpcyBpcyB0aGUgIk1pbmltdW0KPiA+IG51bWJlciBvZiBmcmFtZXMgZm9sbG93aW5nIFBTUiBl eGl0IHRoYXQgdGhlIFNvdXJjZSBkZXZpY2UgbmVlZHMgdG8KPiA+IHdhaXQgZm9yIFBTUiBlbnRy eS4iCj4gPiAKPiA+IFdlIGN1cnJlbnRseSB1c2UgdGhpcyB2YWx1ZSBvbmx5IHRvIHNldHVwIHRo ZSBudW1iZXIgZnJhbWVzIHRvIHdhaXQgYmVmb3JlCj4gPiBQU1IyIHNlbGVjdGl2ZSB1cGRhdGUu IEJ1dCwgYmFzZWQgb24gdGhlIGFib3ZlIGRlc2NyaXB0aW9uIGl0IG1ha2VzIG1vcmUKPiA+IHNl bnNlIHRvIHVzZSB0aGlzIHRvIGNvbmZpZ3VyZSBpZGxlIGZyYW1lcyBmb3IgYm90aCBQU1IxIGFu ZCBhbmQgUFNSMi4gVGhpcwo+ID4gd2lsbCBlbnN1cmUgd2Ugd2FpdCB0aGUgcmVxdWlyZWQgbnVt YmVyIG9mIGZyYW1lcyBiZWZvcmUKPiA+IGFjdGl2YXRpb24gd2hldGhlciBpdCBpcyBQU1IxIG9y IFBTUjIuCj4gPiAKPiA+IFRoZSBtaW5pbXVtIG51bWJlciBvZiBpZGxlIGZyYW1lcyByZW1haW5z IDYsIHdoaWxlIGFsbG93aW5nIHNpbmsKPiA+IHN5bmNocm9uaXphdGlvbiBsYXRlbmN5IGFuZCBW QlQgdG8gaW5jcmVhc2UgdGhpcyB2YWx1ZS4KPiA+IAo+ID4gVGhpcyBhbHNvIHNvbHZlcyB0aGUg ZmxpcC1mbG9wIGJldHdlZW4gc2luayBhbmQgc291cmNlIGZyYW1lcyB0aGF0IEkKPiA+IG5vdGlj ZWQgb24gbXkgVGhpbmtwYWQgWDI2MCBkdXJpbmcgUFNSIGV4aXQuIFRoaXMgc3BlY2lmaWMgcGFu ZWwgaGFzIGEKPiA+IHZhbHVlIG9mIDhoLCB3aGljaCBhY2NvcmRpbmcgdG8gdGhlIHNwZWMgbWVh bnMgdGhlICJTb3VyY2UgZGV2aWNlIG11c3QKPiA+IHdhaXQgZm9yIG1vcmUgdGhhbiBlaWdodCBh Y3RpdmUgZnJhbWVzIGFmdGVyIFBTUiBleGl0IGJlZm9yZSBpbml0aWF0aW5nIFBTUgo+ID4gZW50 cnkuIChJbiB0aGlzIGNhc2UsIHNob3VsZCBiZSBwcm92aWRlZCBieSB0aGUgcGFuZWwgc3VwcGxp ZXIuKSIgVkJUCj4gPiBob3dldmVyIGhhcyBhIHZhbHVlIG9mIDAuCj4gPiAKPiA+IENjOiBKYW5p IE5pa3VsYSA8amFuaS5uaWt1bGFAaW50ZWwuY29tPgo+ID4gQ2M6IEpvc2UgUm9iZXJ0byBkZSBT b3V6YSA8am9zZS5zb3V6YUBpbnRlbC5jb20+Cj4gPiBDYzogUm9kcmlnbyBWaXZpIDxyb2RyaWdv LnZpdmlAaW50ZWwuY29tPgo+ID4gU2lnbmVkLW9mZi1ieTogRGhpbmFrYXJhbiBQYW5kaXlhbiA8 ZGhpbmFrYXJhbi5wYW5kaXlhbkBpbnRlbC5jb20+Cj4gPiAtLS0KPiA+ICBkcml2ZXJzL2dwdS9k cm0vaTkxNS9pbnRlbF9wc3IuYyB8IDQwICsrKysrKysrKysrKysrKysrKysrLS0tLS0tLS0tLS0t LS0tLS0tLS0KPiA+ICAxIGZpbGUgY2hhbmdlZCwgMjAgaW5zZXJ0aW9ucygrKSwgMjAgZGVsZXRp b25zKC0pCj4gPiAKPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9w c3IuYyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3Bzci5jCj4gPiBpbmRleCBlYmM0ODNm MDZjNmYuLjcxZGZlNTQxNzQwZiAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1 L2ludGVsX3Bzci5jCj4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wc3IuYwo+ ID4gQEAgLTI0Nyw2ICsyNDcsOCBAQCB2b2lkIGludGVsX3Bzcl9pbml0X2RwY2Qoc3RydWN0IGlu dGVsX2RwICppbnRlbF9kcCkKPiA+ICAJCXJldHVybjsKPiA+ICAJfQo+ID4gIAlkZXZfcHJpdi0+ cHNyLnNpbmtfc3VwcG9ydCA9IHRydWU7Cj4gPiArCWRldl9wcml2LT5wc3Iuc2lua19zeW5jX2xh dGVuY3kgPQo+ID4gKwkJaW50ZWxfZHBfZ2V0X3Npbmtfc3luY19sYXRlbmN5KGludGVsX2RwKTsK PiA+ICAKPiA+ICAJaWYgKElOVEVMX0dFTihkZXZfcHJpdikgPj0gOSAmJgo+ID4gIAkgICAgKGlu dGVsX2RwLT5wc3JfZHBjZFswXSA9PSBEUF9QU1IyX1dJVEhfWV9DT09SRF9JU19TVVBQT1JURUQp KSB7Cj4gPiBAQCAtMjcyLDggKzI3NCw2IEBAIHZvaWQgaW50ZWxfcHNyX2luaXRfZHBjZChzdHJ1 Y3QgaW50ZWxfZHAgKmludGVsX2RwKQo+ID4gIAkJaWYgKGRldl9wcml2LT5wc3Iuc2lua19wc3Iy X3N1cHBvcnQpIHsKPiA+ICAJCQlkZXZfcHJpdi0+cHNyLmNvbG9yaW1ldHJ5X3N1cHBvcnQgPQo+ ID4gIAkJCQlpbnRlbF9kcF9nZXRfY29sb3JpbWV0cnlfc3RhdHVzKGludGVsX2RwKTsKPiA+IC0J CQlkZXZfcHJpdi0+cHNyLnNpbmtfc3luY19sYXRlbmN5ID0KPiA+IC0JCQkJaW50ZWxfZHBfZ2V0 X3Npbmtfc3luY19sYXRlbmN5KGludGVsX2RwKTsKPiA+ICAJCX0KPiA+ICAJfQo+ID4gIH0KPiA+ IEBAIC0zNzAsMjEgKzM3MCwyMSBAQCBzdGF0aWMgdm9pZCBoc3dfYWN0aXZhdGVfcHNyMShzdHJ1 Y3QgaW50ZWxfZHAgKmludGVsX2RwKQo+ID4gIAlzdHJ1Y3QgaW50ZWxfZGlnaXRhbF9wb3J0ICpk aWdfcG9ydCA9IGRwX3RvX2RpZ19wb3J0KGludGVsX2RwKTsKPiA+ICAJc3RydWN0IGRybV9kZXZp Y2UgKmRldiA9IGRpZ19wb3J0LT5iYXNlLmJhc2UuZGV2Owo+ID4gIAlzdHJ1Y3QgZHJtX2k5MTVf cHJpdmF0ZSAqZGV2X3ByaXYgPSB0b19pOTE1KGRldik7Cj4gPiArCXUzMiBtYXhfc2xlZXBfdGlt ZSA9IDB4MWY7Cj4gPiArCXUzMiB2YWwgPSBFRFBfUFNSX0VOQUJMRTsKPiA+ICAKPiA+IC0JdWlu dDMyX3QgbWF4X3NsZWVwX3RpbWUgPSAweDFmOwo+ID4gLQkvKgo+ID4gLQkgKiBMZXQncyByZXNw ZWN0IFZCVCBpbiBjYXNlIFZCVCBhc2tzIGEgaGlnaGVyIGlkbGVfZnJhbWUgdmFsdWUuCj4gPiAt CSAqIExldCdzIHVzZSA2IGFzIHRoZSBtaW5pbXVtIHRvIGNvdmVyIGFsbCBrbm93biBjYXNlcyBp bmNsdWRpbmcKPiA+IC0JICogdGhlIG9mZi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29t ZSBjYXNlcy4gQWxzbyB0aGVyZSBhcmUKPiA+IC0JICogY2FzZXMgd2hlcmUgc2luayBzaG91bGQg YmUgYWJsZSB0byB0cmFpbgo+ID4gLQkgKiB3aXRoIHRoZSA1IG9yIDYgaWRsZSBwYXR0ZXJucy4K PiA+ICsJLyogTGV0J3MgdXNlIDYgYXMgdGhlIG1pbmltdW0gdG8gY292ZXIgYWxsIGtub3duIGNh c2VzIGluY2x1ZGluZyB0aGUKPiA+ICsJICogb2ZmLWJ5LW9uZSBpc3N1ZSB0aGF0IEhXIGhhcyBp biBzb21lIGNhc2VzLgo+ID4gIAkgKi8KPiA+IC0JdWludDMyX3QgaWRsZV9mcmFtZXMgPSBtYXgo NiwgZGV2X3ByaXYtPnZidC5wc3IuaWRsZV9mcmFtZXMpOwo+ID4gLQl1aW50MzJfdCB2YWwgPSBF RFBfUFNSX0VOQUJMRTsKPiA+ICsJaW50IGlkbGVfZnJhbWVzID0gbWF4KDYsIGRldl9wcml2LT52 YnQucHNyLmlkbGVfZnJhbWVzKTsKPiA+ICAKPiA+IC0JdmFsIHw9IG1heF9zbGVlcF90aW1lIDw8 IEVEUF9QU1JfTUFYX1NMRUVQX1RJTUVfU0hJRlQ7Cj4gPiArCS8qIHNpbmtfc3luY19sYXRlbmN5 IG9mIDggbWVhbnMgc291cmNlIGhhcyB0byB3YWl0IGZvciBtb3JlIHRoYW4gOAo+ID4gKwkgKiBm cmFtZXMsIHdlJ2xsIGdvIHdpdGggOSBmcmFtZXMgZm9yIG5vdwo+ID4gKwkgKi8KPiA+ICsJaWRs ZV9mcmFtZXMgPSBtYXgoaWRsZV9mcmFtZXMsIGRldl9wcml2LT5wc3Iuc2lua19zeW5jX2xhdGVu Y3kgKyAxKTsKPiA+ICAJdmFsIHw9IGlkbGVfZnJhbWVzIDw8IEVEUF9QU1JfSURMRV9GUkFNRV9T SElGVDsKPiA+ICAKPiA+ICsJdmFsIHw9IG1heF9zbGVlcF90aW1lIDw8IEVEUF9QU1JfTUFYX1NM RUVQX1RJTUVfU0hJRlQ7Cj4gPiAgCWlmIChJU19IQVNXRUxMKGRldl9wcml2KSkKPiA+ICAJCXZh bCB8PSBFRFBfUFNSX01JTl9MSU5LX0VOVFJZX1RJTUVfOF9MSU5FUzsKPiA+ICAKPiA+IEBAIC00 MjQsMTUgKzQyNCwxNSBAQCBzdGF0aWMgdm9pZCBoc3dfYWN0aXZhdGVfcHNyMihzdHJ1Y3QgaW50 ZWxfZHAgKmludGVsX2RwKQo+ID4gIAlzdHJ1Y3QgaW50ZWxfZGlnaXRhbF9wb3J0ICpkaWdfcG9y dCA9IGRwX3RvX2RpZ19wb3J0KGludGVsX2RwKTsKPiA+ICAJc3RydWN0IGRybV9kZXZpY2UgKmRl diA9IGRpZ19wb3J0LT5iYXNlLmJhc2UuZGV2Owo+ID4gIAlzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0 ZSAqZGV2X3ByaXYgPSB0b19pOTE1KGRldik7Cj4gPiAtCS8qCj4gPiAtCSAqIExldCdzIHJlc3Bl Y3QgVkJUIGluIGNhc2UgVkJUIGFza3MgYSBoaWdoZXIgaWRsZV9mcmFtZSB2YWx1ZS4KPiA+IC0J ICogTGV0J3MgdXNlIDYgYXMgdGhlIG1pbmltdW0gdG8gY292ZXIgYWxsIGtub3duIGNhc2VzIGlu Y2x1ZGluZwo+ID4gLQkgKiB0aGUgb2ZmLWJ5LW9uZSBpc3N1ZSB0aGF0IEhXIGhhcyBpbiBzb21l IGNhc2VzLiBBbHNvIHRoZXJlIGFyZQo+ID4gLQkgKiBjYXNlcyB3aGVyZSBzaW5rIHNob3VsZCBi ZSBhYmxlIHRvIHRyYWluCj4gPiAtCSAqIHdpdGggdGhlIDUgb3IgNiBpZGxlIHBhdHRlcm5zLgo+ ID4gKwl1MzIgdmFsOwo+ID4gKwo+ID4gKwkvKiBMZXQncyB1c2UgNiBhcyB0aGUgbWluaW11bSB0 byBjb3ZlciBhbGwga25vd24gY2FzZXMgaW5jbHVkaW5nIHRoZQo+ID4gKwkgKiBvZmYtYnktb25l IGlzc3VlIHRoYXQgSFcgaGFzIGluIHNvbWUgY2FzZXMuCj4gPiAgCSAqLwo+ID4gLQl1aW50MzJf dCBpZGxlX2ZyYW1lcyA9IG1heCg2LCBkZXZfcHJpdi0+dmJ0LnBzci5pZGxlX2ZyYW1lcyk7Cj4g PiAtCXUzMiB2YWwgPSBpZGxlX2ZyYW1lcyA8PCBFRFBfUFNSMl9JRExFX0ZSQU1FX1NISUZUOwo+ ID4gKwlpbnQgaWRsZV9mcmFtZXMgPSBtYXgoNiwgZGV2X3ByaXYtPnZidC5wc3IuaWRsZV9mcmFt ZXMpOwo+ID4gKwo+ID4gKwlpZGxlX2ZyYW1lcyA9IG1heChpZGxlX2ZyYW1lcywgZGV2X3ByaXYt PnBzci5zaW5rX3N5bmNfbGF0ZW5jeSArIDEpOwo+ID4gKwl2YWwgPSBpZGxlX2ZyYW1lcyA8PCBF RFBfUFNSMl9JRExFX0ZSQU1FX1NISUZUOwo+IAo+IEkgd29uZGVyIGlmIHdlIHNob3VsZCBjb25z b2xpZGF0ZSBhbGwgdGhpcyBsb2dpbiBpbiBhIHNpbmdsZSBmdW5jdGlvbiBzaW5jZSB0aGV5Cj4g YXJlIGlkZW50aWNhbCBhbmQgb25seSB0aGUgc2hpZnQgaXMgZGlmZmVyZW50Li4uCj4gCj4gQnV0 IHRoZSBsb2dpYyBpcyBjb3JyZWN0LiBJIGNoZWNrZWQgZURQIDEuMyBhbmQgMS40IHNwZWNzIGFu ZCBJIGJlbGlldmUgd2UKPiBuZWVkIHRvIG1vdmUgZm9yd2FyZCB3aXRoIHRoaXMgY2hhbmdlIGFu ZCBkbyBjbGVhbi11cHMgb24gZm9sbG93LXVwcy4KPiAKPiAKPiBSZXZpZXdlZC1ieTogUm9kcmln byBWaXZpIDxyb2RyaWdvLnZpdmlAaW50ZWwuY29tPgoKYW5kIHB1c2hlZCB0byBkaW5xLiB0aGFu a3MKCj4gCj4gQWxzbyBJIGRvbid0IGJlbGlldmUgdGhhdCB3ZSBuZWVkIGNjOnN0YWJsZSBiZWNh dXNlIHdlIG5ldmVyIGVuYWJsZWQgaXQgYW55d2hlcmUKPiBhbnl3YXkgYmVzaWRlcyB0aGUgZmFj dCB0aGF0IGl0IHdvbnQgYmUgYSBjbGVhbiBiYWNrcG9ydC4KPiAKPiA+ICAKPiA+ICAJLyogRklY TUU6IHNlbGVjdGl2ZSB1cGRhdGUgaXMgcHJvYmFibHkgdG90YWxseSBicm9rZW4gYmVjYXVzZSBp dCBkb2Vzbid0Cj4gPiAgCSAqIG1lc2ggYXQgYWxsIHdpdGggb3VyIGZyb250YnVmZmVyIHRyYWNr aW5nLiBBbmQgdGhlIGh3IGFsb25lIGlzbid0Cj4gPiAtLSAKPiA+IDIuMTQuMQo+ID4gCj4gPiBf X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwo+ID4gSW50ZWwt Z2Z4IG1haWxpbmcgbGlzdAo+ID4gSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwo+ID4g aHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4 IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==