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:51:23 -0700 Message-ID: <20180529195123.GB21995@intel.com> References: <20180525033047.7596-1-dhinakaran.pandiyan@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" 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 921E8896E4 for ; Tue, 29 May 2018 19:51:23 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20180525033047.7596-1-dhinakaran.pandiyan@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 T24gVGh1LCBNYXkgMjQsIDIwMTggYXQgMDg6MzA6NDdQTSAtMDcwMCwgRGhpbmFrYXJhbiBQYW5k aXlhbiB3cm90ZToKPiBEUENEIDIwMDloICJTeW5jaHJvbml6YXRpb24gbGF0ZW5jeSBpbiBzaW5r IiBoYXMgYml0cyB0aGF0IHRlbGwgdXMgdGhlCj4gbWF4aW11bSBudW1iZXIgb2YgZnJhbWVzIHNp bmsgY2FuIHRha2UgdG8gcmVzeW5jaHJvbml6ZSB0byBzb3VyY2UgdGltaW5nCj4gd2hlbiBleGl0 aW5nIFBTUi4gTW9yZSBpbXBvcnRhbnRseSwgYXMgcGVyIGVEUCAxLjRiLCB0aGlzIGlzIHRoZSAi TWluaW11bQo+IG51bWJlciBvZiBmcmFtZXMgZm9sbG93aW5nIFBTUiBleGl0IHRoYXQgdGhlIFNv dXJjZSBkZXZpY2UgbmVlZHMgdG8KPiB3YWl0IGZvciBQU1IgZW50cnkuIgo+IAo+IFdlIGN1cnJl bnRseSB1c2UgdGhpcyB2YWx1ZSBvbmx5IHRvIHNldHVwIHRoZSBudW1iZXIgZnJhbWVzIHRvIHdh aXQgYmVmb3JlCj4gUFNSMiBzZWxlY3RpdmUgdXBkYXRlLiBCdXQsIGJhc2VkIG9uIHRoZSBhYm92 ZSBkZXNjcmlwdGlvbiBpdCBtYWtlcyBtb3JlCj4gc2Vuc2UgdG8gdXNlIHRoaXMgdG8gY29uZmln dXJlIGlkbGUgZnJhbWVzIGZvciBib3RoIFBTUjEgYW5kIGFuZCBQU1IyLiBUaGlzCj4gd2lsbCBl bnN1cmUgd2Ugd2FpdCB0aGUgcmVxdWlyZWQgbnVtYmVyIG9mIGZyYW1lcyBiZWZvcmUKPiBhY3Rp dmF0aW9uIHdoZXRoZXIgaXQgaXMgUFNSMSBvciBQU1IyLgo+IAo+IFRoZSBtaW5pbXVtIG51bWJl ciBvZiBpZGxlIGZyYW1lcyByZW1haW5zIDYsIHdoaWxlIGFsbG93aW5nIHNpbmsKPiBzeW5jaHJv bml6YXRpb24gbGF0ZW5jeSBhbmQgVkJUIHRvIGluY3JlYXNlIHRoaXMgdmFsdWUuCj4gCj4gVGhp cyBhbHNvIHNvbHZlcyB0aGUgZmxpcC1mbG9wIGJldHdlZW4gc2luayBhbmQgc291cmNlIGZyYW1l cyB0aGF0IEkKPiBub3RpY2VkIG9uIG15IFRoaW5rcGFkIFgyNjAgZHVyaW5nIFBTUiBleGl0LiBU aGlzIHNwZWNpZmljIHBhbmVsIGhhcyBhCj4gdmFsdWUgb2YgOGgsIHdoaWNoIGFjY29yZGluZyB0 byB0aGUgc3BlYyBtZWFucyB0aGUgIlNvdXJjZSBkZXZpY2UgbXVzdAo+IHdhaXQgZm9yIG1vcmUg dGhhbiBlaWdodCBhY3RpdmUgZnJhbWVzIGFmdGVyIFBTUiBleGl0IGJlZm9yZSBpbml0aWF0aW5n IFBTUgo+IGVudHJ5LiAoSW4gdGhpcyBjYXNlLCBzaG91bGQgYmUgcHJvdmlkZWQgYnkgdGhlIHBh bmVsIHN1cHBsaWVyLikiIFZCVAo+IGhvd2V2ZXIgaGFzIGEgdmFsdWUgb2YgMC4KPiAKPiBDYzog SmFuaSBOaWt1bGEgPGphbmkubmlrdWxhQGludGVsLmNvbT4KPiBDYzogSm9zZSBSb2JlcnRvIGRl IFNvdXphIDxqb3NlLnNvdXphQGludGVsLmNvbT4KPiBDYzogUm9kcmlnbyBWaXZpIDxyb2RyaWdv LnZpdmlAaW50ZWwuY29tPgo+IFNpZ25lZC1vZmYtYnk6IERoaW5ha2FyYW4gUGFuZGl5YW4gPGRo aW5ha2FyYW4ucGFuZGl5YW5AaW50ZWwuY29tPgo+IC0tLQo+ICBkcml2ZXJzL2dwdS9kcm0vaTkx NS9pbnRlbF9wc3IuYyB8IDQwICsrKysrKysrKysrKysrKysrKysrLS0tLS0tLS0tLS0tLS0tLS0t LS0KPiAgMSBmaWxlIGNoYW5nZWQsIDIwIGluc2VydGlvbnMoKyksIDIwIGRlbGV0aW9ucygtKQo+ IAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wc3IuYyBiL2RyaXZl cnMvZ3B1L2RybS9pOTE1L2ludGVsX3Bzci5jCj4gaW5kZXggZWJjNDgzZjA2YzZmLi43MWRmZTU0 MTc0MGYgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcHNyLmMKPiAr KysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wc3IuYwo+IEBAIC0yNDcsNiArMjQ3LDgg QEAgdm9pZCBpbnRlbF9wc3JfaW5pdF9kcGNkKHN0cnVjdCBpbnRlbF9kcCAqaW50ZWxfZHApCj4g IAkJcmV0dXJuOwo+ICAJfQo+ICAJZGV2X3ByaXYtPnBzci5zaW5rX3N1cHBvcnQgPSB0cnVlOwo+ ICsJZGV2X3ByaXYtPnBzci5zaW5rX3N5bmNfbGF0ZW5jeSA9Cj4gKwkJaW50ZWxfZHBfZ2V0X3Np bmtfc3luY19sYXRlbmN5KGludGVsX2RwKTsKPiAgCj4gIAlpZiAoSU5URUxfR0VOKGRldl9wcml2 KSA+PSA5ICYmCj4gIAkgICAgKGludGVsX2RwLT5wc3JfZHBjZFswXSA9PSBEUF9QU1IyX1dJVEhf WV9DT09SRF9JU19TVVBQT1JURUQpKSB7Cj4gQEAgLTI3Miw4ICsyNzQsNiBAQCB2b2lkIGludGVs X3Bzcl9pbml0X2RwY2Qoc3RydWN0IGludGVsX2RwICppbnRlbF9kcCkKPiAgCQlpZiAoZGV2X3By aXYtPnBzci5zaW5rX3BzcjJfc3VwcG9ydCkgewo+ICAJCQlkZXZfcHJpdi0+cHNyLmNvbG9yaW1l dHJ5X3N1cHBvcnQgPQo+ICAJCQkJaW50ZWxfZHBfZ2V0X2NvbG9yaW1ldHJ5X3N0YXR1cyhpbnRl bF9kcCk7Cj4gLQkJCWRldl9wcml2LT5wc3Iuc2lua19zeW5jX2xhdGVuY3kgPQo+IC0JCQkJaW50 ZWxfZHBfZ2V0X3Npbmtfc3luY19sYXRlbmN5KGludGVsX2RwKTsKPiAgCQl9Cj4gIAl9Cj4gIH0K PiBAQCAtMzcwLDIxICszNzAsMjEgQEAgc3RhdGljIHZvaWQgaHN3X2FjdGl2YXRlX3BzcjEoc3Ry dWN0IGludGVsX2RwICppbnRlbF9kcCkKPiAgCXN0cnVjdCBpbnRlbF9kaWdpdGFsX3BvcnQgKmRp Z19wb3J0ID0gZHBfdG9fZGlnX3BvcnQoaW50ZWxfZHApOwo+ICAJc3RydWN0IGRybV9kZXZpY2Ug KmRldiA9IGRpZ19wb3J0LT5iYXNlLmJhc2UuZGV2Owo+ICAJc3RydWN0IGRybV9pOTE1X3ByaXZh dGUgKmRldl9wcml2ID0gdG9faTkxNShkZXYpOwo+ICsJdTMyIG1heF9zbGVlcF90aW1lID0gMHgx ZjsKPiArCXUzMiB2YWwgPSBFRFBfUFNSX0VOQUJMRTsKPiAgCj4gLQl1aW50MzJfdCBtYXhfc2xl ZXBfdGltZSA9IDB4MWY7Cj4gLQkvKgo+IC0JICogTGV0J3MgcmVzcGVjdCBWQlQgaW4gY2FzZSBW QlQgYXNrcyBhIGhpZ2hlciBpZGxlX2ZyYW1lIHZhbHVlLgo+IC0JICogTGV0J3MgdXNlIDYgYXMg dGhlIG1pbmltdW0gdG8gY292ZXIgYWxsIGtub3duIGNhc2VzIGluY2x1ZGluZwo+IC0JICogdGhl IG9mZi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29tZSBjYXNlcy4gQWxzbyB0aGVyZSBh cmUKPiAtCSAqIGNhc2VzIHdoZXJlIHNpbmsgc2hvdWxkIGJlIGFibGUgdG8gdHJhaW4KPiAtCSAq IHdpdGggdGhlIDUgb3IgNiBpZGxlIHBhdHRlcm5zLgo+ICsJLyogTGV0J3MgdXNlIDYgYXMgdGhl IG1pbmltdW0gdG8gY292ZXIgYWxsIGtub3duIGNhc2VzIGluY2x1ZGluZyB0aGUKPiArCSAqIG9m Zi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29tZSBjYXNlcy4KPiAgCSAqLwo+IC0JdWlu dDMyX3QgaWRsZV9mcmFtZXMgPSBtYXgoNiwgZGV2X3ByaXYtPnZidC5wc3IuaWRsZV9mcmFtZXMp Owo+IC0JdWludDMyX3QgdmFsID0gRURQX1BTUl9FTkFCTEU7Cj4gKwlpbnQgaWRsZV9mcmFtZXMg PSBtYXgoNiwgZGV2X3ByaXYtPnZidC5wc3IuaWRsZV9mcmFtZXMpOwo+ICAKPiAtCXZhbCB8PSBt YXhfc2xlZXBfdGltZSA8PCBFRFBfUFNSX01BWF9TTEVFUF9USU1FX1NISUZUOwo+ICsJLyogc2lu a19zeW5jX2xhdGVuY3kgb2YgOCBtZWFucyBzb3VyY2UgaGFzIHRvIHdhaXQgZm9yIG1vcmUgdGhh biA4Cj4gKwkgKiBmcmFtZXMsIHdlJ2xsIGdvIHdpdGggOSBmcmFtZXMgZm9yIG5vdwo+ICsJICov Cj4gKwlpZGxlX2ZyYW1lcyA9IG1heChpZGxlX2ZyYW1lcywgZGV2X3ByaXYtPnBzci5zaW5rX3N5 bmNfbGF0ZW5jeSArIDEpOwo+ICAJdmFsIHw9IGlkbGVfZnJhbWVzIDw8IEVEUF9QU1JfSURMRV9G UkFNRV9TSElGVDsKPiAgCj4gKwl2YWwgfD0gbWF4X3NsZWVwX3RpbWUgPDwgRURQX1BTUl9NQVhf U0xFRVBfVElNRV9TSElGVDsKPiAgCWlmIChJU19IQVNXRUxMKGRldl9wcml2KSkKPiAgCQl2YWwg fD0gRURQX1BTUl9NSU5fTElOS19FTlRSWV9USU1FXzhfTElORVM7Cj4gIAo+IEBAIC00MjQsMTUg KzQyNCwxNSBAQCBzdGF0aWMgdm9pZCBoc3dfYWN0aXZhdGVfcHNyMihzdHJ1Y3QgaW50ZWxfZHAg KmludGVsX2RwKQo+ICAJc3RydWN0IGludGVsX2RpZ2l0YWxfcG9ydCAqZGlnX3BvcnQgPSBkcF90 b19kaWdfcG9ydChpbnRlbF9kcCk7Cj4gIAlzdHJ1Y3QgZHJtX2RldmljZSAqZGV2ID0gZGlnX3Bv cnQtPmJhc2UuYmFzZS5kZXY7Cj4gIAlzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYg PSB0b19pOTE1KGRldik7Cj4gLQkvKgo+IC0JICogTGV0J3MgcmVzcGVjdCBWQlQgaW4gY2FzZSBW QlQgYXNrcyBhIGhpZ2hlciBpZGxlX2ZyYW1lIHZhbHVlLgo+IC0JICogTGV0J3MgdXNlIDYgYXMg dGhlIG1pbmltdW0gdG8gY292ZXIgYWxsIGtub3duIGNhc2VzIGluY2x1ZGluZwo+IC0JICogdGhl IG9mZi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29tZSBjYXNlcy4gQWxzbyB0aGVyZSBh cmUKPiAtCSAqIGNhc2VzIHdoZXJlIHNpbmsgc2hvdWxkIGJlIGFibGUgdG8gdHJhaW4KPiAtCSAq IHdpdGggdGhlIDUgb3IgNiBpZGxlIHBhdHRlcm5zLgo+ICsJdTMyIHZhbDsKPiArCj4gKwkvKiBM ZXQncyB1c2UgNiBhcyB0aGUgbWluaW11bSB0byBjb3ZlciBhbGwga25vd24gY2FzZXMgaW5jbHVk aW5nIHRoZQo+ICsJICogb2ZmLWJ5LW9uZSBpc3N1ZSB0aGF0IEhXIGhhcyBpbiBzb21lIGNhc2Vz Lgo+ICAJICovCj4gLQl1aW50MzJfdCBpZGxlX2ZyYW1lcyA9IG1heCg2LCBkZXZfcHJpdi0+dmJ0 LnBzci5pZGxlX2ZyYW1lcyk7Cj4gLQl1MzIgdmFsID0gaWRsZV9mcmFtZXMgPDwgRURQX1BTUjJf SURMRV9GUkFNRV9TSElGVDsKPiArCWludCBpZGxlX2ZyYW1lcyA9IG1heCg2LCBkZXZfcHJpdi0+ dmJ0LnBzci5pZGxlX2ZyYW1lcyk7Cj4gKwo+ICsJaWRsZV9mcmFtZXMgPSBtYXgoaWRsZV9mcmFt ZXMsIGRldl9wcml2LT5wc3Iuc2lua19zeW5jX2xhdGVuY3kgKyAxKTsKPiArCXZhbCA9IGlkbGVf ZnJhbWVzIDw8IEVEUF9QU1IyX0lETEVfRlJBTUVfU0hJRlQ7CgpJIHdvbmRlciBpZiB3ZSBzaG91 bGQgY29uc29saWRhdGUgYWxsIHRoaXMgbG9naW4gaW4gYSBzaW5nbGUgZnVuY3Rpb24gc2luY2Ug dGhleQphcmUgaWRlbnRpY2FsIGFuZCBvbmx5IHRoZSBzaGlmdCBpcyBkaWZmZXJlbnQuLi4KCkJ1 dCB0aGUgbG9naWMgaXMgY29ycmVjdC4gSSBjaGVja2VkIGVEUCAxLjMgYW5kIDEuNCBzcGVjcyBh bmQgSSBiZWxpZXZlIHdlCm5lZWQgdG8gbW92ZSBmb3J3YXJkIHdpdGggdGhpcyBjaGFuZ2UgYW5k IGRvIGNsZWFuLXVwcyBvbiBmb2xsb3ctdXBzLgoKClJldmlld2VkLWJ5OiBSb2RyaWdvIFZpdmkg PHJvZHJpZ28udml2aUBpbnRlbC5jb20+CgpBbHNvIEkgZG9uJ3QgYmVsaWV2ZSB0aGF0IHdlIG5l ZWQgY2M6c3RhYmxlIGJlY2F1c2Ugd2UgbmV2ZXIgZW5hYmxlZCBpdCBhbnl3aGVyZQphbnl3YXkg YmVzaWRlcyB0aGUgZmFjdCB0aGF0IGl0IHdvbnQgYmUgYSBjbGVhbiBiYWNrcG9ydC4KCj4gIAo+ ICAJLyogRklYTUU6IHNlbGVjdGl2ZSB1cGRhdGUgaXMgcHJvYmFibHkgdG90YWxseSBicm9rZW4g YmVjYXVzZSBpdCBkb2Vzbid0Cj4gIAkgKiBtZXNoIGF0IGFsbCB3aXRoIG91ciBmcm9udGJ1ZmZl ciB0cmFja2luZy4gQW5kIHRoZSBodyBhbG9uZSBpc24ndAo+IC0tIAo+IDIuMTQuMQo+IAo+IF9f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCj4gSW50ZWwtZ2Z4 IG1haWxpbmcgbGlzdAo+IEludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKPiBodHRwczov L2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeApfX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGlu ZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVl ZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK