From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Liu, Hong" Subject: Re: [PATCH] drm/i915: tidy up request alloc Date: Mon, 4 Jul 2016 04:08:34 +0000 Message-ID: <1467605321.2003.9.camel@intel.com> References: <1467250532-23644-1-git-send-email-hong.liu@intel.com> <5774DD4C.6030702@linux.intel.com> <20160630102241.GY28577@nuc-i3427.alporthouse.com> <5775154A.9070005@linux.intel.com> <5776A12A.7040103@intel.com> <20160701183451.GB27799@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTP id AD55A6E389 for ; Mon, 4 Jul 2016 04:08:43 +0000 (UTC) In-Reply-To: <20160701183451.GB27799@nuc-i3427.alporthouse.com> Content-Language: en-US Content-ID: <5D5EAEDDF3092C41B1A83E33F2C023CE@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "Gordon, David S" , "chris@chris-wilson.co.uk" Cc: "intel-gfx@lists.freedesktop.org" List-Id: intel-gfx@lists.freedesktop.org T24gRnJpLCAyMDE2LTA3LTAxIGF0IDE5OjM0ICswMTAwLCBDaHJpcyBXaWxzb24gd3JvdGU6DQo+ IE9uIEZyaSwgSnVsIDAxLCAyMDE2IGF0IDA1OjU4OjE4UE0gKzAxMDAsIERhdmUgR29yZG9uIHdy b3RlOg0KPiA+IE9uIDMwLzA2LzE2IDEzOjQ5LCBUdnJ0a28gVXJzdWxpbiB3cm90ZToNCj4gPiA+ IA0KPiA+ID4gT24gMzAvMDYvMTYgMTE6MjIsIENocmlzIFdpbHNvbiB3cm90ZToNCj4gPiA+ID4g T24gVGh1LCBKdW4gMzAsIDIwMTYgYXQgMDk6NTA6MjBBTSArMDEwMCwgVHZydGtvIFVyc3VsaW4g d3JvdGU6DQo+ID4gPiA+ID4gDQo+ID4gPiA+ID4gT24gMzAvMDYvMTYgMDI6MzUsIEhvbmcgTGl1 IHdyb3RlOg0KPiA+ID4gPiA+ID4gUmV0dXJuIHRoZSBhbGxvY2F0ZWQgcmVxdWVzdCBwb2ludGVy IGRpcmVjdGx5IHRvIHJlbW92ZQ0KPiA+ID4gPiA+ID4gdGhlIGRvdWJsZSBwb2ludGVyIHBhcmFt ZXRlci4NCj4gPiA+ID4gPiA+IA0KPiA+ID4gPiA+ID4gU2lnbmVkLW9mZi1ieTogSG9uZyBMaXUg PGhvbmcubGl1QGludGVsLmNvbT4NCj4gPiA+ID4gPiA+IC0tLQ0KPiA+ID4gPiA+ID4gwqBkcml2 ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbS5jIHwgMjUgKysrKysrKy0tLS0tLS0tLS0tLQ0KPiA+ ID4gPiA+ID4gLS0tLS0tDQo+ID4gPiA+ID4gPiDCoDEgZmlsZSBjaGFuZ2VkLCA3IGluc2VydGlv bnMoKyksIDE4IGRlbGV0aW9ucygtKQ0KPiA+ID4gPiA+ID4gDQo+ID4gPiA+ID4gPiBkaWZmIC0t Z2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW0uYw0KPiA+ID4gPiA+ID4gYi9kcml2 ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2dlbS5jDQo+ID4gPiA+ID4gPiBpbmRleCAxZDk4NzgyLi45 ODgxNDU1IDEwMDY0NA0KPiA+ID4gPiA+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkx NV9nZW0uYw0KPiA+ID4gPiA+ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9nZW0u Yw0KPiA+ID4gPiA+ID4gQEAgLTI5ODgsMzIgKzI5ODgsMjYgQEAgdm9pZCBpOTE1X2dlbV9yZXF1 ZXN0X2ZyZWUoc3RydWN0DQo+ID4gPiA+ID4gPiBrcmVmDQo+ID4gPiA+ID4gPiAqcmVxX3JlZikN Cj4gPiA+ID4gPiA+IMKgwqDCoMKgwqBrbWVtX2NhY2hlX2ZyZWUocmVxLT5pOTE1LT5yZXF1ZXN0 cywgcmVxKTsNCj4gPiA+ID4gPiA+IMKgfQ0KPiA+ID4gPiA+ID4gDQo+ID4gPiA+ID4gPiAtc3Rh dGljIGlubGluZSBpbnQNCj4gPiA+ID4gPiA+ICtzdGF0aWMgaW5saW5lIHN0cnVjdCBkcm1faTkx NV9nZW1fcmVxdWVzdCAqDQo+ID4gPiA+ID4gPiDCoF9faTkxNV9nZW1fcmVxdWVzdF9hbGxvYyhz dHJ1Y3QgaW50ZWxfZW5naW5lX2NzICplbmdpbmUsDQo+ID4gPiA+ID4gPiAtwqDCoMKgwqDCoMKg wqDCoMKgwqDCoMKgwqBzdHJ1Y3QgaTkxNV9nZW1fY29udGV4dCAqY3R4LA0KPiA+ID4gPiA+ID4g LcKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgc3RydWN0IGRybV9pOTE1X2dlbV9yZXF1ZXN0ICoq cmVxX291dCkNCj4gPiA+ID4gPiA+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoHN0cnVjdCBp OTE1X2dlbV9jb250ZXh0ICpjdHgpDQo+ID4gPiA+ID4gPiDCoHsNCj4gPiA+ID4gPiA+IMKgwqDC oMKgwqBzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYgPSBlbmdpbmUtPmk5MTU7DQo+ ID4gPiA+ID4gPiDCoMKgwqDCoMKgdW5zaWduZWQgcmVzZXRfY291bnRlciA9DQo+ID4gPiA+ID4g PiBpOTE1X3Jlc2V0X2NvdW50ZXIoJmRldl9wcml2LT5ncHVfZXJyb3IpOw0KPiA+ID4gPiA+ID4g wqDCoMKgwqDCoHN0cnVjdCBkcm1faTkxNV9nZW1fcmVxdWVzdCAqcmVxOw0KPiA+ID4gPiA+ID4g wqDCoMKgwqDCoGludCByZXQ7DQo+ID4gPiA+ID4gPiANCj4gPiA+ID4gPiA+IC3CoMKgwqDCoGlm ICghcmVxX291dCkNCj4gPiA+ID4gPiA+IC3CoMKgwqDCoMKgwqDCoMKgcmV0dXJuIC1FSU5WQUw7 DQo+ID4gPiA+ID4gPiAtDQo+ID4gPiA+ID4gPiAtwqDCoMKgwqAqcmVxX291dCA9IE5VTEw7DQo+ ID4gPiA+ID4gPiAtDQo+ID4gPiA+ID4gPiDCoMKgwqDCoMKgLyogQUJJOiBCZWZvcmUgdXNlcnNw YWNlIGFjY2Vzc2VzIHRoZSBHUFUgKGUuZy4NCj4gPiA+ID4gPiA+IGV4ZWNidWZmZXIpLA0KPiA+ ID4gPiA+ID4gcmVwb3J0DQo+ID4gPiA+ID4gPiDCoMKgwqDCoMKgwqAqIEVJTyBpZiB0aGUgR1BV IGlzIGFscmVhZHkgd2VkZ2VkLCBvciBFQUdBSU4gdG8gZHJvcA0KPiA+ID4gPiA+ID4gdGhlDQo+ ID4gPiA+ID4gPiBzdHJ1Y3RfbXV0ZXgNCj4gPiA+ID4gPiA+IMKgwqDCoMKgwqDCoCogYW5kIHJl c3RhcnQuDQo+ID4gPiA+ID4gPiDCoMKgwqDCoMKgwqAqLw0KPiA+ID4gPiA+ID4gwqDCoMKgwqDC oHJldCA9IGk5MTVfZ2VtX2NoZWNrX3dlZGdlKHJlc2V0X2NvdW50ZXIsDQo+ID4gPiA+ID4gPiBk ZXZfcHJpdi0+bW0uaW50ZXJydXB0aWJsZSk7DQo+ID4gPiA+ID4gPiDCoMKgwqDCoMKgaWYgKHJl dCkNCj4gPiA+ID4gPiA+IC3CoMKgwqDCoMKgwqDCoMKgcmV0dXJuIHJldDsNCj4gPiA+ID4gPiA+ ICvCoMKgwqDCoMKgwqDCoMKgcmV0dXJuIEVSUl9QVFIocmV0KTsNCj4gPiA+ID4gPiA+IA0KPiA+ ID4gPiA+ID4gwqDCoMKgwqDCoHJlcSA9IGttZW1fY2FjaGVfemFsbG9jKGRldl9wcml2LT5yZXF1 ZXN0cywNCj4gPiA+ID4gPiA+IEdGUF9LRVJORUwpOw0KPiA+ID4gPiA+ID4gwqDCoMKgwqDCoGlm IChyZXEgPT0gTlVMTCkNCj4gPiA+ID4gPiA+IC3CoMKgwqDCoMKgwqDCoMKgcmV0dXJuIC1FTk9N RU07DQo+ID4gPiA+ID4gPiArwqDCoMKgwqDCoMKgwqDCoHJldHVybiBFUlJfUFRSKC1FTk9NRU0p Ow0KPiA+ID4gPiA+ID4gDQo+ID4gPiA+ID4gPiDCoMKgwqDCoMKgcmV0ID0gaTkxNV9nZW1fZ2V0 X3NlcW5vKGVuZ2luZS0+aTkxNSwgJnJlcS0+c2Vxbm8pOw0KPiA+ID4gPiA+ID4gwqDCoMKgwqDC oGlmIChyZXQpDQo+ID4gPiA+ID4gPiBAQCAtMzA0MSwxNCArMzAzNSwxMyBAQCBfX2k5MTVfZ2Vt X3JlcXVlc3RfYWxsb2Moc3RydWN0DQo+ID4gPiA+ID4gPiBpbnRlbF9lbmdpbmVfY3MgKmVuZ2lu ZSwNCj4gPiA+ID4gPiA+IMKgwqDCoMKgwqBpZiAocmV0KQ0KPiA+ID4gPiA+ID4gwqDCoMKgwqDC oMKgwqDCoMKgZ290byBlcnJfY3R4Ow0KPiA+ID4gPiA+ID4gDQo+ID4gPiA+ID4gPiAtwqDCoMKg wqAqcmVxX291dCA9IHJlcTsNCj4gPiA+ID4gPiA+IC3CoMKgwqDCoHJldHVybiAwOw0KPiA+ID4g PiA+ID4gK8KgwqDCoMKgcmV0dXJuIHJlcTsNCj4gPiA+ID4gPiA+IA0KPiA+ID4gPiA+ID4gwqBl cnJfY3R4Og0KPiA+ID4gPiA+ID4gwqDCoMKgwqDCoGk5MTVfZ2VtX2NvbnRleHRfdW5yZWZlcmVu Y2UoY3R4KTsNCj4gPiA+ID4gPiA+IMKgZXJyOg0KPiA+ID4gPiA+ID4gwqDCoMKgwqDCoGttZW1f Y2FjaGVfZnJlZShkZXZfcHJpdi0+cmVxdWVzdHMsIHJlcSk7DQo+ID4gPiA+ID4gPiAtwqDCoMKg wqByZXR1cm4gcmV0Ow0KPiA+ID4gPiA+ID4gK8KgwqDCoMKgcmV0dXJuIEVSUl9QVFIocmV0KTsN Cj4gPiA+ID4gPiA+IMKgfQ0KPiA+ID4gPiA+ID4gDQo+ID4gPiA+ID4gPiDCoC8qKg0KPiA+ID4g PiA+ID4gQEAgLTMwNjcsMTMgKzMwNjAsOSBAQCBzdHJ1Y3QgZHJtX2k5MTVfZ2VtX3JlcXVlc3Qg Kg0KPiA+ID4gPiA+ID4gwqBpOTE1X2dlbV9yZXF1ZXN0X2FsbG9jKHN0cnVjdCBpbnRlbF9lbmdp bmVfY3MgKmVuZ2luZSwNCj4gPiA+ID4gPiA+IMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oMKgc3RydWN0IGk5MTVfZ2VtX2NvbnRleHQgKmN0eCkNCj4gPiA+ID4gPiA+IMKgew0KPiA+ID4g PiA+ID4gLcKgwqDCoMKgc3RydWN0IGRybV9pOTE1X2dlbV9yZXF1ZXN0ICpyZXE7DQo+ID4gPiA+ ID4gPiAtwqDCoMKgwqBpbnQgZXJyOw0KPiA+ID4gPiA+ID4gLQ0KPiA+ID4gPiA+ID4gwqDCoMKg wqDCoGlmIChjdHggPT0gTlVMTCkNCj4gPiA+ID4gPiA+IMKgwqDCoMKgwqDCoMKgwqDCoGN0eCA9 IGVuZ2luZS0+aTkxNS0+a2VybmVsX2NvbnRleHQ7DQo+ID4gPiA+ID4gPiAtwqDCoMKgwqBlcnIg PSBfX2k5MTVfZ2VtX3JlcXVlc3RfYWxsb2MoZW5naW5lLCBjdHgsICZyZXEpOw0KPiA+ID4gPiA+ ID4gLcKgwqDCoMKgcmV0dXJuIGVyciA/IEVSUl9QVFIoZXJyKSA6IHJlcTsNCj4gPiA+ID4gPiA+ ICvCoMKgwqDCoHJldHVybiBfX2k5MTVfZ2VtX3JlcXVlc3RfYWxsb2MoZW5naW5lLCBjdHgpOw0K PiA+ID4gPiA+ID4gwqB9DQo+ID4gPiA+ID4gPiANCj4gPiA+ID4gPiA+IMKgc3RydWN0IGRybV9p OTE1X2dlbV9yZXF1ZXN0ICoNCj4gPiA+ID4gPiA+IA0KPiA+ID4gPiA+IA0KPiA+ID4gPiA+IExv b2tzIGdvb2QgdG8gbWUuIEFuZCBoYXZlIHRoaXMgZmVlbGluZyBJJ3ZlIHNlZW4gdGhpcw0KPiA+ ID4gPiA+IHNvbWV3aGVyZSBiZWZvcmUuDQo+ID4gPiA+IA0KPiA+ID4gPiBTZXZlcmFsIHRpbWVz LiBUaGlzIGlzIG5vdCB0aGUgZnVsbCB0aWR5LCBub3IgZG9lcyBpdCByZWFsaXNlDQo+ID4gPiA+ IHRoZQ0KPiA+ID4gPiByYW1pZmFjdGlvbnMgb2YgcmVxdWVzdCBhbGxvYyB0aHJvdWdoIHRoZSBz dGFjay4NCj4gPiA+IA0KPiA+ID4gSG0gSSBjYW4ndCBzcG90IHRoYXQgaXQgaXMgZG9pbmcgYW55 dGhpbmcgd3Jvbmcgb3IgbWFraW5nDQo+ID4gPiBhbnl0aGluZw0KPiA+ID4gd29yc2UuIFlvdSBk b24ndCB3YW50IHRvIGxldCB0aGUgc21hbGwgY2xlYW51cCBpbj8NCj4gPiA+IA0KPiA+ID4gUmVn YXJkcywNCj4gPiA+IFR2cnRrbw0KPiA+IA0KPiA+IEl0IG91Z2h0IHRvIG1ha2UgYWxtb3N0IG5v IGRpZmZlcmVuY2UsIGJlY2F1c2UgdGhlICpvbmx5KiBwbGFjZSB0aGUNCj4gPiBpbm5lciBmdW5j dGlvbiBpcyBjYWxsZWQgaXMgZnJvbSB0aGUgb3V0ZXIgb25lLCB3aGljaCBwYXNzZXMgYQ0KPiA+ IHBvaW50ZXIgdG8gYSBsb2NhbCBmb3IgdGhlIHJldHVybmVkIG9iamVjdDsgYW5kIHRoZSBpbm5l ciBvbmUgaXMNCj4gPiB0aGVuIGlubGluZWQsIHNvIHRoZSBjb21waWxlciBkb2Vzbid0IGFjdHVh bGx5IHB1dCBpdCBvbiB0aGUgc3RhY2sNCj4gPiBhbmQgY2FsbCB0byB0aGUgaW5uZXIgYWxsb2Nh dG9yIGFueXdheS4NCj4gPiANCj4gPiBTdHJhbmdlbHksIGhvd2V2ZXIsIHdpdGggdGhpcyBjaGFu Z2UgdGhlIGNvZGUgYmVjb21lcyB+NDAwIGJ5dGVzDQo+ID4gYmlnZ2VyIQ0KPiA+IA0KPiA+IERp c2Fzc2VtYmx5IHJldmVhbHMgdGhhdCB3aGlsZSB0aGUgY29kZSBmb3IgdGhlIGV4dGVybmFsbHkt Y2FsbGFibGUNCj4gPiBvdXRlciBmdW5jdGlvbiBpcyBpbmRlZWQgYWxtb3N0IGlkZW50aWNhbCwg YSBzZWNvbmQgY29weSBvZiBpdCBoYXMNCj4gPiBhbHNvIGJlZW4gaW5saW5lZCBhdCB0aGUgb25l IGNhbGxzaXRlIGluIHRoaXMgZmlsZToNCj4gPiANCj4gPiBfX2k5MTVfZ2VtX29iamVjdF9zeW5j KCkgLi4uDQo+ID4gCXJlcSA9IGk5MTVfZ2VtX3JlcXVlc3RfYWxsb2ModG8sIE5VTEwpOw0KPiA+ IA0KPiA+IEkgZG9uJ3QgdGhpbmsgdGhhdCdzIGEgY3JpdGljYWwgcGF0aCBhbmQgd291bGQgcmF0 aGVyIGhhdmUgNDAwDQo+ID4gYnl0ZXMNCj4gPiBzbWFsbGVyIGNvZGVzcGFjZS4gV2UgY2FuIGdl dCB0aGF0IGJhY2sgYnkgYWRkaW5nIC9ub2lubGluZS8gdG8gdGhlDQo+ID4gb3V0ZXIgZnVuY3Rp b24gaTkxNV9nZW1fcmVxdWVzdF9hbGxvYygpIChub3QsIG9mIGNvdXJzZSwgdG8gdGhlDQo+ID4g aW5uZXIgb25lLCB0aGF0IGRlZmluaXRlbHkgKnNob3VsZCogYmUgaW5saW5lKS4NCj4gDQo+IF9f aTkxNV9nZW1fb2JqZWN0X3N5bmMoKSBzaG91bGQgbm90IGJlIGNhbGxpbmcNCj4gaTkxNV9nZW1f cmVxdWVzdF9hbGxvYygpLg0KPiANCj4gVGhhdCdzIHRoZSBpc3N1ZSB3aXRoIHRoaXMgcGF0Y2gs IHlvdXIgcGF0Y2ggYW5kIEpvaG4ncyBwYXRjaC4NCg0KU28gd2Ugd3JvdGUgdGhlIGk5MTVfZ2Vt X3JlcXVlc3RfYWxsb2MoKSB0aGlzIHdheSBpcyB0byBhdm9pZCBiZWluZw0KaW5saW5lZCBpbnRv IGNhbGxlcnMgbGlrZSBfX2k5MTVfZ2VtX29iamVjdF9zeW5jKCk/DQoNCkkgY2hlY2tlZCB0aGUg ZmlsZSB3aXRoIEdDQyA0LjguNSBvbiBteSBjZW50b3MgZW52aXJvbm1lbnQsIGl0IGlzIGxpa2UN CndoYXQgRGF2ZSBmb3VuZC4gV2l0aCB0aGUgcGF0Y2gsIGk5MTVfZ2VtX29iamVjdF9zeW5jKCkg aXMgMzY4IGJ5dGVzDQpiaWdnZXIuDQoNCkJ1dCB3aGVuIEkgY2hlY2tlZCBpdCB3aXRoIEdDQyA2 LjEuMSBvbiBGZWRvcmEgMjQsIGl0IHNlZW1zIGl0IGlubGluZXMNCnRoZSBpOTE1X2dlbV9yZXF1 ZXN0X2FsbG9jKCkgZXZlbiB3aXRoIHRoZSBjdXJyZW50IGltcGxlbWVudGF0aW9uLg0KV2l0aCB0 aGUgcGF0Y2gsIHRoZSBpOTE1X2dlbV9vYmplY3Rfc3luYygpIGlzIDgwIGJ5dGVzIHNtYWxsZXIu DQoNCg0KPiAtQ2hyaXMNCj4gCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNr dG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2lu dGVsLWdmeAo=