From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 21/22] drm/i915: Enable lockless lookup of request tracking via RCU Date: Fri, 29 Jul 2016 10:43:17 +0100 Message-ID: <20160729094317.GJ19065@nuc-i3427.alporthouse.com> References: <1469618100-15298-1-git-send-email-chris@chris-wilson.co.uk> <1469618100-15298-22-git-send-email-chris@chris-wilson.co.uk> <20160728102340.GH6232@phenom.ffwll.local> <20160728204958.GA17037@nuc-i3427.alporthouse.com> <20160729084114.GV6232@phenom.ffwll.local> <20160729084954.GI19065@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from fireflyinternet.com (mail.fireflyinternet.com [109.228.58.192]) by gabe.freedesktop.org (Postfix) with ESMTPS id 9037A6E977 for ; Fri, 29 Jul 2016 09:43:28 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20160729084954.GI19065@nuc-i3427.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter , intel-gfx@lists.freedesktop.org, "Goel, Akash" , Josh Triplett List-Id: intel-gfx@lists.freedesktop.org T24gRnJpLCBKdWwgMjksIDIwMTYgYXQgMDk6NDk6NTRBTSArMDEwMCwgQ2hyaXMgV2lsc29uIHdy b3RlOgo+IE9uIEZyaSwgSnVsIDI5LCAyMDE2IGF0IDEwOjQxOjE0QU0gKzAyMDAsIERhbmllbCBW ZXR0ZXIgd3JvdGU6Cj4gPiBJIGd1ZXNzIGl0IGRvZXNuJ3QgaHVydCB0byBtYWtlIHRoaXMgcmVh bGx5LCByZWFsbHkgY2xlYXIuIFBlcmZlY3QhIFdlbGwKPiA+IGFsbW9zdCwgb25lIG5pdDoKPiA+ IAo+ID4gPiAKPiA+ID4gICAgICAgICAgICAgICAgIC8qIFdoYXQgc3RvcHMgdGhlIGZvbGxvd2lu ZyByY3VfZGVyZWZlcmVuY2UoKSBmcm9tIG9jY3VyaW5nCj4gPiA+ICAgICAgICAgICAgICAgICAg KiBiZWZvcmUgdGhlIGFib3ZlIGk5MTVfZ2VtX3JlcXVlc3RfZ2V0X3JjdSgpPyBJZiB3ZSB3ZXJl Cj4gPiA+ICAgICAgICAgICAgICAgICAgKiB0byByZWFkIHRoZSB2YWx1ZSBiZWZvcmUgcGF1c2lu ZyB0byBnZXQgdGhlIHJlZmVyZW5jZSB0bwo+ID4gPiAgICAgICAgICAgICAgICAgICogdGhlIHJl cXVlc3QsIHdlIG1heSBub3Qgbm90aWNlIGEgY2hhbmdlIGluIHRoZSBhY3RpdmUKPiA+ID4gICAg ICAgICAgICAgICAgICAqIHRyYWNrZXIuCj4gPiA+ICAgICAgICAgICAgICAgICAgKgo+ID4gPiAg ICAgICAgICAgICAgICAgICogVGhlIHJjdV9kZXJlZmVyZW5jZSgpIGlzIGEgbWVyZSByZWFkIGJh cnJpZXIsIHdoaWNoIG1lYW5zCj4gPiAKPiA+IHMvcmVhZCBiYXJyaWVyL2JhcnJpZXIgb2YgZGVw ZW5kaW5nIHJlYWRzLywgcmN1X2RlcmVmZXJlbmNlIGlzIG5vdCBldmVuIGEKPiA+IGZ1bGwgcm1i IQo+ID4gCj4gPiA+ICAgICAgICAgICAgICAgICAgKiB0aGF0IG9wZXJhdGlvbnMgYWZ0ZXIgaXQg d2lsbCBhcHBlYXIgYWZ0ZXIsIG5laXRoZXIgdGhlCj4gPiAKPiA+IGhlbmNlIGFsc286IHMvb3Bl cmF0aW9ucy9hbnkgb3BlcmF0aW9ucyB0aHJvdWdoIHRoZSByZWFkIHBvaW50ZXIvCj4gCj4gQWgg cmlnaHQsIHRoYXQgbmVlZHMgdG8gYmUgZGVwZW5kZW50IHJlYWRzLiBDaGFuZ2VzIGxvb2sgZ29v ZC4KCgogICAgICAgZG8gewogICAgICAgICAgICAgICAgc3RydWN0IGRybV9pOTE1X2dlbV9yZXF1 ZXN0ICpyZXF1ZXN0OwoKICAgICAgICAgICAgICAgIHJlcXVlc3QgPSByY3VfZGVyZWZlcmVuY2Uo YWN0aXZlLT5yZXF1ZXN0KTsKICAgICAgICAgICAgICAgIGlmICghcmVxdWVzdCB8fCBpOTE1X2dl bV9yZXF1ZXN0X2NvbXBsZXRlZChyZXF1ZXN0KSkKICAgICAgICAgICAgICAgICAgICAgICAgcmV0 dXJuIE5VTEw7CgogICAgICAgICAgICAgICAgcmVxdWVzdCA9IGk5MTVfZ2VtX3JlcXVlc3RfZ2V0 X3JjdShyZXF1ZXN0KTsKCiAgICAgICAgICAgICAgICAvKiBXaGF0IHN0b3BzIHRoZSBmb2xsb3dp bmcgcmN1X2FjY2Vzc19wb2ludGVyKCkgZnJvbSBvY2N1cnJpbmcKICAgICAgICAgICAgICAgICAq IGJlZm9yZSB0aGUgYWJvdmUgaTkxNV9nZW1fcmVxdWVzdF9nZXRfcmN1KCk/IElmIHdlIHdlcmUK ICAgICAgICAgICAgICAgICAqIHRvIHJlYWQgdGhlIHZhbHVlIGJlZm9yZSBwYXVzaW5nIHRvIGdl dCB0aGUgcmVmZXJlbmNlIHRvCiAgICAgICAgICAgICAgICAgKiB0aGUgcmVxdWVzdCwgd2UgbWF5 IG5vdCBub3RpY2UgYSBjaGFuZ2UgaW4gdGhlIGFjdGl2ZQogICAgICAgICAgICAgICAgICogdHJh Y2tlci4KICAgICAgICAgICAgICAgICAqCiAgICAgICAgICAgICAgICAgKiBUaGUgcmN1X2FjY2Vz c19wb2ludGVyKCkgaXMgYSBtZXJlIGNvbXBpbGVyIGJhcnJpZXIsIHdoaWNoCiAgICAgICAgICAg ICAgICAgKiBtZWFucyBib3RoIHRoZSBDUFUgYW5kIGNvbXBpbGVyIGFyZSBmcmVlIHRvIHBlcmZv cm0gdGhlCiAgICAgICAgICAgICAgICAgKiBtZW1vcnkgcmVhZCB3aXRob3V0IGNvbnN0cmFpbnQu IFRoZSBjb21waWxlciBvbmx5IGhhcyB0bwogICAgICAgICAgICAgICAgICogZW5zdXJlIHRoYXQg YW55IG9wZXJhdGlvbnMgYWZ0ZXIgdGhlIHJjdV9hY2Nlc3NfcG9pbnRlcigpCiAgICAgICAgICAg ICAgICAgKiBvY2N1ciBhZnRlcndhcmRzIGluIHByb2dyYW0gb3JkZXIuIFRoaXMgbWVhbnMgdGhl IHJlYWQgbWF5CiAgICAgICAgICAgICAgICAgKiBiZSBwZXJmb3JtZWQgZWFybGllciBieSBhbiBv dXQtb2Ytb3JkZXIgQ1BVLCBvciBhZHZlbnR1cm91cwogICAgICAgICAgICAgICAgICogY29tcGls ZXIuCiAgICAgICAgICAgICAgICAgKgogICAgICAgICAgICAgICAgICogVGhlIGF0b21pYyBvcGVy YXRpb24gYXQgdGhlIGhlYXJ0IG9mCiAgICAgICAgICAgICAgICAgKiBpOTE1X2dlbV9yZXF1ZXN0 X2dldF9yY3UoKSwgc2VlIGZlbmNlX2dldF9yY3UoKSwgaXMKICAgICAgICAgICAgICAgICAqIGF0 b21pY19pbmNfbm90X3plcm8oKSB3aGljaCBpcyBvbmx5IGEgZnVsbCBtZW1vcnkgYmFycmllcgog ICAgICAgICAgICAgICAgICogd2hlbiBzdWNjZXNzZnVsLiBUaGF0IGlzLCBpZiBpOTE1X2dlbV9y ZXF1ZXN0X2dldF9yY3UoKQogICAgICAgICAgICAgICAgICogcmV0dXJucyB0aGUgcmVxdWVzdCAo YW5kIHNvIHdpdGggdGhlIHJlZmVyZW5jZSBjb3VudGVkCiAgICAgICAgICAgICAgICAgKiBpbmNy ZW1lbnRlZCkgdGhlbiB0aGUgZm9sbG93aW5nIHJlYWQgZm9yIHJjdV9hY2Nlc3NfcG9pbnRlcigp CiAgICAgICAgICAgICAgICAgKiBtdXN0IG9jY3VyIGFmdGVyIHRoZSBhdG9taWMgb3BlcmF0aW9u IGFuZCBzbyBjb25maXJtCiAgICAgICAgICAgICAgICAgKiB0aGF0IHRoaXMgcmVxdWVzdCBpcyB0 aGUgb25lIGN1cnJlbnRseSBiZWluZyB0cmFja2VkLgogICAgICAgICAgICAgICAgICovCiAgICAg ICAgICAgICAgICBpZiAoIXJlcXVlc3QgfHwgcmVxdWVzdCA9PSByY3VfYWNjZXNzX3BvaW50ZXIo YWN0aXZlLT5yZXF1ZXN0KSkKICAgICAgICAgICAgICAgICAgICAgICAgcmV0dXJuIHJjdV9wb2lu dGVyX2hhbmRvZmYocmVxdWVzdCk7CgogICAgICAgICAgICAgICAgaTkxNV9nZW1fcmVxdWVzdF9w dXQocmVxdWVzdCk7CiAgICAgICAgfSB3aGlsZSAoMSk7CgoKLS0gCkNocmlzIFdpbHNvbiwgSW50 ZWwgT3BlbiBTb3VyY2UgVGVjaG5vbG9neSBDZW50cmUKX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==