From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH] drm/amd/amdgpu: Bail out of BO node creation if not enough VRAM (v3) Date: Wed, 12 Jun 2019 13:57:00 +0200 Message-ID: <122212cb-277c-5de5-56f9-3941beb77f09@gmail.com> References: <20190612110829.4699-1-tom.stdenis@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: <20190612110829.4699-1-tom.stdenis-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: "StDenis, Tom" , "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" QW0gMTIuMDYuMTkgdW0gMTM6MDggc2NocmllYiBTdERlbmlzLCBUb206Cj4gKHYyKTogUmV0dXJu IDAgYW5kIHNldCBtZW0tPm1tX25vZGUgdG8gTlVMTC4KPiAodjMpOiBVc2UgYXRvbWljNjRfYWRk X3JldHVybiBpbnN0ZWFkLgo+Cj4gU2lnbmVkLW9mZi1ieTogVG9tIFN0IERlbmlzIDx0b20uc3Rk ZW5pc0BhbWQuY29tPgoKUmV2aWV3ZWQtYnk6IENocmlzdGlhbiBLw7ZuaWcgPGNocmlzdGlhbi5r b2VuaWdAYW1kLmNvbT4KCj4gLS0tCj4gICBkcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRn cHVfdnJhbV9tZ3IuYyB8IDE3ICsrKysrKysrKysrKy0tLS0tCj4gICAxIGZpbGUgY2hhbmdlZCwg MTIgaW5zZXJ0aW9ucygrKSwgNSBkZWxldGlvbnMoLSkKPgo+IGRpZmYgLS1naXQgYS9kcml2ZXJz L2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfdnJhbV9tZ3IuYyBiL2RyaXZlcnMvZ3B1L2RybS9h bWQvYW1kZ3B1L2FtZGdwdV92cmFtX21nci5jCj4gaW5kZXggOGFlYTJmMjFiMjAyLi5jOTYzYWQ4 NjA3MmUgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X3Zy YW1fbWdyLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfdnJhbV9t Z3IuYwo+IEBAIC0yNzYsNyArMjc2LDcgQEAgc3RhdGljIGludCBhbWRncHVfdnJhbV9tZ3JfbmV3 KHN0cnVjdCB0dG1fbWVtX3R5cGVfbWFuYWdlciAqbWFuLAo+ICAgCXN0cnVjdCBkcm1fbW1fbm9k ZSAqbm9kZXM7Cj4gICAJZW51bSBkcm1fbW1faW5zZXJ0X21vZGUgbW9kZTsKPiAgIAl1bnNpZ25l ZCBsb25nIGxwZm4sIG51bV9ub2RlcywgcGFnZXNfcGVyX25vZGUsIHBhZ2VzX2xlZnQ7Cj4gLQl1 aW50NjRfdCB1c2FnZSA9IDAsIHZpc191c2FnZSA9IDA7Cj4gKwl1aW50NjRfdCB2aXNfdXNhZ2Ug PSAwOwo+ICAgCXVuc2lnbmVkIGk7Cj4gICAJaW50IHI7Cj4gICAKPiBAQCAtMjg0LDYgKzI4NCwx MyBAQCBzdGF0aWMgaW50IGFtZGdwdV92cmFtX21ncl9uZXcoc3RydWN0IHR0bV9tZW1fdHlwZV9t YW5hZ2VyICptYW4sCj4gICAJaWYgKCFscGZuKQo+ICAgCQlscGZuID0gbWFuLT5zaXplOwo+ICAg Cj4gKwkvKiBiYWlsIG91dCBxdWlja2x5IGlmIHRoZXJlJ3MgbGlrZWx5IG5vdCBlbm91Z2ggVlJB TSBmb3IgdGhpcyBCTyAqLwo+ICsJaWYgKGF0b21pYzY0X2FkZF9yZXR1cm4obWVtLT5udW1fcGFn ZXMgPDwgUEFHRV9TSElGVCwgJm1nci0+dXNhZ2UpID4gYWRldi0+Z21jLm1jX3ZyYW1fc2l6ZSkg ewo+ICsJCWF0b21pYzY0X3N1YihtZW0tPm51bV9wYWdlcyA8PCBQQUdFX1NISUZULCAmbWdyLT51 c2FnZSk7Cj4gKwkJbWVtLT5tbV9ub2RlID0gTlVMTDsKPiArCQlyZXR1cm4gMDsKPiArCX0KPiAr Cj4gICAJaWYgKHBsYWNlLT5mbGFncyAmIFRUTV9QTF9GTEFHX0NPTlRJR1VPVVMpIHsKPiAgIAkJ cGFnZXNfcGVyX25vZGUgPSB+MHVsOwo+ICAgCQludW1fbm9kZXMgPSAxOwo+IEBAIC0zMDAsOCAr MzA3LDEwIEBAIHN0YXRpYyBpbnQgYW1kZ3B1X3ZyYW1fbWdyX25ldyhzdHJ1Y3QgdHRtX21lbV90 eXBlX21hbmFnZXIgKm1hbiwKPiAgIAo+ICAgCW5vZGVzID0ga3ZtYWxsb2NfYXJyYXkoKHVpbnQz Ml90KW51bV9ub2Rlcywgc2l6ZW9mKCpub2RlcyksCj4gICAJCQkgICAgICAgR0ZQX0tFUk5FTCB8 IF9fR0ZQX1pFUk8pOwo+IC0JaWYgKCFub2RlcykKPiArCWlmICghbm9kZXMpIHsKPiArCQlhdG9t aWM2NF9zdWIobWVtLT5udW1fcGFnZXMgPDwgUEFHRV9TSElGVCwgJm1nci0+dXNhZ2UpOwo+ICAg CQlyZXR1cm4gLUVOT01FTTsKPiArCX0KPiAgIAo+ICAgCW1vZGUgPSBEUk1fTU1fSU5TRVJUX0JF U1Q7Cj4gICAJaWYgKHBsYWNlLT5mbGFncyAmIFRUTV9QTF9GTEFHX1RPUERPV04pCj4gQEAgLTMy MSw3ICszMzAsNiBAQCBzdGF0aWMgaW50IGFtZGdwdV92cmFtX21ncl9uZXcoc3RydWN0IHR0bV9t ZW1fdHlwZV9tYW5hZ2VyICptYW4sCj4gICAJCWlmICh1bmxpa2VseShyKSkKPiAgIAkJCWJyZWFr Owo+ICAgCj4gLQkJdXNhZ2UgKz0gbm9kZXNbaV0uc2l6ZSA8PCBQQUdFX1NISUZUOwo+ICAgCQl2 aXNfdXNhZ2UgKz0gYW1kZ3B1X3ZyYW1fbWdyX3Zpc19zaXplKGFkZXYsICZub2Rlc1tpXSk7Cj4g ICAJCWFtZGdwdV92cmFtX21ncl92aXJ0X3N0YXJ0KG1lbSwgJm5vZGVzW2ldKTsKPiAgIAkJcGFn ZXNfbGVmdCAtPSBwYWdlczsKPiBAQCAtMzQxLDE0ICszNDksMTIgQEAgc3RhdGljIGludCBhbWRn cHVfdnJhbV9tZ3JfbmV3KHN0cnVjdCB0dG1fbWVtX3R5cGVfbWFuYWdlciAqbWFuLAo+ICAgCQlp ZiAodW5saWtlbHkocikpCj4gICAJCQlnb3RvIGVycm9yOwo+ICAgCj4gLQkJdXNhZ2UgKz0gbm9k ZXNbaV0uc2l6ZSA8PCBQQUdFX1NISUZUOwo+ICAgCQl2aXNfdXNhZ2UgKz0gYW1kZ3B1X3ZyYW1f bWdyX3Zpc19zaXplKGFkZXYsICZub2Rlc1tpXSk7Cj4gICAJCWFtZGdwdV92cmFtX21ncl92aXJ0 X3N0YXJ0KG1lbSwgJm5vZGVzW2ldKTsKPiAgIAkJcGFnZXNfbGVmdCAtPSBwYWdlczsKPiAgIAl9 Cj4gICAJc3Bpbl91bmxvY2soJm1nci0+bG9jayk7Cj4gICAKPiAtCWF0b21pYzY0X2FkZCh1c2Fn ZSwgJm1nci0+dXNhZ2UpOwo+ICAgCWF0b21pYzY0X2FkZCh2aXNfdXNhZ2UsICZtZ3ItPnZpc191 c2FnZSk7Cj4gICAKPiAgIAltZW0tPm1tX25vZGUgPSBub2RlczsKPiBAQCAtMzU5LDYgKzM2NSw3 IEBAIHN0YXRpYyBpbnQgYW1kZ3B1X3ZyYW1fbWdyX25ldyhzdHJ1Y3QgdHRtX21lbV90eXBlX21h bmFnZXIgKm1hbiwKPiAgIAl3aGlsZSAoaS0tKQo+ICAgCQlkcm1fbW1fcmVtb3ZlX25vZGUoJm5v ZGVzW2ldKTsKPiAgIAlzcGluX3VubG9jaygmbWdyLT5sb2NrKTsKPiArCWF0b21pYzY0X3N1Yiht ZW0tPm51bV9wYWdlcyA8PCBQQUdFX1NISUZULCAmbWdyLT51c2FnZSk7Cj4gICAKPiAgIAlrdmZy ZWUobm9kZXMpOwo+ICAgCXJldHVybiByID09IC1FTk9TUEMgPyAwIDogcjsKCl9fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmFtZC1nZnggbWFpbGluZyBsaXN0 CmFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Au b3JnL21haWxtYW4vbGlzdGluZm8vYW1kLWdmeA==