From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm/atomic: Try to preserve the crtc enabled state in drm_atomic_remove_fb, v2. Date: Mon, 6 Nov 2017 18:20:50 +0200 Message-ID: <20171106162050.GL10981@intel.com> References: <20171101150433.10777-1-maarten.lankhorst@linux.intel.com> <20171101152903.GF10981@intel.com> <20171101170019.GG10981@intel.com> <20171106140120.GH10981@intel.com> <20171106140607.GI10981@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: 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: Daniel Vetter , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gTW9uLCBOb3YgMDYsIDIwMTcgYXQgMDQ6NDM6MTdQTSArMDEwMCwgTWFhcnRlbiBMYW5raG9y c3Qgd3JvdGU6Cj4gT3AgMDYtMTEtMTcgb20gMTU6MDYgc2NocmVlZiBWaWxsZSBTeXJqw6Rsw6Q6 Cj4gPiBPbiBNb24sIE5vdiAwNiwgMjAxNyBhdCAwNDowMToyMFBNICswMjAwLCBWaWxsZSBTeXJq w6Rsw6Qgd3JvdGU6Cj4gPj4gT24gVGh1LCBOb3YgMDIsIDIwMTcgYXQgMDk6NTU6NDBBTSArMDEw MCwgTWFhcnRlbiBMYW5raG9yc3Qgd3JvdGU6Cj4gPj4+IE9wIDAxLTExLTE3IG9tIDE4OjAwIHNj aHJlZWYgVmlsbGUgU3lyasOkbMOkOgo+ID4+Pj4gT24gV2VkLCBOb3YgMDEsIDIwMTcgYXQgMDQ6 NTU6MDZQTSArMDEwMCwgTWFhcnRlbiBMYW5raG9yc3Qgd3JvdGU6Cj4gPj4+Pj4gT3AgMDEtMTEt MTcgb20gMTY6Mjkgc2NocmVlZiBWaWxsZSBTeXJqw6Rsw6Q6Cj4gPj4+Pj4+IE9uIFdlZCwgTm92 IDAxLCAyMDE3IGF0IDA0OjA0OjMzUE0gKzAxMDAsIE1hYXJ0ZW4gTGFua2hvcnN0IHdyb3RlOgo+ ID4+Pj4+Pj4gVGhpcyBpbnRyb2R1Y2VzIGEgc2xpZ2h0IGJlaGF2aW9yYWwgY2hhbmdlIHRvIHJt ZmIuIEluc3RlYWQgb2YKPiA+Pj4+Pj4+IGRpc2FibGluZyBhIGNydGMgd2hlbiB0aGUgcHJpbWFy eSBwbGFuZSBpcyBkaXNhYmxlZCwgd2UgdHJ5IHRvCj4gPj4+Pj4+PiBwcmVzZXJ2ZSBpdC4KPiA+ Pj4+Pj4+Cj4gPj4+Pj4+PiBBcGFydCBmcm9tIG9sZCB2ZXJzaW9ucyBvZiB0aGUgdm13Z2Z4IHhv cmcgZHJpdmVyLCB0aGVyZSBpcwo+ID4+Pj4+Pj4gbm90aGluZyBkZXBlbmRpbmcgb24gcm1mYiBk aXNhYmxpbmcgYSBjcnRjLgo+ID4+Pj4+Pj4KPiA+Pj4+Pj4+IFZtd2dmeCcgYW5kIHNpbXBsZSBr bXMgaGVscGVyIGF0b21pYyBpbXBsZW1lbnRhdGlvbiByZWplY3RzIENSVEMKPiA+Pj4+Pj4+IGVu YWJsZWQgd2l0aG91dCBwbGFuZSwgc28gd2UgY2FuIGRvIHRoaXMgc2FmZWx5Lgo+ID4+Pj4gVGhl IGNvZGUgZm9yIHRob3NlIHNlZW1zIGEgYml0IGluY29uc2lzdGVudC4gVGhlIGNydGMgY2hlY2sg cmVxdWlyZXMKPiA+Pj4+IHRoYXQgdGhlIGNydGMgc3RhdGUgYW5kIHBsYW5lIHN0YXRlIG1hdGNo LiBCdXQgdGhlIHBsYW5lIGNoZWNrIGFsbG93cwo+ID4+Pj4gdGhlIHBsYW5lIHRvIGJlIGVuYWJs ZWQgdy9vIHRoZSBjcnRjIGJlaW5nIGVuYWJsZWQuIEkgZ3Vlc3MgaXQgZG9lc24ndAo+ID4+Pj4g bWF0dGVyIHJlYWxseSBzaW5jZSB5b3UgY2FuJ3QgZW5hYmxlIHRoZSBwbGFuZSB3aXRob3V0IGEg Y3J0YywgYW5kIHRoZQo+ID4+Pj4gY3J0YyBjaGVjayB3b3VsZCB0aGVuIGNhdGNoIHRoZSBjYXNl IHdoZXJlIHRoZSBjcnRjIHdvdWxkIGJlIGRpc2FibGVkLgo+ID4+PiBFeGFjdGx5LiA6LSkgT25s eSB3aGVuIGEgcGxhbmUgaXMgdW5ib3VuZCBhbmQgc3RheXMgdW5ib3VuZCwgdGhlIGNydGMgY2hl Y2sgd29uJ3QKPiA+Pj4gYmUgaW52b2tlZC4gSGVuY2UgaXQncyB0aGUgbW9zdCBhY2N1cmF0ZSB3 YXkgb2YgbWFraW5nIHN1cmUgdGhhdAo+ID4+PiBjcnRjIGVuYWJsZWQgPD0+IHByaW1hcnkgcGxh bmUgYm91bmQuCj4gPj4+Cj4gPj4+IElmIHRoZSBjaGVjayB3YXMgZG9uZSBpbiB0aGUgcHJpbWFy eSBwbGFuZSwgYW4gYXRvbWljIG1vZGVzZXQgY291bGQgZW5hYmxlCj4gPj4+IHRoZSBjcnRjIHdp dGhvdXQgZW5hYmxpbmcgdGhlIHByaW1hcnkgcGxhbmUsIHdoaWNoIHNob3VsZG4ndCBiZSBhbGxv d2VkIGJ1dAo+ID4+PiBhIHBsYW5lIGNoZWNrIHdvbid0IGNhdGNoIGl0Lgo+ID4+PiBUaGlzIGhh cyBiZWVuIGEgYnVnIGluIHNpbXBsZS1rbXMtaGVscGVyLCBmaXhlZCBpbiB0aGUgYmVsb3cgY29t bWl0Ogo+ID4+Pgo+ID4+PiBjb21taXQgNzY1ODMxZGMyN2FiMTQxYjNhMGJlMWFiNTViOTIyYjAx MjQyNzkwMgo+ID4+PiBBdXRob3I6IE1hYXJ0ZW4gTGFua2hvcnN0IDxtYWFydGVuLmxhbmtob3Jz dEBsaW51eC5pbnRlbC5jb20+Cj4gPj4+IERhdGU6ICAgV2VkIEp1bCAxMiAxMDoxMzoyOSAyMDE3 ICswMjAwCj4gPj4+Cj4gPj4+ICAgICBkcm0vc2ltcGxlLWttcy1oZWxwZXI6IEZpeCB0aGUgY2hl Y2sgZm9yIHRoZSBtaXNtYXRjaCBiZXR3ZWVuIHBsYW5lIGFuZCBDUlRDIGVuYWJsZWQuCj4gPj4g SG1tLiBPSyB0aGF0IHBhcnQgbG9va3MgT0suIFdoYXQgZG9lcyBzZWVtIGEgYml0IGluY29uc2lz dGVudCBpcyB0aGUKPiA+PiBmYWN0IHRoYXQgd2UgcGFzcyBjYW5fdXBkYXRlX2Rpc2FibGVkPXRy dWUsIGJ1dCBsYXRlciBvbiB3ZSByZWplY3QgdGhlCj4gPj4gdXBkYXRlIHdoZW4gdmlzaWJsZT09 ZmFsc2UuIFRoZSBvbGQgY29kZSB3b3VsZCBoYXZlIGFjY2VwdGVkIHRoYXQKPiA+PiBiZWNhdXNl IGl0IGRpZG4ndCBldmVuIGNhbGwgZHJtX3BsYW5lX2hlbHBlcl9jaGVja19zdGF0ZSgpIHdoZW4g dGhlCj4gPj4gY3J0YyAoYW5kIHRodXMgYWxzbyB0aGUgcGxhbmUpIHdhcyBkaXNhYmxlZC4KPiA+ IEFjdHVhbGx5IGhvdyBkb2VzIHRoaXMgd29yayBhdCBhbGw/IElmIHlvdSB0dXJuIG9mZiBib3Ro IHRoZSBjcnRjIGFuZAo+ID4gcGxhbmUsIHRoZW4gdGhlIHBsYW5lIGNoZWNrIHdpbGwgYWx3YXlz IHJldHVybiAtRUlOVkFMIG9uIGFjY291bnQgb2YKPiA+IHZpc2libGU9PWZhbHNlPwo+ID4KPiBJ ZiB0aGUgY3J0YyBpcyB0dXJuZWQgb2ZmLCBlbmFibGVkID0gZmFsc2UgYW5kIHRoZSBwcmltYXJ5 IHBsYW5lIGlzIG5vdCBpbiBjcnRjX3N0YXRlLT5wbGFuZV9tYXNrLgo+Cj4gVmlzaWJpbGl0eSBp cyBpZ25vcmVkIGluIHRoaXMgY2hlY2ssIHRoYXQgcGFydCBpcyBoYW5kbGVkIGluIHBsYW5lIGNo ZWNrIHRoYXQgdGhlIGZyYW1lYnVmZmVyIGhhcyB0byBzcGFuIHRoZSBlbnRpcmUgY3J0YyBpZiBl bmFibGVkLgoKSG1tLiBJIHRob3VnaHQgdGhlICFjcnRjLT5lbmFibGVkIGNoZWNrIGRpc2FwcGVh cmVkIGZyb20KZHJtX3NpbXBsZV9rbXNfcGxhbmVfYXRvbWljX2NoZWNrKCkgYnV0IGxvb2tzIGxp a2UgSSB3YXMgd3JvbmcgYW5kIGl0J3MKc3RpbGwgdGhlcmUuIE9LLCBqdXN0IHRvdGFsbHkgaWdu b3JlIHdoYXQgSSB3cm90ZSBiZWZvcmUuIEkgZ3Vlc3Mgb25lCnNob3VsZG4ndCB0cnkgdG8gZmln dXJlIG91dCB3aGF0IHRoZSBjb2RlIGlzIGdvaW5nIHRvIGJlIGRvaW5nIHdoaWxlIGluCnRoZSBt aWRkbGUgb2YgYW4gdW5yZWxhdGVkIGJpc2VjdC4KClRob3VnaCBmb3Igc29tZSBleHRyYSBjb25z aXN0ZW5jeSB3ZSBtaWdodCB3YW50IHRvIGNoYW5nZSB0aGF0IHRvIGNoZWNrCnRvIGxvb2sgZm9y ICFmYiBhZnRlciBjYWxsaW5nIGRybV9wbGFuZV9oZWxwZXJfY2hlY2tfc3RhdGUoKS4gVGhhdCB3 YXkKd2Ugd291bGRuJ3QgaGF2ZSB0byBjaGFuZ2UgZHJpdmVycy9zaW1wbGVfa21zX2hlbHBlciBp ZiBjb21lIHVwIHdpdGgKc29tZXRoaW5nIHRoYXQgaGFzIHRvIGJlIGNoZWNrZWQgZXZlbiBmb3Ig ZGlzYWJsZWQgcGxhbmVzCmluIGRybV9wbGFuZV9oZWxwZXJfY2hlY2tfc3RhdGUoKS4KCi0tIApW aWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZy ZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3Rp bmZvL2ludGVsLWdmeAo=