On 03/01/2017 04:28 PM, ozeng wrote: > Dump register COMPUTE_PGM_RSRC1 and COMPUTE_PGM_RSRC2 > in function kgd_hqd_dump > > Change-Id: I8e007f5bcc095d3283b642791860dd072dd43b1c > Signed-off-by: Oak Zeng <Oak.Zeng at amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c | 2 ++ > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.c | 2 ++ > 2 files changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c > index 7732def..d7bcdc3 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c > @@ -439,6 +439,8 @@ static int kgd_hqd_dump(struct kgd_dev *kgd, > > acquire_queue(kgd, pipe_id, queue_id); > > + DUMP_REG(mmCOMPUTE_PGM_RSRC1); > + DUMP_REG(mmCOMPUTE_PGM_RSRC2); > DUMP_REG(mmCOMPUTE_STATIC_THREAD_MGMT_SE0); > DUMP_REG(mmCOMPUTE_STATIC_THREAD_MGMT_SE1); > DUMP_REG(mmCOMPUTE_STATIC_THREAD_MGMT_SE2); > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.c > index 289c188..2aff607 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.c > @@ -433,6 +433,8 @@ static int kgd_hqd_dump(struct kgd_dev *kgd, > > acquire_queue(kgd, pipe_id, queue_id); > > + DUMP_REG(mmCOMPUTE_PGM_RSRC1); > + DUMP_REG(mmCOMPUTE_PGM_RSRC2); > DUMP_REG(mmCOMPUTE_STATIC_THREAD_MGMT_SE0); > DUMP_REG(mmCOMPUTE_STATIC_THREAD_MGMT_SE1); > DUMP_REG(mmCOMPUTE_STATIC_THREAD_MGMT_SE2); > Why not just read these via the debugfs interface? Or is this part of a standard (re: not debugfs) interface? Cheers, Tom
