From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Engestrom Subject: Re: [PATCH] drm: make drm_get_format_name atomic/irq safe again Date: Sat, 5 Nov 2016 01:23:45 +0000 Message-ID: <20161105012344.GA28349@engestrom.ch> References: <1478273565-6113-1-git-send-email-robdclark@gmail.com> <20161104162738.GQ4617@intel.com> <20161104173224.GJ25290@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wm0-f65.google.com (mail-wm0-f65.google.com [74.125.82.65]) by gabe.freedesktop.org (Postfix) with ESMTPS id B210A6E111 for ; Sat, 5 Nov 2016 01:23:51 +0000 (UTC) Received: by mail-wm0-f65.google.com with SMTP id u144so6326556wmu.0 for ; Fri, 04 Nov 2016 18:23:51 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Rob Clark Cc: "dri-devel@lists.freedesktop.org" List-Id: dri-devel@lists.freedesktop.org T24gRnJpZGF5LCAyMDE2LTExLTA0IDEzOjUwOjI1IC0wNDAwLCBSb2IgQ2xhcmsgd3JvdGU6Cj4g T24gRnJpLCBOb3YgNCwgMjAxNiBhdCAxOjMyIFBNLCBFcmljIEVuZ2VzdHJvbQo+IDxlcmljLmVu Z2VzdHJvbUBpbWd0ZWMuY29tPiB3cm90ZToKPiA+IE9uIEZyaWRheSwgMjAxNi0xMS0wNCAxMzox Mjo1MSAtMDQwMCwgUm9iIENsYXJrIHdyb3RlOgo+ID4+IE9uIEZyaSwgTm92IDQsIDIwMTYgYXQg MTI6MjcgUE0sIFZpbGxlIFN5cmrDpGzDpAo+ID4+IDx2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVs LmNvbT4gd3JvdGU6Cj4gPj4gPiBPbiBGcmksIE5vdiAwNCwgMjAxNiBhdCAxMTozMjo0NUFNIC0w NDAwLCBSb2IgQ2xhcmsgd3JvdGU6Cj4gPj4gPj4gRml4ZXM6IDkwODQ0ZjAwMDQ5ZTlmNDI1NzNm ZDMxZDdjMzJlOGZkMzFkM2ZkMDcKPiA+PiA+Pgo+ID4+ID4+ICAgICBkcm06IG1ha2UgZHJtX2dl dF9mb3JtYXRfbmFtZSB0aHJlYWQtc2FmZQo+ID4+ID4+Cj4gPj4gPj4gICAgIFNpZ25lZC1vZmYt Ynk6IEVyaWMgRW5nZXN0cm9tIDxlcmljQGVuZ2VzdHJvbS5jaD4KPiA+PiA+PiAgICAgW2RhbnZl dDogQ2xhcmlmeSB0aGF0IHRoZSByZXR1cm5lZCBwb2ludGVyIG11c3QgYmUgZnJlZWQgd2l0aAo+ ID4+ID4+ICAgICBrZnJlZSgpLl0KPiA+PiA+PiAgICAgU2lnbmVkLW9mZi1ieTogRGFuaWVsIFZl dHRlciA8ZGFuaWVsLnZldHRlckBmZndsbC5jaD4KPiA+PiA+Pgo+ID4+ID4+IE5vdGU6IEkgdGhp bmsgd2UgbmVlZCB0byBiZSBhIGJpdCBjYXJlZnVsIGFib3V0IGZvbGxvdy11cCBhdWRpdHMgb2YK PiA+PiA+PiBjYWxsZXJzIG9mIHRoaXMuLiAgbm93IHRoYXQgeW91IG5lZWQgdG8ga2ZyZWUgdGhl IHJldHVybiB2YWx1ZSBJIHRoaW5rCj4gPj4gPj4gaXQgaXMgZmFpcmx5IGVhc3kgZm9yIG5ldyBw YXRjaGVzIHRvIGludHJvZHVjZSBuZXcgY2FsbGVycyB3aGljaCBsZWFrCj4gPj4gPj4gdGhlIHJl dHVybiB2YWx1ZS4gIFdlIHByb2JhYmx5IHNob3VsZCBoYXZlIGxlZnQgdGhlIGZ1bmN0aW9uIGFz LWlzIGFuZAo+ID4+ID4+IGludHJvZHVjZSBhIG5ldyB2YXJpYW50LCBvciBzb21ldGhpbmcgbGlr ZSB0aGF0Lgo+ID4+ID4+Cj4gPj4gPj4gU2lnbmVkLW9mZi1ieTogUm9iIENsYXJrIDxyb2JkY2xh cmtAZ21haWwuY29tPgo+ID4+ID4+IC0tLQo+ID4+ID4+ICBkcml2ZXJzL2dwdS9kcm0vZHJtX2Zv dXJjYy5jIHwgNSArKysrLQo+ID4+ID4+ICAxIGZpbGUgY2hhbmdlZCwgNCBpbnNlcnRpb25zKCsp LCAxIGRlbGV0aW9uKC0pCj4gPj4gPj4KPiA+PiA+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUv ZHJtL2RybV9mb3VyY2MuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZm91cmNjLmMKPiA+PiA+PiBp bmRleCBjYmI4Yjc3Li4yYmU5ZWE4IDEwMDY0NAo+ID4+ID4+IC0tLSBhL2RyaXZlcnMvZ3B1L2Ry bS9kcm1fZm91cmNjLmMKPiA+PiA+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2ZvdXJjYy5j Cj4gPj4gPj4gQEAgLTg3LDcgKzg3LDEwIEBAIEVYUE9SVF9TWU1CT0woZHJtX21vZGVfbGVnYWN5 X2ZiX2Zvcm1hdCk7Cj4gPj4gPj4gICAqLwo+ID4+ID4+ICBjaGFyICpkcm1fZ2V0X2Zvcm1hdF9u YW1lKHVpbnQzMl90IGZvcm1hdCkKPiA+PiA+PiAgewo+ID4+ID4+IC0gICAgIGNoYXIgKmJ1ZiA9 IGttYWxsb2MoMzIsIEdGUF9LRVJORUwpOwo+ID4+ID4+ICsgICAgIGNoYXIgKmJ1ZiA9IGttYWxs b2MoMzIsIEdGUF9BVE9NSUMpOwo+ID4+ID4+ICsKPiA+PiA+PiArICAgICBpZiAoIWJ1ZikKPiA+ PiA+PiArICAgICAgICAgICAgIHJldHVybiBOVUxMOwo+ID4KPiA+IFVucmVsYXRlZCBidWcsIHRo YW5rcyBmb3IgdGhhdCBmaXggdG9vIDopCj4gPgo+ID4+ID4+Cj4gPj4gPj4gICAgICAgc25wcmlu dGYoYnVmLCAzMiwKPiA+PiA+PiAgICAgICAgICAgICAgICAiJWMlYyVjJWMgJXMtZW5kaWFuICgw eCUwOHgpIiwKPiA+PiA+Cj4gPj4gPiBXaHkgYXJlbid0IHdlIHVzaW5nIGthc3ByaW50ZigpPwo+ ID4KPiA+IEJlY2F1c2UgSSBkaWRuJ3Qga25vdyBpdCB3YXMgYSB0aGluZyA6KAo+ID4gUGF0Y2gg dXNpbmcgaXQgaW5jb21pbmcuCj4gCj4gVGhhbmtzCj4gCj4gPj4gPgo+ID4+ID4gT3Igd2UgY291 bGQgaGF2ZSBqdXN0IG1hZGUgdGhlIGNhbGxlciBwcm92aWRlIHRoZSBidWZmZXIuLi4KPiA+Pgo+ ID4+IEkgZ3Vlc3Mga2FzcHJpbnRmKCkgd291bGQgZG8gdGhlIGpvYiAoYWx0aG91Z2ggc3RpbGwg bm90IGFkZHJlc3MgdGhlCj4gPj4gZmFjdCB0aGF0IHdlIHN0aWxsIGFsd2F5cyBkbyB0aGUgc3By aW50ZiBiaXRzLCByYXRoZXIgdGhhbiBvbmx5IGRvaW5nCj4gPj4gaXQgd2hlbiBkZWJ1ZyBsb2dz IGFyZSBlbmFibGVkKS4uICBjYWxsZXIgcHJvdmlkZWQgYnVmZmVyIHdvdWxkIGJlCj4gPj4gYmV0 dGVyLiAgQW5kIG1ha2UgaXQgbW9yZSBvYnZpb3VzIHRoYXQgc29tZXRoaW5nIG5lZWRzIHRvIGJl IGZpeGVkCj4gPj4gd2hlbiBtZXJnaW5nL3JlYmFzaW5nIHBhdGNoZXMgdGhhdCBzdGFydGVkIGxp ZmUgYmVmb3JlIHRoaXMgY2hhbmdlCj4gPj4gbGFuZGVkLgo+ID4+Cj4gPj4gSSBzdGlsbCBraW5k YSBsaWtlIHRoZSBpZGVhIG9mIG1ha2luZyB2c3ByaW50ZiBrbm93IGFib3V0IGZvdXJjYydzCj4g Pj4gd2l0aCBhIG5ldyBmb3JtYXQgc3RyaW5nLCBhbmQganVzdCBtYWtpbmcgZHJtX2dldF9mb3Jt YXRfbmFtZSgpIGdvCj4gPj4gYXdheS4KPiA+Pgo+ID4+IEJ1dCBJIGRvbid0IHJlYWxseSBoYXZl IHRpbWUgYXRtIHRvIHJlLXdvcmsgYWxsIHRoZSBjYWxsZXJzIG9mCj4gPj4gZHJtX2dldF9mb3Jt YXRfbmFtZSgpLiAgU28gSSBndWVzcyB1bmxlc3Mgc29tZW9uZSBlbHNlIGRvZXMsIEknZCBnbyB3 Lwo+ID4+IGthc3ByaW50ZigpIGZvciBub3cuCj4gPgo+ID4gVGhhdCBzb3VuZHMgY2xlYW5lciB0 byBtZSBpbmRlZWQsIEknbGwgc2VuZCBhIHBhdGNoIGRvaW5nIHRoYXQgdG9uaWdodC4KPiA+IEFu eSBpZGVhIGZvciB0aGUgbmFtZT8gZHJtX2dldF9mb3JtYXRfbmFtZV9zYWZlKHVpbnQzMl90LCBj aGFyKik/Cj4gPiBJIGNvdWxkIGFsc28ga2VlcCB0aGUgc2FtZSBuYW1lIGFuZCByZWx5IG9uIHRo ZSBmdW5jdGlvbiBzaWduYXR1cmUKPiA+IGNoYW5nZSB0byBtYWtlIHN1cmUgYW55IG1lcmdpbmcg b2YgdGhlIG9sZCBmdW5jdGlvbiBjYWxsIHdpbGwgYnJlYWsgdGhlCj4gPiBjb21waWxhdGlvbiBh bmQgYmUgbm90aWNlZCB0aGF0IHdheS4KPiAKPiBJIHRoaW5rIGlmIGZ4biBzaWduYXR1cmUgY2hh bmdlcywgd2UgY2FuIGtlZXAgdGhlIHNhbWUgbmFtZS4gIFdoYXQgSQo+IGdldCBuZXJ2b3VzIGFi b3V0IGlzIGZ4biBzaWcrbmFtZSBzdGF5cyBzYW1lIGJ1dCBzZW1hbnRpY3MgY2hhbmdlIDstKQo+ IAo+IGJ0dywgaWYgdGhpcyBpc24ndCB3aGF0IHlvdSB3ZXJlIGFscmVhZHkgcGxhbm5pbmcsIEkn ZCBzdWdnZXN0Ogo+IAo+ICAgIGNoYXIgKmRybV9nZXRfZm9ybWF0X25hbWUodWludDMyX3QsIGNo YXIgKik7Cj4gCj4gc28gd2UgY2FuIGtlZXAgaXQgaW5saW5lIHcvIERSTV9ERUJVR194eXooKSBt YWNyb3MuCgpUaGlzIHdhcyBleGFjdGx5IG15IGlkZWEuCgo+IE9yIG1heWJlIHNvbWV0aGluZyBs aWtlOgo+IAo+ICAgIHR5cGVkZWYgY2hhciBkcm1fZm9ybWF0X25hbWVfYnVmW01BWF9TSVpFXQo+ ICAgIGNoYXIgKmRybV9nZXRfZm9ybWF0X25hbWUodWludDMyX3QsIGRybV9mb3JtYXRfbmFtZV9i dWYpOwo+IAo+ID8/CgpJIGRpZG4ndCB0aGluayBvZiB0aGF0LCBidXQgdGhhdCB3b3VsZCBhdm9p ZCBtaXN0YWtlcyBhYm91dCB0aGUgc2l6ZSBvZgp0aGUgYnVmZmVyLiBNb2RpZnlpbmcgdGhlIGNv ZGUgbm93LCBJJ2xsIHNlbmQgdGhlIHBhdGNoIGluIGEgZmV3IG1pbnV0ZXMuCgpDaGVlcnMsCiAg RXJpYwpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmkt ZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6 Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK