From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Koenig, Christian" Subject: Re: [RFC PATCH v2 4/5] drm, cgroup: Add total GEM buffer allocation limit Date: Thu, 16 May 2019 14:08:38 +0000 Message-ID: <65075485-3c94-41cf-d39a-1fd451ecc430@amd.com> References: <20181120185814.13362-1-Kenny.Ho@amd.com> <20190509210410.5471-1-Kenny.Ho@amd.com> <20190509210410.5471-5-Kenny.Ho@amd.com> <7db2caae-7eab-7c6a-fe90-89cb9cae30b4@amd.com> <6e124f5e-f83f-5ca1-4616-92538f202653@gmail.com> <20190516122858.GB3851@phenom.ffwll.local> Mime-Version: 1.0 Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20190516122858.GB3851@phenom.ffwll.local> Content-Language: en-US Content-ID: <303281B957BB504189E859716D523295@namprd12.prod.outlook.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Content-Type: text/plain; charset="iso-8859-1" To: Daniel Vetter Cc: "sunnanyong@huawei.com" , "Ho, Kenny" , "Welty, Brian" , "dri-devel@lists.freedesktop.org" , "Deucher, Alexander" , Kenny Ho , "amd-gfx@lists.freedesktop.org" , Tejun Heo , "cgroups@vger.kernel.org" QW0gMTYuMDUuMTkgdW0gMTQ6Mjggc2NocmllYiBEYW5pZWwgVmV0dGVyOg0KPiBbQ0FVVElPTjog RXh0ZXJuYWwgRW1haWxdDQo+DQo+IE9uIFRodSwgTWF5IDE2LCAyMDE5IGF0IDA5OjI1OjMxQU0g KzAyMDAsIENocmlzdGlhbiBLw7ZuaWcgd3JvdGU6DQo+PiBBbSAxNi4wNS4xOSB1bSAwOToxNiBz Y2hyaWViIEtvZW5pZywgQ2hyaXN0aWFuOg0KPj4+IEFtIDE2LjA1LjE5IHVtIDA0OjI5IHNjaHJp ZWIgS2VubnkgSG86DQo+Pj4+IFtDQVVUSU9OOiBFeHRlcm5hbCBFbWFpbF0NCj4+Pj4NCj4+Pj4g T24gV2VkLCBNYXkgMTUsIDIwMTkgYXQgNToyNiBQTSBXZWx0eSwgQnJpYW4gPGJyaWFuLndlbHR5 QGludGVsLmNvbT4gd3JvdGU6DQo+Pj4+PiBPbiA1LzkvMjAxOSAyOjA0IFBNLCBLZW5ueSBIbyB3 cm90ZToNCj4+Pj4+PiBUaGVyZSBhcmUgZm91ciBjb250cm9sIGZpbGUgdHlwZXMsDQo+Pj4+Pj4g c3RhdHMgKHJvKSAtIGRpc3BsYXkgY3VycmVudCBtZWFzdXJlZCB2YWx1ZXMgZm9yIGEgcmVzb3Vy Y2UNCj4+Pj4+PiBtYXggKHJ3KSAtIGxpbWl0cyBmb3IgYSByZXNvdXJjZQ0KPj4+Pj4+IGRlZmF1 bHQgKHJvLCByb290IGNncm91cCBvbmx5KSAtIGRlZmF1bHQgdmFsdWVzIGZvciBhIHJlc291cmNl DQo+Pj4+Pj4gaGVscCAocm8sIHJvb3QgY2dyb3VwIG9ubHkpIC0gaGVscCBzdHJpbmcgZm9yIGEg cmVzb3VyY2UNCj4+Pj4+Pg0KPj4+Pj4+IEVhY2ggZmlsZSBpcyBtdWx0aS1saW5lZCB3aXRoIG9u ZSBlbnRyeS9saW5lIHBlciBkcm0gZGV2aWNlLg0KPj4+Pj4gTXVsdGktbGluZSBpcyBjb3JyZWN0 IGZvciBtdWx0aXBsZSBkZXZpY2VzLCBidXQgSSBiZWxpZXZlIHlvdSBuZWVkDQo+Pj4+PiB0byB1 c2UgYSBLRVkgdG8gZGVub3RlIGRldmljZSBmb3IgYm90aCB5b3VyIHNldCBhbmQgZ2V0IHJvdXRp bmVzLg0KPj4+Pj4gSSBkaWRuJ3Qgc2VlIHlvdXIgc2V0IGZ1bmN0aW9ucyByZWFkaW5nIGEga2V5 LCBvciB0aGUgZ2V0IGZ1bmN0aW9ucw0KPj4+Pj4gcHJpbnRpbmcgdGhlIGtleSBpbiBvdXRwdXQu DQo+Pj4+PiBjZ3JvdXBzLXYyIGNvbnZlbnRpb25zIG1lbnRpb24gdXNpbmcgS0VZIG9mIG1ham9y Om1pbm9yLCBidXQgSSB0aGluaw0KPj4+Pj4geW91IGNhbiB1c2UgZHJtX21pbm9yIGFzIGtleT8N Cj4+Pj4gR2l2ZW4gdGhpcyBjb250cm9sbGVyIGlzIHNwZWNpZmljIHRvIHRoZSBkcm0ga2VybmVs IHN1YnN5c3RlbSB3aGljaA0KPj4+PiB1c2VzIG1pbm9yIHRvIGlkZW50aWZ5IGRybSBkZXZpY2Us DQo+Pj4gV2FpdCBhIHNlY29uZCwgdXNpbmcgdGhlIERSTSBtaW5vciBpcyBhIGdvb2QgaWRlYSBp biB0aGUgZmlyc3QgcGxhY2UuDQo+PiBXZWxsIHRoYXQgc2hvdWxkIGhhdmUgcmVhZCAiaXMgbm90 IGEgZ29vZCBpZGVhIi4uDQo+IFdoYXQgZWxzZSBzaG91bGQgd2UgdXNlPw0KDQpXZWxsIHdoYXQg ZG9lcyBmb3IgZXhhbXBsZSB1ZGV2IHVzZXMgdG8gaWRlbnRpZnkgYSBkZXZpY2U/DQoNCj4+IENo cmlzdGlhbi4NCj4+DQo+Pj4gSSBoYXZlIGEgdGVzdCBzeXN0ZW0gd2l0aCBhIFZlZ2ExMCBhbmQg YSBWZWdhMjAuIFdoaWNoIGRldmljZSBnZXRzIHdoaWNoDQo+Pj4gbWlub3IgaXMgbm90IHN0YWJs ZSwgYnV0IHJhdGhlciBkZWZpbmVkIGJ5IHRoZSBzY2FuIG9yZGVyIG9mIHRoZSBQQ0llIGJ1cy4N Cj4+Pg0KPj4+IE5vcm1hbGx5IHRoZSBzY2FuIG9yZGVyIGlzIGFsd2F5cyB0aGUgc2FtZSwgYnV0 IGFkZGluZyBvciByZW1vdmluZw0KPj4+IGRldmljZXMgb3IgZGVsYXlpbmcgdGhpbmdzIGp1c3Qg YSBsaXR0bGUgYml0IGR1cmluZyBpbml0IGlzIGVub3VnaCB0bw0KPj4+IGNoYW5nZSB0aGlzLg0K Pj4+DQo+Pj4gV2UgbmVlZCBzb21ldGhpbmcgbGlrZSB0aGUgTGludXggc3lzZnMgbG9jYXRpb24g b3Igc2ltaWxhciB0byBoYXZlIGENCj4+PiBzdGFibGUgaW1wbGVtZW50YXRpb24uDQo+IFlvdSBj YW4gZ28gZnJvbSBzeXNmcyBsb2NhdGlvbiB0byBkcm0gY2xhc3MgZGlyZWN0b3J5IChpbiBzeXNm cykgYW5kIGJhY2suDQo+IFRoYXQgbWVhbnMgaWYgeW91IGNhcmUgeW91IG5lZWQgdG8gd2FsayBz eXNmcyB5b3Vyc2VsZiBhIGJpdCwgYnV0IHVzaW5nDQo+IHRoZSBkcm0gbWlub3IgaXNuJ3QgYSBi bG9ja2VyIGl0c2VsZi4NCg0KWWVhaCwgYWdyZWVkIHRoYXQgdXNlcnNwYWNlIGNvdWxkIGRvIHRo aXMuIEJ1dCBJIHRoaW5rIGlmIHRoZXJlIGlzIGFuIG9mIA0KaGFuZCBhbHRlcm5hdGl2ZSB3ZSBz aG91bGQgdXNlIHRoaXMgaW5zdGVhZC4NCg0KPiBPbmUgZG93bnNpZGUgd2l0aCB0aGUgZHJtIG1p bm9yIGlzIHRoYXQgaXQncyBwcmV0dHkgZ29vZCBub25zZW5zZSBvbmNlIHlvdQ0KPiBoYXZlIG1v cmUgdGhhbiA2NCBncHVzIHRob3VnaCwgZHVlIHRvIGhvdyB3ZSBzcGFjZSByZW5kZXIgYW5kIGxl Z2FjeSBub2Rlcw0KPiBpbiB0aGUgbWlub3IgaWRzIDotKQ0KDQpPaywgYW5vdGhlciBnb29kIHJl YXNvbiB0byBhdCBsZWFzdCBub3QgdXNlIHRoZSBtaW5vcj1saW5lbnVtIGFwcHJvYWNoLg0KDQpD aHJpc3RpYW4uDQoNCj4gLURhbmllbA0KPj4+IFJlZ2FyZHMsDQo+Pj4gQ2hyaXN0aWFuLg0KPj4+ DQo+Pj4+ICAgICBJIGRvbid0IHNlZSBhIG5lZWQgdG8gY29tcGxpY2F0ZQ0KPj4+PiB0aGUgaW50 ZXJmYWNlcyBtb3JlIGJ5IGhhdmluZyBtYWpvciBhbmQgYSBrZXkuICBBcyB5b3UgY2FuIHNlZSBp biB0aGUNCj4+Pj4gZXhhbXBsZXMgYmVsb3csIHRoZSBkcm0gZGV2aWNlIG1pbm9yIGNvcnJlc3Bv bmRzIHRvIHRoZSBsaW5lIG51bWJlci4NCj4+Pj4gSSBhbSBub3Qgc3VyZSBob3cgc3RyaWN0IGNn cm91cCB1cHN0cmVhbSBpcyBhYm91dCB0aGUgY29udmVudGlvbiBidXQgSQ0KPj4+PiBhbSBob3Bp bmcgdGhlcmUgYXJlIGZsZXhpYmlsaXR5IGhlcmUgdG8gYWxsb3cgZm9yIHdoYXQgSSBoYXZlDQo+ Pj4+IGltcGxlbWVudGVkLiAgVGhlcmUgYXJlIGEgY291cGxlIG9mIG90aGVyIHRoaW5ncyBJIGhh dmUgZG9uZSB0aGF0IGlzDQo+Pj4+IG5vdCBkZXNjcmliZWQgaW4gdGhlIGNvbnZlbnRpb246IDEp IGluY2x1c2lvbiBvZiByZWFkLW9ubHkgKi5oZWxwIGZpbGUNCj4+Pj4gYXQgdGhlIHJvb3QgY2dy b3VwLCAyKSB1c2UgcmVhZC1vbmx5ICh3aGljaCBJIGNhbiBwb3RlbnRpYWxseSBtYWtlIHJ3KQ0K Pj4+PiAqLmRlZmF1bHQgZmlsZSBpbnN0ZWFkIG9mIGhhdmluZyBhIGRlZmF1bHQgZW50cmllcyAo c2luY2UgdGhlIGRlZmF1bHQNCj4+Pj4gY2FuIGJlIGRpZmZlcmVudCBmb3IgZGlmZmVyZW50IGRl dmljZXMpIGluc2lkZSB0aGUgY29udHJvbCBmaWxlcyAodGhpcw0KPj4+PiB3YXksIHRoZSByZXNl dHRpbmcgb2YgY2dyb3VwIHZhbHVlcyBmb3IgYWxsIHRoZSBkcm0gZGV2aWNlcywgY2FuIGJlDQo+ Pj4+IGRvbmUgYnkgYSBzaW1wbGUgJ2NwJy4pDQo+Pj4+DQo+Pj4+Pj4gVXNhZ2UgZXhhbXBsZXM6 DQo+Pj4+Pj4gLy8gc2V0IGxpbWl0IGZvciBjYXJkMSB0byAxR0INCj4+Pj4+PiBzZWQgLWkgJzJz Ly4qLzEwNzM3NDE4MjQvJyAvc3lzL2ZzL2Nncm91cC88Y2dyb3VwPi9kcm0uYnVmZmVyLnRvdGFs Lm1heA0KPj4+Pj4+DQo+Pj4+Pj4gLy8gc2V0IGxpbWl0IGZvciBjYXJkMCB0byA1MTJNQg0KPj4+ Pj4+IHNlZCAtaSAnMXMvLiovNTM2ODcwOTEyLycgL3N5cy9mcy9jZ3JvdXAvPGNncm91cD4vZHJt LmJ1ZmZlci50b3RhbC5tYXgNCj4+Pj4+PiAgICAgLyoqIEBmaWxlIGRybV9nZW0uYw0KPj4+Pj4+ IEBAIC0xNTQsNiArMTU2LDkgQEAgdm9pZCBkcm1fZ2VtX3ByaXZhdGVfb2JqZWN0X2luaXQoc3Ry dWN0IGRybV9kZXZpY2UgKmRldiwNCj4+Pj4+PiAgICAgICAgICBvYmotPmhhbmRsZV9jb3VudCA9 IDA7DQo+Pj4+Pj4gICAgICAgICAgb2JqLT5zaXplID0gc2l6ZTsNCj4+Pj4+PiAgICAgICAgICBk cm1fdm1hX25vZGVfcmVzZXQoJm9iai0+dm1hX25vZGUpOw0KPj4+Pj4+ICsNCj4+Pj4+PiArICAg ICBvYmotPmRybWNncnAgPSBnZXRfZHJtY2dycChjdXJyZW50KTsNCj4+Pj4+PiArICAgICBkcm1j Z3JwX2NoZ19ib19hbGxvYyhvYmotPmRybWNncnAsIGRldiwgc2l6ZSk7DQo+Pj4+PiBXaHkgZG8g dGhlIGNoYXJnaW5nIGhlcmU/DQo+Pj4+PiBUaGVyZSBpcyBubyBiYWNraW5nIHN0b3JlIHlldCBm b3IgdGhlIGJ1ZmZlciwgc28gdGhpcyBpcyByZWFsbHkgdHJhY2tpbmcgc29tZXRoaW5nIGFraW4g dG8gYWxsb3dlZCB2aXJ0dWFsIG1lbW9yeSBmb3IgR0VNIG9iamVjdHM/DQo+Pj4+PiBJcyB0aGlz IHJlYWxseSB1c2VmdWwgZm9yIGFuIGFkbWluaXN0cmF0b3IgdG8gY29udHJvbD8NCj4+Pj4+IElz bid0IHRoZSByZXNvdXJjZSB3ZSB3YW50IHRvIGNvbnRyb2wgYWN0dWFsbHkgdGhlIHBoeXNpY2Fs IGJhY2tpbmcgc3RvcmU/DQo+Pj4+IFRoYXQncyBjb3JyZWN0LiAgVGhpcyBpcyBqdXN0IHRoZSBm aXJzdCBsZXZlbCBvZiBjb250cm9sIHNpbmNlIHRoZQ0KPj4+PiBiYWNraW5nIHN0b3JlIGNhbiBi ZSBiYWNrZWQgYnkgZGlmZmVyZW50IHR5cGUgb2YgbWVtb3J5LiAgSSBhbSBpbiB0aGUNCj4+Pj4g cHJvY2VzcyBvZiBhZGRpbmcgYXQgbGVhc3QgdHdvIG1vcmUgcmVzb3VyY2VzLiAgU3RheSB0dW5l ZC4gIEkgYW0NCj4+Pj4gZG9pbmcgdGhlIGNoYXJnZSBoZXJlIHRvIGVuZm9yY2UgdGhlIGlkZWEg b2YgImNyZWF0b3IgaXMgZGVlbWVkIG93bmVyIg0KPj4+PiBhdCBhIHBsYWNlIHdoZXJlIHRoZSBj b2RlIGlzIHNoYXJlZCBieSBhbGwgKHRoZSBpbml0IGZ1bmN0aW9uLikNCj4+Pj4NCj4+Pj4+PiAr ICAgICB3aGlsZSAoaSA8PSBtYXhfbWlub3IgJiYgbGltaXRzICE9IE5VTEwpIHsNCj4+Pj4+PiAr ICAgICAgICAgICAgIHN2YWwgPSAgc3Ryc2VwKCZsaW1pdHMsICJcbiIpOw0KPj4+Pj4+ICsgICAg ICAgICAgICAgcmMgPSBrc3RydG9sbChzdmFsLCAwLCAmdmFsKTsNCj4+Pj4+IElucHV0IHNob3Vs ZCBiZSAiS0VZIFZBTFVFIiwgc28gS0VZIHdpbGwgZGV0ZXJtaW5lIGRldmljZSB0byBhcHBseSB0 aGlzIHRvLg0KPj4+Pj4gQWxzbywgcGVyIGNncm91cHMtdjIgZG9jdW1lbnRhdGlvbiBvZiBsaW1p dHMsIEkgYmVsaWV2ZSBuZWVkIHRvIHBhcnNlIGFuZCBoYW5kbGUgdGhlIHNwZWNpYWwgIm1heCIg aW5wdXQgdmFsdWUuDQo+Pj4+Pg0KPj4+Pj4gcGFyc2VfcmVzb3VyY2VzKCkgaW4gcmRtYSBjb250 cm9sbGVyIGlzIGV4YW1wbGUgZm9yIGJvdGggb2YgYWJvdmUuDQo+Pj4+IFBsZWFzZSBzZWUgbXkg cHJldmlvdXMgcmVwbHkgZm9yIHRoZSByYXRpb25hbGUgb2YgbXkgaG9wZSB0byBub3QgbmVlZA0K Pj4+PiBhIGtleS4gIEkgY2FuIGNlcnRhaW5seSBhZGQgaGFuZGxpbmcgb2YgIm1heCIgYW5kICJk ZWZhdWx0Ii4NCj4+Pj4NCj4+Pj4NCj4+Pj4+PiArdm9pZCBkcm1jZ3JwX2NoZ19ib19hbGxvYyhz dHJ1Y3QgZHJtY2dycCAqZHJtY2dycCwgc3RydWN0IGRybV9kZXZpY2UgKmRldiwNCj4+Pj4+PiAr ICAgICAgICAgICAgIHNpemVfdCBzaXplKQ0KPj4+Pj4gU2hvdWxkbid0IHRoaXMgcmV0dXJuIGFu IGVycm9yIGFuZCBiZSBpbXBsZW1lbnRlZCB3aXRoIHNhbWUgc2VtYW50aWNzIGFzIHRoZQ0KPj4+ Pj4gdHJ5X2NoYXJnZSgpIGZ1bmN0aW9ucyBvZiBvdGhlciBjb250cm9sbGVycz8NCj4+Pj4+IEJl bG93IHdpbGwgYWxsb3cgc3RhdHNfdG90YWxfYWxsb2NhdGVkIHRvIG92ZXJydW4gbGltaXRzX3Rv dGFsX2FsbG9jYXRlZC4NCj4+Pj4gVGhpcyBpcyBiZWNhdXNlIEkgYW0gY2hhcmdpbmcgdGhlIGJ1 ZmZlciBhdCB0aGUgaW5pdCBvZiB0aGUgYnVmZmVyDQo+Pj4+IHdoaWNoIGRvZXMgbm90IGZhaWwg c28gdGhlICJ0cnkiIChkcm1jZ3JwX2JvX2Nhbl9hbGxvY2F0ZSkgaXMgc2VwYXJhdGUNCj4+Pj4g YW5kIHBsYWNlZCBlYXJsaWVyIGFuZCBuZWFyZXIgb3RoZXIgY29uZGl0aW9uIHdoZXJlIGdlbSBv YmplY3QNCj4+Pj4gYWxsb2NhdGlvbiBtYXkgZmFpbC4gIEluIG90aGVyIHdvcmRzLCB0aGVyZSBh cmUgbXVsdGlwbGUgcG9zc2liaWxpdGllcw0KPj4+PiBmb3Igd2hpY2ggZ2VtIGFsbG9jYXRpb24g bWF5IGZhaWwgKGNncm91cCBsaW1pdCBiZWluZyBvbmUgb2YgdGhlbSkgYW5kDQo+Pj4+IHNhdGlz ZnlpbmcgY2dyb3VwIGxpbWl0IGRvZXMgbm90IG1lYW4gYSBjaGFyZ2UgaXMgbmVlZGVkLiAgSSBj YW4NCj4+Pj4gY2VydGFpbmx5IGNvbWJpbmUgdGhlIHR3byBmdW5jdGlvbnMgdG8gaGF2ZSBhbiBh ZGRpdGlvbmFsIHRyeV9jaGFyZ2UNCj4+Pj4gc2VtYW50aWMgYXMgd2VsbCBpZiB0aGF0IGlzIHJl YWxseSBuZWVkZWQuDQo+Pj4+DQo+Pj4+IFJlZ2FyZHMsDQo+Pj4+IEtlbm55DQo+Pj4gX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18NCj4+PiBhbWQtZ2Z4IG1h aWxpbmcgbGlzdA0KPj4+IGFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnDQo+Pj4gaHR0cHM6 Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9hbWQtZ2Z4DQo+PiBfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXw0KPj4gZHJpLWRldmVs IG1haWxpbmcgbGlzdA0KPj4gZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZw0KPj4gaHR0 cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwNCj4g LS0NCj4gRGFuaWVsIFZldHRlcg0KPiBTb2Z0d2FyZSBFbmdpbmVlciwgSW50ZWwgQ29ycG9yYXRp b24NCj4gaHR0cDovL2Jsb2cuZmZ3bGwuY2gNCg0KX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlz dHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4v bGlzdGluZm8vZHJpLWRldmVs