From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Gordon Subject: Re: [PATCH v3 1/3] drm/i915: simplify allocation of driver-internal requests Date: Tue, 12 Jan 2016 13:02:27 +0000 Message-ID: <5694F963.50906@intel.com> References: <1452162052-22573-1-git-send-email-david.s.gordon@intel.com> <1452162052-22573-2-git-send-email-david.s.gordon@intel.com> <20160107115846.GI652@nuc-i3427.alporthouse.com> <568E9722.8010309@virtuousgeek.org> <20160107165300.GR652@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 mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 1B3256E638 for ; Tue, 12 Jan 2016 05:02:29 -0800 (PST) In-Reply-To: <20160107165300.GR652@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: Chris Wilson , Jesse Barnes , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gMDcvMDEvMTYgMTY6NTMsIENocmlzIFdpbHNvbiB3cm90ZToKPiBPbiBUaHUsIEphbiAwNywg MjAxNiBhdCAwODo0OTozOEFNIC0wODAwLCBKZXNzZSBCYXJuZXMgd3JvdGU6Cj4+IE9uIDAxLzA3 LzIwMTYgMDM6NTggQU0sIENocmlzIFdpbHNvbiB3cm90ZToKPj4+IE9uIFRodSwgSmFuIDA3LCAy MDE2IGF0IDEwOjIwOjUwQU0gKzAwMDAsIERhdmUgR29yZG9uIHdyb3RlOgo+Pj4+IFRoZXJlIGFy ZSBhIG51bWJlciBvZiBwbGFjZXMgd2hlcmUgdGhlIGRyaXZlciBuZWVkcyBhIHJlcXVlc3QsIGJ1 dCBpc24ndAo+Pj4+IHdvcmtpbmcgb24gYmVoYWxmIG9mIGFueSBzcGVjaWZpYyB1c2VyIG9yIGlu IGEgc3BlY2lmaWMgY29udGV4dC4gQXQKPj4+PiBwcmVzZW50LCB3ZSBhc3NvY2lhdGUgdGhlbSB3 aXRoIHRoZSBwZXItZW5naW5lIGRlZmF1bHQgY29udGV4dC4gQSBmdXR1cmUKPj4+PiBwYXRjaCB3 aWxsIGFib2xpc2ggdGhvc2UgcGVyLWVuZ2luZSBjb250ZXh0IHBvaW50ZXJzOyBidXQgd2UgY2Fu IGFscmVhZHkKPj4+PiBlbGltaW5hdGUgYSBsb3Qgb2YgdGhlIHJlZmVyZW5jZXMgdG8gdGhlbSwg anVzdCBieSBtYWtpbmcgdGhlIGFsbG9jYXRvcgo+Pj4+IGFsbG93IE5VTEwgYXMgYSBzaG9ydGhh bmQgZm9yICJhbiBhcHByb3ByaWF0ZSBjb250ZXh0IGZvciB0aGlzIHJpbmciLAo+Pj4+IHdoaWNo IHdpbGwgbWVhbiB0aGF0IHRoZSBjYWxsZXJzIGRvbid0IG5lZWQgdG8ga25vdyBhbnl0aGluZyBh Ym91dCBob3cKPj4+PiB0aGUgImFwcHJvcHJpYXRlIGNvbnRleHQiIGlzIGZvdW5kIChlLmcuIHBl ci1yaW5nIHZzIHBlci1kZXZpY2UsIGV0YykuCj4+Pj4KPj4+PiBTbyB0aGlzIHBhdGNoIHJlbmFt ZXMgdGhlIGV4aXN0aW5nIGk5MTVfZ2VtX3JlcXVlc3RfYWxsb2MoKSwgYW5kIG1ha2VzCj4+Pj4g aXQgbG9jYWwgKHN0YXRpYyBpbmxpbmUpLCBhbmQgcmVwbGFjZXMgaXQgd2l0aCBhIHdyYXBwZXIg dGhhdCBwcm92aWRlcwo+Pj4+IGEgZGVmYXVsdCBpZiB0aGUgY29udGV4dCBpcyBOVUxMLCBhbmQg YWxzbyBoYXMgYSBuaWNlciBjYWxsaW5nCj4+Pj4gY29udmVudGlvbiAoZG9lc24ndCByZXF1aXJl IGEgcG9pbnRlciB0byBhbiBvdXRwdXQgcGFyYW1ldGVyKS4gVGhlbiB3ZQo+Pj4+IGNoYW5nZSBh bGwgY2FsbGVycyB0byB1c2UgdGhlIG5ldyBjb252ZW50aW9uOgo+Pj4+IE9MRDoKPj4+PiAJZXJy ID0gaTkxNV9nZW1fcmVxdWVzdF9hbGxvYyhyaW5nLCB1c2VyX2N0eCwgJnJlcSk7Cj4+Pj4gCWlm IChlcnIpIC4uLgo+Pj4+IE5FVzoKPj4+PiAJcmVxID0gaTkxNV9nZW1fcmVxdWVzdF9hbGxvYyhy aW5nLCB1c2VyX2N0eCk7Cj4+Pj4gCWlmIChJU19FUlIocmVxKSkgLi4uCj4+Pj4gT0xEOgo+Pj4+ IAllcnIgPSBpOTE1X2dlbV9yZXF1ZXN0X2FsbG9jKHJpbmcsIHJpbmctPmRlZmF1bHRfY29udGV4 dCwgJnJlcSk7Cj4+Pj4gCWlmIChlcnIpIC4uLgo+Pj4+IE5FVzoKPj4+PiAJcmVxID0gaTkxNV9n ZW1fcmVxdWVzdF9hbGxvYyhyaW5nLCBOVUxMKTsKPj4+PiAJaWYgKElTX0VSUihyZXEpKSAuLi4K Pj4+Cj4+PiBOYWsuIFlvdSBoYXZlbid0IGZpeGVkIGk5MTVfZ2VtX3JlcXVlc3RfYWxsb2MoKSBh dCBhbGwuCj4+Pgo+Pj4gaHR0cDovL2NnaXQuZnJlZWRlc2t0b3Aub3JnL35pY2tsZS9saW51eC0y LjYvY29tbWl0Lz9oPWJyZWFkY3J1bWJzJmlkPTgyYzcyZTFhMmI0Mzg1ZjBhYjA3ZGNjZWU0NWFj ZWYzODMwM2U5NmYKPj4+IGlzIHRoZSBwYXRjaCBJIGhhdmUgYmVlbiBjYXJyeWluZyBldmVyIHNp bmNlLgo+Pgo+PiBDYW4gd2Ugc3RvcCB3aXRoIHRoZSAibmFrIj8gIFRoaXMgcGF0Y2ggd3JhcHMg dGhlIHJlcXVlc3QgYWxsb2MKPj4gZGlmZmVyZW50bHkgdGhhbiB5b3VycywgYnV0IHlvdSBoYXZl bid0IGdpdmVuIGRldGFpbHMgYXMgdG8gd2h5IHlvdQo+PiB0aGluayBpdCdzIGluY29ycmVjdCAo c2VlIERhdmUncyByZXBseSkuCj4KPiBJIGFtIGFubm95ZWQgdGhhdCBwZW9wbGUgZG8gbm90IHJl dmlldyBteSBwYXRjaGVzIGFuZCBhcmUgZHVwbGljYXRpbmcKPiB3b3JrIEkgZGlkIGxhc3QgeWVh ciBhbmQgdGhlIHllYXIgYmVmb3JlLCB3aXRob3V0IGF0dGVtcHRpbmcgdG8gZml4Cj4gcmVhbCBi dWdzLgo+IC1DaHJpcwoKQ2hyaXMsIHRoaXMgcGF0Y2hzZXQgaXMgdG90YWxseSBkaXJlY3RlZCB0 b3dhcmRzIGZpeGluZyBhIHNwZWNpZmljIGJ1ZywgCm9uZSB3aGljaCwgbW9yZW92ZXIsIGFyb3Nl IGEgY29uc2VxdWVuY2Ugb2YgYSBwYXRjaCBZT1Ugd3JvdGU6CmIwMzY2YTUgZHJtL2k5MTU6IGlu dGVsX3JpbmdfaW5pdGlhbGl6ZWQoKSBtdXN0IGJlIHNpbXBsZSBhbmQgaW5saW5lCihtZWEgY3Vs cGEgdG9vLCBvYnZpb3VzbHksIHNpbmNlIEkgd2FzIHRoZSBvbmUgd2hvIHJlYmFzZWQgJiBwdXNo ZWQgaXQpLgpOaWNrIGhhcyBhIGZpeCBmb3IgdGhlIG9yaWdpbmFsIGJ1Zywgd2hpY2ggaW52b2x2 ZXMgcmV2ZXJzaW5nIHRoZSAKdGVhcmRvd24gb3JkZXIsIGJ1dCBjYW4ndCBub3cgdXNlIGl0IHNp bmNlIGIwMzY2YTUsIHNvIHRoZSBidWcgcmVtYWlucy4gCk5pY2sncyBmaXggY2FuIGJlIG1hZGUg dG8gd29yayBpZiB3ZSByZXBsYWNlIHRoZSBwZXItZW5naW5lIGRlZmF1bHQgCmNvbnRleHRzIHdp dGggdGhlIGdsb2JhbCBvbmUsIHdoaWNoIHlvdSd2ZSBhbHJlYWR5IGFncmVlZCBpcyBhIGdvb2Qg aWRlYSAKKEkgdGhpbmsgaXQgd2FzIHlvdXIgaWRlYSBpbiB0aGUgZmlyc3QgcGxhY2UhKS4KCldl IGNhbid0IHRha2UgeW91ciBwYXRjaCBiZWNhdXNlIGl0IGRvZXNuJ3QgYXBwbHkgdG8gbmlnaHRs eS4gSWYgeW91IApwcm92aWRlIGEgc3RhbmRhbG9uZSB2ZXJzaW9uIHRoYXQncyBub3QgZW50YW5n bGVkIHdpdGggMTAwIG90aGVyIHBhdGNoZXMgCkknbGwgaGFwcGlseSByZXZpZXcgaXQuIE9yIEkg bWlnaHQgY2hlcnJ5LXBpY2sgeW91ciBleGlzdGluZyBvbmUgb3V0IG9mIAp0aGUgMTkwLWVsZW1l bnQgcGF0Y2hzZXQgYW5kIHRyeSB0byByZWJhc2UgaXQgb250byBuaWdodGx5LCB3aGljaCBpcyBo b3cgCmIwMzY2YTUgZ290IGluIGluIHRoZSBmaXJzdCBwbGFjZS4gSSBzdXNwZWN0IGl0IHdvdWxk IGxvb2sgdmVyeSBtdWNoIApsaWtlIG1pbmUgdGhlbiAuLi4KCk1heWJlIHdlIHNob3VsZCBqdXN0 IHJldmVydCBiMDM2NmE1IGluc3RlYWQ/IEV2ZW4gdGhvdWdoIGl0IHdhcyBxdWl0ZSAKbmljZSBp biBpdHNlbGYgLi4uCgouRGF2ZS4KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRl c2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9p bnRlbC1nZngK