| 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
| ||
|
Message-ID: <f46f73db6594c7cd40149e35da9f188baa5961a2.camel@mailbox.org> Date: Wed, 28 May 2025 16:18:30 +0200 From: Philipp Stanner <phasta@...lbox.org> To: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@....com> Cc: Christian König <christian.koenig@....com>, Maíra Canal <mcanal@...lia.com>, Thomas Hellström <thomas.hellstrom@...ux.intel.com>, Abhinav Kumar <quic_abhinavk@...cinc.com>, Alex Deucher <alexander.deucher@....com>, Boris Brezillon <boris.brezillon@...labora.com>, Danilo Krummrich <dakr@...nel.org>, David Airlie <airlied@...il.com>, Dmitry Baryshkov <lumag@...nel.org>, Felix Kuehling <Felix.Kuehling@....com>, Frank Binns <frank.binns@...tec.com>, Jonathan Corbet <corbet@....net>, Liviu Dudau <liviu.dudau@....com>, Lizhi Hou <lizhi.hou@....com>, Lucas De Marchi <lucas.demarchi@...el.com>, Lucas Stach <l.stach@...gutronix.de>, Lyude Paul <lyude@...hat.com>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, Matt Coster <matt.coster@...tec.com>, Matthew Brost <matthew.brost@...el.com>, Maxime Ripard <mripard@...nel.org>, Melissa Wen <mwen@...lia.com>, Min Ma <min.ma@....com>, Oded Gabbay <ogabbay@...nel.org>, Philipp Stanner <phasta@...nel.org>, Qiang Yu <yuq825@...il.com>, Rob Clark <robdclark@...il.com>, Rob Herring <robh@...nel.org>, Rodrigo Vivi <rodrigo.vivi@...el.com>, Simona Vetter <simona@...ll.ch>, Steven Price <steven.price@....com>, Sumit Semwal <sumit.semwal@...aro.org>, Thomas Zimmermann <tzimmermann@...e.de>, amd-gfx@...ts.freedesktop.org, dri-devel@...ts.freedesktop.org, etnaviv@...ts.freedesktop.org, freedreno@...ts.freedesktop.org, intel-xe@...ts.freedesktop.org, lima@...ts.freedesktop.org, linaro-mm-sig@...ts.linaro.org, linux-arm-msm@...r.kernel.org, linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org, linux-media@...r.kernel.org, nouveau@...ts.freedesktop.org Subject: Re: [PATCH v11 00/10] Improve gpu_scheduler trace events + UAPI On Mon, 2025-05-26 at 14:54 +0200, Pierre-Eric Pelloux-Prayer wrote: > Hi, > > The initial goal of this series was to improve the drm and amdgpu > trace events to be able to expose more of the inner workings of > the scheduler and drivers to developers via tools. > > Then, the series evolved to become focused only on gpu_scheduler. > The changes around vblank events will be part of a different > series, as well as the amdgpu ones. > > Moreover Sima suggested to make some trace events stable uAPI, > so tools can rely on them long term. > > The first patches extend and cleanup the gpu scheduler events, > then add a documentation entry in drm-uapi.rst. > > The last 2 patches are new in v8. One is based on a suggestion > from Tvrtko and gets rid of drm_sched_job::id. The other is a > cleanup of amdgpu trace events to use the fence=%llu:%llu format. > > The drm_sched_job patches don't affect gpuvis which has code to parse > the gpu_scheduler events but these events are not enabled. > > Changes since v10: > * fixed 2 errors reported by kernel test robot > * rebased on drm-misc-next > > Changes since v9: > * fixed documentation link syntax > * fixed typos in commit messages > * spelled out that these events cannot be used before > drm_sched_job_arm has been called > > Changes since v8: > * swapped patches 8 & 9 > * rebased on drm-next > > Changes since v7: > * uint64_t -> u64 > * reworked dependencies tracing (Tvrtko) > * use common name prefix for all events (Tvrtko) > * dropped drm_sched_job::id (Tvrtko) > > Useful links: > - userspace tool using the updated events: > https://gitlab.freedesktop.org/tomstdenis/umr/-/merge_requests/37 > - v8: > https://lists.freedesktop.org/archives/dri-devel/2025-March/496781.html > > Pierre-Eric Pelloux-Prayer (10): > drm/debugfs: Output client_id in in drm_clients_info > drm/sched: Store the drm client_id in drm_sched_fence > drm/sched: Add device name to the drm_sched_process_job event > drm/sched: Cleanup gpu_scheduler trace events > drm/sched: Trace dependencies for GPU jobs > drm/sched: Add the drm_client_id to the drm_sched_run/exec_job > events > drm/sched: Cleanup event names > drm: Get rid of drm_sched_job.id > drm/doc: Document some tracepoints as uAPI > drm/amdgpu: update trace format to match gpu_scheduler_trace Applied to drm-misc-next Thanks P. > > Documentation/gpu/drm-uapi.rst | 19 ++++ > drivers/accel/amdxdna/aie2_ctx.c | 3 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 8 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_job.h | 3 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 32 ++---- > drivers/gpu/drm/drm_debugfs.c | 10 +- > drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c | 2 +- > drivers/gpu/drm/imagination/pvr_job.c | 2 +- > drivers/gpu/drm/imagination/pvr_queue.c | 5 +- > drivers/gpu/drm/imagination/pvr_queue.h | 2 +- > drivers/gpu/drm/lima/lima_gem.c | 2 +- > drivers/gpu/drm/lima/lima_sched.c | 6 +- > drivers/gpu/drm/lima/lima_sched.h | 3 +- > drivers/gpu/drm/lima/lima_trace.h | 6 +- > drivers/gpu/drm/msm/msm_gem_submit.c | 8 +- > drivers/gpu/drm/nouveau/nouveau_sched.c | 3 +- > drivers/gpu/drm/panfrost/panfrost_drv.c | 2 +- > drivers/gpu/drm/panthor/panthor_drv.c | 3 +- > drivers/gpu/drm/panthor/panthor_mmu.c | 2 +- > drivers/gpu/drm/panthor/panthor_sched.c | 5 +- > drivers/gpu/drm/panthor/panthor_sched.h | 3 +- > .../gpu/drm/scheduler/gpu_scheduler_trace.h | 103 +++++++++++++--- > -- > drivers/gpu/drm/scheduler/sched_entity.c | 16 ++- > drivers/gpu/drm/scheduler/sched_fence.c | 4 +- > drivers/gpu/drm/scheduler/sched_internal.h | 2 +- > drivers/gpu/drm/scheduler/sched_main.c | 12 +- > .../gpu/drm/scheduler/tests/mock_scheduler.c | 3 +- > drivers/gpu/drm/v3d/v3d_submit.c | 2 +- > drivers/gpu/drm/xe/xe_sched_job.c | 3 +- > include/drm/gpu_scheduler.h | 13 ++- > 32 files changed, 191 insertions(+), 101 deletions(-) >
Powered by blists - more mailing lists