From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/2] drm/atomic: Make atomic helper track newly assigned planes correctly. Date: Mon, 16 Oct 2017 18:37:09 +0300 Message-ID: <20171016153709.GN10981@intel.com> References: <20171016132928.6498-1-maarten.lankhorst@linux.intel.com> <20171016134210.GK10981@intel.com> <20171016144844.GM10981@intel.com> <8525dcf7-b978-7e5f-2734-df7e7a452d10@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <8525dcf7-b978-7e5f-2734-df7e7a452d10@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Maarten Lankhorst Cc: Gustavo Padovan , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Daniel Vetter List-Id: intel-gfx@lists.freedesktop.org T24gTW9uLCBPY3QgMTYsIDIwMTcgYXQgMDU6Mjg6MjdQTSArMDIwMCwgTWFhcnRlbiBMYW5raG9y c3Qgd3JvdGU6Cj4gT3AgMTYtMTAtMTcgb20gMTY6NDggc2NocmVlZiBWaWxsZSBTeXJqw6Rsw6Q6 Cj4gPiBPbiBNb24sIE9jdCAxNiwgMjAxNyBhdCAwMzo1OTozOFBNICswMjAwLCBNYWFydGVuIExh bmtob3JzdCB3cm90ZToKPiA+PiBPcCAxNi0xMC0xNyBvbSAxNTo0MiBzY2hyZWVmIFZpbGxlIFN5 cmrDpGzDpDoKPiA+Pj4gT24gTW9uLCBPY3QgMTYsIDIwMTcgYXQgMDM6Mjk6MjdQTSArMDIwMCwg TWFhcnRlbiBMYW5raG9yc3Qgd3JvdGU6Cj4gPj4+PiBDb21taXQgNjY5YzkyMTVhZmVhICgiZHJt L2F0b21pYzogTWFrZSBhc3luYyBwbGFuZSB1cGRhdGUgY2hlY2tzIHdvcmsgYXMKPiA+Pj4+IGlu dGVuZGVkLCB2Mi4iKSBmb3JjZWQgcGxhbmVzIHRvIGFsd2F5cyBiZSB0cmFja2VkLCBidXQgZm9y Z290IHRvCj4gPj4+PiBleHBsaWNpdGx5IGdldCB0aGUgY3J0YyBjb21taXQgZnJvbSB0aGUgbmV3 IGNydGMgd2hlbiBhdmFpbGFibGUuCj4gPj4+Pgo+ID4+Pj4gVGhpcyBicm9rZSBwbGFuZSBjb21t aXQgdHJhY2tpbmcsIGFuZCBjYXVzZWQga21zX2F0b21pY190cmFuc2l0aW9ucwo+ID4+Pj4gdG8g cmFuZG9tbHkgZmFpbCB3aXRoIC1FQlVTWS4KPiA+Pj4+Cj4gPj4+PiBTaWduZWQtb2ZmLWJ5OiBN YWFydGVuIExhbmtob3JzdCA8bWFhcnRlbi5sYW5raG9yc3RAbGludXguaW50ZWwuY29tPgo+ID4+ Pj4gRml4ZXM6IDY2OWM5MjE1YWZlYSAoImRybS9hdG9taWM6IE1ha2UgYXN5bmMgcGxhbmUgdXBk YXRlIGNoZWNrcyB3b3JrIGFzIGludGVuZGVkLCB2Mi4iKQo+ID4+Pj4gQ2M6IEd1c3Rhdm8gUGFk b3ZhbiA8Z3VzdGF2by5wYWRvdmFuQGNvbGxhYm9yYS5jb20+Cj4gPj4+PiBDYzogRGFuaWVsIFZl dHRlciA8ZGFuaWVsLnZldHRlckBmZndsbC5jaD4KPiA+Pj4+IEJ1Z3ppbGxhOiBodHRwczovL2J1 Z3MuZnJlZWRlc2t0b3Aub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDI2NzEKPiA+Pj4+IFRlc3RjYXNl OiBrbXNfYXRvbWljX3RyYW5zaXRpb25zCj4gPj4+PiAtLS0KPiA+Pj4+ICBkcml2ZXJzL2dwdS9k cm0vZHJtX2F0b21pY19oZWxwZXIuYyB8IDIgKy0KPiA+Pj4+ICAxIGZpbGUgY2hhbmdlZCwgMSBp bnNlcnRpb24oKyksIDEgZGVsZXRpb24oLSkKPiA+Pj4+Cj4gPj4+PiBkaWZmIC0tZ2l0IGEvZHJp dmVycy9ncHUvZHJtL2RybV9hdG9taWNfaGVscGVyLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2F0 b21pY19oZWxwZXIuYwo+ID4+Pj4gaW5kZXggZDU5NDQxZjFkY2Q0Li5iNjRjOGY1YmM5NDAgMTAw NjQ0Cj4gPj4+PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2F0b21pY19oZWxwZXIuYwo+ID4+ Pj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2RybV9hdG9taWNfaGVscGVyLmMKPiA+Pj4+IEBAIC0x ODA0LDcgKzE4MDQsNyBAQCBpbnQgZHJtX2F0b21pY19oZWxwZXJfc2V0dXBfY29tbWl0KHN0cnVj dCBkcm1fYXRvbWljX3N0YXRlICpzdGF0ZSwKPiA+Pj4+ICAJCSAgICAhdHJ5X3dhaXRfZm9yX2Nv bXBsZXRpb24oJm9sZF9wbGFuZV9zdGF0ZS0+Y29tbWl0LT5mbGlwX2RvbmUpKQo+ID4+Pj4gIAkJ CXJldHVybiAtRUJVU1k7Cj4gPj4+PiAgCj4gPj4+PiAtCQljb21taXQgPSBjcnRjX29yX2Zha2Vf Y29tbWl0KHN0YXRlLCBvbGRfcGxhbmVfc3RhdGUtPmNydGMpOwo+ID4+Pj4gKwkJY29tbWl0ID0g Y3J0Y19vcl9mYWtlX2NvbW1pdChzdGF0ZSwgb2xkX3BsYW5lX3N0YXRlLT5jcnRjID86IG5ld19w bGFuZV9zdGF0ZS0+Y3J0Yyk7Cj4gPj4+IFNob3VsZG4ndCBvbGQgdnMuIG5ldyBzdGF0ZSBiZSB0 aGUgb3RoZXIgd2F5IGFyb3VuZD8KPiA+PiBIbW0gdG8gYmUgaG9uZXN0LCBjb3VsZCBiZS4gV2Ug ZG9uJ3QgYWxsb3cgY3J0YydzIHRvIHN3aXRjaCBwbGFuZXMgZGlyZWN0bHkuIFNvIGluIHByYWN0 aWNlIGl0IGRvZXNuJ3QgbWF0dGVyLgo+ID4gTm90IHN1cmUgd2hlcmUgd2UgYWN0dWFsbHkgcHJl dmVudCB0aGF0LiBBIHF1aWNrIHRyYXdsIHRocm91Z2ggdGhlIGNvZGUKPiA+IGRpZG4ndCByZXZl YWwgYW55dGhpbmcgbGlrZSB0aGF0Lgo+IHBsYW5lX3N3aXRjaGluZ19jcnRjKCksIGNhbGxlZCBm cm9tIGRybV9hdG9taWNfY2hlY2tfb25seS0+ZHJtX2F0b21pY19wbGFuZV9jaGVjaygpLgoKQWgs IHRoYXQncyB3aGVyZSBpdCB3YXMgaGlkaW5nLgoKPiAKPiBTbyBJIHdvdWxkbid0IHdvcnJ5LiA6 KQo+ID4+IEJ1dCBpZiB3ZSBldmVyIGRpZCBhbGxvdyBtb3ZpbmcgY3J0YydzLCBpdCdzIHVwIGZv ciBkZWJhdGUgd2hhdCBjcnRjIHdlIHdhbnQgdG8gdXNlIGhlcmUuLgo+ID4gbmV3IGlzIHRoZSBv bmUgaXQnZCBiZSBoYW5naW5nIG9mZiBhdCB0aGUgZW5kIHNvIHRoYXQgc2VlbXMgbGlrZSB0aGUK PiA+IHJpZ2h0IGNob2ljZS4gSXQgd291bGQgYWxzbyBtYXRjaCB3aGF0IHdlIGRvIGluIGk5MTUg Y29kZS4KPiBPayBuZXcgb25lIGl0IGlzLCB3aXRoIHRoYXQgY2hhbmdlZCBkbyBJIGhhdmUgeW91 ciByLWI/Cj4gCgpZZXMuCgotLSAKVmlsbGUgU3lyasOkbMOkCkludGVsIE9UQwpfX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBs aXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVz a3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK