From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dhinakaran Pandiyan Subject: Re: [PATCH] drm/i915/psr: Set idle frame count based on sink synchronization latency Date: Tue, 29 May 2018 14:30:51 -0700 Message-ID: <1527629451.31662.6.camel@intel.com> References: <20180525033047.7596-1-dhinakaran.pandiyan@intel.com> <20180529195123.GB21995@intel.com> Reply-To: dhinakaran.pandiyan@intel.com Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by gabe.freedesktop.org (Postfix) with ESMTPS id 456496E6DB for ; Tue, 29 May 2018 21:05:33 +0000 (UTC) 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: Rodrigo Vivi Cc: Jani Nikula , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gVHVlLCAyMDE4LTA1LTI5IGF0IDEyOjUxIC0wNzAwLCBSb2RyaWdvIFZpdmkgd3JvdGU6Cj4g T24gVGh1LCBNYXkgMjQsIDIwMTggYXQgMDg6MzA6NDdQTSAtMDcwMCwgRGhpbmFrYXJhbiBQYW5k aXlhbiB3cm90ZToKPiA+IAo+ID4gRFBDRCAyMDA5aCAiU3luY2hyb25pemF0aW9uIGxhdGVuY3kg aW4gc2luayIgaGFzIGJpdHMgdGhhdCB0ZWxsIHVzCj4gPiB0aGUKPiA+IG1heGltdW0gbnVtYmVy IG9mIGZyYW1lcyBzaW5rIGNhbiB0YWtlIHRvIHJlc3luY2hyb25pemUgdG8gc291cmNlCj4gPiB0 aW1pbmcKPiA+IHdoZW4gZXhpdGluZyBQU1IuIE1vcmUgaW1wb3J0YW50bHksIGFzIHBlciBlRFAg MS40YiwgdGhpcyBpcyB0aGUKPiA+ICJNaW5pbXVtCj4gPiBudW1iZXIgb2YgZnJhbWVzIGZvbGxv d2luZyBQU1IgZXhpdCB0aGF0IHRoZSBTb3VyY2UgZGV2aWNlIG5lZWRzIHRvCj4gPiB3YWl0IGZv ciBQU1IgZW50cnkuIgo+ID4gCj4gPiBXZSBjdXJyZW50bHkgdXNlIHRoaXMgdmFsdWUgb25seSB0 byBzZXR1cCB0aGUgbnVtYmVyIGZyYW1lcyB0byB3YWl0Cj4gPiBiZWZvcmUKPiA+IFBTUjIgc2Vs ZWN0aXZlIHVwZGF0ZS4gQnV0LCBiYXNlZCBvbiB0aGUgYWJvdmUgZGVzY3JpcHRpb24gaXQgbWFr ZXMKPiA+IG1vcmUKPiA+IHNlbnNlIHRvIHVzZSB0aGlzIHRvIGNvbmZpZ3VyZSBpZGxlIGZyYW1l cyBmb3IgYm90aCBQU1IxIGFuZCBhbmQKPiA+IFBTUjIuIFRoaXMKPiA+IHdpbGwgZW5zdXJlIHdl IHdhaXQgdGhlIHJlcXVpcmVkIG51bWJlciBvZiBmcmFtZXMgYmVmb3JlCj4gPiBhY3RpdmF0aW9u IHdoZXRoZXIgaXQgaXMgUFNSMSBvciBQU1IyLgo+ID4gCj4gPiBUaGUgbWluaW11bSBudW1iZXIg b2YgaWRsZSBmcmFtZXMgcmVtYWlucyA2LCB3aGlsZSBhbGxvd2luZyBzaW5rCj4gPiBzeW5jaHJv bml6YXRpb24gbGF0ZW5jeSBhbmQgVkJUIHRvIGluY3JlYXNlIHRoaXMgdmFsdWUuCj4gPiAKPiA+ IFRoaXMgYWxzbyBzb2x2ZXMgdGhlIGZsaXAtZmxvcCBiZXR3ZWVuIHNpbmsgYW5kIHNvdXJjZSBm cmFtZXMgdGhhdAo+ID4gSQo+ID4gbm90aWNlZCBvbiBteSBUaGlua3BhZCBYMjYwIGR1cmluZyBQ U1IgZXhpdC4gVGhpcyBzcGVjaWZpYyBwYW5lbAo+ID4gaGFzIGEKPiA+IHZhbHVlIG9mIDhoLCB3 aGljaCBhY2NvcmRpbmcgdG8gdGhlIHNwZWMgbWVhbnMgdGhlICJTb3VyY2UgZGV2aWNlCj4gPiBt dXN0Cj4gPiB3YWl0IGZvciBtb3JlIHRoYW4gZWlnaHQgYWN0aXZlIGZyYW1lcyBhZnRlciBQU1Ig ZXhpdCBiZWZvcmUKPiA+IGluaXRpYXRpbmcgUFNSCj4gPiBlbnRyeS4gKEluIHRoaXMgY2FzZSwg c2hvdWxkIGJlIHByb3ZpZGVkIGJ5IHRoZSBwYW5lbCBzdXBwbGllci4pIgo+ID4gVkJUCj4gPiBo b3dldmVyIGhhcyBhIHZhbHVlIG9mIDAuCj4gPiAKPiA+IENjOiBKYW5pIE5pa3VsYSA8amFuaS5u aWt1bGFAaW50ZWwuY29tPgo+ID4gQ2M6IEpvc2UgUm9iZXJ0byBkZSBTb3V6YSA8am9zZS5zb3V6 YUBpbnRlbC5jb20+Cj4gPiBDYzogUm9kcmlnbyBWaXZpIDxyb2RyaWdvLnZpdmlAaW50ZWwuY29t Pgo+ID4gU2lnbmVkLW9mZi1ieTogRGhpbmFrYXJhbiBQYW5kaXlhbiA8ZGhpbmFrYXJhbi5wYW5k aXlhbkBpbnRlbC5jb20+Cj4gPiAtLS0KPiA+IMKgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxf cHNyLmMgfCA0MCArKysrKysrKysrKysrKysrKysrKy0tLS0tLQo+ID4gLS0tLS0tLS0tLS0tLS0K PiA+IMKgMSBmaWxlIGNoYW5nZWQsIDIwIGluc2VydGlvbnMoKyksIDIwIGRlbGV0aW9ucygtKQo+ ID4gCj4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcHNyLmMKPiA+ IGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcHNyLmMKPiA+IGluZGV4IGViYzQ4M2YwNmM2 Zi4uNzFkZmU1NDE3NDBmIDEwMDY0NAo+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50 ZWxfcHNyLmMKPiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3Bzci5jCj4gPiBA QCAtMjQ3LDYgKzI0Nyw4IEBAIHZvaWQgaW50ZWxfcHNyX2luaXRfZHBjZChzdHJ1Y3QgaW50ZWxf ZHAKPiA+ICppbnRlbF9kcCkKPiA+IMKgCQlyZXR1cm47Cj4gPiDCoAl9Cj4gPiDCoAlkZXZfcHJp di0+cHNyLnNpbmtfc3VwcG9ydCA9IHRydWU7Cj4gPiArCWRldl9wcml2LT5wc3Iuc2lua19zeW5j X2xhdGVuY3kgPQo+ID4gKwkJaW50ZWxfZHBfZ2V0X3Npbmtfc3luY19sYXRlbmN5KGludGVsX2Rw KTsKPiA+IMKgCj4gPiDCoAlpZiAoSU5URUxfR0VOKGRldl9wcml2KSA+PSA5ICYmCj4gPiDCoAnC oMKgwqDCoChpbnRlbF9kcC0+cHNyX2RwY2RbMF0gPT0KPiA+IERQX1BTUjJfV0lUSF9ZX0NPT1JE X0lTX1NVUFBPUlRFRCkpIHsKPiA+IEBAIC0yNzIsOCArMjc0LDYgQEAgdm9pZCBpbnRlbF9wc3Jf aW5pdF9kcGNkKHN0cnVjdCBpbnRlbF9kcAo+ID4gKmludGVsX2RwKQo+ID4gwqAJCWlmIChkZXZf cHJpdi0+cHNyLnNpbmtfcHNyMl9zdXBwb3J0KSB7Cj4gPiDCoAkJCWRldl9wcml2LT5wc3IuY29s b3JpbWV0cnlfc3VwcG9ydCA9Cj4gPiDCoAkJCQlpbnRlbF9kcF9nZXRfY29sb3JpbWV0cnlfc3Rh dHVzKGluCj4gPiB0ZWxfZHApOwo+ID4gLQkJCWRldl9wcml2LT5wc3Iuc2lua19zeW5jX2xhdGVu Y3kgPQo+ID4gLQkJCQlpbnRlbF9kcF9nZXRfc2lua19zeW5jX2xhdGVuY3koaW50Cj4gPiBlbF9k cCk7Cj4gPiDCoAkJfQo+ID4gwqAJfQo+ID4gwqB9Cj4gPiBAQCAtMzcwLDIxICszNzAsMjEgQEAg c3RhdGljIHZvaWQgaHN3X2FjdGl2YXRlX3BzcjEoc3RydWN0IGludGVsX2RwCj4gPiAqaW50ZWxf ZHApCj4gPiDCoAlzdHJ1Y3QgaW50ZWxfZGlnaXRhbF9wb3J0ICpkaWdfcG9ydCA9Cj4gPiBkcF90 b19kaWdfcG9ydChpbnRlbF9kcCk7Cj4gPiDCoAlzdHJ1Y3QgZHJtX2RldmljZSAqZGV2ID0gZGln X3BvcnQtPmJhc2UuYmFzZS5kZXY7Cj4gPiDCoAlzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2 X3ByaXYgPSB0b19pOTE1KGRldik7Cj4gPiArCXUzMiBtYXhfc2xlZXBfdGltZSA9IDB4MWY7Cj4g PiArCXUzMiB2YWwgPSBFRFBfUFNSX0VOQUJMRTsKPiA+IMKgCj4gPiAtCXVpbnQzMl90IG1heF9z bGVlcF90aW1lID0gMHgxZjsKPiA+IC0JLyoKPiA+IC0JwqAqIExldCdzIHJlc3BlY3QgVkJUIGlu IGNhc2UgVkJUIGFza3MgYSBoaWdoZXIgaWRsZV9mcmFtZQo+ID4gdmFsdWUuCj4gPiAtCcKgKiBM ZXQncyB1c2UgNiBhcyB0aGUgbWluaW11bSB0byBjb3ZlciBhbGwga25vd24gY2FzZXMKPiA+IGlu Y2x1ZGluZwo+ID4gLQnCoCogdGhlIG9mZi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29t ZSBjYXNlcy4gQWxzbwo+ID4gdGhlcmUgYXJlCj4gPiAtCcKgKiBjYXNlcyB3aGVyZSBzaW5rIHNo b3VsZCBiZSBhYmxlIHRvIHRyYWluCj4gPiAtCcKgKiB3aXRoIHRoZSA1IG9yIDYgaWRsZSBwYXR0 ZXJucy4KPiA+ICsJLyogTGV0J3MgdXNlIDYgYXMgdGhlIG1pbmltdW0gdG8gY292ZXIgYWxsIGtu b3duIGNhc2VzCj4gPiBpbmNsdWRpbmcgdGhlCj4gPiArCcKgKiBvZmYtYnktb25lIGlzc3VlIHRo YXQgSFcgaGFzIGluIHNvbWUgY2FzZXMuCj4gPiDCoAnCoCovCj4gPiAtCXVpbnQzMl90IGlkbGVf ZnJhbWVzID0gbWF4KDYsIGRldl9wcml2LQo+ID4gPnZidC5wc3IuaWRsZV9mcmFtZXMpOwo+ID4g LQl1aW50MzJfdCB2YWwgPSBFRFBfUFNSX0VOQUJMRTsKPiA+ICsJaW50IGlkbGVfZnJhbWVzID0g bWF4KDYsIGRldl9wcml2LT52YnQucHNyLmlkbGVfZnJhbWVzKTsKPiA+IMKgCj4gPiAtCXZhbCB8 PSBtYXhfc2xlZXBfdGltZSA8PCBFRFBfUFNSX01BWF9TTEVFUF9USU1FX1NISUZUOwo+ID4gKwkv KiBzaW5rX3N5bmNfbGF0ZW5jeSBvZiA4IG1lYW5zIHNvdXJjZSBoYXMgdG8gd2FpdCBmb3IKPiA+ IG1vcmUgdGhhbiA4Cj4gPiArCcKgKiBmcmFtZXMsIHdlJ2xsIGdvIHdpdGggOSBmcmFtZXMgZm9y IG5vdwo+ID4gKwnCoCovCj4gPiArCWlkbGVfZnJhbWVzID0gbWF4KGlkbGVfZnJhbWVzLCBkZXZf cHJpdi0KPiA+ID5wc3Iuc2lua19zeW5jX2xhdGVuY3kgKyAxKTsKPiA+IMKgCXZhbCB8PSBpZGxl X2ZyYW1lcyA8PCBFRFBfUFNSX0lETEVfRlJBTUVfU0hJRlQ7Cj4gPiDCoAo+ID4gKwl2YWwgfD0g bWF4X3NsZWVwX3RpbWUgPDwgRURQX1BTUl9NQVhfU0xFRVBfVElNRV9TSElGVDsKPiA+IMKgCWlm IChJU19IQVNXRUxMKGRldl9wcml2KSkKPiA+IMKgCQl2YWwgfD0gRURQX1BTUl9NSU5fTElOS19F TlRSWV9USU1FXzhfTElORVM7Cj4gPiDCoAo+ID4gQEAgLTQyNCwxNSArNDI0LDE1IEBAIHN0YXRp YyB2b2lkIGhzd19hY3RpdmF0ZV9wc3IyKHN0cnVjdCBpbnRlbF9kcAo+ID4gKmludGVsX2RwKQo+ ID4gwqAJc3RydWN0IGludGVsX2RpZ2l0YWxfcG9ydCAqZGlnX3BvcnQgPQo+ID4gZHBfdG9fZGln X3BvcnQoaW50ZWxfZHApOwo+ID4gwqAJc3RydWN0IGRybV9kZXZpY2UgKmRldiA9IGRpZ19wb3J0 LT5iYXNlLmJhc2UuZGV2Owo+ID4gwqAJc3RydWN0IGRybV9pOTE1X3ByaXZhdGUgKmRldl9wcml2 ID0gdG9faTkxNShkZXYpOwo+ID4gLQkvKgo+ID4gLQnCoCogTGV0J3MgcmVzcGVjdCBWQlQgaW4g Y2FzZSBWQlQgYXNrcyBhIGhpZ2hlciBpZGxlX2ZyYW1lCj4gPiB2YWx1ZS4KPiA+IC0JwqAqIExl dCdzIHVzZSA2IGFzIHRoZSBtaW5pbXVtIHRvIGNvdmVyIGFsbCBrbm93biBjYXNlcwo+ID4gaW5j bHVkaW5nCj4gPiAtCcKgKiB0aGUgb2ZmLWJ5LW9uZSBpc3N1ZSB0aGF0IEhXIGhhcyBpbiBzb21l IGNhc2VzLiBBbHNvCj4gPiB0aGVyZSBhcmUKPiA+IC0JwqAqIGNhc2VzIHdoZXJlIHNpbmsgc2hv dWxkIGJlIGFibGUgdG8gdHJhaW4KPiA+IC0JwqAqIHdpdGggdGhlIDUgb3IgNiBpZGxlIHBhdHRl cm5zLgo+ID4gKwl1MzIgdmFsOwo+ID4gKwo+ID4gKwkvKiBMZXQncyB1c2UgNiBhcyB0aGUgbWlu aW11bSB0byBjb3ZlciBhbGwga25vd24gY2FzZXMKPiA+IGluY2x1ZGluZyB0aGUKPiA+ICsJwqAq IG9mZi1ieS1vbmUgaXNzdWUgdGhhdCBIVyBoYXMgaW4gc29tZSBjYXNlcy4KPiA+IMKgCcKgKi8K PiA+IC0JdWludDMyX3QgaWRsZV9mcmFtZXMgPSBtYXgoNiwgZGV2X3ByaXYtCj4gPiA+dmJ0LnBz ci5pZGxlX2ZyYW1lcyk7Cj4gPiAtCXUzMiB2YWwgPSBpZGxlX2ZyYW1lcyA8PCBFRFBfUFNSMl9J RExFX0ZSQU1FX1NISUZUOwo+ID4gKwlpbnQgaWRsZV9mcmFtZXMgPSBtYXgoNiwgZGV2X3ByaXYt PnZidC5wc3IuaWRsZV9mcmFtZXMpOwo+ID4gKwo+ID4gKwlpZGxlX2ZyYW1lcyA9IG1heChpZGxl X2ZyYW1lcywgZGV2X3ByaXYtCj4gPiA+cHNyLnNpbmtfc3luY19sYXRlbmN5ICsgMSk7Cj4gPiAr CXZhbCA9IGlkbGVfZnJhbWVzIDw8IEVEUF9QU1IyX0lETEVfRlJBTUVfU0hJRlQ7Cj4gSSB3b25k ZXIgaWYgd2Ugc2hvdWxkIGNvbnNvbGlkYXRlIGFsbCB0aGlzIGxvZ2luIGluIGEgc2luZ2xlIGZ1 bmN0aW9uCj4gc2luY2UgdGhleQo+IGFyZSBpZGVudGljYWwgYW5kIG9ubHkgdGhlIHNoaWZ0IGlz IGRpZmZlcmVudC4uLgo+IAo+IEJ1dCB0aGUgbG9naWMgaXMgY29ycmVjdC4gSSBjaGVja2VkIGVE UCAxLjMgYW5kIDEuNCBzcGVjcyBhbmQgSQo+IGJlbGlldmUgd2UKPiBuZWVkIHRvIG1vdmUgZm9y d2FyZCB3aXRoIHRoaXMgY2hhbmdlIGFuZCBkbyBjbGVhbi11cHMgb24gZm9sbG93LXVwcy4KPiAK PiAKPiBSZXZpZXdlZC1ieTogUm9kcmlnbyBWaXZpIDxyb2RyaWdvLnZpdmlAaW50ZWwuY29tPgo+ IAo+IEFsc28gSSBkb24ndCBiZWxpZXZlIHRoYXQgd2UgbmVlZCBjYzpzdGFibGUgYmVjYXVzZSB3 ZSBuZXZlciBlbmFibGVkCj4gaXQgYW55d2hlcmUKPiBhbnl3YXkgYmVzaWRlcyB0aGUgZmFjdCB0 aGF0IGl0IHdvbnQgYmUgYSBjbGVhbiBiYWNrcG9ydC4KClllYWgsIHdlJ3ZlIGhhZCBhIGNvdXBs ZSBvZiBjb21taXRzIHRoYXQgdG91Y2ggdGhpcyBjb2RlLCB3b24ndCBiZSBhCmNsZWFuIGJhY2sg cG9ydC4KwqAKVGhhbmtzIGZvciByZXZpZXdpbmcgYW5kIG1lcmdpbmcgdGhpcyBwYXRjaC4KCj4g Cj4gPiAKPiA+IMKgCj4gPiDCoAkvKiBGSVhNRTogc2VsZWN0aXZlIHVwZGF0ZSBpcyBwcm9iYWJs eSB0b3RhbGx5IGJyb2tlbgo+ID4gYmVjYXVzZSBpdCBkb2Vzbid0Cj4gPiDCoAnCoCogbWVzaCBh dCBhbGwgd2l0aCBvdXIgZnJvbnRidWZmZXIgdHJhY2tpbmcuIEFuZCB0aGUgaHcKPiA+IGFsb25l IGlzbid0Cl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCklu dGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRw czovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=