From mboxrd@z Thu Jan 1 00:00:00 1970 From: Felix Kuehling Subject: Re: [PATCH 1/1] drm/amdkfd: Do not ignore requested queue size during allocation Date: Tue, 21 Nov 2017 11:30:48 -0500 Message-ID: <77963976-376a-beb5-ba2c-8b40c5a690da@amd.com> References: <20171116213631.3987-1-jan.vesely@rutgers.edu> <21e77adc-4fbe-a3e9-0a02-5d84eb201561@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: Content-Language: en-CA List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "amd-gfx" To: Oded Gabbay Cc: amd-gfx list , Jan Vesely , Maling list - DRI developers List-Id: dri-devel@lists.freedesktop.org Ck9uIDIwMTctMTEtMjEgMDY6NDQgQU0sIE9kZWQgR2FiYmF5IHdyb3RlOgo+IFRoYW5rcyBGZWxp eCBmb3IgY2F0Y2hpbmcgdGhhdCwgRm9yIHNvbWUgcmVhc29uIEkgcmVtZW1iZXJlZCAgRU9QCj4g YnVmZmVyIHNob3VsZCBiZSB0aGUgc2FtZSBzaXplIG9mIHRoZSBxdWV1ZS4KClRoZSBFT1AgcXVl dWUgc2l6ZSBpcyBoYXJkLWNvZGVkIHRvIHByb3AuZW9wX3JpbmdfYnVmZmVyX3NpemUgPQpQQUdF X1NJWkUgZm9yIGtlcm5lbCBxdWV1ZXMgaW4gaW5pdGlhbGl6ZSBpbiBrZmRfa2VybmVsX3F1ZXVl LmMuIEknbSBub3QKdG9vIGZhbWlsaWFyIHdpdGggdGhlIEhXL0ZXIGRldGFpbHMuIEJ1dCBJIHNl ZSB0aGlzIGNvbW1lbnQgaW4Ka2ZkX21xZF9tYW5hZ2VyX3ZpLmM6CgogICAgICAgIC8qCiAgICAg ICAgICogSFcgZG9lcyBub3QgY2xhbXAgdGhpcyBmaWVsZCBjb3JyZWN0bHkuIE1heGltdW0gRU9Q IHF1ZXVlIHNpemUKICAgICAgICAgKiBpcyBjb25zdHJhaW5lZCBieSBwZXItU0UgRU9QIGRvbmUg c2lnbmFsIGNvdW50LCB3aGljaCBpcyA4LWJpdC4KICAgICAgICAgKiBMaW1pdCBpcyAweEZGIEVP UCBlbnRyaWVzICg9IDB4N0Y4IGR3b3JkcykuIENQIHdpbGwgbm90IHN1Ym1pdAogICAgICAgICAq IG1vcmUgdGhhbiAoRU9QIGVudHJ5IGNvdW50IC0gMSkgc28gYSBxdWV1ZSBzaXplIG9mIDB4ODAw IGR3b3JkcwogICAgICAgICAqIGlzIHNhZmUsIGdpdmluZyBhIG1heGltdW0gZmllbGQgdmFsdWUg b2YgMHhBLgogICAgICAgICAqLwoKV2l0aCB0aGF0IHRoZSBtYXhpbXVtIHBvc3NpYmxlIEVPUCBx dWV1ZSBzaXplIHdvdWxkIGJlIHR3byBwYWdlcywKcmVnYXJkbGVzcyBvZiB0aGUgcXVldWUgc2l6 ZS4KCj4gVGhlbiB3ZSBjYW4gcmVtb3ZlIHRoZSBxdWV1ZSBzaXplIHBhcmFtZXRlciBmcm9tIHRo YXQgZnVuY3Rpb24gPwoKTm90IHRoZSB3YXkgdGhlIGNvZGUgaXMgY3VycmVudGx5IG9yZ2FuaXpl ZC4gQ3VycmVudGx5IHN0cnVjdAprZXJuZWxfcXVldWVfb3BzIGlzIHNoYXJlZCBmb3IgQVNJQy1p bmRlcGVuZGVudCBhbmQgQVNJQy1zcGVjaWZpYyBxdWV1ZQpvcHMuIFRoZSBBU0lDLWluZGVwZW5k ZW50IGluaXRpYWxpemUgZnVuY3Rpb24gaW4ga2ZkX2tlcm5lbF9xdWV1ZS5jCnN0aWxsIG5lZWRz IHRoaXMgcGFyYW1ldGVyLgoKVGhhdCBzYWlkLCB0aGUga2VybmVsX3F1ZXVlIHN0dWZmIGNvdWxk IGJlIGNsZWFuZWQgdXAgYSBiaXQgaW4gZ2VuZXJhbC4KSU1PIHRoZSBoYXJkd2FyZS1pbmRlcGVu ZGVudCBmdW5jdGlvbnMgZG9uJ3QgcmVhbGx5IG5lZWQgdG8gYmUgY2FsbGVkCnRocm91Z2ggZnVu Y3Rpb24gcG9pbnRlcnMuIFRoZSBBU0lDLXNwZWNpZmljIGZ1bmN0aW9uIHBvaW50ZXJzIGRvbid0 Cm5lZWQgdG8gYmUgaW4gdGhlIGtlcm5lbF9xdWV1ZSBzdHJ1Y3R1cmUsIHRoZXkgY291bGQgYmUg aW4ga2ZkX2Rldi4KClJlZ2FyZHMsCsKgIEZlbGl4Cgo+Cj4gT24gTW9uLCBOb3YgMjAsIDIwMTcg YXQgOToyMiBQTSwgRmVsaXggS3VlaGxpbmcgPGZlbGl4Lmt1ZWhsaW5nQGFtZC5jb20+IHdyb3Rl Ogo+PiBJIHRoaW5rIHRoaXMgcGF0Y2ggaXMgbm90IGNvcnJlY3QuIFRoZSBFT1AtbWVtIGlzIG5v dCBhc3NvY2lhdGVkIHdpdGgKPj4gdGhlIHF1ZXVlIHNpemUuIFRoZSBFT1AgYnVmZmVyIGlzIGEg c2VwYXJhdGUgYnVmZmVyIHVzZWQgYnkgdGhlIGZpcm13YXJlCj4+IHRvIGhhbmRsZSBjb21tYW5k IGNvbXBsZXRpb24uIEFzIEkgdW5kZXJzdGFuZCBpdCwgdGhpcyBhbGxvd3MgbW9yZQo+PiBjb25j dXJyZW5jeSwgd2hpbGUgc3RpbGwgbWFraW5nIGl0IGxvb2sgbGlrZSBhbGwgY29tbWFuZHMgaW4g dGhlIHF1ZXVlCj4+IGFyZSBjb21wbGV0aW5nIGluIG9yZGVyLgo+Pgo+PiBSZWdhcmRzLAo+PiAg IEZlbGl4Cj4+Cj4+Cj4+IE9uIDIwMTctMTEtMTkgMDM6MTkgQU0sIE9kZWQgR2FiYmF5IHdyb3Rl Ogo+Pj4gT24gVGh1LCBOb3YgMTYsIDIwMTcgYXQgMTE6MzYgUE0sIEphbiBWZXNlbHkgPGphbi52 ZXNlbHlAcnV0Z2Vycy5lZHU+IHdyb3RlOgo+Pj4+IFNpZ25lZC1vZmYtYnk6IEphbiBWZXNlbHkg PGphbi52ZXNlbHlAcnV0Z2Vycy5lZHU+Cj4+Pj4gLS0tCj4+Pj4gIGRyaXZlcnMvZ3B1L2RybS9h bWQvYW1ka2ZkL2tmZF9rZXJuZWxfcXVldWVfdmkuYyB8IDUgKysrLS0KPj4+PiAgMSBmaWxlIGNo YW5nZWQsIDMgaW5zZXJ0aW9ucygrKSwgMiBkZWxldGlvbnMoLSkKPj4+Pgo+Pj4+IGRpZmYgLS1n aXQgYS9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGtmZC9rZmRfa2VybmVsX3F1ZXVlX3ZpLmMgYi9k cml2ZXJzL2dwdS9kcm0vYW1kL2FtZGtmZC9rZmRfa2VybmVsX3F1ZXVlX3ZpLmMKPj4+PiBpbmRl eCBmMWQ0ODI4MWUzMjIuLmIzYmVlMzk2NjFhYiAxMDA2NDQKPj4+PiAtLS0gYS9kcml2ZXJzL2dw dS9kcm0vYW1kL2FtZGtmZC9rZmRfa2VybmVsX3F1ZXVlX3ZpLmMKPj4+PiArKysgYi9kcml2ZXJz L2dwdS9kcm0vYW1kL2FtZGtmZC9rZmRfa2VybmVsX3F1ZXVlX3ZpLmMKPj4+PiBAQCAtMzcsMTUg KzM3LDE2IEBAIHN0YXRpYyBib29sIGluaXRpYWxpemVfdmkoc3RydWN0IGtlcm5lbF9xdWV1ZSAq a3EsIHN0cnVjdCBrZmRfZGV2ICpkZXYsCj4+Pj4gICAgICAgICAgICAgICAgICAgICAgICAgZW51 bSBrZmRfcXVldWVfdHlwZSB0eXBlLCB1bnNpZ25lZCBpbnQgcXVldWVfc2l6ZSkKPj4+PiAgewo+ Pj4+ICAgICAgICAgaW50IHJldHZhbDsKPj4+PiArICAgICAgIHVuc2lnbmVkIGludCBzaXplID0g QUxJR04ocXVldWVfc2l6ZSwgUEFHRV9TSVpFKTsKPj4+Pgo+Pj4+IC0gICAgICAgcmV0dmFsID0g a2ZkX2d0dF9zYV9hbGxvY2F0ZShkZXYsIFBBR0VfU0laRSwgJmtxLT5lb3BfbWVtKTsKPj4+PiAr ICAgICAgIHJldHZhbCA9IGtmZF9ndHRfc2FfYWxsb2NhdGUoZGV2LCBzaXplLCAma3EtPmVvcF9t ZW0pOwo+Pj4+ICAgICAgICAgaWYgKHJldHZhbCAhPSAwKQo+Pj4+ICAgICAgICAgICAgICAgICBy ZXR1cm4gZmFsc2U7Cj4+Pj4KPj4+PiAgICAgICAgIGtxLT5lb3BfZ3B1X2FkZHIgPSBrcS0+ZW9w X21lbS0+Z3B1X2FkZHI7Cj4+Pj4gICAgICAgICBrcS0+ZW9wX2tlcm5lbF9hZGRyID0ga3EtPmVv cF9tZW0tPmNwdV9wdHI7Cj4+Pj4KPj4+PiAtICAgICAgIG1lbXNldChrcS0+ZW9wX2tlcm5lbF9h ZGRyLCAwLCBQQUdFX1NJWkUpOwo+Pj4+ICsgICAgICAgbWVtc2V0KGtxLT5lb3Bfa2VybmVsX2Fk ZHIsIDAsIHNpemUpOwo+Pj4+Cj4+Pj4gICAgICAgICByZXR1cm4gdHJ1ZTsKPj4+PiAgfQo+Pj4+ IC0tCj4+Pj4gMi4xMy42Cj4+Pj4KPj4+PiBfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fXwo+Pj4+IGFtZC1nZnggbWFpbGluZyBsaXN0Cj4+Pj4gYW1kLWdmeEBs aXN0cy5mcmVlZGVza3RvcC5vcmcKPj4+PiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2FtZC1nZngKPj4+IFRoYW5rcyEKPj4+IEFwcGxpZWQgdG8gLW5leHQg dHJlZQo+Pj4gT2RlZAo+Pj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX18KPj4+IGFtZC1nZnggbWFpbGluZyBsaXN0Cj4+PiBhbWQtZ2Z4QGxpc3RzLmZyZWVk ZXNrdG9wLm9yZwo+Pj4gaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0 aW5mby9hbWQtZ2Z4CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fXwphbWQtZ2Z4IG1haWxpbmcgbGlzdAphbWQtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpo dHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2FtZC1nZngK