From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Zhang, Jerry (Junwei)" Subject: Re: [PATCH 05/15] drm/amdgpu: handle multi level PD size calculation Date: Tue, 28 Mar 2017 10:41:13 +0800 Message-ID: <58D9CD49.1040809@amd.com> References: <1490594005-14553-1-git-send-email-David1.Zhou@amd.com> <1490594005-14553-6-git-send-email-David1.Zhou@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1490594005-14553-6-git-send-email-David1.Zhou-5C7GfCeVMHo@public.gmane.org> List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "amd-gfx" To: Chunming Zhou , amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, Christian.Koenig-5C7GfCeVMHo@public.gmane.org, Alexander.Deucher-5C7GfCeVMHo@public.gmane.org T24gMDMvMjcvMjAxNyAwMTo1MyBQTSwgQ2h1bm1pbmcgWmhvdSB3cm90ZToKPiBGcm9tOiBDaHJp c3RpYW4gS8O2bmlnIDxjaHJpc3RpYW4ua29lbmlnQGFtZC5jb20+Cj4KPiBBbGxvd3MgdXMgdG8g Z2V0IHRoZSBzaXplIGZvciBhbGwgbGV2ZWxzIGFzIHdlbGwuCj4KPiBDaGFuZ2UtSWQ6IElhZjJm OWIyYmYxOWMzNjIzMDE4YTIyMTVmOGNmMDFhNjFiZGJlOGVhCj4gU2lnbmVkLW9mZi1ieTogQ2hy aXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFuLmtvZW5pZ0BhbWQuY29tPgo+IFJldmlld2VkLWJ5OiBB bGV4IERldWNoZXIgPGFsZXhhbmRlci5kZXVjaGVyQGFtZC5jb20+Cj4gLS0tCj4gICBkcml2ZXJz L2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfdm0uYyB8IDM0ICsrKysrKysrKysrKysrKysrKysr KystLS0tLS0tLS0tLS0KPiAgIDEgZmlsZSBjaGFuZ2VkLCAyMiBpbnNlcnRpb25zKCspLCAxMiBk ZWxldGlvbnMoLSkKPgo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9h bWRncHVfdm0uYyBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV92bS5jCj4gaW5k ZXggOTE3Mjk1NC4uOTA0OTRjZSAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vYW1kL2Ft ZGdwdS9hbWRncHVfdm0uYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdw dV92bS5jCj4gQEAgLTc2LDI3ICs3NiwzNyBAQCBzdHJ1Y3QgYW1kZ3B1X3B0ZV91cGRhdGVfcGFy YW1zIHsKPiAgIH07Cj4KPiAgIC8qKgo+IC0gKiBhbWRncHVfdm1fbnVtX3BkZSAtIHJldHVybiB0 aGUgbnVtYmVyIG9mIHBhZ2UgZGlyZWN0b3J5IGVudHJpZXMKPiArICogYW1kZ3B1X3ZtX251bV9l bnRyaWVzIC0gcmV0dXJuIHRoZSBudW1iZXIgb2YgZW50cmllcyBpbiBhIFBEL1BUCj4gICAgKgo+ ICAgICogQGFkZXY6IGFtZGdwdV9kZXZpY2UgcG9pbnRlcgo+ICAgICoKPiAtICogQ2FsY3VsYXRl IHRoZSBudW1iZXIgb2YgcGFnZSBkaXJlY3RvcnkgZW50cmllcy4KPiArICogQ2FsY3VsYXRlIHRo ZSBudW1iZXIgb2YgZW50cmllcyBpbiBhIHBhZ2UgZGlyZWN0b3J5IG9yIHBhZ2UgdGFibGUuCj4g ICAgKi8KPiAtc3RhdGljIHVuc2lnbmVkIGFtZGdwdV92bV9udW1fcGRlcyhzdHJ1Y3QgYW1kZ3B1 X2RldmljZSAqYWRldikKPiArc3RhdGljIHVuc2lnbmVkIGFtZGdwdV92bV9udW1fZW50cmllcyhz dHJ1Y3QgYW1kZ3B1X2RldmljZSAqYWRldiwKPiArCQkJCSAgICAgIHVuc2lnbmVkIGxldmVsKQo+ ICAgewo+IC0JcmV0dXJuIGFkZXYtPnZtX21hbmFnZXIubWF4X3BmbiA+PiBhbWRncHVfdm1fYmxv Y2tfc2l6ZTsKPiArCWlmIChsZXZlbCA9PSAwKQo+ICsJCS8qIEZvciB0aGUgcm9vdCBkaXJlY3Rv cnkgKi8KPiArCQlyZXR1cm4gYWRldi0+dm1fbWFuYWdlci5tYXhfcGZuID4+Cj4gKwkJCShhbWRn cHVfdm1fYmxvY2tfc2l6ZSAqIGFkZXYtPnZtX21hbmFnZXIubnVtX2xldmVsKTsKPiArCWVsc2Ug aWYgKGxldmVsID09IGFkZXYtPnZtX21hbmFnZXIubnVtX2xldmVsKQo+ICsJCS8qIEZvciB0aGUg cGFnZSB0YWJsZXMgb24gdGhlIGxlYXZlcyAqLwo+ICsJCXJldHVybiBBTURHUFVfVk1fUFRFX0NP VU5UOwo+ICsJZWxzZQo+ICsJCS8qIEV2ZXJ5dGhpbmcgaW4gYmV0d2VlbiAqLwo+ICsJCXJldHVy biAxIDw8IGFtZGdwdV92bV9ibG9ja19zaXplOwoKV2UgbWF5IHVzZSBBTURHUFVfVk1fUFRFX0NP VU5UIGRpcmVjdGx5LgoKQlRXLCBlYWNoIFBUIHNpemUgaXMgc2FtZSBhcyBhbWRncHVfdm1fYmxv Y2tfc2l6ZS4KCkplcnJ5Cgo+ICAgfQo+Cj4gICAvKioKPiAtICogYW1kZ3B1X3ZtX2RpcmVjdG9y eV9zaXplIC0gcmV0dXJucyB0aGUgc2l6ZSBvZiB0aGUgcGFnZSBkaXJlY3RvcnkgaW4gYnl0ZXMK PiArICogYW1kZ3B1X3ZtX2JvX3NpemUgLSByZXR1cm5zIHRoZSBzaXplIG9mIHRoZSBCT3MgaW4g Ynl0ZXMKPiAgICAqCj4gICAgKiBAYWRldjogYW1kZ3B1X2RldmljZSBwb2ludGVyCj4gICAgKgo+ IC0gKiBDYWxjdWxhdGUgdGhlIHNpemUgb2YgdGhlIHBhZ2UgZGlyZWN0b3J5IGluIGJ5dGVzLgo+ ICsgKiBDYWxjdWxhdGUgdGhlIHNpemUgb2YgdGhlIEJPIGZvciBhIHBhZ2UgZGlyZWN0b3J5IG9y IHBhZ2UgdGFibGUgaW4gYnl0ZXMuCj4gICAgKi8KPiAtc3RhdGljIHVuc2lnbmVkIGFtZGdwdV92 bV9kaXJlY3Rvcnlfc2l6ZShzdHJ1Y3QgYW1kZ3B1X2RldmljZSAqYWRldikKPiArc3RhdGljIHVu c2lnbmVkIGFtZGdwdV92bV9ib19zaXplKHN0cnVjdCBhbWRncHVfZGV2aWNlICphZGV2LCB1bnNp Z25lZCBsZXZlbCkKPiAgIHsKPiAtCXJldHVybiBBTURHUFVfR1BVX1BBR0VfQUxJR04oYW1kZ3B1 X3ZtX251bV9wZGVzKGFkZXYpICogOCk7Cj4gKwlyZXR1cm4gQU1ER1BVX0dQVV9QQUdFX0FMSUdO KGFtZGdwdV92bV9udW1fZW50cmllcyhhZGV2LCBsZXZlbCkgKiA4KTsKPiAgIH0KPgo+ICAgLyoq Cj4gQEAgLTEzOTMsNyArMTQwMyw3IEBAIGludCBhbWRncHVfdm1fYm9fbWFwKHN0cnVjdCBhbWRn cHVfZGV2aWNlICphZGV2LAo+ICAgCXNhZGRyID4+PSBhbWRncHVfdm1fYmxvY2tfc2l6ZTsKPiAg IAllYWRkciA+Pj0gYW1kZ3B1X3ZtX2Jsb2NrX3NpemU7Cj4KPiAtCUJVR19PTihlYWRkciA+PSBh bWRncHVfdm1fbnVtX3BkZXMoYWRldikpOwo+ICsJQlVHX09OKGVhZGRyID49IGFtZGdwdV92bV9u dW1fZW50cmllcyhhZGV2LCAwKSk7Cj4KPiAgIAlpZiAoZWFkZHIgPiB2bS0+cm9vdC5sYXN0X2Vu dHJ5X3VzZWQpCj4gICAJCXZtLT5yb290Lmxhc3RfZW50cnlfdXNlZCA9IGVhZGRyOwo+IEBAIC0x NTc2LDggKzE1ODYsOCBAQCBpbnQgYW1kZ3B1X3ZtX2luaXQoc3RydWN0IGFtZGdwdV9kZXZpY2Ug KmFkZXYsIHN0cnVjdCBhbWRncHVfdm0gKnZtKQo+ICAgCUlOSVRfTElTVF9IRUFEKCZ2bS0+Y2xl YXJlZCk7Cj4gICAJSU5JVF9MSVNUX0hFQUQoJnZtLT5mcmVlZCk7Cj4KPiAtCXBkX3NpemUgPSBh bWRncHVfdm1fZGlyZWN0b3J5X3NpemUoYWRldik7Cj4gLQlwZF9lbnRyaWVzID0gYW1kZ3B1X3Zt X251bV9wZGVzKGFkZXYpOwo+ICsJcGRfc2l6ZSA9IGFtZGdwdV92bV9ib19zaXplKGFkZXYsIDAp Owo+ICsJcGRfZW50cmllcyA9IGFtZGdwdV92bV9udW1fZW50cmllcyhhZGV2LCAwKTsKPgo+ICAg CS8qIGFsbG9jYXRlIHBhZ2UgdGFibGUgYXJyYXkgKi8KPiAgIAl2bS0+cm9vdC5lbnRyaWVzID0g ZHJtX2NhbGxvY19sYXJnZShwZF9lbnRyaWVzLAo+IEBAIC0xNjYyLDcgKzE2NzIsNyBAQCB2b2lk IGFtZGdwdV92bV9maW5pKHN0cnVjdCBhbWRncHVfZGV2aWNlICphZGV2LCBzdHJ1Y3QgYW1kZ3B1 X3ZtICp2bSkKPiAgIAkJa2ZyZWUobWFwcGluZyk7Cj4gICAJfQo+Cj4gLQlmb3IgKGkgPSAwOyBp IDwgYW1kZ3B1X3ZtX251bV9wZGVzKGFkZXYpOyBpKyspIHsKPiArCWZvciAoaSA9IDA7IGkgPCBh bWRncHVfdm1fbnVtX2VudHJpZXMoYWRldiwgMCk7IGkrKykgewo+ICAgCQlzdHJ1Y3QgYW1kZ3B1 X2JvICpwdCA9IHZtLT5yb290LmVudHJpZXNbaV0uYm87Cj4KPiAgIAkJaWYgKCFwdCkKPgpfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwphbWQtZ2Z4IG1haWxp bmcgbGlzdAphbWQtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVk ZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2FtZC1nZngK