lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 19 Oct 2022 18:32:43 +0100 From: Tvrtko Ursulin <tvrtko.ursulin@...ux.intel.com> To: Intel-gfx@...ts.freedesktop.org Cc: cgroups@...r.kernel.org, linux-kernel@...r.kernel.org, Tejun Heo <tj@...nel.org>, Johannes Weiner <hannes@...xchg.org>, Zefan Li <lizefan.x@...edance.com>, Dave Airlie <airlied@...hat.com>, Daniel Vetter <daniel.vetter@...ll.ch>, Rob Clark <robdclark@...omium.org>, Stéphane Marchesin <marcheu@...omium.org>, "T . J . Mercier" <tjmercier@...gle.com>, Kenny.Ho@....com, Christian König <christian.koenig@....com>, Brian Welty <brian.welty@...el.com>, Tvrtko Ursulin <tvrtko.ursulin@...el.com> Subject: [RFC 06/17] drm: Only track clients which are providing drm_cgroup_ops From: Tvrtko Ursulin <tvrtko.ursulin@...el.com> To reduce the number of tracking going on, especially with drivers which will not support any sort of control from the drm cgroup controller side, lets express the funcionality as opt-in and use the presence of drm_cgroup_ops as activation criteria. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@...el.com> --- drivers/gpu/drm/drm_cgroup.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/drm_cgroup.c b/drivers/gpu/drm/drm_cgroup.c index 0fbb88f08cef..7ed9c7150cae 100644 --- a/drivers/gpu/drm/drm_cgroup.c +++ b/drivers/gpu/drm/drm_cgroup.c @@ -16,6 +16,9 @@ void drm_clients_close(struct drm_file *file_priv) lockdep_assert_held(&dev->filelist_mutex); + if (!dev->driver->cg_ops) + return; + clients = xa_load(&drm_pid_clients, pid); list_del_rcu(&file_priv->clink); if (atomic_dec_and_test(&clients->num)) { @@ -40,6 +43,9 @@ int drm_clients_open(struct drm_file *file_priv) lockdep_assert_held(&dev->filelist_mutex); + if (!dev->driver->cg_ops) + return 0; + clients = xa_load(&drm_pid_clients, pid); if (!clients) { clients = kmalloc(sizeof(*clients), GFP_KERNEL); -- 2.34.1
Powered by blists - more mailing lists