From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH] drm/amdgpu: update ib_start/size_alignment same as windows used Date: Fri, 15 Jun 2018 09:09:02 +0200 Message-ID: References: <20180615064542.16504-1-david1.zhou@amd.com> Reply-To: christian.koenig-5C7GfCeVMHo@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20180615064542.16504-1-david1.zhou-5C7GfCeVMHo@public.gmane.org> Content-Language: en-US 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 Cc: Marek.Olsak-5C7GfCeVMHo@public.gmane.org, Sean.Ryan-5C7GfCeVMHo@public.gmane.org QW0gMTUuMDYuMjAxOCB1bSAwODo0NSBzY2hyaWViIENodW5taW5nIFpob3U6Cj4gUEFHRV9TSVpF IGZvciBzdGFydF9hbGlnbm1lbnQgaXMgZmFyIG11Y2ggdGhhbiBodyByZXF1aXJlbWVudCwKPiBB bmQgbm93LCB1cGRhdGUgdG8gZXhwZXJlaW5jZSB2YWx1ZSBmcm9tIHdpbmRvdyBzaWRlLgo+Cj4g Q2hhbmdlLUlkOiBJMDhhN2U3MjA3NjM4NmMzMmZhZjM2ZWM0ODEyYjMwZTY4ZGRlMjNlNQo+IFNp Z25lZC1vZmYtYnk6IENodW5taW5nIFpob3UgPGRhdmlkMS56aG91QGFtZC5jb20+CgpBY2tlZC1i eTogQ2hyaXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFuLmtvZW5pZ0BhbWQuY29tPgoKPiAtLS0KPiAg IGRyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV9rbXMuYyB8IDI4ICsrKysrKysrKysr KysrLS0tLS0tLS0tLS0tLS0KPiAgIDEgZmlsZSBjaGFuZ2VkLCAxNCBpbnNlcnRpb25zKCspLCAx NCBkZWxldGlvbnMoLSkKPgo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdw dS9hbWRncHVfa21zLmMgYi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfa21zLmMK PiBpbmRleCAzOTJkZDI0ZTgzZjUuLmQwNDFkZGRhYWQwYyAxMDA2NDQKPiAtLS0gYS9kcml2ZXJz L2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfa21zLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0v YW1kL2FtZGdwdS9hbWRncHVfa21zLmMKPiBAQCAtMzI5LDM1ICszMjksMzUgQEAgc3RhdGljIGlu dCBhbWRncHVfaW5mb19pb2N0bChzdHJ1Y3QgZHJtX2RldmljZSAqZGV2LCB2b2lkICpkYXRhLCBz dHJ1Y3QgZHJtX2ZpbGUKPiAgIAkJCXR5cGUgPSBBTURfSVBfQkxPQ0tfVFlQRV9HRlg7Cj4gICAJ CQlmb3IgKGkgPSAwOyBpIDwgYWRldi0+Z2Z4Lm51bV9nZnhfcmluZ3M7IGkrKykKPiAgIAkJCQly aW5nX21hc2sgfD0gKChhZGV2LT5nZnguZ2Z4X3JpbmdbaV0ucmVhZHkgPyAxIDogMCkgPDwgaSk7 Cj4gLQkJCWliX3N0YXJ0X2FsaWdubWVudCA9IEFNREdQVV9HUFVfUEFHRV9TSVpFOwo+IC0JCQlp Yl9zaXplX2FsaWdubWVudCA9IDg7Cj4gKwkJCWliX3N0YXJ0X2FsaWdubWVudCA9IDMyOwo+ICsJ CQlpYl9zaXplX2FsaWdubWVudCA9IDMyOwo+ICAgCQkJYnJlYWs7Cj4gICAJCWNhc2UgQU1ER1BV X0hXX0lQX0NPTVBVVEU6Cj4gICAJCQl0eXBlID0gQU1EX0lQX0JMT0NLX1RZUEVfR0ZYOwo+ICAg CQkJZm9yIChpID0gMDsgaSA8IGFkZXYtPmdmeC5udW1fY29tcHV0ZV9yaW5nczsgaSsrKQo+ICAg CQkJCXJpbmdfbWFzayB8PSAoKGFkZXYtPmdmeC5jb21wdXRlX3JpbmdbaV0ucmVhZHkgPyAxIDog MCkgPDwgaSk7Cj4gLQkJCWliX3N0YXJ0X2FsaWdubWVudCA9IEFNREdQVV9HUFVfUEFHRV9TSVpF Owo+IC0JCQlpYl9zaXplX2FsaWdubWVudCA9IDg7Cj4gKwkJCWliX3N0YXJ0X2FsaWdubWVudCA9 IDMyOwo+ICsJCQlpYl9zaXplX2FsaWdubWVudCA9IDMyOwo+ICAgCQkJYnJlYWs7Cj4gICAJCWNh c2UgQU1ER1BVX0hXX0lQX0RNQToKPiAgIAkJCXR5cGUgPSBBTURfSVBfQkxPQ0tfVFlQRV9TRE1B Owo+ICAgCQkJZm9yIChpID0gMDsgaSA8IGFkZXYtPnNkbWEubnVtX2luc3RhbmNlczsgaSsrKQo+ ICAgCQkJCXJpbmdfbWFzayB8PSAoKGFkZXYtPnNkbWEuaW5zdGFuY2VbaV0ucmluZy5yZWFkeSA/ IDEgOiAwKSA8PCBpKTsKPiAtCQkJaWJfc3RhcnRfYWxpZ25tZW50ID0gQU1ER1BVX0dQVV9QQUdF X1NJWkU7Cj4gLQkJCWliX3NpemVfYWxpZ25tZW50ID0gMTsKPiArCQkJaWJfc3RhcnRfYWxpZ25t ZW50ID0gMjU2Owo+ICsJCQlpYl9zaXplX2FsaWdubWVudCA9IDQ7Cj4gICAJCQlicmVhazsKPiAg IAkJY2FzZSBBTURHUFVfSFdfSVBfVVZEOgo+ICAgCQkJdHlwZSA9IEFNRF9JUF9CTE9DS19UWVBF X1VWRDsKPiAgIAkJCWZvciAoaSA9IDA7IGkgPCBhZGV2LT51dmQubnVtX3V2ZF9pbnN0OyBpKysp Cj4gICAJCQkJcmluZ19tYXNrIHw9ICgoYWRldi0+dXZkLmluc3RbaV0ucmluZy5yZWFkeSA/IDEg OiAwKSA8PCBpKTsKPiAtCQkJaWJfc3RhcnRfYWxpZ25tZW50ID0gQU1ER1BVX0dQVV9QQUdFX1NJ WkU7Cj4gLQkJCWliX3NpemVfYWxpZ25tZW50ID0gMTY7Cj4gKwkJCWliX3N0YXJ0X2FsaWdubWVu dCA9IDY0Owo+ICsJCQlpYl9zaXplX2FsaWdubWVudCA9IDY0Owo+ICAgCQkJYnJlYWs7Cj4gICAJ CWNhc2UgQU1ER1BVX0hXX0lQX1ZDRToKPiAgIAkJCXR5cGUgPSBBTURfSVBfQkxPQ0tfVFlQRV9W Q0U7Cj4gICAJCQlmb3IgKGkgPSAwOyBpIDwgYWRldi0+dmNlLm51bV9yaW5nczsgaSsrKQo+ICAg CQkJCXJpbmdfbWFzayB8PSAoKGFkZXYtPnZjZS5yaW5nW2ldLnJlYWR5ID8gMSA6IDApIDw8IGkp Owo+IC0JCQlpYl9zdGFydF9hbGlnbm1lbnQgPSBBTURHUFVfR1BVX1BBR0VfU0laRTsKPiArCQkJ aWJfc3RhcnRfYWxpZ25tZW50ID0gNDsKPiAgIAkJCWliX3NpemVfYWxpZ25tZW50ID0gMTsKPiAg IAkJCWJyZWFrOwo+ICAgCQljYXNlIEFNREdQVV9IV19JUF9VVkRfRU5DOgo+IEBAIC0zNjcsMjYg KzM2NywyNiBAQCBzdGF0aWMgaW50IGFtZGdwdV9pbmZvX2lvY3RsKHN0cnVjdCBkcm1fZGV2aWNl ICpkZXYsIHZvaWQgKmRhdGEsIHN0cnVjdCBkcm1fZmlsZQo+ICAgCQkJCQlyaW5nX21hc2sgfD0K PiAgIAkJCQkJKChhZGV2LT51dmQuaW5zdFtpXS5yaW5nX2VuY1tqXS5yZWFkeSA/IDEgOiAwKSA8 PAo+ICAgCQkJCQkoaiArIGkgKiBhZGV2LT51dmQubnVtX2VuY19yaW5ncykpOwo+IC0JCQlpYl9z dGFydF9hbGlnbm1lbnQgPSBBTURHUFVfR1BVX1BBR0VfU0laRTsKPiAtCQkJaWJfc2l6ZV9hbGln bm1lbnQgPSAxOwo+ICsJCQlpYl9zdGFydF9hbGlnbm1lbnQgPSA2NDsKPiArCQkJaWJfc2l6ZV9h bGlnbm1lbnQgPSA2NDsKPiAgIAkJCWJyZWFrOwo+ICAgCQljYXNlIEFNREdQVV9IV19JUF9WQ05f REVDOgo+ICAgCQkJdHlwZSA9IEFNRF9JUF9CTE9DS19UWVBFX1ZDTjsKPiAgIAkJCXJpbmdfbWFz ayA9IGFkZXYtPnZjbi5yaW5nX2RlYy5yZWFkeSA/IDEgOiAwOwo+IC0JCQlpYl9zdGFydF9hbGln bm1lbnQgPSBBTURHUFVfR1BVX1BBR0VfU0laRTsKPiArCQkJaWJfc3RhcnRfYWxpZ25tZW50ID0g MTY7Cj4gICAJCQlpYl9zaXplX2FsaWdubWVudCA9IDE2Owo+ICAgCQkJYnJlYWs7Cj4gICAJCWNh c2UgQU1ER1BVX0hXX0lQX1ZDTl9FTkM6Cj4gICAJCQl0eXBlID0gQU1EX0lQX0JMT0NLX1RZUEVf VkNOOwo+ICAgCQkJZm9yIChpID0gMDsgaSA8IGFkZXYtPnZjbi5udW1fZW5jX3JpbmdzOyBpKysp Cj4gICAJCQkJcmluZ19tYXNrIHw9ICgoYWRldi0+dmNuLnJpbmdfZW5jW2ldLnJlYWR5ID8gMSA6 IDApIDw8IGkpOwo+IC0JCQlpYl9zdGFydF9hbGlnbm1lbnQgPSBBTURHUFVfR1BVX1BBR0VfU0la RTsKPiArCQkJaWJfc3RhcnRfYWxpZ25tZW50ID0gNjQ7Cj4gICAJCQlpYl9zaXplX2FsaWdubWVu dCA9IDE7Cj4gICAJCQlicmVhazsKPiAgIAkJY2FzZSBBTURHUFVfSFdfSVBfVkNOX0pQRUc6Cj4g ICAJCQl0eXBlID0gQU1EX0lQX0JMT0NLX1RZUEVfVkNOOwo+ICAgCQkJcmluZ19tYXNrID0gYWRl di0+dmNuLnJpbmdfanBlZy5yZWFkeSA/IDEgOiAwOwo+IC0JCQlpYl9zdGFydF9hbGlnbm1lbnQg PSBBTURHUFVfR1BVX1BBR0VfU0laRTsKPiArCQkJaWJfc3RhcnRfYWxpZ25tZW50ID0gMTY7Cj4g ICAJCQlpYl9zaXplX2FsaWdubWVudCA9IDE2Owo+ICAgCQkJYnJlYWs7Cj4gICAJCWRlZmF1bHQ6 CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwphbWQtZ2Z4 IG1haWxpbmcgbGlzdAphbWQtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3Rz LmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2FtZC1nZngK