From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rodrigo Vivi Subject: Re: [PATCH v2 1/3] drm/i915/frontbuffer: Pull frontbuffer_flush out of gem_obj_pin_to_display Date: Wed, 7 Mar 2018 14:46:30 -0800 Message-ID: <20180307224628.GL4440@intel.com> References: <20180307033420.3086-1-dhinakaran.pandiyan@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 397C46E00C for ; Wed, 7 Mar 2018 22:46:31 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20180307033420.3086-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: intel-gfx@lists.freedesktop.org, Paulo Zanoni List-Id: intel-gfx@lists.freedesktop.org T24gVHVlLCBNYXIgMDYsIDIwMTggYXQgMDc6MzQ6MThQTSAtMDgwMCwgRGhpbmFrYXJhbiBQYW5k aXlhbiB3cm90ZToKPiBGcm9tOiAiUGFuZGl5YW4sIERoaW5ha2FyYW4iIDxkaGluYWthcmFuLnBh bmRpeWFuQGludGVsLmNvbT4KPiAKPiBpOTE1X2dlbV9vYmpfcGluX3RvX2Rpc3BsYXkoKSBjYWxs cyBmcm9udGJ1ZmZlcl9mbHVzaCB3aXRoIG9yaWdpbiBzZXQgdG8KPiBESVJUWUZCLiBUaGUgY2Fs bGVycyBob3dldmVyIGFyZSBhdCBhIHZhbnRhZ2UgcG9pbnQgdG8gZGVjaWRlIGlmIGhhcmR3YXJl Cj4gZnJvbnRidWZmZXIgdHJhY2tpbmcgY2FuIGRvIHRoZSBmbHVzaCBmb3IgdXMuIEZvciBleGFt cGxlLCBsZWdhY3kgY3Vyc29yCj4gdXBkYXRlcywgbGlrZSBmbGlwcywgd3JpdGUgdG8gTU1JTyBy ZWdpc3RlcnMsIHdoaWNoIHRoZW4gdHJpZ2dlcnMgUFNSIGZsdXNoCj4gYnkgdGhlIGhhcmR3YXJl LiBNb3ZpbmcgZnJvbnRidWZmZXJfZmx1c2ggb3V0IHdpbGwgZW5hYmxlIHVzIHRvIHNraXAgYQo+ IHNvZnR3YXJlIGluaXRpYXRlZCBmbHVzaCBieSBzZXR0aW5nIG9yaWdpbiB0byBGTElQLiBUaGFu a3MgdG8gQ2hyaXMgZm9yIHRoZQo+IGlkZWEuCj4gCj4gdjI6Cj4gUmViYXNlZCBkdWUgdG8gVmls bGUgYWRkaW5nIGludGVsX3BsYW5lX3Bpbl9mYigpLgo+IE1pbm9yIGNvZGUgcmVvcmRlcmluZyBh cyBmYl9vYmpfZmx1c2ggZG9lc24ndCBuZWVkIHN0cnVjdF9tdXRleCAoQ2hyaXMpCj4gCj4gQ2M6 IENocmlzIFdpbHNvbiA8Y2hyaXNAY2hyaXMtd2lsc29uLmNvLnVrPgo+IENjOiBWaWxsZSBTeXJq w6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+IENjOiBQYXVsbyBaYW5vbmkg PHBhdWxvLnIuemFub25pQGludGVsLmNvbT4KPiBTaWduZWQtb2ZmLWJ5OiBEaGluYWthcmFuIFBh bmRpeWFuIDxkaGluYWthcmFuLnBhbmRpeWFuQGludGVsLmNvbT4KPiAtLS0KPiAgZHJpdmVycy9n cHUvZHJtL2k5MTUvaTkxNV9nZW0uYyAgICAgIHwgOSArKysrLS0tLS0KPiAgZHJpdmVycy9ncHUv ZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jIHwgOSArKysrKysrLS0KPiAgZHJpdmVycy9ncHUvZHJt L2k5MTUvaW50ZWxfZmJkZXYuYyAgIHwgNSArKystLQo+ICBkcml2ZXJzL2dwdS9kcm0vaTkxNS9p bnRlbF9vdmVybGF5LmMgfCAxICsKPiAgNCBmaWxlcyBjaGFuZ2VkLCAxNSBpbnNlcnRpb25zKCsp LCA5IGRlbGV0aW9ucygtKQo+IAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9p OTE1X2dlbS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW0uYwo+IGluZGV4IGE1YmQw NzMzOGI0Ni4uZTRjNWExYTIyZDhiIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1 L2k5MTVfZ2VtLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbS5jCj4gQEAg LTQwNzEsOSArNDA3MSwxMCBAQCBpbnQgaTkxNV9nZW1fc2V0X2NhY2hpbmdfaW9jdGwoc3RydWN0 IGRybV9kZXZpY2UgKmRldiwgdm9pZCAqZGF0YSwKPiAgfQo+ICAKPiAgLyoKPiAtICogUHJlcGFy ZSBidWZmZXIgZm9yIGRpc3BsYXkgcGxhbmUgKHNjYW5vdXQsIGN1cnNvcnMsIGV0YykuCj4gLSAq IENhbiBiZSBjYWxsZWQgZnJvbSBhbiB1bmludGVycnVwdGlibGUgcGhhc2UgKG1vZGVzZXR0aW5n KSBhbmQgYWxsb3dzCj4gLSAqIGFueSBmbHVzaGVzIHRvIGJlIHBpcGVsaW5lZCAoZm9yIHBhZ2Vm bGlwcykuCj4gKyAqIFByZXBhcmUgYnVmZmVyIGZvciBkaXNwbGF5IHBsYW5lIChzY2Fub3V0LCBj dXJzb3JzLCBldGMpLiBDYW4gYmUgY2FsbGVkIGZyb20KPiArICogYW4gdW5pbnRlcnJ1cHRpYmxl IHBoYXNlIChtb2Rlc2V0dGluZykgYW5kIGFsbG93cyBhbnkgZmx1c2hlcyB0byBiZSBwaXBlbGlu ZWQKPiArICogKGZvciBwYWdlZmxpcHMpLiBXZSBvbmx5IGZsdXNoIHRoZSBjYWNoZXMgd2hpbGUg cHJlcGFyaW5nIHRoZSBidWZmZXIgZm9yCj4gKyAqIGRpc3BsYXksIHRoZSBjYWxsZXJzIGFyZSBy ZXNwb25zaWJsZSBmb3IgZnJvbnRidWZmZXIgZmx1c2guCj4gICAqLwo+ICBzdHJ1Y3QgaTkxNV92 bWEgKgo+ICBpOTE1X2dlbV9vYmplY3RfcGluX3RvX2Rpc3BsYXlfcGxhbmUoc3RydWN0IGRybV9p OTE1X2dlbV9vYmplY3QgKm9iaiwKPiBAQCAtNDEyOSw5ICs0MTMwLDcgQEAgaTkxNV9nZW1fb2Jq ZWN0X3Bpbl90b19kaXNwbGF5X3BsYW5lKHN0cnVjdCBkcm1faTkxNV9nZW1fb2JqZWN0ICpvYmos Cj4gIAo+ICAJdm1hLT5kaXNwbGF5X2FsaWdubWVudCA9IG1heF90KHU2NCwgdm1hLT5kaXNwbGF5 X2FsaWdubWVudCwgYWxpZ25tZW50KTsKPiAgCj4gLQkvKiBUcmVhdCB0aGlzIGFzIGFuIGVuZC1v Zi1mcmFtZSwgbGlrZSBpbnRlbF91c2VyX2ZyYW1lYnVmZmVyX2RpcnR5KCkgKi8KCkFsdGhvdWdo IGZsdXNoIGJ5IHNvbWUgZGVmaW5pdGlvbiBoZXJlIGlzIGludmFsaWRhdGUgKyBmbHVzaApJIHNl ZSBmbHVzaCBvcGVyYXRpb25zIGFzIHRoZSBvcGVyYXRpb24gdG8gYmUgcGVyZm9ybWVkIGF0IHRo ZSAiZW5kLW9mLWZyYW1lIi4KQWZ0ZXIgdGhlIG9wZXJhdGlvbiB3YXMgZG9uZSBhbmQgd2hhdGV2 ZXIgaGFkIHRvIGJlIG1vZGlmaWVkIHdhcyBtb2RpZmllZC4KCkkgc2VlIHlvdSBwYXRjaCBjaGFu Z2luZyB0aGUgZmx1c2ggdG8gdGhlIGNyZWF0aW9uIGFuZCBiZWdpbiBvZiB0aGUgZmIgaGFuZGxp bmcKbGlrZSBvbiBjcmVhdGluZyBhbmQgaW5pdCBmYiBmdW5jdGlvbnMuIEkgYmVsaWV2ZSBieSB0 aGF0IHRpbWUgd2UgYXJlIG5vdCByZWFkeQp0byBleGl0IFBTUiB5ZXQuCgpXaGF0IGFtIEkgbWlz c2luZz8KCj4gIAlfX2k5MTVfZ2VtX29iamVjdF9mbHVzaF9mb3JfZGlzcGxheShvYmopOwo+IC0J aW50ZWxfZmJfb2JqX2ZsdXNoKG9iaiwgT1JJR0lOX0RJUlRZRkIpOwo+ICAKPiAgCS8qIEl0IHNo b3VsZCBub3cgYmUgb3V0IG9mIGFueSBvdGhlciB3cml0ZSBkb21haW5zLCBhbmQgd2UgY2FuIHVw ZGF0ZQo+ICAJICogdGhlIGRvbWFpbiB2YWx1ZXMgZm9yIG91ciBjaGFuZ2VzLgo+IGRpZmYgLS1n aXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMgYi9kcml2ZXJzL2dwdS9k cm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPiBpbmRleCAzMzEwODQwODI1NDUuLjkxY2U4YTA1MjJh MyAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPiAr KysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPiBAQCAtMjg1OCw2ICsy ODU4LDkgQEAgaW50ZWxfZmluZF9pbml0aWFsX3BsYW5lX29iaihzdHJ1Y3QgaW50ZWxfY3J0YyAq aW50ZWxfY3J0YywKPiAgCQlyZXR1cm47Cj4gIAl9Cj4gIAo+ICsJb2JqID0gaW50ZWxfZmJfb2Jq KGZiKTsKPiArCWludGVsX2ZiX29ial9mbHVzaChvYmosIE9SSUdJTl9ESVJUWUZCKTsKPiArCj4g IAlwbGFuZV9zdGF0ZS0+c3JjX3ggPSAwOwo+ICAJcGxhbmVfc3RhdGUtPnNyY195ID0gMDsKPiAg CXBsYW5lX3N0YXRlLT5zcmNfdyA9IGZiLT53aWR0aCA8PCAxNjsKPiBAQCAtMjg3MSw3ICsyODc0 LDYgQEAgaW50ZWxfZmluZF9pbml0aWFsX3BsYW5lX29iaihzdHJ1Y3QgaW50ZWxfY3J0YyAqaW50 ZWxfY3J0YywKPiAgCWludGVsX3N0YXRlLT5iYXNlLnNyYyA9IGRybV9wbGFuZV9zdGF0ZV9zcmMo cGxhbmVfc3RhdGUpOwo+ICAJaW50ZWxfc3RhdGUtPmJhc2UuZHN0ID0gZHJtX3BsYW5lX3N0YXRl X2Rlc3QocGxhbmVfc3RhdGUpOwo+ICAKPiAtCW9iaiA9IGludGVsX2ZiX29iaihmYik7Cj4gIAlp ZiAoaTkxNV9nZW1fb2JqZWN0X2lzX3RpbGVkKG9iaikpCj4gIAkJZGV2X3ByaXYtPnByZXNlcnZl X2Jpb3Nfc3dpenpsZSA9IHRydWU7Cj4gIAo+IEBAIC0xMjc4NSw2ICsxMjc4Nyw4IEBAIGludGVs X3ByZXBhcmVfcGxhbmVfZmIoc3RydWN0IGRybV9wbGFuZSAqcGxhbmUsCj4gIAlpZiAocmV0KQo+ ICAJCXJldHVybiByZXQ7Cj4gIAo+ICsJaW50ZWxfZmJfb2JqX2ZsdXNoKG9iaiwgT1JJR0lOX0RJ UlRZRkIpOwo+ICsKPiAgCWlmICghbmV3X3N0YXRlLT5mZW5jZSkgeyAvKiBpbXBsaWNpdCBmZW5j aW5nICovCj4gIAkJc3RydWN0IGRtYV9mZW5jZSAqZmVuY2U7Cj4gIAo+IEBAIC0xMzE3Miw4ICsx MzE3Niw5IEBAIGludGVsX2xlZ2FjeV9jdXJzb3JfdXBkYXRlKHN0cnVjdCBkcm1fcGxhbmUgKnBs YW5lLAo+ICAJaWYgKHJldCkKPiAgCQlnb3RvIG91dF91bmxvY2s7Cj4gIAo+IC0Jb2xkX2ZiID0g b2xkX3BsYW5lX3N0YXRlLT5mYjsKPiArCWludGVsX2ZiX29ial9mbHVzaChpbnRlbF9mYl9vYmoo ZmIpLCBPUklHSU5fRElSVFlGQik7Cj4gIAo+ICsJb2xkX2ZiID0gb2xkX3BsYW5lX3N0YXRlLT5m YjsKPiAgCWk5MTVfZ2VtX3RyYWNrX2ZiKGludGVsX2ZiX29iaihvbGRfZmIpLCBpbnRlbF9mYl9v YmooZmIpLAo+ICAJCQkgIGludGVsX3BsYW5lLT5mcm9udGJ1ZmZlcl9iaXQpOwo+ICAKPiBkaWZm IC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZmJkZXYuYyBiL2RyaXZlcnMvZ3B1 L2RybS9pOTE1L2ludGVsX2ZiZGV2LmMKPiBpbmRleCA2ZjEyYWRjMDYzNjUuLjY1YTMzMTM3MjNj OSAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9mYmRldi5jCj4gKysr IGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZmJkZXYuYwo+IEBAIC0yMjEsNiArMjIxLDkg QEAgc3RhdGljIGludCBpbnRlbGZiX2NyZWF0ZShzdHJ1Y3QgZHJtX2ZiX2hlbHBlciAqaGVscGVy LAo+ICAJCWdvdG8gb3V0X3VubG9jazsKPiAgCX0KPiAgCj4gKwlmYiA9ICZpZmJkZXYtPmZiLT5i YXNlOwo+ICsJaW50ZWxfZmJfb2JqX2ZsdXNoKGludGVsX2ZiX29iaihmYiksIE9SSUdJTl9ESVJU WUZCKTsKPiArCj4gIAlpbmZvID0gZHJtX2ZiX2hlbHBlcl9hbGxvY19mYmkoaGVscGVyKTsKPiAg CWlmIChJU19FUlIoaW5mbykpIHsKPiAgCQlEUk1fRVJST1IoIkZhaWxlZCB0byBhbGxvY2F0ZSBm Yl9pbmZvXG4iKTsKPiBAQCAtMjMwLDggKzIzMyw2IEBAIHN0YXRpYyBpbnQgaW50ZWxmYl9jcmVh dGUoc3RydWN0IGRybV9mYl9oZWxwZXIgKmhlbHBlciwKPiAgCj4gIAlpbmZvLT5wYXIgPSBoZWxw ZXI7Cj4gIAo+IC0JZmIgPSAmaWZiZGV2LT5mYi0+YmFzZTsKPiAtCj4gIAlpZmJkZXYtPmhlbHBl ci5mYiA9IGZiOwo+ICAKPiAgCXN0cmNweShpbmZvLT5maXguaWQsICJpbnRlbGRybWZiIik7Cj4g ZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX292ZXJsYXkuYyBiL2RyaXZl cnMvZ3B1L2RybS9pOTE1L2ludGVsX292ZXJsYXkuYwo+IGluZGV4IDM2NjcxYTkzN2ZhNC4uYzJm MTBkODk5MzI5IDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX292ZXJs YXkuYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX292ZXJsYXkuYwo+IEBAIC04 MDcsNiArODA3LDcgQEAgc3RhdGljIGludCBpbnRlbF9vdmVybGF5X2RvX3B1dF9pbWFnZShzdHJ1 Y3QgaW50ZWxfb3ZlcmxheSAqb3ZlcmxheSwKPiAgCQlyZXQgPSBQVFJfRVJSKHZtYSk7Cj4gIAkJ Z290byBvdXRfcGluX3NlY3Rpb247Cj4gIAl9Cj4gKwlpbnRlbF9mYl9vYmpfZmx1c2gobmV3X2Jv LCBPUklHSU5fRElSVFlGQik7Cj4gIAo+ICAJcmV0ID0gaTkxNV92bWFfcHV0X2ZlbmNlKHZtYSk7 Cj4gIAlpZiAocmV0KQo+IC0tIAo+IDIuMTQuMQo+IAo+IF9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fCj4gSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdAo+IEludGVs LWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKPiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9y Zy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeApfX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0 cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9s aXN0aW5mby9pbnRlbC1nZngK