From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [RFC PATCH v3 04/11] drm, cgroup: Add total GEM buffer allocation limit Date: Wed, 26 Jun 2019 23:41:13 +0200 Message-ID: <20190626214113.GA12905@phenom.ffwll.local> References: <20190626150522.11618-1-Kenny.Ho@amd.com> <20190626150522.11618-5-Kenny.Ho@amd.com> <20190626160553.GR12905@phenom.ffwll.local> Mime-Version: 1.0 Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "amd-gfx" Content-Type: text/plain; charset="us-ascii" To: Kenny Ho Cc: joseph.greathouse-5C7GfCeVMHo@public.gmane.org, Kenny Ho , jsparks-WVYJKLFxKCc@public.gmane.org, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, lkaplan-WVYJKLFxKCc@public.gmane.org, Alex Deucher , dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, Daniel Vetter , Tejun Heo , cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Christian =?iso-8859-1?Q?K=F6nig?= T24gV2VkLCBKdW4gMjYsIDIwMTkgYXQgMDU6Mjc6NDhQTSAtMDQwMCwgS2VubnkgSG8gd3JvdGU6 Cj4gT24gV2VkLCBKdW4gMjYsIDIwMTkgYXQgMTI6MDUgUE0gRGFuaWVsIFZldHRlciA8ZGFuaWVs QGZmd2xsLmNoPiB3cm90ZToKPiA+Cj4gPiA+IGRybS5idWZmZXIuZGVmYXVsdAo+ID4gPiAgICAg ICAgIEEgcmVhZC1vbmx5IGZsYXQta2V5ZWQgZmlsZSB3aGljaCBleGlzdHMgb24gdGhlIHJvb3Qg Y2dyb3VwLgo+ID4gPiAgICAgICAgIEVhY2ggZW50cnkgaXMga2V5ZWQgYnkgdGhlIGRybSBkZXZp Y2UncyBtYWpvcjptaW5vci4KPiA+ID4KPiA+ID4gICAgICAgICBEZWZhdWx0IGxpbWl0cyBvbiB0 aGUgdG90YWwgR0VNIGJ1ZmZlciBhbGxvY2F0aW9uIGluIGJ5dGVzLgo+ID4KPiA+IERvbid0IHdl IG5lZWQgYSAiMCBtZWFucyBubyBsaW1pdCIgc2VtYW50aWNzIGhlcmU/Cj4gCj4gSSBiZWxpZXZl IHRoZSBjb252ZW50aW9uIGlzIHRvIHVzZSB0aGUgJ21heCcga2V5d29yZC4KPiAKPiA+Cj4gPiBJ IHRoaW5rIHdlIG5lZWQgYSBuZXcgZHJtLWNncm91cC5yc3Qgd2hpY2ggY29udGFpbnMgYWxsIHRo aXMKPiA+IGRvY3VtZW50YXRpb24uCj4gCj4gWWVzIEkgcGxhbm5lZCB0byBkbyB0aGF0IHdoZW4g dGhpbmdzIGFyZSBtb3JlIGZpbmFsaXplZC4gIEkgYW0KPiBhY3R1YWxseSB3cml0aW5nIHRoZSBj b21taXQgbWVzc2FnZSBmb2xsb3dpbmcgdGhlIGN1cnJlbnQgZG9jIGZvcm1hdAo+IHNvIEkgY2Fu IHJldXNlIGl0IGluIHRoZSByc3QuCgpBd2Vzb21lLgoKPiA+IFdpdGggbXVsdGlwbGUgR1BVcywg ZG8gd2UgbmVlZCBhbiBvdmVyYWxsIEdFTSBibyBsaW1pdCwgYWNyb3NzIGFsbCBncHVzPwo+ID4g Rm9yIG90aGVyIHN0dWZmIGxhdGVyIG9uIGxpa2UgdnJhbS90dC8uLi4gYW5kIGFsbCB0aGF0IGl0 IG5lZWRzIHRvIGJlCj4gPiBwZXItZGV2aWNlLCBidXQgSSB0aGluayBvbmUgb3ZlcmFsbCBsaW1p dCBjb3VsZCBiZSB1c2VmdWwuCj4gCj4gVGhpcyBvbmUgSSBhbSBub3Qgc3VyZSBidXQgc2hvdWxk IGJlIGZhaXJseSBzdHJhaWdodGZvcndhcmQgdG8gYWRkLgo+IEknZCBsb3ZlIHRvIGhlYXIgbW9y ZSBmZWVkYmFja3Mgb24gdGhpcyBhcyB3ZWxsLgo+IAo+ID4gPiAgICAgICBpZiAoIWFtZGdwdV9i b192YWxpZGF0ZV9zaXplKGFkZXYsIHNpemUsIGJwLT5kb21haW4pKQo+ID4gPiAgICAgICAgICAg ICAgIHJldHVybiAtRU5PTUVNOwo+ID4gPgo+ID4gPiArICAgICBpZiAoIWRybWNncnBfYm9fY2Fu X2FsbG9jYXRlKGN1cnJlbnQsIGFkZXYtPmRkZXYsIHNpemUpKQo+ID4gPiArICAgICAgICAgICAg IHJldHVybiAtRU5PTUVNOwo+ID4KPiA+IFNvIHdoYXQgaGFwcGVucyB3aGVuIHlvdSBzdGFydCBh IGxvdCBvZiB0aHJlYWRzIGFsbCBhdCB0aGUgc2FtZSB0aW1lLAo+ID4gYWxsb2NhdGluZyBnZW0g Ym8/IEFsc28gd291bGQgYmUgbmljZSBpZiB3ZSBjb3VsZCByb2xsIG91dCBhdCBsZWFzdCB0aGUK PiA+IGFjY291bnRpbmcgcGFydCBvZiB0aGlzIGNncm91cCB0byBhbGwgR0VNIGRyaXZlcnMuCj4g Cj4gV2hlbiB0aGVyZSBpcyBhIGxhcmdlIG51bWJlciBvZiBhbGxvY2F0aW9uLCB0aGUgYWxsb2Nh dGlvbiB3aWxsIGJlCj4gY2hlY2tlZCBpbiBzZXF1ZW5jZSB3aXRoaW4gYSBkZXZpY2UgKHNpbmNl IEkgdXNlZCBhIHBlciBkZXZpY2UgbXV0ZXgKPiBpbiB0aGUgY2hlY2suKSAgQXJlIHlvdSBzdWdn ZXN0aW5nIHRoZSBvdmVyaGVhZCBoZXJlIGlzIHNpZ25pZmljYW50Cj4gZW5vdWdoIHRvIGJlIGEg Ym90dGxlbmVjaz8gIFRoZSBhY2NvdW50aW5nIHBhcnQgc2hvdWxkIGJlIGF2YWlsYWJsZSB0bwo+ IGFsbCBHRU0gZHJpdmVycyAodW5sZXNzIEkgbWlzc2VkIHNvbWV0aGluZykgc2luY2UgdGhlIGNo ZyBhbmQgdW5jaGcKPiBmdW5jdGlvbiBpcyBjYWxsZWQgdmlhIHRoZSBnZW5lcmljIGRybV9nZW1f cHJpdmF0ZV9vYmplY3RfaW5pdCBhbmQKPiBkcm1fZ2VtX29iamVjdF9yZWxlYXNlLgoKdGhyZWFk IDE6IGNoZWNrcyBsaW1pdHMsIHN0aWxsIHVuZGVyIHRoZSB0b3RhbAoKdGhyZWFkIDI6IGNoZWNr cyBsaW1pdHMsIHN0aWxsIHVuZGVyIHRoZSB0b3RhbAoKdGhyZWFkIDE6IGFsbG9jYXRlcywgc3Rp bGwgdW5kZXIKCnRocmVhZCAyOiBhbGxvY2F0ZXMsIG5vdyBvdmVyIHRoZSBsaW1pdAoKSSB0aGlu ayB0aGUgY2hlY2sgYW5kIGNoZyBuZWVkIHRvIGJlIG9uZSBzdGVwLCBvciB0aGlzIHdvbnQgd29y ay4gT3IgSSdtCm1pc3Npbmcgc29tZXRoaW5nIHNvbWV3aGVyZS4KCldydCByb2xsaW5nIG91dCB0 aGUgYWNjb3VudGluZyBmb3IgYWxsIGRyaXZlcnM6IFNpbmNlIHlvdSBhbHNvIHJvbGwgb3V0CmVu Zm9yY2VtZW50IGluIHRoaXMgcGF0Y2ggSSdtIG5vdCBzdXJlIHdoZXRoZXIgdGhlIGFjY291bnRp bmcgcGFydCBpcwpmdWxseSBzdGFuZC1hbG9uZS4gQW5kIGFzIGRpc2N1c3NlZCBhIGJpdCBvbiBh biBlYXJsaWVyIHBhdGNoLCBJIHRoaW5rIGZvcgpEUklWRVJfR0VNIHdlIHNob3VsZCBzZXQgdXAg dGhlIGFjY291bnRpbmcgY2dyb3VwIGF1dG9tYXRpY2FsbHkuCgo+ID4gPiArICAgICAvKiBvbmx5 IGFsbG93IGJvIGZyb20gdGhlIHNhbWUgY2dyb3VwIG9yIGl0cyBhbmNlc3RvciB0byBiZSBpbXBv cnRlZCAqLwo+ID4gPiArICAgICBpZiAoZHJtY2dycCAhPSBOVUxMICYmCj4gPgo+ID4gUXVpdGUg YSBzZXJpb3VzIGxpbWl0YXRpb24gaGVyZSAuLi4KPiA+Cj4gPiA+ICsgICAgICAgICAgICAgICAg ICAgICAhZHJtY2dycF9pc19zZWxmX29yX2FuY2VzdG9yKGRybWNncnAsIG9iai0+ZHJtY2dycCkp IHsKPiA+Cj4gPiBBbHNvIHdoYXQgaGFwcGVucyBpZiB5b3UgYWN0dWFsbHkgc2hhcmUgYWNyb3Nz IGRldmljZXM/IFRoZW4gaW1wb3J0aW5nIGluCj4gPiB0aGUgMm5kIGdyb3VwIGlzIHN1ZGRlbmx5 IHBvc3NpYmxlLCBhbmQgSSB0aGluayB3aWxsIGJlIGRvdWJsZS1jb3VudGVkLgo+ID4KPiA+IFdo YXQncyB0aGUgdW5kZXJseWluZyB0ZWNobmljYWwgcmVhc29uIGZvciBub3QgYWxsb3dpbmcgc2hh cmluZyBhY3Jvc3MKPiA+IGNncm91cHM/Cj4gCj4gV2l0aCB0aGUgY3VycmVudCBpbXBsZW1lbnRh dGlvbiwgdGhlcmUgc2hvdWxkbid0IGJlIGRvdWJsZSBjb3VudGluZyBhcwo+IHRoZSBjb3VudGlu ZyBpcyBkb25lIGR1cmluZyB0aGUgYnVmZmVyIGluaXQuCgpJZiB5b3Ugc2hhcmUgYWNyb3NzIGRl dmljZXMgdGhlcmUgd2lsbCBiZSB0d28gZHJtX2dlbV9vYmVjdCBzdHJ1Y3R1cmVzLCBvbgpvbiBl YWNoIGRldmljZS4gQnV0IG9ubHkgb25lIHVuZGVybHlpbmcgYm8uCgpOb3cgdGhlIGJvIGxpbWl0 IGlzIHBlci1kZXZpY2UgdG9vLCBzbyB0aGF0J3MgYWxsIGZpbmUsIGJ1dCBmb3IgYSBnbG9iYWwK Ym8gbGltaXQgd2UnZCBuZWVkIHRvIG1ha2Ugc3VyZSB3ZSBjb3VudCB0aGVzZSBvbmx5IG9uY2Uu Cgo+IFRvIGJlIGNsZWFyLCBzaGFyaW5nIGFjcm9zcyBjZ3JvdXAgaXMgYWxsb3dlZCwgdGhlIGJ1 ZmZlciBqdXN0IG5lZWRzCj4gdG8gYmUgYWxsb2NhdGVkIGJ5IGEgcHJvY2VzcyB0aGF0IGlzIHBh cmVudCB0byB0aGUgY2dyb3VwLiAgU28gaW4gdGhlCj4gY2FzZSBvZiB4b3JnIGFsbG9jYXRpbmcg YnVmZmVyIGZvciBjbGllbnQsIHRoZSB4b3JnIHdvdWxkIGJlIGluIHRoZQo+IHJvb3QgY2dyb3Vw IGFuZCB0aGUgYnVmZmVyIGNhbiBiZSBwYXNzZWQgYXJvdW5kIGJ5IGRpZmZlcmVudCBjbGllbnRz Cj4gKGluIHJvb3Qgb3Igb3RoZXIgY2dyb3VwLikgIFRoZSBpZGVhIGhlcmUgaXMgdG8gZXN0YWJs aXNoIHNvbWUgZm9ybSBvZgo+IG93bmVyc2hpcCwgb3RoZXJ3aXNlIHRoZXJlIHdvdWxkbid0IGJl IGEgd2F5IHRvIGFjY291bnQgZm9yIG9yIGxpbWl0Cj4gdGhlIHVzYWdlLgoKQnV0IHdoeT8gV2hh dCdzIHRoZSBwcm9ibGVtIGlmIEkgYWxsb2NhdGUgc29tZXRoaW5nIGFuZCB0aGVuIGhhbmQgaXQg dG8Kc29tZW9uZSBlbHNlLiBFLmcuIG9uZSBwb3B1bGFyIHVzZSBvZiBjZ3JvdXBzIGlzIHRvIGlz b2xhdGUgY2xpZW50cywgc28KbWF5YmUgeW91J2QgZG8gYSBjZ3JvdXAgKyBuYW1lc3BhY2UgZm9y IGVhY2ggWDExIGNsaWVudCAob2sgd2F5bGFuZCwgd2l0aApYMTEgdGhpcyBpcyBwcm9iYWJseSBw b2ludGxlc3MpLgoKQnV0IHdpdGggeW91ciBjdXJyZW50IGxpbWl0YXRpb24gdGhvc2UgY2xpZW50 cyBjYW4ndCBwYXNzIGJ1ZmZlcnMgdG8gdGhlCmNvbXBvc2l0b3IgYW55bW9yZSwgbWFraW5nIGNn cm91cHMgdXNlbGVzcy4gWW91ciBleGFtcGxlIGhlcmUgb25seSB3b3JrcwppZiBYb3JnIGlzIGlu IHRoZSByb290IGFuZCBhbGxvY2F0ZXMgYWxsIHRoZSBidWZmZXJzLiBUaGF0J3Mgbm90IGV2ZW4g dHJ1ZQpmb3IgRFJJMyBhbnltb3JlLgoKU28gcHJldHR5IHNlcmlvdXMgbGltaXRhdGlvbiBvbiBj Z3JvdXBzLCBhbmQgSSdtIG5vdCByZWFsbHkgdW5kZXJzdGFuZGluZwp3aHkgd2UgbmVlZCB0aGlz LiBJIHRoaW5rIGlmIHdlIHdhbnQgdG8gcHJldmVudCBidWZmZXIgc2hhcmluZywgd2hhdCB3ZQpu ZWVkIGFyZSBzb21lIHNlbGludXggaG9va3MgYW5kIHN0dWZmIHNvIHlvdSBjYW4gcHJldmVudCBh biBpbXBvcnQvYWNjZXNzCmJ5IHNvbWVvbmUgd2hvJ3Mgbm90IGFsbG93ZWQgdG8gdG91Y2ggYSBi dWZmZXIuIEJ1dCB0aGF0IGtpbmQgb2YgYWNjZXNzCnJpZ2h0IG1hbmFnZW1lbnQgc2hvdWxkIGJl IHNlcGFyYXRlIGZyb20gcmVzb3VyY2UgY29udHJvbCBpbW8uCi1EYW5pZWwKLS0gCkRhbmllbCBW ZXR0ZXIKU29mdHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9uCmh0dHA6Ly9ibG9nLmZm d2xsLmNoCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmFt ZC1nZnggbWFpbGluZyBsaXN0CmFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8v bGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vYW1kLWdmeA==