From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julia Lawall Subject: Re: [Outreachy kernel] [PATCH] drm/gem: use new idr deletion interface to cleanup drm_gem_handle_delete() Date: Tue, 26 Sep 2017 12:20:18 +0200 (CEST) Message-ID: References: <20170925184728.GA8861@aishwarya> <20170926082047.eqsd2lslpqp4pi5f@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail3-relais-sop.national.inria.fr (mail3-relais-sop.national.inria.fr [192.134.164.104]) by gabe.freedesktop.org (Postfix) with ESMTPS id B4F666E507 for ; Tue, 26 Sep 2017 10:20:23 +0000 (UTC) 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: Daniel Vetter Cc: dri-devel , outreachy-kernel , Daniel Vetter , Aishwarya Pant List-Id: dri-devel@lists.freedesktop.org CgpPbiBUdWUsIDI2IFNlcCAyMDE3LCBEYW5pZWwgVmV0dGVyIHdyb3RlOgoKPiBPbiBUdWUsIFNl cCAyNiwgMjAxNyBhdCAxMDo0NyBBTSwgRGFuaWVsIFZldHRlciA8ZGFuaWVsQGZmd2xsLmNoPiB3 cm90ZToKPiA+IE9uIFR1ZSwgU2VwIDI2LCAyMDE3IGF0IDEwOjM4IEFNLCBKdWxpYSBMYXdhbGwg PGp1bGlhLmxhd2FsbEBsaXA2LmZyPiB3cm90ZToKPiA+Pgo+ID4+Cj4gPj4gT24gVHVlLCAyNiBT ZXAgMjAxNywgRGFuaWVsIFZldHRlciB3cm90ZToKPiA+Pgo+ID4+PiBPbiBUdWUsIFNlcCAyNiwg MjAxNyBhdCAxMjoxNzoyOEFNICswNTMwLCBBaXNod2FyeWEgUGFudCB3cm90ZToKPiA+Pj4gPiBU aGUgSURSIGRlbGV0aW9uIGludGVyZmFjZSBub3cgcmV0dXJucyB0aGUgZGVsZXRlZCBlbnRyeSBv ciBOVUxMIGlmIGl0IHdhcyBub3QKPiA+Pj4gPiBwcmVzZW50LiBTbyB3ZSBkb24ndCBoYXZlIHRv IGRvIHRoZSBleHRyYSB3b3JrIG9mIGNoZWNraW5nIGlmIHdlIGhhdmUgYQo+ID4+PiA+IHJlZmVy ZW5jZSBvbiB0aGUgZHJtX2dlbV9vYmplY3QsIHRoaXMgY2FuIGJlIGhhbmRsZWQgYnkgY2hlY2tp bmcgdGhlIHJldHVybgo+ID4+PiA+IHZhbHVlIGZyb20gaWRyX3JlbW92ZSgpIGFuZCB0aGUgZXh0 cmEgbG9ja3MgY2FuIGJlIGRyb3BwZWQuCj4gPj4+ID4KPiA+Pj4gPiBTaWduZWQtb2ZmLWJ5OiBB aXNod2FyeWEgUGFudCA8YWlzaHBhbnRAZ21haWwuY29tPgo+ID4+Pgo+ID4+PiBIYW5lZW4gaXMg YWxyZWFkeSB3b3JraW5nIG9uIHRoaXMgdGFzaywgd2l0aCB0aGUgcGF0Y2gganVzdCBtZXJnZWQu IFBsZWFzZQo+ID4+PiBjb29yZGluYXRlIHdpdGggbWVudG9ycyAobWUgb3IgU2VhbiBQYXVsKSBi ZWZvcmUgc3RhcnRpbmcgb24gc29tZXRoaW5nIHRvCj4gPj4+IGF2b2lkIHN1Y2ggY2xhc2hlcyBp biB0aGUgZnV0dXJlLgo+ID4+Pgo+ID4+PiBOb3RlIGFsc28gdGhhdCBzb21lIHRvZG8gaXRlbXMg YXJlIGp1c3QgaWRlYXMsIGFuZCBuZWVkIGNvbmZpcm1hdGlvbiB3aXRoCj4gPj4+IHJlbGV2YW50 IG1haW50YWluZXJzIGZpcnN0Lgo+ID4+Cj4gPj4gTm90IHN1cmUgd2hlcmUgdGhlIG1peHVwIGhh cHBlbmVkLCBidXQgYW55b25lIHdobyBzdGFydHMgb24gYSBwcm9qZWN0Cj4gPj4gc3BlY2lmaWMg dGFzayBzaG91bGQgbm90ZSB0aGF0IG9uIHRoZSBvdXRyZWFjaHkga2VybmVsIHdpa2kgYW5kIGJl Zm9yZQo+ID4+IHN0YXJ0aW5nIG9uIGEgcHJvamVjdCBzcGVjaWZpYyB0YXNrIG9uZSBzaG91bGQg Y2hlY2sgdGhhdCBubyBvbmUgaXMKPiA+PiB3b3JraW5nIG9uIGl0LiAgSWYgc29tZW9uZSBzdGFy dGVkIHNvbWUgdGltZSBhZ28sIGFuZCB0aGUgY29kZSBkb2Vzbid0Cj4gPj4gc2VlbSB0byBoYXZl IGNoYW5nZWQsIGFzayB0aGUgcGVyc29uIHdobyBjbGFpbWVkIHRoZSB0YXNrIG9yIHRoZSBtZW50 b3IuCj4gPgo+ID4gSG0sIHRoZSBkcmktZGV2ZWwgdGFza3MgYXJlbid0IG9uIHRoZSB3aWtpLCBi dXQgaW4gdGhlIGtlcm5lbCBzb3VyY2VzLgo+ID4gV2hlcmUgc2hvdWxkIHdlIHB1dCB0aGUgc2Ny YXRjaC1wYWQgdG8gYXZvaWQgc3VjaCBjb25mbGljdHMgaW4gdGhlCj4gPiBmdXR1cmU/IEUuZy4g dGhlIElJTyBzdWJzeXRlbXMgaGFzIGl0J3MgdGFzayBvbiB0aGUgd2lraSwgd2hlcmUgdGhpcwo+ ID4gd29ya3MgbXVjaCBiZXR0ZXIuCj4KPiBPaywgSSBmb3VuZCBpdC4gTG9va3MgbGlrZSBhIGZl dyBzaWduZWQgdXAgZm9yIHN0dWZmLCBidXQgd2l0aG91dAo+IGNoYXR0aW5nIHdpdGggdXMgZmly c3QsIGFuZCBlbmRlZCB1cCBwaWNraW5nIHRhc2tzIHRoYXQgbmVlZCBzb21lCj4gc2VyaW91cyBH UFUgZXhwZXJ0aXNlLiBJLmUuIG1heWJlIHN1aXRhYmxlIGZvciBhIGZ1bGwgaW50ZXJuc2hpcCwK PiBkZWZpbml0ZWx5IG5vdCBhcyBzdGFydGVyIHRhc2tzLiBJJ2xsIHNlbmQgb3V0IG1haWxzLgoK VGhhbmtzLiAgRXZlcnlvbmUsIHBsZWFzZSBkaXNjdXNzIHdpdGggdGhlIG1lbnRvciBiZWZvcmUg cGlja2luZyBhIHRhc2suCgpqdWxpYQoKPiAtRGFuaWVsCj4KPgo+ID4gLURhbmllbAo+ID4KPiA+ Pgo+ID4+IHRoYW5rcywKPiA+Pgo+ID4+IGp1bGlhCj4gPj4KPiA+Pj4KPiA+Pj4gVGhhbmtzLCBE YW5pZWwKPiA+Pj4gPiAtLS0KPiA+Pj4gPiAgZHJpdmVycy9ncHUvZHJtL2RybV9nZW0uYyB8IDIx ICsrLS0tLS0tLS0tLS0tLS0tLS0tLQo+ID4+PiA+ICAxIGZpbGUgY2hhbmdlZCwgMiBpbnNlcnRp b25zKCspLCAxOSBkZWxldGlvbnMoLSkKPiA+Pj4gPgo+ID4+PiA+IGRpZmYgLS1naXQgYS9kcml2 ZXJzL2dwdS9kcm0vZHJtX2dlbS5jIGIvZHJpdmVycy9ncHUvZHJtL2RybV9nZW0uYwo+ID4+PiA+ IGluZGV4IGM1NWYzMzguLmY2Mjc1N2EgMTAwNjQ0Cj4gPj4+ID4gLS0tIGEvZHJpdmVycy9ncHUv ZHJtL2RybV9nZW0uYwo+ID4+PiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZ2VtLmMKPiA+ Pj4gPiBAQCAtMjgyLDI5ICsyODIsMTIgQEAgZHJtX2dlbV9oYW5kbGVfZGVsZXRlKHN0cnVjdCBk cm1fZmlsZSAqZmlscCwgdTMyIGhhbmRsZSkKPiA+Pj4gPiAgewo+ID4+PiA+ICAgICBzdHJ1Y3Qg ZHJtX2dlbV9vYmplY3QgKm9iajsKPiA+Pj4gPgo+ID4+PiA+IC0gICAvKiBUaGlzIGlzIGdyb3Nz LiBUaGUgaWRyIHN5c3RlbSBkb2Vzbid0IGxldCB1cyB0cnkgYSBkZWxldGUgYW5kCj4gPj4+ID4g LSAgICAqIHJldHVybiBhbiBlcnJvciBjb2RlLiAgSXQganVzdCBzcGV3cyBpZiB5b3UgZmFpbCBh dCBkZWxldGluZy4KPiA+Pj4gPiAtICAgICogU28sIHdlIGhhdmUgdG8gZ3JhYiBhIGxvY2sgYXJv dW5kIGZpbmRpbmcgdGhlIG9iamVjdCBhbmQgdGhlbgo+ID4+PiA+IC0gICAgKiBkb2luZyB0aGUg ZGVsZXRlIG9uIGl0IGFuZCBkcm9wcGluZyB0aGUgcmVmY291bnQsIG9yIHRoZSB1c2VyCj4gPj4+ ID4gLSAgICAqIGNvdWxkIHJhY2UgdXMgdG8gZG91YmxlLWRlY3JlbWVudCB0aGUgcmVmY291bnQg YW5kIGNhdXNlIGEKPiA+Pj4gPiAtICAgICogdXNlLWFmdGVyLWZyZWUgbGF0ZXIuICBHaXZlbiB0 aGUgZnJlcXVlbmN5IG9mIG91ciBoYW5kbGUgbG9va3VwcywKPiA+Pj4gPiAtICAgICogd2UgbWF5 IHdhbnQgdG8gdXNlIGlkYSBmb3IgbnVtYmVyIGFsbG9jYXRpb24gYW5kIGEgaGFzaCB0YWJsZQo+ ID4+PiA+IC0gICAgKiBmb3IgdGhlIHBvaW50ZXJzLCBhbnl3YXkuCj4gPj4+ID4gLSAgICAqLwo+ ID4+PiA+ICAgICBzcGluX2xvY2soJmZpbHAtPnRhYmxlX2xvY2spOwo+ID4+PiA+IC0KPiA+Pj4g PiAtICAgLyogQ2hlY2sgaWYgd2UgY3VycmVudGx5IGhhdmUgYSByZWZlcmVuY2Ugb24gdGhlIG9i amVjdCAqLwo+ID4+PiA+IC0gICBvYmogPSBpZHJfcmVwbGFjZSgmZmlscC0+b2JqZWN0X2lkciwg TlVMTCwgaGFuZGxlKTsKPiA+Pj4gPiAtICAgc3Bpbl91bmxvY2soJmZpbHAtPnRhYmxlX2xvY2sp Owo+ID4+PiA+IC0gICBpZiAoSVNfRVJSX09SX05VTEwob2JqKSkKPiA+Pj4gPiArICAgb2JqID0g aWRyX3JlbW92ZSgmZmlscC0+b2JqZWN0X2lkciwgaGFuZGxlKTsKPiA+Pj4gPiArICAgaWYgKCFv YmopCj4gPj4+ID4gICAgICAgICAgICAgcmV0dXJuIC1FSU5WQUw7Cj4gPj4+ID4gLQo+ID4+PiA+ ICAgICAvKiBSZWxlYXNlIGRyaXZlcidzIHJlZmVyZW5jZSBhbmQgZGVjcmVtZW50IHJlZmNvdW50 LiAqLwo+ID4+PiA+ICAgICBkcm1fZ2VtX29iamVjdF9yZWxlYXNlX2hhbmRsZShoYW5kbGUsIG9i aiwgZmlscCk7Cj4gPj4+ID4gLQo+ID4+PiA+IC0gICAvKiBBbmQgZmluYWxseSBtYWtlIHRoZSBo YW5kbGUgYXZhaWxhYmxlIGZvciBmdXR1cmUgYWxsb2NhdGlvbnMuICovCj4gPj4+ID4gLSAgIHNw aW5fbG9jaygmZmlscC0+dGFibGVfbG9jayk7Cj4gPj4+ID4gLSAgIGlkcl9yZW1vdmUoJmZpbHAt Pm9iamVjdF9pZHIsIGhhbmRsZSk7Cj4gPj4+ID4gICAgIHNwaW5fdW5sb2NrKCZmaWxwLT50YWJs ZV9sb2NrKTsKPiA+Pj4gPgo+ID4+PiA+ICAgICByZXR1cm4gMDsKPiA+Pj4gPiAtLQo+ID4+PiA+ IDIuNy40Cj4gPj4+ID4KPiA+Pj4gPiAtLQo+ID4+PiA+IFlvdSByZWNlaXZlZCB0aGlzIG1lc3Nh Z2UgYmVjYXVzZSB5b3UgYXJlIHN1YnNjcmliZWQgdG8gdGhlIEdvb2dsZSBHcm91cHMgIm91dHJl YWNoeS1rZXJuZWwiIGdyb3VwLgo+ID4+PiA+IFRvIHVuc3Vic2NyaWJlIGZyb20gdGhpcyBncm91 cCBhbmQgc3RvcCByZWNlaXZpbmcgZW1haWxzIGZyb20gaXQsIHNlbmQgYW4gZW1haWwgdG8gb3V0 cmVhY2h5LWtlcm5lbCt1bnN1YnNjcmliZUBnb29nbGVncm91cHMuY29tLgo+ID4+PiA+IFRvIHBv c3QgdG8gdGhpcyBncm91cCwgc2VuZCBlbWFpbCB0byBvdXRyZWFjaHkta2VybmVsQGdvb2dsZWdy b3Vwcy5jb20uCj4gPj4+ID4gVG8gdmlldyB0aGlzIGRpc2N1c3Npb24gb24gdGhlIHdlYiB2aXNp dCBodHRwczovL2dyb3Vwcy5nb29nbGUuY29tL2QvbXNnaWQvb3V0cmVhY2h5LWtlcm5lbC8yMDE3 MDkyNTE4NDcyOC5HQTg4NjElNDBhaXNod2FyeWEuCj4gPj4+ID4gRm9yIG1vcmUgb3B0aW9ucywg dmlzaXQgaHR0cHM6Ly9ncm91cHMuZ29vZ2xlLmNvbS9kL29wdG91dC4KPiA+Pj4KPiA+Pj4gLS0K PiA+Pj4gRGFuaWVsIFZldHRlcgo+ID4+PiBTb2Z0d2FyZSBFbmdpbmVlciwgSW50ZWwgQ29ycG9y YXRpb24KPiA+Pj4gaHR0cDovL2Jsb2cuZmZ3bGwuY2gKPiA+Pj4KPiA+Pj4gLS0KPiA+Pj4gWW91 IHJlY2VpdmVkIHRoaXMgbWVzc2FnZSBiZWNhdXNlIHlvdSBhcmUgc3Vic2NyaWJlZCB0byB0aGUg R29vZ2xlIEdyb3VwcyAib3V0cmVhY2h5LWtlcm5lbCIgZ3JvdXAuCj4gPj4+IFRvIHVuc3Vic2Ny aWJlIGZyb20gdGhpcyBncm91cCBhbmQgc3RvcCByZWNlaXZpbmcgZW1haWxzIGZyb20gaXQsIHNl bmQgYW4gZW1haWwgdG8gb3V0cmVhY2h5LWtlcm5lbCt1bnN1YnNjcmliZUBnb29nbGVncm91cHMu Y29tLgo+ID4+PiBUbyBwb3N0IHRvIHRoaXMgZ3JvdXAsIHNlbmQgZW1haWwgdG8gb3V0cmVhY2h5 LWtlcm5lbEBnb29nbGVncm91cHMuY29tLgo+ID4+PiBUbyB2aWV3IHRoaXMgZGlzY3Vzc2lvbiBv biB0aGUgd2ViIHZpc2l0IGh0dHBzOi8vZ3JvdXBzLmdvb2dsZS5jb20vZC9tc2dpZC9vdXRyZWFj aHkta2VybmVsLzIwMTcwOTI2MDgyMDQ3LmVxc2QybHNscHFwNHBpNWYlNDBwaGVub20uZmZ3bGwu bG9jYWwuCj4gPj4+IEZvciBtb3JlIG9wdGlvbnMsIHZpc2l0IGh0dHBzOi8vZ3JvdXBzLmdvb2ds ZS5jb20vZC9vcHRvdXQuCj4gPj4+Cj4gPgo+ID4KPiA+Cj4gPiAtLQo+ID4gRGFuaWVsIFZldHRl cgo+ID4gU29mdHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9uCj4gPiArNDEgKDApIDc5 IDM2NSA1NyA0OCAtIGh0dHA6Ly9ibG9nLmZmd2xsLmNoCj4KPgo+Cj4gLS0KPiBEYW5pZWwgVmV0 dGVyCj4gU29mdHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9uCj4gKzQxICgwKSA3OSAz NjUgNTcgNDggLSBodHRwOi8vYmxvZy5mZndsbC5jaAo+Cj4gLS0KPiBZb3UgcmVjZWl2ZWQgdGhp cyBtZXNzYWdlIGJlY2F1c2UgeW91IGFyZSBzdWJzY3JpYmVkIHRvIHRoZSBHb29nbGUgR3JvdXBz ICJvdXRyZWFjaHkta2VybmVsIiBncm91cC4KPiBUbyB1bnN1YnNjcmliZSBmcm9tIHRoaXMgZ3Jv dXAgYW5kIHN0b3AgcmVjZWl2aW5nIGVtYWlscyBmcm9tIGl0LCBzZW5kIGFuIGVtYWlsIHRvIG91 dHJlYWNoeS1rZXJuZWwrdW5zdWJzY3JpYmVAZ29vZ2xlZ3JvdXBzLmNvbS4KPiBUbyBwb3N0IHRv IHRoaXMgZ3JvdXAsIHNlbmQgZW1haWwgdG8gb3V0cmVhY2h5LWtlcm5lbEBnb29nbGVncm91cHMu Y29tLgo+IFRvIHZpZXcgdGhpcyBkaXNjdXNzaW9uIG9uIHRoZSB3ZWIgdmlzaXQgaHR0cHM6Ly9n cm91cHMuZ29vZ2xlLmNvbS9kL21zZ2lkL291dHJlYWNoeS1rZXJuZWwvQ0FLTUs3dUhpYTlUQzJt VkFHTG54MjJTMkpHMlZRUWNvJTJCZzRQRWRuUDhHY1VOTzhDNUElNDBtYWlsLmdtYWlsLmNvbS4K PiBGb3IgbW9yZSBvcHRpb25zLCB2aXNpdCBodHRwczovL2dyb3Vwcy5nb29nbGUuY29tL2Qvb3B0 b3V0Lgo+Cl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRy aS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRw czovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=