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: Mon, 11 Jan 2016 12:45:20 +0000 Message-ID: <5693A3E0.9000309@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> <568E5B5F.8020109@intel.com> <20160107165639.GS652@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 mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTP id E26DB6E363 for ; Mon, 11 Jan 2016 04:45:23 -0800 (PST) In-Reply-To: <20160107165639.GS652@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 , intel-gfx@lists.freedesktop.org, Jesse Barnes List-Id: intel-gfx@lists.freedesktop.org T24gMDcvMDEvMTYgMTY6NTYsIENocmlzIFdpbHNvbiB3cm90ZToKPiBPbiBUaHUsIEphbiAwNywg MjAxNiBhdCAxMjozNDozOVBNICswMDAwLCBEYXZlIEdvcmRvbiB3cm90ZToKPj4gT24gMDcvMDEv MTYgMTE6NTgsIENocmlzIFdpbHNvbiB3cm90ZToKPj4+IE9uIFRodSwgSmFuIDA3LCAyMDE2IGF0 IDEwOjIwOjUwQU0gKzAwMDAsIERhdmUgR29yZG9uIHdyb3RlOgo+Pj4+IFRoZXJlIGFyZSBhIG51 bWJlciBvZiBwbGFjZXMgd2hlcmUgdGhlIGRyaXZlciBuZWVkcyBhIHJlcXVlc3QsIGJ1dCBpc24n dAo+Pj4+IHdvcmtpbmcgb24gYmVoYWxmIG9mIGFueSBzcGVjaWZpYyB1c2VyIG9yIGluIGEgc3Bl Y2lmaWMgY29udGV4dC4gQXQKPj4+PiBwcmVzZW50LCB3ZSBhc3NvY2lhdGUgdGhlbSB3aXRoIHRo ZSBwZXItZW5naW5lIGRlZmF1bHQgY29udGV4dC4gQSBmdXR1cmUKPj4+PiBwYXRjaCB3aWxsIGFi b2xpc2ggdGhvc2UgcGVyLWVuZ2luZSBjb250ZXh0IHBvaW50ZXJzOyBidXQgd2UgY2FuIGFscmVh ZHkKPj4+PiBlbGltaW5hdGUgYSBsb3Qgb2YgdGhlIHJlZmVyZW5jZXMgdG8gdGhlbSwganVzdCBi eSBtYWtpbmcgdGhlIGFsbG9jYXRvcgo+Pj4+IGFsbG93IE5VTEwgYXMgYSBzaG9ydGhhbmQgZm9y ICJhbiBhcHByb3ByaWF0ZSBjb250ZXh0IGZvciB0aGlzIHJpbmciLAo+Pj4+IHdoaWNoIHdpbGwg bWVhbiB0aGF0IHRoZSBjYWxsZXJzIGRvbid0IG5lZWQgdG8ga25vdyBhbnl0aGluZyBhYm91dCBo b3cKPj4+PiB0aGUgImFwcHJvcHJpYXRlIGNvbnRleHQiIGlzIGZvdW5kIChlLmcuIHBlci1yaW5n IHZzIHBlci1kZXZpY2UsIGV0YykuCj4+Pj4KPj4+PiBTbyB0aGlzIHBhdGNoIHJlbmFtZXMgdGhl IGV4aXN0aW5nIGk5MTVfZ2VtX3JlcXVlc3RfYWxsb2MoKSwgYW5kIG1ha2VzCj4+Pj4gaXQgbG9j YWwgKHN0YXRpYyBpbmxpbmUpLCBhbmQgcmVwbGFjZXMgaXQgd2l0aCBhIHdyYXBwZXIgdGhhdCBw cm92aWRlcwo+Pj4+IGEgZGVmYXVsdCBpZiB0aGUgY29udGV4dCBpcyBOVUxMLCBhbmQgYWxzbyBo YXMgYSBuaWNlciBjYWxsaW5nCj4+Pj4gY29udmVudGlvbiAoZG9lc24ndCByZXF1aXJlIGEgcG9p bnRlciB0byBhbiBvdXRwdXQgcGFyYW1ldGVyKS4gVGhlbiB3ZQo+Pj4+IGNoYW5nZSBhbGwgY2Fs bGVycyB0byB1c2UgdGhlIG5ldyBjb252ZW50aW9uOgo+Pj4+IE9MRDoKPj4+PiAJZXJyID0gaTkx NV9nZW1fcmVxdWVzdF9hbGxvYyhyaW5nLCB1c2VyX2N0eCwgJnJlcSk7Cj4+Pj4gCWlmIChlcnIp IC4uLgo+Pj4+IE5FVzoKPj4+PiAJcmVxID0gaTkxNV9nZW1fcmVxdWVzdF9hbGxvYyhyaW5nLCB1 c2VyX2N0eCk7Cj4+Pj4gCWlmIChJU19FUlIocmVxKSkgLi4uCj4+Pj4gT0xEOgo+Pj4+IAllcnIg PSBpOTE1X2dlbV9yZXF1ZXN0X2FsbG9jKHJpbmcsIHJpbmctPmRlZmF1bHRfY29udGV4dCwgJnJl cSk7Cj4+Pj4gCWlmIChlcnIpIC4uLgo+Pj4+IE5FVzoKPj4+PiAJcmVxID0gaTkxNV9nZW1fcmVx dWVzdF9hbGxvYyhyaW5nLCBOVUxMKTsKPj4+PiAJaWYgKElTX0VSUihyZXEpKSAuLi4KPj4+Cj4+ PiBOYWsuIFlvdSBoYXZlbid0IGZpeGVkIGk5MTVfZ2VtX3JlcXVlc3RfYWxsb2MoKSBhdCBhbGwu Cj4+Pgo+Pj4gaHR0cDovL2NnaXQuZnJlZWRlc2t0b3Aub3JnL35pY2tsZS9saW51eC0yLjYvY29t bWl0Lz9oPWJyZWFkY3J1bWJzJmlkPTgyYzcyZTFhMmI0Mzg1ZjBhYjA3ZGNjZWU0NWFjZWYzODMw M2U5NmYKPj4+IGlzIHRoZSBwYXRjaCBJIGhhdmUgYmVlbiBjYXJyeWluZyBldmVyIHNpbmNlLgo+ Pj4gLUNocmlzCj4+Cj4+IEkgdGhpbmsgeW91J2xsIGZpbmQgdGhhdCB0aGUgdmVyc2lvbiBvZiBp OTE1X2dlbV9yZXF1ZXN0X2FsbG9jKCkKPj4gSSd2ZSBpbXBsZW1lbnRlZCBpcyBlcXVpdmFsZW50 IHRvIHlvdXJzLCB3aXRoIHRoZSAqYWRkaXRpb25hbCogKGFuZAo+PiBiZXR0ZXIpIHNlbWFudGlj IG9mIG5vdCByZXF1aXJpbmcgdGhlIGNhbGxlciB0byBzcGVjaWZ5Cj4+IChyaW5nLT5kZWZhdWx0 X3BhcmFtKSBhcyB0aGUgY29udGV4dCBwYXJhbWV0ZXIgKHdoaWNoIGlzIHRoZSBtYWluCj4+IHBv aW50LCBhcyBmYXIgYXMgSSdtIGNvbmNlcm5lZDsgbWFraW5nIHRoZSBjYWxsaW5nIGNvbnZlbnRp b24gbmljZXIKPj4gd2FzIGp1c3QgaW5jaWRlbnRhbCkuCj4KPiBOby4gU3BlY2lmeWluZyB0aGUg Y29udGV4dCBpcyBjcnVjaWFsIHRvIHJlcXVlc3QgYWxsb2NhdGlvbi4gVGhlIGlzc3VlCj4gaW4g dGhlIGN1cnJlbnQgZnVuY3Rpb24gY2FsbCBjaGFpbnMgYXJlIHRoZSByZXF1ZXN0cyBhcHBlYXIg b3V0IG9mCj4gbm93aGVyZSByYXRoZXIgdGhhbiBiZWluZyBjcmVhdGVkIHdpdGggZXhwbGljaXQg Y29udGV4dC4KPgo+IEhpZGluZyB0aGUgY29udGV4dCBpcyBiYWQuCj4gLUNocmlzCgpXaHkgd291 bGQgaXQgYmUgYmV0dGVyIHRvIG5hbWUgZGV2X3ByaXYtPmtlcm5lbF9jb250ZXh0IGF0IGV2ZXJ5 IApjYWxsc2l0ZSByYXRoZXIgdGhhbiBvbmNlIGluIHRoZSBjYWxsZWQgZnVuY3Rpb24gKGVzcGVj aWFsbHkgaW4gbXkgCnZlcnNpb24gd2hlcmUgaXQncyB2ZXJ5IG9idmlvdXMgdGhhdCB0aGUgbmV3 IHdyYXBwZXIgcHJvdmlkZXMgdGhhdCAKZGVmYXVsdCBiZWZvcmUgY2FsbGluZyB0aGUgb3JpZ2lu YWwgdW5kZXJseWluZyBhbGxvY2F0b3IpLgoKVGhlIGNvbnRleHQgaXMgb25seSBpbXBvcnRhbnQg dG8gY2FsbGVycyB3aG8gaGF2ZSBvbmUgaW4gaGFuZC4gVG8gdGhlIApvdGhlcnMsIGl0J3MgYW4g aXJyZWxldmFuY2U7IHRoZXkncmUganVzdCBzYXlpbmcsICJoZXksIEkgd2FudCBhIHJlcXVlc3Qg CmJ1dCBpdCdzIG5vdCBjb25uZWN0ZWQgdG8gYW55IGNvbnRleHQgSSBrbm93IG9mLCBzbyBnaW1t ZSBhIGRlZmF1bHQiLiAKVGhlIHByb29mIG9mIHRoYXQgaXMgdGhhdCB3ZSBjYW4gc3dpdGNoIHRo ZW0gZnJvbSBhIHBlci1lbmdpbmUgZGVmYXVsdCAKdG8gYSBwZXItZHJpdmVyIGRlZmF1bHQgd2l0 aG91dCBjaGFuZ2luZyBhbnl0aGluZyBlbHNlIGFib3V0IHRoZW0uIFRoZXkgCnJlYWxseSBkb24n dCBuZWVkIHRvIGtub3cgaG93IHRvIGZpbmQgdGhlIGRlZmF1bHQgY29udGV4dCwgc28gbGV0IHRo ZSAKYWxsb2NhdG9yIGZpbmQgaXQgZm9yIHRoZW0uCgo8YXNpZGU+ClN1cHBvc2Ugd2Ugc29tZSBk YXkgZGVjaWRlIHRoYXQgcGVyaGFwcyB0aGVyZSBzaG91bGQgYmUgKm11bHRpcGxlKiAKL2tlcm5l bCBjb250ZXh0cy8gKG1heWJlIGZvciBncmVhdGVyIHBhcmFsbGVsaXNtKTsgbXkgYXBwcm9hY2gg d2lsbCBsZXQgCnVzIHNwcmVhZCBub24tdXNlciByZXF1ZXN0cyBhY3Jvc3MgYWxsIHN1aXRhYmxl IGNvbnRleHRzIGp1c3QgYnkgCmltcGxlbWVudGluZyB0aGF0IGNob2ljZSBpbiBvbmUgcGxhY2Ug KHRoZSBuZXcgd3JhcHBlcikuIEJ5IGNvbnRyYXN0LCAKbmFtaW5nIHRoZSBkZWZhdWx0IGtlcm5l bCBjb250ZXh0IGF0IGV2ZXJ5IGNhbGxzaXRlIHdvdWxkIG1ha2UgdGhpcyBhIApmYXIgbW9yZSBp bnZhc2l2ZSBjaGFuZ2UuCjwvYXNpZGU+CgpJdCBpcyBhIGNvbW1vbnBsYWNlIG9mIG1vZGVybiBz b2Z0d2FyZSBwcmFjdGljZSB0aGF0IGVuY2Fwc3VsYXRpbmcgKGFuZCAKdGh1cyBoaWRpbmcpIGly cmVsZXZhbnQgZGV0YWlsIGlzIEdPT0QgcHJhY3RpY2U7IGl0IGlzIHRoZSBleHBvc2luZyBvZiAK aW50ZXJuYWwgaW1wbGVtZW50YXRpb24gZGV0YWlsIHRoYXQgaXMgQkFELgoKLkRhdmUuCl9fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWls aW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJl ZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==