On my TODO list, but behind about two or three other items I need to tackle first. Christian. Am 25.04.2017 um 11:07 schrieb zhoucm1: > ping... anyone can give the review? > > Thanks, > David Zhou > > On 2017å¹´04æ??24æ?¥ 13:57, Chunming Zhou wrote: >> The current kernel implementation, which grabs the idle VMID from >> pool when emitting the job may: >> >> The back-to-back submission from one process could use different >> VMID. >> The submission to different queues from single process could use >> different VMID >> >> It works well in most case but cannot work for the SQ thread trace >> capture. >> >> The VMID for the submission that set the {SQTT}_BASE, which refers to >> the address of the trace buffer, is stored in shader engine. >> >> If the profiling application have to use different VMIDs to submit >> IBs in its life cycle: >> >> Some trace is not captured since it actually uses different VMID >> to submit jobs. >> Some part of captured trace may come from different application >> since they are accidentally uses the ownerâ??s VMID to submit jobs. >> >> V2: >> 1. address Christian's comments: >> a. drop context flags for tag process, instead, add vm ioctl. >> b. change order of patches. >> c. sync waiting only when vm flush needs. >> >> 2. address Alex's comments; >> bump module version >> >> Chunming Zhou (6): >> drm/amdgpu: add vm ioctl >> drm/amdgpu: add dedicated vmid field in vm struct >> drm/amdgpu: reserve vmid by vm ioctl >> drm/amdgpu: add limitation for dedicated vm number v2 >> drm/amdgpu: implement grab dedicated vmid V2 >> drm/amdgpu: bump module verion for reserved vmid >> >> drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 1 + >> drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 3 +- >> drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 1 + >> drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 159 >> ++++++++++++++++++++++++++++- >> drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h | 5 + >> include/uapi/drm/amdgpu_drm.h | 20 ++++ >> 7 files changed, 188 insertions(+), 2 deletions(-) >> > > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx