From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kuehling, Felix" Subject: Re: [PATCH] drm/amd/amdgpu: Bail out of BO node creation if not enough VRAM (v3) Date: Wed, 12 Jun 2019 18:02:31 +0000 Message-ID: <977f7a62-9d5c-ded8-8c03-7950e095bde3@amd.com> References: <20190612110829.4699-1-tom.stdenis@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20190612110829.4699-1-tom.stdenis-5C7GfCeVMHo@public.gmane.org> Content-Language: en-US Content-ID: <2900175B99CEFD4883B99B68C39D9544-asWib9pRmPqcE4WynfumptQqCkab/8FMAL8bYrjMMd8@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: "StDenis, Tom" , "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" T24gMjAxOS0wNi0xMiA0OjA4LCBTdERlbmlzLCBUb20gd3JvdGU6DQo+ICh2Mik6IFJldHVybiAw IGFuZCBzZXQgbWVtLT5tbV9ub2RlIHRvIE5VTEwuDQo+ICh2Myk6IFVzZSBhdG9taWM2NF9hZGRf cmV0dXJuIGluc3RlYWQuDQo+DQo+IFNpZ25lZC1vZmYtYnk6IFRvbSBTdCBEZW5pcyA8dG9tLnN0 ZGVuaXNAYW1kLmNvbT4NCj4gLS0tDQo+ICAgZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1k Z3B1X3ZyYW1fbWdyLmMgfCAxNyArKysrKysrKysrKystLS0tLQ0KPiAgIDEgZmlsZSBjaGFuZ2Vk LCAxMiBpbnNlcnRpb25zKCspLCA1IGRlbGV0aW9ucygtKQ0KPg0KPiBkaWZmIC0tZ2l0IGEvZHJp dmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X3ZyYW1fbWdyLmMgYi9kcml2ZXJzL2dwdS9k cm0vYW1kL2FtZGdwdS9hbWRncHVfdnJhbV9tZ3IuYw0KPiBpbmRleCA4YWVhMmYyMWIyMDIuLmM5 NjNhZDg2MDcyZSAxMDA2NDQNCj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1k Z3B1X3ZyYW1fbWdyLmMNCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1 X3ZyYW1fbWdyLmMNCj4gQEAgLTI3Niw3ICsyNzYsNyBAQCBzdGF0aWMgaW50IGFtZGdwdV92cmFt X21ncl9uZXcoc3RydWN0IHR0bV9tZW1fdHlwZV9tYW5hZ2VyICptYW4sDQo+ICAgCXN0cnVjdCBk cm1fbW1fbm9kZSAqbm9kZXM7DQo+ICAgCWVudW0gZHJtX21tX2luc2VydF9tb2RlIG1vZGU7DQo+ ICAgCXVuc2lnbmVkIGxvbmcgbHBmbiwgbnVtX25vZGVzLCBwYWdlc19wZXJfbm9kZSwgcGFnZXNf bGVmdDsNCj4gLQl1aW50NjRfdCB1c2FnZSA9IDAsIHZpc191c2FnZSA9IDA7DQo+ICsJdWludDY0 X3QgdmlzX3VzYWdlID0gMDsNCj4gICAJdW5zaWduZWQgaTsNCj4gICAJaW50IHI7DQo+ICAgDQo+ IEBAIC0yODQsNiArMjg0LDEzIEBAIHN0YXRpYyBpbnQgYW1kZ3B1X3ZyYW1fbWdyX25ldyhzdHJ1 Y3QgdHRtX21lbV90eXBlX21hbmFnZXIgKm1hbiwNCj4gICAJaWYgKCFscGZuKQ0KPiAgIAkJbHBm biA9IG1hbi0+c2l6ZTsNCj4gICANCj4gKwkvKiBiYWlsIG91dCBxdWlja2x5IGlmIHRoZXJlJ3Mg bGlrZWx5IG5vdCBlbm91Z2ggVlJBTSBmb3IgdGhpcyBCTyAqLw0KPiArCWlmIChhdG9taWM2NF9h ZGRfcmV0dXJuKG1lbS0+bnVtX3BhZ2VzIDw8IFBBR0VfU0hJRlQsICZtZ3ItPnVzYWdlKSA+IGFk ZXYtPmdtYy5tY192cmFtX3NpemUpIHsNCj4gKwkJYXRvbWljNjRfc3ViKG1lbS0+bnVtX3BhZ2Vz IDw8IFBBR0VfU0hJRlQsICZtZ3ItPnVzYWdlKTsNCg0KT24gMzItYml0IHN5c3RlbXMgbWVtLT5u dW1fcGFnZXMgaXMgb25seSAzMi1iaXQgKHVuc2lnbmVkIGxvbmcpIGFuZCB0aGUgDQo8PCBQQUdF X1NISUZUIG1heSBvdmVyZmxvdy4gWW91IHByb2JhYmx5IHdhbnQgdG8gY2FzdCB0aGlzIHRvIHU2 NCANCmV4cGxpY2l0bHkgYmVmb3JlIHRoZSBzaGlmdC4NCg0KUmVnYXJkcywNCiDCoCBGZWxpeA0K DQoNCj4gKwkJbWVtLT5tbV9ub2RlID0gTlVMTDsNCj4gKwkJcmV0dXJuIDA7DQo+ICsJfQ0KPiAr DQo+ICAgCWlmIChwbGFjZS0+ZmxhZ3MgJiBUVE1fUExfRkxBR19DT05USUdVT1VTKSB7DQo+ICAg CQlwYWdlc19wZXJfbm9kZSA9IH4wdWw7DQo+ICAgCQludW1fbm9kZXMgPSAxOw0KPiBAQCAtMzAw LDggKzMwNywxMCBAQCBzdGF0aWMgaW50IGFtZGdwdV92cmFtX21ncl9uZXcoc3RydWN0IHR0bV9t ZW1fdHlwZV9tYW5hZ2VyICptYW4sDQo+ICAgDQo+ICAgCW5vZGVzID0ga3ZtYWxsb2NfYXJyYXko KHVpbnQzMl90KW51bV9ub2Rlcywgc2l6ZW9mKCpub2RlcyksDQo+ICAgCQkJICAgICAgIEdGUF9L RVJORUwgfCBfX0dGUF9aRVJPKTsNCj4gLQlpZiAoIW5vZGVzKQ0KPiArCWlmICghbm9kZXMpIHsN Cj4gKwkJYXRvbWljNjRfc3ViKG1lbS0+bnVtX3BhZ2VzIDw8IFBBR0VfU0hJRlQsICZtZ3ItPnVz YWdlKTsNCj4gICAJCXJldHVybiAtRU5PTUVNOw0KPiArCX0NCj4gICANCj4gICAJbW9kZSA9IERS TV9NTV9JTlNFUlRfQkVTVDsNCj4gICAJaWYgKHBsYWNlLT5mbGFncyAmIFRUTV9QTF9GTEFHX1RP UERPV04pDQo+IEBAIC0zMjEsNyArMzMwLDYgQEAgc3RhdGljIGludCBhbWRncHVfdnJhbV9tZ3Jf bmV3KHN0cnVjdCB0dG1fbWVtX3R5cGVfbWFuYWdlciAqbWFuLA0KPiAgIAkJaWYgKHVubGlrZWx5 KHIpKQ0KPiAgIAkJCWJyZWFrOw0KPiAgIA0KPiAtCQl1c2FnZSArPSBub2Rlc1tpXS5zaXplIDw8 IFBBR0VfU0hJRlQ7DQo+ICAgCQl2aXNfdXNhZ2UgKz0gYW1kZ3B1X3ZyYW1fbWdyX3Zpc19zaXpl KGFkZXYsICZub2Rlc1tpXSk7DQo+ICAgCQlhbWRncHVfdnJhbV9tZ3JfdmlydF9zdGFydChtZW0s ICZub2Rlc1tpXSk7DQo+ICAgCQlwYWdlc19sZWZ0IC09IHBhZ2VzOw0KPiBAQCAtMzQxLDE0ICsz NDksMTIgQEAgc3RhdGljIGludCBhbWRncHVfdnJhbV9tZ3JfbmV3KHN0cnVjdCB0dG1fbWVtX3R5 cGVfbWFuYWdlciAqbWFuLA0KPiAgIAkJaWYgKHVubGlrZWx5KHIpKQ0KPiAgIAkJCWdvdG8gZXJy b3I7DQo+ICAgDQo+IC0JCXVzYWdlICs9IG5vZGVzW2ldLnNpemUgPDwgUEFHRV9TSElGVDsNCj4g ICAJCXZpc191c2FnZSArPSBhbWRncHVfdnJhbV9tZ3JfdmlzX3NpemUoYWRldiwgJm5vZGVzW2ld KTsNCj4gICAJCWFtZGdwdV92cmFtX21ncl92aXJ0X3N0YXJ0KG1lbSwgJm5vZGVzW2ldKTsNCj4g ICAJCXBhZ2VzX2xlZnQgLT0gcGFnZXM7DQo+ICAgCX0NCj4gICAJc3Bpbl91bmxvY2soJm1nci0+ bG9jayk7DQo+ICAgDQo+IC0JYXRvbWljNjRfYWRkKHVzYWdlLCAmbWdyLT51c2FnZSk7DQo+ICAg CWF0b21pYzY0X2FkZCh2aXNfdXNhZ2UsICZtZ3ItPnZpc191c2FnZSk7DQo+ICAgDQo+ICAgCW1l bS0+bW1fbm9kZSA9IG5vZGVzOw0KPiBAQCAtMzU5LDYgKzM2NSw3IEBAIHN0YXRpYyBpbnQgYW1k Z3B1X3ZyYW1fbWdyX25ldyhzdHJ1Y3QgdHRtX21lbV90eXBlX21hbmFnZXIgKm1hbiwNCj4gICAJ d2hpbGUgKGktLSkNCj4gICAJCWRybV9tbV9yZW1vdmVfbm9kZSgmbm9kZXNbaV0pOw0KPiAgIAlz cGluX3VubG9jaygmbWdyLT5sb2NrKTsNCj4gKwlhdG9taWM2NF9zdWIobWVtLT5udW1fcGFnZXMg PDwgUEFHRV9TSElGVCwgJm1nci0+dXNhZ2UpOw0KPiAgIA0KPiAgIAlrdmZyZWUobm9kZXMpOw0K PiAgIAlyZXR1cm4gciA9PSAtRU5PU1BDID8gMCA6IHI7DQpfX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fXwphbWQtZ2Z4IG1haWxpbmcgbGlzdAphbWQtZ2Z4QGxp c3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFu L2xpc3RpbmZvL2FtZC1nZng=