From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v3) Date: Thu, 6 Jun 2019 14:12:22 +0200 Message-ID: <0ad90e37-afd1-9328-bd2a-2c37e6826a30@gmail.com> References: <20190606105023.13750-1-tom.stdenis@amd.com> <3fd2e057-0284-2bc7-7b76-8280646aad52@gmail.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: 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: "StDenis, Tom" , "Koenig, Christian" , "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" QW0gMDYuMDYuMTkgdW0gMTM6NTEgc2NocmllYiBTdERlbmlzLCBUb206Cj4gT24gMjAxOS0wNi0w NiA3OjQ5IGEubS4sIENocmlzdGlhbiBLw7ZuaWcgd3JvdGU6Cj4+IEFtIDA2LjA2LjE5IHVtIDEy OjUwIHNjaHJpZWIgU3REZW5pcywgVG9tOgo+Pj4gVGhpcyBvcHRpb24gaXMgbm8gbG9uZ2VyIG5l ZWRlZC7CoCBUaGUgZGVmYXVsdCBjb2RlIHBhdGhzCj4+PiBhcmUgbm93IHRoZSBvbmx5IG9wdGlv bi4KPj4+Cj4+PiB2MjogQWRkIEhQQUdFIHN1cHBvcnQgYW5kIGEgZGVmYXVsdCBmb3Igbm9uIGNv bnRpZ3VvdXMgbWFwcwo+Pj4gdjM6IE1pc3JlYWQgNTEyIHBhZ2VzIGFzIE1pQiAuLi4KPj4+Cj4+ PiBTaWduZWQtb2ZmLWJ5OiBUb20gU3QgRGVuaXMgPHRvbS5zdGRlbmlzQGFtZC5jb20+Cj4+PiAt LS0KPj4+ICDCoCBkcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHUuaMKgwqDCoMKgwqDC oMKgwqDCoCB8wqAgMSAtCj4+PiAgwqAgZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1 X2RldmljZS5jwqDCoCB8wqAgNyAtLS0tLS0tCj4+PiAgwqAgZHJpdmVycy9ncHUvZHJtL2FtZC9h bWRncHUvYW1kZ3B1X2Rydi5jwqDCoMKgwqDCoCB8wqAgOCAtLS0tLS0tLQo+Pj4gIMKgIGRyaXZl cnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV92cmFtX21nci5jIHwgMTQgKysrKysrKysrLS0t LS0KPj4+ICDCoCA0IGZpbGVzIGNoYW5nZWQsIDkgaW5zZXJ0aW9ucygrKSwgMjEgZGVsZXRpb25z KC0pCj4+Pgo+Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdw dS5oCj4+PiBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdS5oCj4+PiBpbmRleCAx OWEwMDI4MmUzNGMuLmU1NGYzMTg2NWY2MCAxMDA2NDQKPj4+IC0tLSBhL2RyaXZlcnMvZ3B1L2Ry bS9hbWQvYW1kZ3B1L2FtZGdwdS5oCj4+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdw dS9hbWRncHUuaAo+Pj4gQEAgLTE0Miw3ICsxNDIsNiBAQCBleHRlcm4gdWludCBhbWRncHVfc2Rt YV9waGFzZV9xdWFudHVtOwo+Pj4gIMKgIGV4dGVybiBjaGFyICphbWRncHVfZGlzYWJsZV9jdTsK Pj4+ICDCoCBleHRlcm4gY2hhciAqYW1kZ3B1X3ZpcnR1YWxfZGlzcGxheTsKPj4+ICDCoCBleHRl cm4gdWludCBhbWRncHVfcHBfZmVhdHVyZV9tYXNrOwo+Pj4gLWV4dGVybiBpbnQgYW1kZ3B1X3Zy YW1fcGFnZV9zcGxpdDsKPj4+ICDCoCBleHRlcm4gaW50IGFtZGdwdV9uZ2c7Cj4+PiAgwqAgZXh0 ZXJuIGludCBhbWRncHVfcHJpbV9idWZfcGVyX3NlOwo+Pj4gIMKgIGV4dGVybiBpbnQgYW1kZ3B1 X3Bvc19idWZfcGVyX3NlOwo+Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1k Z3B1L2FtZGdwdV9kZXZpY2UuYwo+Pj4gYi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRn cHVfZGV2aWNlLmMKPj4+IGluZGV4IDAyMTJjOWVlMzE3Yy4uMmUxM2I4ZWY2NjgxIDEwMDY0NAo+ Pj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2RldmljZS5jCj4+PiAr KysgYi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfZGV2aWNlLmMKPj4+IEBAIC05 NzMsMTMgKzk3Myw2IEBAIHN0YXRpYyBpbnQgYW1kZ3B1X2RldmljZV9jaGVja19hcmd1bWVudHMo c3RydWN0Cj4+PiBhbWRncHVfZGV2aWNlICphZGV2KQo+Pj4gIMKgIMKgwqDCoMKgwqAgYW1kZ3B1 X2RldmljZV9jaGVja19ibG9ja19zaXplKGFkZXYpOwo+Pj4gIMKgIC3CoMKgwqAgaWYgKGFtZGdw dV92cmFtX3BhZ2Vfc3BsaXQgIT0gLTEgJiYgKGFtZGdwdV92cmFtX3BhZ2Vfc3BsaXQgPAo+Pj4g MTYgfHwKPj4+IC3CoMKgwqDCoMKgwqDCoCAhaXNfcG93ZXJfb2ZfMihhbWRncHVfdnJhbV9wYWdl X3NwbGl0KSkpIHsKPj4+IC3CoMKgwqDCoMKgwqDCoCBkZXZfd2FybihhZGV2LT5kZXYsICJpbnZh bGlkIFZSQU0gcGFnZSBzcGxpdCAoJWQpXG4iLAo+Pj4gLcKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oCBhbWRncHVfdnJhbV9wYWdlX3NwbGl0KTsKPj4+IC3CoMKgwqDCoMKgwqDCoCBhbWRncHVfdnJh bV9wYWdlX3NwbGl0ID0gMTAyNDsKPj4+IC3CoMKgwqAgfQo+Pj4gLQo+Pj4gIMKgwqDCoMKgwqAg cmV0ID0gYW1kZ3B1X2RldmljZV9nZXRfam9iX3RpbWVvdXRfc2V0dGluZ3MoYWRldik7Cj4+PiAg wqDCoMKgwqDCoCBpZiAocmV0KSB7Cj4+PiAgwqDCoMKgwqDCoMKgwqDCoMKgIGRldl9lcnIoYWRl di0+ZGV2LCAiaW52YWxpZCBsb2NrdXBfdGltZW91dCBwYXJhbWV0ZXIKPj4+IHN5bnRheFxuIik7 Cj4+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2Rydi5j Cj4+PiBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV9kcnYuYwo+Pj4gaW5kZXgg MWYzOGQ2ZmMxZmUzLi5lZjIyYTJhMjU0NDggMTAwNjQ0Cj4+PiAtLS0gYS9kcml2ZXJzL2dwdS9k cm0vYW1kL2FtZGdwdS9hbWRncHVfZHJ2LmMKPj4+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9hbWQv YW1kZ3B1L2FtZGdwdV9kcnYuYwo+Pj4gQEAgLTEwOCw3ICsxMDgsNiBAQCBpbnQgYW1kZ3B1X3Zt X2ZyYWdtZW50X3NpemUgPSAtMTsKPj4+ICDCoCBpbnQgYW1kZ3B1X3ZtX2Jsb2NrX3NpemUgPSAt MTsKPj4+ICDCoCBpbnQgYW1kZ3B1X3ZtX2ZhdWx0X3N0b3AgPSAwOwo+Pj4gIMKgIGludCBhbWRn cHVfdm1fZGVidWcgPSAwOwo+Pj4gLWludCBhbWRncHVfdnJhbV9wYWdlX3NwbGl0ID0gNTEyOwo+ Pj4gIMKgIGludCBhbWRncHVfdm1fdXBkYXRlX21vZGUgPSAtMTsKPj4+ICDCoCBpbnQgYW1kZ3B1 X2V4cF9od19zdXBwb3J0ID0gMDsKPj4+ICDCoCBpbnQgYW1kZ3B1X2RjID0gLTE7Cj4+PiBAQCAt MzQyLDEzICszNDEsNiBAQCBtb2R1bGVfcGFyYW1fbmFtZWQodm1fZGVidWcsIGFtZGdwdV92bV9k ZWJ1ZywKPj4+IGludCwgMDY0NCk7Cj4+PiAgwqAgTU9EVUxFX1BBUk1fREVTQyh2bV91cGRhdGVf bW9kZSwgIlZNIHVwZGF0ZSB1c2luZyBDUFUgKDAgPSBuZXZlcgo+Pj4gKGRlZmF1bHQgZXhjZXB0 IGZvciBsYXJnZSBCQVIoTEIpKSwgMSA9IEdyYXBoaWNzIG9ubHksIDIgPSBDb21wdXRlCj4+PiBv bmx5IChkZWZhdWx0IGZvciBMQiksIDMgPSBCb3RoIik7Cj4+PiAgwqAgbW9kdWxlX3BhcmFtX25h bWVkKHZtX3VwZGF0ZV9tb2RlLCBhbWRncHVfdm1fdXBkYXRlX21vZGUsIGludCwgMDQ0NCk7Cj4+ PiAgwqAgLS8qKgo+Pj4gLSAqIERPQzogdnJhbV9wYWdlX3NwbGl0IChpbnQpCj4+PiAtICogT3Zl cnJpZGUgdGhlIG51bWJlciBvZiBwYWdlcyBhZnRlciB3ZSBzcGxpdCBWUkFNIGFsbG9jYXRpb25z Cj4+PiAoZGVmYXVsdCA1MTIsIC0xID0gZGlzYWJsZSkuIFRoZSBkZWZhdWx0IGlzIDUxMi4KPj4+ IC0gKi8KPj4+IC1NT0RVTEVfUEFSTV9ERVNDKHZyYW1fcGFnZV9zcGxpdCwgIk51bWJlciBvZiBw YWdlcyBhZnRlciB3ZSBzcGxpdAo+Pj4gVlJBTSBhbGxvY2F0aW9ucyAoZGVmYXVsdCA1MTIsIC0x ID0gZGlzYWJsZSkiKTsKPj4+IC1tb2R1bGVfcGFyYW1fbmFtZWQodnJhbV9wYWdlX3NwbGl0LCBh bWRncHVfdnJhbV9wYWdlX3NwbGl0LCBpbnQsIDA0NDQpOwo+Pj4gLQo+Pj4gIMKgIC8qKgo+Pj4g IMKgwqAgKiBET0M6IGV4cF9od19zdXBwb3J0IChpbnQpCj4+PiAgwqDCoCAqIEVuYWJsZSBleHBl cmltZW50YWwgaHcgc3VwcG9ydCAoMSA9IGVuYWJsZSkuIFRoZSBkZWZhdWx0IGlzIDAKPj4+IChk aXNhYmxlZCkuCj4+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1k Z3B1X3ZyYW1fbWdyLmMKPj4+IGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X3Zy YW1fbWdyLmMKPj4+IGluZGV4IGVjOWVhM2ZkYmI0YS4uOGFlYTJmMjFiMjAyIDEwMDY0NAo+Pj4g LS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X3ZyYW1fbWdyLmMKPj4+ICsr KyBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV92cmFtX21nci5jCj4+PiBAQCAt Mjg0LDE3ICsyODQsMjEgQEAgc3RhdGljIGludCBhbWRncHVfdnJhbV9tZ3JfbmV3KHN0cnVjdAo+ Pj4gdHRtX21lbV90eXBlX21hbmFnZXIgKm1hbiwKPj4+ICDCoMKgwqDCoMKgIGlmICghbHBmbikK Pj4+ICDCoMKgwqDCoMKgwqDCoMKgwqAgbHBmbiA9IG1hbi0+c2l6ZTsKPj4+ICDCoCAtwqDCoMKg IGlmIChwbGFjZS0+ZmxhZ3MgJiBUVE1fUExfRkxBR19DT05USUdVT1VTIHx8Cj4+PiAtwqDCoMKg wqDCoMKgwqAgYW1kZ3B1X3ZyYW1fcGFnZV9zcGxpdCA9PSAtMSkgewo+Pj4gK8KgwqDCoCBpZiAo cGxhY2UtPmZsYWdzICYgVFRNX1BMX0ZMQUdfQ09OVElHVU9VUykgewo+Pj4gIMKgwqDCoMKgwqDC oMKgwqDCoCBwYWdlc19wZXJfbm9kZSA9IH4wdWw7Cj4+PiAgwqDCoMKgwqDCoMKgwqDCoMKgIG51 bV9ub2RlcyA9IDE7Cj4+PiAgwqDCoMKgwqDCoCB9IGVsc2Ugewo+Pj4gLcKgwqDCoMKgwqDCoMKg IHBhZ2VzX3Blcl9ub2RlID0gbWF4KCh1aW50MzJfdClhbWRncHVfdnJhbV9wYWdlX3NwbGl0LAo+ Pj4gLcKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgbWVtLT5wYWdlX2Fs aWdubWVudCk7Cj4+PiArI2lmZGVmIENPTkZJR19UUkFOU1BBUkVOVF9IVUdFUEFHRQo+Pj4gK8Kg wqDCoMKgwqDCoMKgIHBhZ2VzX3Blcl9ub2RlID0gSFBBR0VfUE1EX05SOwo+Pj4gKyNlbHNlCj4+ PiArwqDCoMKgwqDCoMKgwqAgLyogZGVmYXVsdCB0byAyTUIgKi8KPj4+ICvCoMKgwqDCoMKgwqDC oCBwYWdlc19wZXJfbm9kZSA9ICgyVUwgPDwgKDIwVUwgLSBQQUdFX1NISUZUKSk7Cj4+PiArI2Vu ZGlmCj4+PiArwqDCoMKgwqDCoMKgwqAgcGFnZXNfcGVyX25vZGUgPSBtYXgoKHVpbnQzMl90KXBh Z2VzX3Blcl9ub2RlLAo+Pj4gbWVtLT5wYWdlX2FsaWdubWVudCk7Cj4+PiAgwqDCoMKgwqDCoMKg wqDCoMKgIG51bV9ub2RlcyA9IERJVl9ST1VORF9VUChtZW0tPm51bV9wYWdlcywgcGFnZXNfcGVy X25vZGUpOwo+Pj4gIMKgwqDCoMKgwqAgfQo+Pj4gIMKgIC3CoMKgwqAgbm9kZXMgPSBrdm1hbGxv Y19hcnJheShudW1fbm9kZXMsIHNpemVvZigqbm9kZXMpLAo+Pj4gK8KgwqDCoCBub2RlcyA9IGt2 bWFsbG9jX2FycmF5KCh1aW50MzJfdCludW1fbm9kZXMsIHNpemVvZigqbm9kZXMpLAo+PiBZb3Ug Y2FuIHByb2JhYmx5IGRyb3AgdGhhdCBjYXN0IGhlcmUsIGFwYXJ0IGZyb20gdGhhdCB0aGUgcGF0 Y2ggaXMKPj4gUmV2aWV3ZWQtYnk6IENocmlzdGlhbiBLw7ZuaWcgPGNocmlzdGlhbi5rb2VuaWdA YW1kLmNvbT4uCj4KPiBXaXRob3V0IHRoZSBjYXN0IGl0IHByb2R1Y2VzIGEgc29tZXdoYXQgbGVu Z3RoeSB3YXJuaW5nIGRpYWdub3N0aWMuwqAgVGhlCj4gb3JpZ2luYWwgY29kZSBoYWQgdGhlIGNh c3QgYXMgd2VsbC4KClN0cmFuZ2UsIHRoZSBvcmlnaW5hbCBjb2RlIGhhdCB0aGUgY2FzZSBiZWNh dXNlIGFtZGdwdV92cmFtX3BhZ2Vfc3BsaXQgCndhcyBhIHNpZ25lZCBpbnQsIGJ1dCBudW1fbm9k ZXMgc2hvdWxkIGJlIHVuc2lnbmVkIGFueXdheS4KClNvIHRoYXQgc2hvdWxkbid0IGJlIG5lY2Vz c2FyeSBhbnkgbW9yZS4gQW55d2F5IGtlZXBpbmcgdGhlIGNhc3Qgd29ya3MgCmZvciBtZSBhcyB3 ZWxsLgoKPj4gRG8geW91IHdhbnQgdG8gbG9vayBpbnRvIHRlc3QvYmVuY2htYXJrIGFzIHdlbGw/ Cj4gU3VyZSwgaXMgdGhlIHBsYW4gdG8gY2xlYXZlIGFsbCB0aGF0IG91dCB0b28/CgpXZWxsIGZp cnN0IG9mIGFsbCBkaWcgYSBiaXQgaW50byB3aGF0IHRob3NlIHN0aWxsIGRvLiBJZiBJJ20gbm90 IApjb21wbGV0ZWx5IG1pc3Rha2VuIGF0IGxlYXN0IG9uZSBvZiB0aGVtIGlzIHJlZHVuZGFudCBu b3cuCgpDaHJpc3RpYW4uCgo+Cj4KPiBUaGFua3MsCj4KPiBUb20KPgo+Cj4gX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPiBhbWQtZ2Z4IG1haWxpbmcgbGlz dAo+IGFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCj4gaHR0cHM6Ly9saXN0cy5mcmVlZGVz a3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9hbWQtZ2Z4CgpfX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fXwphbWQtZ2Z4IG1haWxpbmcgbGlzdAphbWQtZ2Z4QGxp c3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFu L2xpc3RpbmZvL2FtZC1nZng=