From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH v11] drm: Unplug drm device when unregistering it (v8) Date: Thu, 1 Jun 2017 14:26:01 +0200 Message-ID: <81db9974-e666-1d04-e18c-776e17296a6f@redhat.com> References: <1492068764-24353-1-git-send-email-jeffy.chen@rock-chips.com> <20170414151503.lmpp3udfuycavfki@art_vandelay> <20170529202512.GF23936@nuc-i3427.alporthouse.com> <47c5eca2-3624-edbc-bb2d-453645b6bf3b@redhat.com> <592E2CD1.8080206@rock-chips.com> <7941eb36-725e-72b5-67c7-1e3a9ae8f422@redhat.com> <20170601122229.GP11742@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by gabe.freedesktop.org (Postfix) with ESMTPS id E29AE6E0BB for ; Thu, 1 Jun 2017 12:26:06 +0000 (UTC) In-Reply-To: <20170601122229.GP11742@nuc-i3427.alporthouse.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Chris Wilson , jeffy , Sean Paul , linux-kernel@vger.kernel.org, briannorris@chromium.org, dianders@chromium.org, tfiga@chromium.org, zyw@rock-chips.com, marcheu@chromium.org, mark.yao@rock-chips.com, hshi@chromium.org, Daniel Vetter , Jani Nikula , dri-devel@lists.freedesktop.org, David Airlie , Tom Gundersen , =?UTF-8?Q?Marco_Diego_Aur=c3=a9lio_Mesquita?= , Patrik Jakobsson , Dave Airlie List-Id: dri-devel@lists.freedesktop.org SGksCgpPbiAwMS0wNi0xNyAxNDoyMiwgQ2hyaXMgV2lsc29uIHdyb3RlOgo+IE9uIFRodSwgSnVu IDAxLCAyMDE3IGF0IDAyOjEzOjI4UE0gKzAyMDAsIEhhbnMgZGUgR29lZGUgd3JvdGU6Cj4+IEhp LAo+Pgo+PiBPbiAzMS0wNS0xNyAwNDozOSwgamVmZnkgd3JvdGU6Cj4+PiBIaSBIYW5zLAo+Pj4K Pj4+IHRoYW54IGZvciBpbnZlc3RpZ2F0aW5nIDopCj4+Pgo+Pj4gT24gMDUvMzAvMjAxNyAwMzow NiBQTSwgSGFucyBkZSBHb2VkZSB3cm90ZToKPj4+PiBIaSwKPj4+Pgo+Pj4+IE9uIDI5LTA1LTE3 IDIyOjI1LCBDaHJpcyBXaWxzb24gd3JvdGU6Cj4+Pj4+IE9uIEZyaSwgQXByIDE0LCAyMDE3IGF0 IDExOjE1OjA0QU0gLTA0MDAsIFNlYW4gUGF1bCB3cm90ZToKPj4+Pj4+IE9uIFRodSwgQXByIDEz LCAyMDE3IGF0IDAzOjMyOjQ0UE0gKzA4MDAsIEplZmZ5IENoZW4gd3JvdGU6Cj4+Pj4+Pj4gQWZ0 ZXIgdW5iaW5kaW5nIGRybSwgdGhlIHVzZXIgc3BhY2UgbWF5IHN0aWxsIG93bnMgdGhlIGRybSBk ZXYgZmQsIGFuZAo+Pj4+Pj4+IG1heSBzdGlsbCBiZSBhYmxlIHRvIGNhbGwgZHJtIGlvY3RsLgo+ Pj4+Pj4+Cj4+Pj4+Pj4gV2UncmUgdXNpbmcgYW4gdW5wbHVnZ2VkIHN0YXRlIHRvIHByZXZlbnQg c29tZXRoaW5nIGxpa2UgdGhhdCwgc28gbGV0J3MKPj4+Pj4+PiByZXVzZSBpdCBoZXJlLgo+Pj4+ Pj4+Cj4+Pj4+Pj4gQWxzbyBkcm9wIGRybV91bnBsdWdfZGV2LCBiZWNhdXNlIGl0IHdvdWxkIGJl IHVudXNlZCBhZnRlciBvdGhlcgo+Pj4+Pj4+IGNoYW5nZXMuCj4+Pj4+Pj4KPj4+Pj4+PiBWZXJp ZmllZCBvbiByazMzOTkgY2hyb21lYm9vayBrZXZpbih3aXRoIGNyb3MgNC40IGtlcm5lbCksIG5v IG1vcmUKPj4+Pj4+PiBjcmFzaGVzCj4+Pj4+Pj4gd2hlbiB1bmJpbmRpbmcgZHJtIHdpdGggdWkg c2VydmljZSBzdGlsbCBydW5uaW5nLgo+Pj4+Pj4+Cj4+Pj4+Pj4gdjI6IEZpeCBzb21lIGNvbW1p dCBtZXNzYWdlcy4KPj4+Pj4+PiB2MzogUmV1c2UgdW5wbHVnIHN0YXR1cy4KPj4+Pj4+PiB2NDog QWRkIGRybV9kZXZpY2Vfc2V0X3BsdWdfc3RhdGUgaGVscGVyLgo+Pj4+Pj4+IHY1OiBGaXggaGFu ZyB3aGVuIHVucmVnaXN0ZXJpbmcgZHJtIGRldiB3aXRoIG9wZW5fY291bnQgMC4KPj4+Pj4+PiB2 NjogTW92ZSBkcm1fZGV2aWNlX3NldF9wbHVnX3N0YXRlIGludG8gZHJtX2Rydi4KPj4+Pj4+PiB2 NzogQWRkIG1pc3NpbmcgZHJtX2Rldl91bnJlZiBpbiB1ZGxfZHJ2Lgo+Pj4+Pj4+IHY4OiBGaXgg Y29tcGlsZXIgZXJyb3JzIGFmdGVyIGVuYWJsZSB1ZGwuCj4+Pj4+Pj4KPj4+Pj4+PiBTaWduZWQt b2ZmLWJ5OiBKZWZmeSBDaGVuIDxqZWZmeS5jaGVuQHJvY2stY2hpcHMuY29tPgo+Pj4+Pj4+Cj4+ Pj4+Pj4gLS0tCj4+Pj4+Pgo+Pj4+Pj4gSGkgSmVmZnksCj4+Pj4+PiBHaXZlbiB0aGUgdHJvdWJs ZSB3ZSd2ZSBoYWQgd2l0aCB0aGlzIHBhdGNoIGFscmVhZHksIGNvdXBsZWQgd2l0aCB0aGUKPj4+ Pj4+IGZhY3QgdGhhdAo+Pj4+Pj4gdW5iaW5kaW5nIHdoaWxlIHVzZXJzcGFjZSBpcyBhY3RpdmUg aXMgYSBjb250cml2ZWQvcGF0aG9sb2dpY2FsIGNhc2UsCj4+Pj4+PiBJIGRvbid0Cj4+Pj4+PiB0 aGluayBpdCdzIHdvcnRoIHBpY2tpbmcgdGhpcyBwYXRjaCB1cHN0cmVhbS4KPj4+Pj4+Cj4+Pj4+ PiBJZiBpdCdzIHJlYWxseSBjYXVzaW5nIGlzc3VlcyBkb3duc3RyZWFtLCB5b3UgY2FuIGFkZCBt eSBSZXZpZXdlZC1ieQo+Pj4+Pj4gZm9yIGEgQ0hST01JVU0KPj4+Pj4+IHBhdGNoLCBidXQgSSdk IHJhdGhlciBub3QgY2FycnkgcGF0Y2hlcyBpbiB0aGUgQ3JPUyByZXBvIGlmIHdlIGRvbid0Cj4+ Pj4+PiBuZWVkIHRvLgo+Pj4+Pgo+Pj4+PiBXb3VsZCBhCj4+Pj4+Cj4+Pj4+IEZpeGVzOiBhMzli ZTYwNmY5OWQgKCJkcm06IERvIGEgZnVsbCBkZXZpY2UgdW5yZWdpc3RlciB3aGVuIHVucGx1Z2dp bmciKQo+Pj4+PiBSZXBvcnRlZC1ieTogTWFyY28gRGllZ28gQXVyw6lsaW8gTWVzcXVpdGEgPG1h cmNvZGllZ29tZXNxdWl0YUBnbWFpbC5jb20+Cj4+Pj4+IENjOiBIYW5zIGRlIEdvZWRlIDxoZGVn b2VkZUByZWRoYXQuY29tPgo+Pj4+Pgo+Pj4+PiBjb252aW5jZSB1cyB0byBsb29rIGludG8gdGhp cyBwYXRjaCBhZ2Fpbj8KPj4+Pgo+Pj4+IFRoZSBwcm9ibGVtIGlzIHRoaXMgcGF0Y2ggaXMgd3Jv bmcsIHNlZSBiZWxvdy4KPj4+Pgo+Pj4+Pj4+ICAgZHJpdmVycy9ncHUvZHJtL2RybV9kcnYuYyAg ICAgfCAyNiArKysrKysrKysrLS0tLS0tLS0tLS0tLS0tLQo+Pj4+Pj4+ICAgZHJpdmVycy9ncHUv ZHJtL3VkbC91ZGxfZHJ2LmMgfCAgMyArKy0KPj4+Pj4+PiAgIGluY2x1ZGUvZHJtL2RybVAuaCAg ICAgICAgICAgIHwgIDYgLS0tLS0tCj4+Pj4+Pj4gICBpbmNsdWRlL2RybS9kcm1fZHJ2LmggICAg ICAgICB8ICAxIC0KPj4+Pj4+PiAgIDQgZmlsZXMgY2hhbmdlZCwgMTIgaW5zZXJ0aW9ucygrKSwg MjQgZGVsZXRpb25zKC0pCj4+Pj4+Pj4KPj4+Pj4+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUv ZHJtL2RybV9kcnYuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZHJ2LmMKPj4+Pj4+PiBpbmRleCBi NWM2YmI0Li5lMWRhNGQxIDEwMDY0NAo+Pj4+Pj4+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9kcm1f ZHJ2LmMKPj4+Pj4+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2Rydi5jCj4+Pj4+Pj4gQEAg LTM1NSwyMiArMzU1LDYgQEAgdm9pZCBkcm1fcHV0X2RldihzdHJ1Y3QgZHJtX2RldmljZSAqZGV2 KQo+Pj4+Pj4+ICAgfQo+Pj4+Pj4+ICAgRVhQT1JUX1NZTUJPTChkcm1fcHV0X2Rldik7Cj4+Pj4+ Pj4gLXZvaWQgZHJtX3VucGx1Z19kZXYoc3RydWN0IGRybV9kZXZpY2UgKmRldikKPj4+Pj4+PiAt ewo+Pj4+Pj4+IC0gICAgLyogZm9yIGEgVVNCIGRldmljZSAqLwo+Pj4+Pj4+IC0gICAgZHJtX2Rl dl91bnJlZ2lzdGVyKGRldik7Cj4+Pj4+Pj4gLQo+Pj4+Pj4+IC0gICAgbXV0ZXhfbG9jaygmZHJt X2dsb2JhbF9tdXRleCk7Cj4+Pj4+Pj4gLQo+Pj4+Pj4+IC0gICAgZHJtX2RldmljZV9zZXRfdW5w bHVnZ2VkKGRldik7Cj4+Pj4+Pj4gLQo+Pj4+Pj4+IC0gICAgaWYgKGRldi0+b3Blbl9jb3VudCA9 PSAwKSB7Cj4+Pj4+Pj4gLSAgICAgICAgZHJtX3B1dF9kZXYoZGV2KTsKPj4+Pj4+PiAtICAgIH0K Pj4+Pj4+PiAtICAgIG11dGV4X3VubG9jaygmZHJtX2dsb2JhbF9tdXRleCk7Cj4+Pj4+Pj4gLX0K Pj4+Pj4+PiAtRVhQT1JUX1NZTUJPTChkcm1fdW5wbHVnX2Rldik7Cj4+Pj4+Pj4gLQo+Pj4+Pj4+ ICAgLyoKPj4+Pj4+PiAgICAqIERSTSBpbnRlcm5hbCBtb3VudAo+Pj4+Pj4+ICAgICogV2Ugd2Fu dCB0byBiZSBhYmxlIHRvIGFsbG9jYXRlIG91ciBvd24gInN0cnVjdCBhZGRyZXNzX3NwYWNlIgo+ Pj4+Pj4+IHRvIGNvbnRyb2wKPj4+Pj4+PiBAQCAtNzMzLDYgKzcxNywxMyBAQCBzdGF0aWMgdm9p ZCByZW1vdmVfY29tcGF0X2NvbnRyb2xfbGluayhzdHJ1Y3QKPj4+Pj4+PiBkcm1fZGV2aWNlICpk ZXYpCj4+Pj4+Pj4gICAgICAga2ZyZWUobmFtZSk7Cj4+Pj4+Pj4gICB9Cj4+Pj4+Pj4gK3N0YXRp YyBpbmxpbmUgdm9pZCBkcm1fZGV2aWNlX3NldF9wbHVnX3N0YXRlKHN0cnVjdCBkcm1fZGV2aWNl ICpkZXYsCj4+Pj4+Pj4gKyAgICAgICAgICAgICAgICAgICAgICAgICBib29sIHBsdWdnZWQpCj4+ Pj4+Pj4gK3sKPj4+Pj4+PiArICAgIHNtcF93bWIoKTsKPj4+Pj4+PiArICAgIGF0b21pY19zZXQo JmRldi0+dW5wbHVnZ2VkLCAhcGx1Z2dlZCk7Cj4+Pj4+Pj4gK30KPj4+Pj4+PiArCj4+Pj4+Pj4g ICAvKioKPj4+Pj4+PiAgICAqIGRybV9kZXZfcmVnaXN0ZXIgLSBSZWdpc3RlciBEUk0gZGV2aWNl Cj4+Pj4+Pj4gICAgKiBAZGV2OiBEZXZpY2UgdG8gcmVnaXN0ZXIKPj4+Pj4+PiBAQCAtNzg3LDYg Kzc3OCw4IEBAIGludCBkcm1fZGV2X3JlZ2lzdGVyKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsCj4+ Pj4+Pj4gdW5zaWduZWQgbG9uZyBmbGFncykKPj4+Pj4+PiAgICAgICBpZiAoZHJtX2NvcmVfY2hl Y2tfZmVhdHVyZShkZXYsIERSSVZFUl9NT0RFU0VUKSkKPj4+Pj4+PiAgICAgICAgICAgZHJtX21v ZGVzZXRfcmVnaXN0ZXJfYWxsKGRldik7Cj4+Pj4+Pj4gKyAgICBkcm1fZGV2aWNlX3NldF9wbHVn X3N0YXRlKGRldiwgdHJ1ZSk7Cj4+Pj4+Pj4gKwo+Pj4+Pj4+ICAgICAgIHJldCA9IDA7Cj4+Pj4+ Pj4gICAgICAgRFJNX0lORk8oIkluaXRpYWxpemVkICVzICVkLiVkLiVkICVzIGZvciAlcyBvbiBt aW5vciAlZFxuIiwKPj4+Pj4+PiBAQCAtODI2LDYgKzgxOSw3IEBAIHZvaWQgZHJtX2Rldl91bnJl Z2lzdGVyKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYpCj4+Pj4+Pj4gICAgICAgZHJtX2xhc3RjbG9z ZShkZXYpOwo+Pj4+Pj4+ICAgICAgIGRldi0+cmVnaXN0ZXJlZCA9IGZhbHNlOwo+Pj4+Pj4+ICsg ICAgZHJtX2RldmljZV9zZXRfcGx1Z19zdGF0ZShkZXYsIGZhbHNlKTsKPj4+Pj4+PiAgICAgICBp ZiAoZHJtX2NvcmVfY2hlY2tfZmVhdHVyZShkZXYsIERSSVZFUl9NT0RFU0VUKSkKPj4+Pj4+PiAg ICAgICAgICAgZHJtX21vZGVzZXRfdW5yZWdpc3Rlcl9hbGwoZGV2KTsKPj4+Pj4+PiBkaWZmIC0t Z2l0IGEvZHJpdmVycy9ncHUvZHJtL3VkbC91ZGxfZHJ2LmMKPj4+Pj4+PiBiL2RyaXZlcnMvZ3B1 L2RybS91ZGwvdWRsX2Rydi5jCj4+Pj4+Pj4gaW5kZXggY2Q4YjAxNy4uZmM3M2UyNCAxMDA2NDQK Pj4+Pj4+PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vdWRsL3VkbF9kcnYuYwo+Pj4+Pj4+ICsrKyBi L2RyaXZlcnMvZ3B1L2RybS91ZGwvdWRsX2Rydi5jCj4+Pj4+Pj4gQEAgLTEwOCw3ICsxMDgsOCBA QCBzdGF0aWMgdm9pZCB1ZGxfdXNiX2Rpc2Nvbm5lY3Qoc3RydWN0Cj4+Pj4+Pj4gdXNiX2ludGVy ZmFjZSAqaW50ZXJmYWNlKQo+Pj4+Pj4+ICAgICAgIGRybV9rbXNfaGVscGVyX3BvbGxfZGlzYWJs ZShkZXYpOwo+Pj4+Pj4+ICAgICAgIHVkbF9mYmRldl91bnBsdWcoZGV2KTsKPj4+Pj4+PiAgICAg ICB1ZGxfZHJvcF91c2IoZGV2KTsKPj4+Pj4+PiAtICAgIGRybV91bnBsdWdfZGV2KGRldik7Cj4+ Pj4+Pj4gKyAgICBkcm1fZGV2X3VucmVnaXN0ZXIoZGV2KTsKPj4+Pj4+PiArICAgIGRybV9kZXZf dW5yZWYoZGV2KTsKPj4+Pgo+Pj4+IFRoZSB1bnJlZiBoZXJlIHdpbGwgY2F1c2UgdGhlIGRldmlj ZSBzdHJ1Y3QgdG8gZ2V0IGZyZWUtZWQgZXZlbiBpZgo+Pj4+IHVzZXJzcGFjZSBzdGlsbCBob2xk cyByZWZlcmVuY2VzIHRvIGl0IHRocm91Z2ggdGhlIGRybV9kZXYuIFRvIGZpeAo+Pj4+IHRoaXMg d2Ugd291bGQgbmVlZCB0byBjYWxsIGRybV9kZXZfcmVmIG9uIGEgb3BlbiBmcm9tIHVzZXJzcGFj ZSBhbmQKPj4+PiBkcm1fZGV2X3VucmVmIGZyb20gZHJtX3JlbGVhc2UuCj4+Pgo+Pj4gcmlnaHQs IGJ1dCBpIHRoaW5rIHdlIGFyZSBhbHJlYWR5IGRpZCB0aGUgcmVmL3VucmVmIGluIHRoZSBvcGVu L3JlbGVhc2UgdGhyb3VnaCBkcm1fbWlub3JfYWNxdWlyZS9kcm1fbWlub3JfcmVsZWFzZS4KPj4K Pj4gQWggeWVzLCBJIHNlZS4gU3RpbGwgY2FsbGluZyBkcm1fZGV2X3VucmVnaXN0ZXIoKSBkaXJl Y3RseSBmcm9tCj4+IHVkbF91c2JfZGlzY29ubmVjdCgpIGlzIG5vdCBnb2luZyB0byB3b3JrLCBz ZWUgdGhlIHBhdGNoIHRpdGxlZAo+PiAiZHJtOiBGaXggb29wcyArIFhzZXJ2ZXIgaGFuZyB3aGVu IHVucGx1Z2dpbmcgVVNCIGRybSBkZXZpY2VzIgo+Pgo+PiBUaGUgcHJvYmxlbSBpcyB0aGF0IGRy bV9kZXZfdW5yZWdpc3RlcigpIHByb2JhYmx5IHNob3VsZCBiZQo+PiBzcGxpdCBpbnRvIGEgZHJt X2Rldl91bnJlZ2lzdGVyKCkgYW5kIGRybV9kZXZfY2xlYW51cCgpCj4+IGZ1bmN0aW9uIHdpdGgg dGhlIGNsZWFudXAgcGFydCBnZXR0aW5nIGNhbGxlZCBieSB0aGUgbGFzdCB1bnJlZiwKPj4gYW5k IGF0IGxlYXN0IGNhbGxpbmcgZHJtX2xhc3RjbG9zZSBhbmQgdGhlIGRyaXZlci0+dW5sb2FkIGNh bGwKPj4gbmVlZHMgdG8gYmUgbW92ZWQgdG8gdGhlIG5ldyBkcm1fZGV2X2NsZWFudXAuCj4gCj4g SXQgYWxyZWFkeSBpcy4gZHJpdmVyLT51bmxvYWQoKSBoYXMgYmVlbiBkZXByZWNhdGVkIGZvciBh IHdoaWxlLCBpbgo+IHNwaXJpdCBhdCBsZWFzdCwgd2UgcHJvYmFibHkgc2hvdWxkIGFkZCBhIHdh cm5pbmcgdG8gaXQsIGFuZCBpcwo+IG5vdCBtZWFudCB0byBiZSB1c2VkIGFueW1vcmUuCgpPaywg d2hhdCBhYm91dCBkcm1fbGFzdGNsb3NlID8gQ2FsbGluZyB0aGF0IHdoaWxlIHVzZXJzcGFjZSBz dGlsbApoYXMgZmRzIG9wZW4gc2VlbXMgd3JvbmcgdG9vLiBTYW1lIGZvciB0aGlzIHBhcnQgb2Yg ZHJtX2Rldl91bnJlZ2lzdGVyKCk6CgogICAgICAgICBsaXN0X2Zvcl9lYWNoX2VudHJ5X3NhZmUo cl9saXN0LCBsaXN0X3RlbXAsICZkZXYtPm1hcGxpc3QsIGhlYWQpCiAgICAgICAgICAgICAgICAg ZHJtX2xlZ2FjeV9ybW1hcChkZXYsIHJfbGlzdC0+bWFwKTsKClJlbW92aW5nIG1tYXBwaW5nIGJh Y2tpbmcgZGF0YSAoaWYgSSB1bmRlcnN0YW5kIGNvcnJlY3RseSkgd2hpbGUKaXQgbWF5IHN0aWxs IGJlIHVzZWQgc2VlbXMgbGlrZSBhIGJhZCBpZGVhIHRvbywgc28gSSBkbyBiZWxpZXZlIHRoYXQK ZHJtX2Rldl91bnJlZ2lzdGVyIG5lZWRzIHNvbWUgbG92ZSAvIHNvbWUgYml0cyBtb3ZlZCB0byB0 aGUgbGFzdApkcm1fZGV2X3B1dCBjYWxsIGJlZm9yZSBpdCB3aWxsIGJlIHNhZmUgdG8gY2FsbCBp dCBkaXJlY3RseQpvbiB1c2ItZGlzY29ubmVjdC4KClJlZ2FyZHMsCgpIYW5zCl9fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxp c3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNr dG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=