[PATCH 0/6] *** Dedicated vmid per process v2 ***

Christian König deathsimple at vodafone.de
Tue Apr 25 10:02:25 UTC 2017


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




More information about the amd-gfx mailing list