From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kazlauskas, Nicholas" Subject: Re: [PATCH 4/4] drm/amd/display: Make pageflip event delivery compatible with VRR. Date: Tue, 19 Mar 2019 13:05:34 +0000 Message-ID: References: <20190318171952.24302-1-mario.kleiner.de@gmail.com> <20190318171952.24302-5-mario.kleiner.de@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20190318171952.24302-5-mario.kleiner.de-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Content-Language: en-US Content-ID: <32BBACFE49DC2441BCC49B876E005788-asWib9pRmPqcE4WynfumptQqCkab/8FMAL8bYrjMMd8@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "amd-gfx" To: Mario Kleiner , "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" Cc: "dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" List-Id: dri-devel@lists.freedesktop.org T24gMy8xOC8xOSAxOjE5IFBNLCBNYXJpbyBLbGVpbmVyIHdyb3RlOg0KPiBXZSB3YW50IHZibGFu ayBjb3VudHMgYW5kIHRpbWVzdGFtcHMgb2YgZmxpcCBjb21wbGV0aW9uIGFzIHNlbnQNCj4gaW4g cGFnZWZsaXAgY29tcGxldGlvbiBldmVudHMgdG8gYmUgY29uc2lzdGVudCB3aXRoIHRoZSB2Ymxh bmsNCj4gY291bnQgYW5kIHRpbWVzdGFtcCBvZiB0aGUgdmJsYW5rIG9mIGZsaXAgY29tcGxldGlv biwgbGlrZSBpbiBub24NCj4gVlJSIG1vZGUuDQo+IA0KPiBJbiBWUlIgbW9kZSwgZHJtX3VwZGF0 ZV92YmxhbmtfY291bnQoKSAtIGFuZCB0aGVyZWJ5IHZibGFuaw0KPiBjb3VudCBhbmQgdGltZXN0 YW1wIHVwZGF0ZXMgLSBtdXN0IGJlIGRlbGF5ZWQgdW50aWwgYWZ0ZXIgdGhlDQo+IGVuZCBvZiBm cm9udC1wb3JjaCBvZiBlYWNoIHZibGFuaywgYXMgaXQgaXMgb25seSBzYWZlIHRvDQo+IGNhbGN1 bGF0ZSB2YmxhbmsgdGltZXN0YW1wcyBvdXRzaWRlIG9mIHRoZSBmcm9udC1wb3JjaCwgd2hlbg0K PiB3ZSBhY3R1YWxseSBrbm93IHdoZW4gdGhlIHZibGFuayB3aWxsIGVuZCBvciBoYXMgZW5kZWQu DQo+IA0KPiBUaGUgZnVuY3Rpb24gZHJtX3VwZGF0ZV92YmxhbmtfY291bnQoKSB3aGljaCB1cGRh dGVzIHRpbWVzdGFtcHMNCj4gYW5kIGNvdW50cyBnZXRzIGNhbGxlZCBieSBkcm1fY3J0Y19hY2N1 cmF0ZV92YmxhbmtfY291bnQoKSBvciBieQ0KPiBkcm1fY3J0Y19oYW5kbGVfdmJsYW5rKCkuDQo+ IA0KPiBUaGVyZWZvcmUgd2UgbXVzdCBtYWtlIHN1cmUgdGhhdCBwYWdlZmxpcCBldmVudHMgZm9y IGEgY29tcGxldGVkDQo+IGZsaXAgYXJlIG9ubHkgc2VudCBvdXQgYWZ0ZXIgZHJtX2NydGNfYWNj dXJhdGVfdmJsYW5rX2NvdW50KCkgb3INCj4gZHJtX2NydGNfaGFuZGxlX3ZibGFuaygpIGlzIGV4 ZWN1dGVkLCBhZnRlciBlbmQgb2YgZnJvbnQtcG9yY2gNCj4gZm9yIHRoZSB2Ymxhbmsgb2YgZmxp cCBjb21wbGV0aW9uLg0KPiANCj4gVHdvIGNhc2VzOg0KPiANCj4gYSkgUGFnZWZsaXAgaXJxIGhh bmRsZXIgZXhlY3V0ZXMgaW5zaWRlIGZyb250LXBvcmNoOg0KPiAgICAgSW4gdGhpcyBjYXNlIHdl IG11c3QgZGVmZXIgc2VuZGluZyBwYWdlZmxpcCBldmVudHMgdW50aWwNCj4gICAgIGRybV9jcnRj X2hhbmRsZV92YmxhbmsoKSBleGVjdXRlcyBhZnRlciBlbmQgb2YgZnJvbnQtcG9yY2gsDQo+ICAg ICBhbmQgdGhlcmVieSBjYWxjdWxhdGVzIHByb3BlciB2YmxhbmsgY291bnQgYW5kIHRpbWVzdGFt cC4NCj4gICAgIElvdy4gdGhlIHBmbGlwIGlycSBoYW5kbGVyIG11c3QganVzdCBhcm0gYSBwYWdl ZmxpcCBldmVudA0KPiAgICAgdG8gYmUgc2VudCBvdXQgYnkgZHJtX2NydGNfaGFuZGxlX3ZibGFu aygpIGxhdGVyIG9uLg0KPiANCj4gYikgUGFnZWZsaXAgaXJxIGhhbmRsZXIgZXhlY3V0ZXMgYWZ0 ZXIgZW5kIG9mIGZyb250LXBvcmNoLCBlLmcuLA0KPiAgICAgYWZ0ZXIgZmxpcCBjb21wbGV0aW9u IGluIGJhY2stcG9yY2ggb3IgZHVlIHRvIGEgbWFzc2l2ZWx5DQo+ICAgICBkZWxheWVkIGhhbmRs ZXIgaW52b2NhdGlvbiBpbnRvIHRoZSBhY3RpdmUgc2Nhbm91dCBvZiB0aGUgbmV3DQo+ICAgICBm cmFtZS4gSW4gdGhpcyBjYXNlIHdlIGNhbiBjYWxsIGRybV9jcnRjX2FjY3VyYXRlX3ZibGFua19j b3VudCgpDQo+ICAgICB0byBzYWZlbHkgZm9yY2UgY2FsY3VsYXRpb24gb2YgYSBwcm9wZXIgdmJs YW5rIGNvdW50IGFuZA0KPiAgICAgdGltZXN0YW1wLCBhbmQgbXVzdCBzZW5kIHRoZSBwYWdlZmxp cCBjb21wbGV0aW9uIGV2ZW50DQo+ICAgICBvdXJzZWx2ZXMgZnJvbSB0aGUgcGFnZWZsaXAgaXJx IGhhbmRsZXIuDQo+IA0KPiAgICAgVGhpcyBpcyB0aGUgc2FtZSBiZWhhdmlvdXIgYXMgbmVlZGVk IGZvciBzdGFuZGFyZCBmaXhlZCByZWZyZXNoDQo+ICAgICByYXRlIG1vZGUuDQo+IA0KPiBUbyBk ZWNpZGUgZnJvbSB3aXRoaW4gcGFnZWZsaXAgaGFuZGxlciBpZiB3ZSBhcmUgaW4gY2FzZSBhKSBv ciBiKSwNCj4gd2UgY2hlY2sgdGhlIGN1cnJlbnQgc2Nhbm91dCBwb3NpdGlvbiBhZ2FpbnN0IHRo ZSBib3VuZGFyeSBvZg0KPiBmcm9udC1wb3JjaC4gSW4gbm9uLVZSUiBtb2RlIHdlIGp1c3QgZG8g d2hhdCB3ZSBkaWQgaW4gdGhlIHBhc3QuDQo+IA0KPiBTaWduZWQtb2ZmLWJ5OiBNYXJpbyBLbGVp bmVyIDxtYXJpby5rbGVpbmVyLmRlQGdtYWlsLmNvbT4NCg0KUmV2aWV3ZWQtYnk6IE5pY2hvbGFz IEthemxhdXNrYXMgPG5pY2hvbGFzLmthemxhdXNrYXNAYW1kLmNvbT4NCg0KVGhpcyBwYXRjaCBs b29rcyBmaW5lIHRvIG1lIGZvciB0aGUgbW9zdCBwYXJ0LCBidXQgaXQncyBzdGlsbCBwZW5kaW5n IG9uIA0KdGhlIG90aGVyIHBhdGNoZXMgaW4gdGhlIHNlcmllcy4NCg0KPiAtLS0NCj4gICBkcml2 ZXJzL2dwdS9kcm0vYW1kL2Rpc3BsYXkvYW1kZ3B1X2RtL2FtZGdwdV9kbS5jIHwgNjggKysrKysr KysrKysrKysrKysrLS0tLS0NCj4gICAxIGZpbGUgY2hhbmdlZCwgNTUgaW5zZXJ0aW9ucygrKSwg MTMgZGVsZXRpb25zKC0pDQo+IA0KPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9k aXNwbGF5L2FtZGdwdV9kbS9hbWRncHVfZG0uYyBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvZGlzcGxh eS9hbWRncHVfZG0vYW1kZ3B1X2RtLmMNCj4gaW5kZXggNTU1ZDllOWYuLjQ5OTI4NGQgMTAwNjQ0 DQo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9hbWQvZGlzcGxheS9hbWRncHVfZG0vYW1kZ3B1X2Rt LmMNCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9kaXNwbGF5L2FtZGdwdV9kbS9hbWRncHVf ZG0uYw0KPiBAQCAtMjYzLDYgKzI2MywxMCBAQCBzdGF0aWMgdm9pZCBkbV9wZmxpcF9oaWdoX2ly cSh2b2lkICppbnRlcnJ1cHRfcGFyYW1zKQ0KPiAgIAlzdHJ1Y3QgY29tbW9uX2lycV9wYXJhbXMg KmlycV9wYXJhbXMgPSBpbnRlcnJ1cHRfcGFyYW1zOw0KPiAgIAlzdHJ1Y3QgYW1kZ3B1X2Rldmlj ZSAqYWRldiA9IGlycV9wYXJhbXMtPmFkZXY7DQo+ICAgCXVuc2lnbmVkIGxvbmcgZmxhZ3M7DQo+ ICsJc3RydWN0IGRybV9wZW5kaW5nX3ZibGFua19ldmVudCAqZTsNCj4gKwlzdHJ1Y3QgZG1fY3J0 Y19zdGF0ZSAqYWNydGNfc3RhdGU7DQo+ICsJdWludDMyX3QgdnBvcywgaHBvcywgdl9ibGFua19z dGFydCwgdl9ibGFua19lbmQ7DQo+ICsJYm9vbCB2cnJfYWN0aXZlOw0KPiAgIA0KPiAgIAlhbWRn cHVfY3J0YyA9IGdldF9jcnRjX2J5X290Z19pbnN0KGFkZXYsIGlycV9wYXJhbXMtPmlycV9zcmMg LSBJUlFfVFlQRV9QRkxJUCk7DQo+ICAgDQo+IEBAIC0yODUsMTggKzI4OSw1NyBAQCBzdGF0aWMg dm9pZCBkbV9wZmxpcF9oaWdoX2lycSh2b2lkICppbnRlcnJ1cHRfcGFyYW1zKQ0KPiAgIAkJcmV0 dXJuOw0KPiAgIAl9DQo+ICAgDQo+IC0JLyogVXBkYXRlIHRvIGNvcnJlY3QgY291bnQocykgaWYg cmFjaW5nIHdpdGggdmJsYW5rIGlycSAqLw0KPiAtCWRybV9jcnRjX2FjY3VyYXRlX3ZibGFua19j b3VudCgmYW1kZ3B1X2NydGMtPmJhc2UpOw0KPiArCS8qIHBhZ2UgZmxpcCBjb21wbGV0ZWQuICov DQo+ICsJZSA9IGFtZGdwdV9jcnRjLT5ldmVudDsNCj4gKwlhbWRncHVfY3J0Yy0+ZXZlbnQgPSBO VUxMOw0KPiAgIA0KPiAtCS8qIHdha2UgdXAgdXNlcnNwYWNlICovDQo+IC0JaWYgKGFtZGdwdV9j cnRjLT5ldmVudCkgew0KPiAtCQlkcm1fY3J0Y19zZW5kX3ZibGFua19ldmVudCgmYW1kZ3B1X2Ny dGMtPmJhc2UsIGFtZGdwdV9jcnRjLT5ldmVudCk7DQo+ICsJaWYgKCFlKQ0KPiArCQlXQVJOX09O KDEpOw0KPiAgIA0KPiAtCQkvKiBwYWdlIGZsaXAgY29tcGxldGVkLiBjbGVhbiB1cCAqLw0KPiAt CQlhbWRncHVfY3J0Yy0+ZXZlbnQgPSBOVUxMOw0KPiArCWFjcnRjX3N0YXRlID0gdG9fZG1fY3J0 Y19zdGF0ZShhbWRncHVfY3J0Yy0+YmFzZS5zdGF0ZSk7DQo+ICsJdnJyX2FjdGl2ZSA9IGFtZGdw dV9kbV92cnJfYWN0aXZlKGFjcnRjX3N0YXRlKTsNCj4gKw0KPiArCS8qIEZpeGVkIHJlZnJlc2gg cmF0ZSwgb3IgVlJSIHNjYW5vdXQgcG9zaXRpb24gb3V0c2lkZSBmcm9udC1wb3JjaD8gKi8NCj4g KwlpZiAoIXZycl9hY3RpdmUgfHwNCj4gKwkgICAgIWRjX3N0cmVhbV9nZXRfc2Nhbm91dHBvcyhh Y3J0Y19zdGF0ZS0+c3RyZWFtLCAmdl9ibGFua19zdGFydCwNCj4gKwkJCQkgICAgICAmdl9ibGFu a19lbmQsICZocG9zLCAmdnBvcykgfHwNCj4gKwkgICAgKHZwb3MgPCB2X2JsYW5rX3N0YXJ0KSkg ew0KPiArCQkvKiBVcGRhdGUgdG8gY29ycmVjdCBjb3VudCBhbmQgdmJsYW5rIHRpbWVzdGFtcCBp ZiByYWNpbmcgd2l0aA0KPiArCQkgKiB2YmxhbmsgaXJxLiBUaGlzIGFsc28gdXBkYXRlcyB0byB0 aGUgY29ycmVjdCB2YmxhbmsgdGltZXN0YW1wDQo+ICsJCSAqIGV2ZW4gaW4gVlJSIG1vZGUsIGFz IHNjYW5vdXQgaXMgcGFzdCB0aGUgZnJvbnQtcG9yY2ggYXRtLg0KPiArCQkgKi8NCj4gKwkJZHJt X2NydGNfYWNjdXJhdGVfdmJsYW5rX2NvdW50KCZhbWRncHVfY3J0Yy0+YmFzZSk7DQo+ICAgDQo+ IC0JfSBlbHNlDQo+IC0JCVdBUk5fT04oMSk7DQo+ICsJCS8qIFdha2UgdXAgdXNlcnNwYWNlIGJ5 IHNlbmRpbmcgdGhlIHBhZ2VmbGlwIGV2ZW50IHdpdGggcHJvcGVyDQo+ICsJCSAqIGNvdW50IGFu ZCB0aW1lc3RhbXAgb2YgdmJsYW5rIG9mIGZsaXAgY29tcGxldGlvbi4NCj4gKwkJICovDQo+ICsJ CWlmIChlKSB7DQo+ICsJCQlkcm1fY3J0Y19zZW5kX3ZibGFua19ldmVudCgmYW1kZ3B1X2NydGMt PmJhc2UsIGUpOw0KPiArDQo+ICsJCQkvKiBFdmVudCBzZW50LCBzbyBkb25lIHdpdGggdmJsYW5r IGZvciB0aGlzIGZsaXAgKi8NCj4gKwkJCWRybV9jcnRjX3ZibGFua19wdXQoJmFtZGdwdV9jcnRj LT5iYXNlKTsNCj4gKwkJfQ0KPiArCX0gZWxzZSBpZiAoZSkgew0KPiArCQkvKiBWUlIgYWN0aXZl IGFuZCBpbnNpZGUgZnJvbnQtcG9yY2g6IHZibGFuayBjb3VudCBhbmQNCj4gKwkJICogdGltZXN0 YW1wIGZvciBwYWdlZmxpcCBldmVudCB3aWxsIG9ubHkgYmUgdXAgdG8gZGF0ZSBhZnRlcg0KPiAr CQkgKiBkcm1fY3J0Y19oYW5kbGVfdmJsYW5rKCkgaGFzIGJlZW4gZXhlY3V0ZWQgZnJvbSBsYXRl IHZibGFuaw0KPiArCQkgKiBpcnEgaGFuZGxlciBhZnRlciBzdGFydCBvZiBiYWNrLXBvcmNoICh2 bGluZSAwKS4gV2UgcXVldWUgdGhlDQo+ICsJCSAqIHBhZ2VmbGlwIGV2ZW50IGZvciBzZW5kLW91 dCBieSBkcm1fY3J0Y19oYW5kbGVfdmJsYW5rKCkgd2l0aA0KPiArCQkgKiB1cGRhdGVkIHRpbWVz dGFtcCBhbmQgY291bnQsIG9uY2UgaXQgcnVucyBhZnRlciB1cy4NCj4gKwkJICoNCj4gKwkJICog V2UgbmVlZCB0byBvcGVuLWNvZGUgdGhpcyBpbnN0ZWFkIG9mIHVzaW5nIHRoZSBoZWxwZXINCj4g KwkJICogZHJtX2NydGNfYXJtX3ZibGFua19ldmVudCgpLCBhcyB0aGF0IGhlbHBlciB3b3VsZA0K PiArCQkgKiBjYWxsIGRybV9jcnRjX2FjY3VyYXRlX3ZibGFua19jb3VudCgpLCB3aGljaCB3ZSBt dXN0DQo+ICsJCSAqIG5vdCBjYWxsIGluIFZSUiBtb2RlIHdoaWxlIHdlIGFyZSBpbiBmcm9udC1w b3JjaCENCj4gKwkJICovDQo+ICsNCj4gKwkJLyogc2VxdWVuY2Ugd2lsbCBiZSByZXBsYWNlZCBi eSByZWFsIGNvdW50IGR1cmluZyBzZW5kLW91dC4gKi8NCj4gKwkJZS0+c2VxdWVuY2UgPSBkcm1f Y3J0Y192YmxhbmtfY291bnQoJmFtZGdwdV9jcnRjLT5iYXNlKTsNCj4gKwkJZS0+cGlwZSA9IGFt ZGdwdV9jcnRjLT5jcnRjX2lkOw0KPiArDQo+ICsJCWxpc3RfYWRkX3RhaWwoJmUtPmJhc2UubGlu aywgJmFkZXYtPmRkZXYtPnZibGFua19ldmVudF9saXN0KTsNCj4gKwkJZSA9IE5VTEw7DQoNCkkg Z3Vlc3MgZHJtX2NydGNfdmJsYW5rX29mZiBoYW5kbGVzIHNlbmRpbmcgYW55IGxlZnRvdmVyIGV2 ZW50cyBoZXJlIGlmIA0KdGhlcmUgYXJlIGFueSBpZiB0aGUgSVJRIGdldHMgZGlzYWJsZWQgaW5j b3JyZWN0bHkuDQoNCkkgd29uZGVyIHdoeSB3ZSBuZXZlciBqdXN0IHVzZWQgdGhpcyBsaXN0IGlu IHRoZSBmaXJzdCBwbGFjZSBmb3IgDQpwYWdlZmxpcCB2YmxhbmsgZXZlbnQgaGFuZGxpbmcgaW5z dGVhZCBvZiB0aGUgbG9ja2luZyBhbmQgTlVMTCBzZXQuIA0KVGhvdWdoIEkgc3VwcG9zZSBpdCdz IG1vcmUgdXNlZnVsIHRvIHVzZSB0aGlzIG5vdyBmb3IgVlJSIHZibGFuayBoYW5kbGluZy4NCg0K PiArCX0NCj4gICANCj4gICAJLyogS2VlcCB0cmFjayBvZiB2Ymxhbmsgb2YgdGhpcyBmbGlwIGZv ciBmbGlwIHRocm90dGxpbmcuIFdlIHVzZSB0aGUNCj4gICAJICogY29va2VkIGh3IGNvdW50ZXIs IGFzIHRoYXQgb25lIGluY3JlbWVudGVkIGF0IHN0YXJ0IG9mIHRoaXMgdmJsYW5rDQo+IEBAIC0z MDksMTAgKzM1Miw5IEBAIHN0YXRpYyB2b2lkIGRtX3BmbGlwX2hpZ2hfaXJxKHZvaWQgKmludGVy cnVwdF9wYXJhbXMpDQo+ICAgCWFtZGdwdV9jcnRjLT5wZmxpcF9zdGF0dXMgPSBBTURHUFVfRkxJ UF9OT05FOw0KPiAgIAlzcGluX3VubG9ja19pcnFyZXN0b3JlKCZhZGV2LT5kZGV2LT5ldmVudF9s b2NrLCBmbGFncyk7DQo+ICAgDQo+IC0JRFJNX0RFQlVHX0RSSVZFUigiJXMgLSBjcnRjIDolZFsl cF0sIHBmbGlwX3N0YXQ6QU1ER1BVX0ZMSVBfTk9ORVxuIiwNCj4gLQkJCQkJX19mdW5jX18sIGFt ZGdwdV9jcnRjLT5jcnRjX2lkLCBhbWRncHVfY3J0Yyk7DQo+IC0NCj4gLQlkcm1fY3J0Y192Ymxh bmtfcHV0KCZhbWRncHVfY3J0Yy0+YmFzZSk7DQo+ICsJRFJNX0RFQlVHX0RSSVZFUigiY3J0Yzol ZFslcF0sIHBmbGlwX3N0YXQ6QU1ER1BVX0ZMSVBfTk9ORSwgdnJyWyVkXS1mcCAlZFxuIiwNCj4g KwkJCSBhbWRncHVfY3J0Yy0+Y3J0Y19pZCwgYW1kZ3B1X2NydGMsDQo+ICsJCQkgdnJyX2FjdGl2 ZSwgKGludCkgIWUpOw0KDQpUaGUgZGVidWcgb3V0cHV0IGZvciB0aGlzIGlzIGEgbGl0dGxlIHN0 cmFuZ2UsIGJ1dCBJIGd1ZXNzIGl0IG1ha2VzIGl0IA0KbWFrZXMgYXMgbXVjaCBzZW5zZSBhcyB0 aGUgb3JpZ2luYWwgZGlkLiBJJ20gZ2xhZCB0aGUgX19mdW5jX18gaXMgZ29uZSANCmF0IGxlYXN0 Lg0KDQo+ICAgfQ0KPiAgIA0KPiAgIHN0YXRpYyB2b2lkIGRtX3Z1cGRhdGVfaGlnaF9pcnEodm9p ZCAqaW50ZXJydXB0X3BhcmFtcykNCj4gDQoNCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fCmFtZC1nZnggbWFpbGluZyBsaXN0CmFtZC1nZnhAbGlzdHMuZnJl ZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGlu Zm8vYW1kLWdmeA==