From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH] drm/i915/psr: Set idle frame count based on sink synchronization latency Date: Fri, 25 May 2018 10:17:28 +0300 Message-ID: <87wovs2ohj.fsf@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 mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5895289B8E for ; Fri, 25 May 2018 07:17:32 +0000 (UTC) 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 , intel-gfx@lists.freedesktop.org Cc: Dhinakaran Pandiyan , Rodrigo Vivi List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCAyNCBNYXkgMjAxOCwgRGhpbmFrYXJhbiBQYW5kaXlhbiA8ZGhpbmFrYXJhbi5wYW5k aXlhbkBnbWFpbC5jb20+IHdyb3RlOgo+IERQQ0QgMjAwOWggIlN5bmNocm9uaXphdGlvbiBsYXRl bmN5IGluIHNpbmsiIGhhcyBiaXRzIHRoYXQgdGVsbCB1cyB0aGUKPiBtYXhpbXVtIG51bWJlciBv ZiBmcmFtZXMgc2luayBjYW4gdGFrZSB0byByZXN5bmNocm9uaXplIHRvIHNvdXJjZSB0aW1pbmcK PiB3aGVuIGV4aXRpbmcgUFNSLiBNb3JlIGltcG9ydGFudGx5LCBhcyBwZXIgZURQIDEuNGIsIHRo aXMgaXMgdGhlICJNaW5pbXVtCj4gbnVtYmVyIG9mIGZyYW1lcyBmb2xsb3dpbmcgUFNSIGV4aXQg dGhhdCB0aGUgU291cmNlIGRldmljZSBuZWVkcyB0bwo+IHdhaXQgZm9yIFBTUiBlbnRyeS4iCj4K PiBXZSBjdXJyZW50bHkgdXNlIHRoaXMgdmFsdWUgb25seSB0byBzZXR1cCB0aGUgbnVtYmVyIGZy YW1lcyB0byB3YWl0IGJlZm9yZQo+IFBTUjIgc2VsZWN0aXZlIHVwZGF0ZS4gQnV0LCBiYXNlZCBv biB0aGUgYWJvdmUgZGVzY3JpcHRpb24gaXQgbWFrZXMgbW9yZQo+IHNlbnNlIHRvIHVzZSB0aGlz IHRvIGNvbmZpZ3VyZSBpZGxlIGZyYW1lcyBmb3IgYm90aCBQU1IxIGFuZCBhbmQgUFNSMi4gVGhp cwo+IHdpbGwgZW5zdXJlIHdlIHdhaXQgdGhlIHJlcXVpcmVkIG51bWJlciBvZiBmcmFtZXMgYmVm b3JlCj4gYWN0aXZhdGlvbiB3aGV0aGVyIGl0IGlzIFBTUjEgb3IgUFNSMi4KPgo+IFRoZSBtaW5p bXVtIG51bWJlciBvZiBpZGxlIGZyYW1lcyByZW1haW5zIDYsIHdoaWxlIGFsbG93aW5nIHNpbmsK PiBzeW5jaHJvbml6YXRpb24gbGF0ZW5jeSBhbmQgVkJUIHRvIGluY3JlYXNlIHRoaXMgdmFsdWUu Cj4KPiBUaGlzIGFsc28gc29sdmVzIHRoZSBmbGlwLWZsb3AgYmV0d2VlbiBzaW5rIGFuZCBzb3Vy Y2UgZnJhbWVzIHRoYXQgSQo+IG5vdGljZWQgb24gbXkgVGhpbmtwYWQgWDI2MCBkdXJpbmcgUFNS IGV4aXQuIFRoaXMgc3BlY2lmaWMgcGFuZWwgaGFzIGEKPiB2YWx1ZSBvZiA4aCwgd2hpY2ggYWNj b3JkaW5nIHRvIHRoZSBzcGVjIG1lYW5zIHRoZSAiU291cmNlIGRldmljZSBtdXN0Cj4gd2FpdCBm b3IgbW9yZSB0aGFuIGVpZ2h0IGFjdGl2ZSBmcmFtZXMgYWZ0ZXIgUFNSIGV4aXQgYmVmb3JlIGlu aXRpYXRpbmcgUFNSCj4gZW50cnkuIChJbiB0aGlzIGNhc2UsIHNob3VsZCBiZSBwcm92aWRlZCBi eSB0aGUgcGFuZWwgc3VwcGxpZXIuKSIgVkJUCj4gaG93ZXZlciBoYXMgYSB2YWx1ZSBvZiAwLgo+ Cj4gQ2M6IEphbmkgTmlrdWxhIDxqYW5pLm5pa3VsYUBpbnRlbC5jb20+Cj4gQ2M6IEpvc2UgUm9i ZXJ0byBkZSBTb3V6YSA8am9zZS5zb3V6YUBpbnRlbC5jb20+Cj4gQ2M6IFJvZHJpZ28gVml2aSA8 cm9kcmlnby52aXZpQGludGVsLmNvbT4KPiBTaWduZWQtb2ZmLWJ5OiBEaGluYWthcmFuIFBhbmRp eWFuIDxkaGluYWthcmFuLnBhbmRpeWFuQGludGVsLmNvbT4KCkhvdyBiYWRseSBkb2VzIHRoaXMg ZGVwZW5kIG9uIHRoZSBsYXN0IHBzciBzZXJpZXMgdGhhdCB3YXMganVzdCBtZXJnZWQ/CkkuZS4g ZG9lcyBjYzogc3RhYmxlIG1ha2UgYW55IHNlbnNlPwoKQlIsCkphbmkuCgoKPiAtLS0KPiAgZHJp dmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcHNyLmMgfCA0MCArKysrKysrKysrKysrKysrKysrKy0t LS0tLS0tLS0tLS0tLS0tLS0tCj4gIDEgZmlsZSBjaGFuZ2VkLCAyMCBpbnNlcnRpb25zKCspLCAy MCBkZWxldGlvbnMoLSkKPgo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRl bF9wc3IuYyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3Bzci5jCj4gaW5kZXggZWJjNDgz ZjA2YzZmLi43MWRmZTU0MTc0MGYgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUv aW50ZWxfcHNyLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wc3IuYwo+IEBA IC0yNDcsNiArMjQ3LDggQEAgdm9pZCBpbnRlbF9wc3JfaW5pdF9kcGNkKHN0cnVjdCBpbnRlbF9k cCAqaW50ZWxfZHApCj4gIAkJcmV0dXJuOwo+ICAJfQo+ICAJZGV2X3ByaXYtPnBzci5zaW5rX3N1 cHBvcnQgPSB0cnVlOwo+ICsJZGV2X3ByaXYtPnBzci5zaW5rX3N5bmNfbGF0ZW5jeSA9Cj4gKwkJ aW50ZWxfZHBfZ2V0X3Npbmtfc3luY19sYXRlbmN5KGludGVsX2RwKTsKPiAgCj4gIAlpZiAoSU5U RUxfR0VOKGRldl9wcml2KSA+PSA5ICYmCj4gIAkgICAgKGludGVsX2RwLT5wc3JfZHBjZFswXSA9 PSBEUF9QU1IyX1dJVEhfWV9DT09SRF9JU19TVVBQT1JURUQpKSB7Cj4gQEAgLTI3Miw4ICsyNzQs NiBAQCB2b2lkIGludGVsX3Bzcl9pbml0X2RwY2Qoc3RydWN0IGludGVsX2RwICppbnRlbF9kcCkK PiAgCQlpZiAoZGV2X3ByaXYtPnBzci5zaW5rX3BzcjJfc3VwcG9ydCkgewo+ICAJCQlkZXZfcHJp di0+cHNyLmNvbG9yaW1ldHJ5X3N1cHBvcnQgPQo+ICAJCQkJaW50ZWxfZHBfZ2V0X2NvbG9yaW1l dHJ5X3N0YXR1cyhpbnRlbF9kcCk7Cj4gLQkJCWRldl9wcml2LT5wc3Iuc2lua19zeW5jX2xhdGVu Y3kgPQo+IC0JCQkJaW50ZWxfZHBfZ2V0X3Npbmtfc3luY19sYXRlbmN5KGludGVsX2RwKTsKPiAg CQl9Cj4gIAl9Cj4gIH0KPiBAQCAtMzcwLDIxICszNzAsMjEgQEAgc3RhdGljIHZvaWQgaHN3X2Fj dGl2YXRlX3BzcjEoc3RydWN0IGludGVsX2RwICppbnRlbF9kcCkKPiAgCXN0cnVjdCBpbnRlbF9k aWdpdGFsX3BvcnQgKmRpZ19wb3J0ID0gZHBfdG9fZGlnX3BvcnQoaW50ZWxfZHApOwo+ICAJc3Ry dWN0IGRybV9kZXZpY2UgKmRldiA9IGRpZ19wb3J0LT5iYXNlLmJhc2UuZGV2Owo+ICAJc3RydWN0 IGRybV9pOTE1X3ByaXZhdGUgKmRldl9wcml2ID0gdG9faTkxNShkZXYpOwo+ICsJdTMyIG1heF9z bGVlcF90aW1lID0gMHgxZjsKPiArCXUzMiB2YWwgPSBFRFBfUFNSX0VOQUJMRTsKPiAgCj4gLQl1 aW50MzJfdCBtYXhfc2xlZXBfdGltZSA9IDB4MWY7Cj4gLQkvKgo+IC0JICogTGV0J3MgcmVzcGVj dCBWQlQgaW4gY2FzZSBWQlQgYXNrcyBhIGhpZ2hlciBpZGxlX2ZyYW1lIHZhbHVlLgo+IC0JICog TGV0J3MgdXNlIDYgYXMgdGhlIG1pbmltdW0gdG8gY292ZXIgYWxsIGtub3duIGNhc2VzIGluY2x1 ZGluZwo+IC0JICogdGhlIG9mZi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29tZSBjYXNl cy4gQWxzbyB0aGVyZSBhcmUKPiAtCSAqIGNhc2VzIHdoZXJlIHNpbmsgc2hvdWxkIGJlIGFibGUg dG8gdHJhaW4KPiAtCSAqIHdpdGggdGhlIDUgb3IgNiBpZGxlIHBhdHRlcm5zLgo+ICsJLyogTGV0 J3MgdXNlIDYgYXMgdGhlIG1pbmltdW0gdG8gY292ZXIgYWxsIGtub3duIGNhc2VzIGluY2x1ZGlu ZyB0aGUKPiArCSAqIG9mZi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29tZSBjYXNlcy4K PiAgCSAqLwo+IC0JdWludDMyX3QgaWRsZV9mcmFtZXMgPSBtYXgoNiwgZGV2X3ByaXYtPnZidC5w c3IuaWRsZV9mcmFtZXMpOwo+IC0JdWludDMyX3QgdmFsID0gRURQX1BTUl9FTkFCTEU7Cj4gKwlp bnQgaWRsZV9mcmFtZXMgPSBtYXgoNiwgZGV2X3ByaXYtPnZidC5wc3IuaWRsZV9mcmFtZXMpOwo+ ICAKPiAtCXZhbCB8PSBtYXhfc2xlZXBfdGltZSA8PCBFRFBfUFNSX01BWF9TTEVFUF9USU1FX1NI SUZUOwo+ICsJLyogc2lua19zeW5jX2xhdGVuY3kgb2YgOCBtZWFucyBzb3VyY2UgaGFzIHRvIHdh aXQgZm9yIG1vcmUgdGhhbiA4Cj4gKwkgKiBmcmFtZXMsIHdlJ2xsIGdvIHdpdGggOSBmcmFtZXMg Zm9yIG5vdwo+ICsJICovCj4gKwlpZGxlX2ZyYW1lcyA9IG1heChpZGxlX2ZyYW1lcywgZGV2X3By aXYtPnBzci5zaW5rX3N5bmNfbGF0ZW5jeSArIDEpOwo+ICAJdmFsIHw9IGlkbGVfZnJhbWVzIDw8 IEVEUF9QU1JfSURMRV9GUkFNRV9TSElGVDsKPiAgCj4gKwl2YWwgfD0gbWF4X3NsZWVwX3RpbWUg PDwgRURQX1BTUl9NQVhfU0xFRVBfVElNRV9TSElGVDsKPiAgCWlmIChJU19IQVNXRUxMKGRldl9w cml2KSkKPiAgCQl2YWwgfD0gRURQX1BTUl9NSU5fTElOS19FTlRSWV9USU1FXzhfTElORVM7Cj4g IAo+IEBAIC00MjQsMTUgKzQyNCwxNSBAQCBzdGF0aWMgdm9pZCBoc3dfYWN0aXZhdGVfcHNyMihz dHJ1Y3QgaW50ZWxfZHAgKmludGVsX2RwKQo+ICAJc3RydWN0IGludGVsX2RpZ2l0YWxfcG9ydCAq ZGlnX3BvcnQgPSBkcF90b19kaWdfcG9ydChpbnRlbF9kcCk7Cj4gIAlzdHJ1Y3QgZHJtX2Rldmlj ZSAqZGV2ID0gZGlnX3BvcnQtPmJhc2UuYmFzZS5kZXY7Cj4gIAlzdHJ1Y3QgZHJtX2k5MTVfcHJp dmF0ZSAqZGV2X3ByaXYgPSB0b19pOTE1KGRldik7Cj4gLQkvKgo+IC0JICogTGV0J3MgcmVzcGVj dCBWQlQgaW4gY2FzZSBWQlQgYXNrcyBhIGhpZ2hlciBpZGxlX2ZyYW1lIHZhbHVlLgo+IC0JICog TGV0J3MgdXNlIDYgYXMgdGhlIG1pbmltdW0gdG8gY292ZXIgYWxsIGtub3duIGNhc2VzIGluY2x1 ZGluZwo+IC0JICogdGhlIG9mZi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29tZSBjYXNl cy4gQWxzbyB0aGVyZSBhcmUKPiAtCSAqIGNhc2VzIHdoZXJlIHNpbmsgc2hvdWxkIGJlIGFibGUg dG8gdHJhaW4KPiAtCSAqIHdpdGggdGhlIDUgb3IgNiBpZGxlIHBhdHRlcm5zLgo+ICsJdTMyIHZh bDsKPiArCj4gKwkvKiBMZXQncyB1c2UgNiBhcyB0aGUgbWluaW11bSB0byBjb3ZlciBhbGwga25v d24gY2FzZXMgaW5jbHVkaW5nIHRoZQo+ICsJICogb2ZmLWJ5LW9uZSBpc3N1ZSB0aGF0IEhXIGhh cyBpbiBzb21lIGNhc2VzLgo+ICAJICovCj4gLQl1aW50MzJfdCBpZGxlX2ZyYW1lcyA9IG1heCg2 LCBkZXZfcHJpdi0+dmJ0LnBzci5pZGxlX2ZyYW1lcyk7Cj4gLQl1MzIgdmFsID0gaWRsZV9mcmFt ZXMgPDwgRURQX1BTUjJfSURMRV9GUkFNRV9TSElGVDsKPiArCWludCBpZGxlX2ZyYW1lcyA9IG1h eCg2LCBkZXZfcHJpdi0+dmJ0LnBzci5pZGxlX2ZyYW1lcyk7Cj4gKwo+ICsJaWRsZV9mcmFtZXMg PSBtYXgoaWRsZV9mcmFtZXMsIGRldl9wcml2LT5wc3Iuc2lua19zeW5jX2xhdGVuY3kgKyAxKTsK PiArCXZhbCA9IGlkbGVfZnJhbWVzIDw8IEVEUF9QU1IyX0lETEVfRlJBTUVfU0hJRlQ7Cj4gIAo+ ICAJLyogRklYTUU6IHNlbGVjdGl2ZSB1cGRhdGUgaXMgcHJvYmFibHkgdG90YWxseSBicm9rZW4g YmVjYXVzZSBpdCBkb2Vzbid0Cj4gIAkgKiBtZXNoIGF0IGFsbCB3aXRoIG91ciBmcm9udGJ1ZmZl ciB0cmFja2luZy4gQW5kIHRoZSBodyBhbG9uZSBpc24ndAoKLS0gCkphbmkgTmlrdWxhLCBJbnRl bCBPcGVuIFNvdXJjZSBHcmFwaGljcyBDZW50ZXIKX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlz dHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4v bGlzdGluZm8vaW50ZWwtZ2Z4Cg==