Re: [PATCH v2 0/5] drm/ttm: Introduce TTM res manager debugfs helpers

2022-04-19 Thread Christian König

Am 18.04.22 um 22:09 schrieb Zack Rusin:

On Tue, 2022-04-12 at 11:15 +0200, Christian König wrote:

Hi Zack,

Reviewed-by: Christian König  for the
entire
series.

One nit pick is that I want to get rid of using ttm_manager_type() in
drivers and use pointers to the members directly. But that's
something
for a later cleanup anyway.

That sounds good to me. Let me know if you'd like me to hold off on
pushing this until the ttm_manager_type changes are ready, otherwise
I'll push it to drm-misc-next tomorrow.


Please go ahead with pushing. That's cleanup work which I will throw in 
when I have time (which rarely happens these days).


Christian.



z





Re: [PATCH v2 0/5] drm/ttm: Introduce TTM res manager debugfs helpers

2022-04-18 Thread Zack Rusin
On Tue, 2022-04-12 at 11:15 +0200, Christian König wrote:
> 
> Hi Zack,
> 
> Reviewed-by: Christian König  for the
> entire
> series.
> 
> One nit pick is that I want to get rid of using ttm_manager_type() in
> drivers and use pointers to the members directly. But that's
> something
> for a later cleanup anyway.

That sounds good to me. Let me know if you'd like me to hold off on
pushing this until the ttm_manager_type changes are ready, otherwise
I'll push it to drm-misc-next tomorrow.

z



Re: [PATCH v2 0/5] drm/ttm: Introduce TTM res manager debugfs helpers

2022-04-12 Thread Christian König

Hi Zack,

Reviewed-by: Christian König  for the entire 
series.


One nit pick is that I want to get rid of using ttm_manager_type() in 
drivers and use pointers to the members directly. But that's something 
for a later cleanup anyway.


Thanks,
Christian.

Am 12.04.22 um 05:35 schrieb Zack Rusin:

From: Zack Rusin 

v2: Switch to using ttm_resource_manager's directly in the debugfs
callbacks

This series introduces generic TTM resource manager debugfs helpers and
refactors TTM drivers which have been using hand rolled out versions
of those to use the new code.

Zack Rusin (5):
   drm/ttm: Add common debugfs code for resource managers
   drm/vmwgfx: Add debugfs entries for various ttm resource managers
   drm/amdgpu: Use TTM builtin resource manager debugfs code
   drm/qxl: Use TTM builtin resource manager debugfs code
   drm/radeon: Use TTM builtin resource manager debugfs code

  drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 86 +
  drivers/gpu/drm/qxl/qxl_ttm.c   | 39 ++-
  drivers/gpu/drm/radeon/radeon_ttm.c | 36 ++-
  drivers/gpu/drm/ttm/ttm_resource.c  | 34 ++
  drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 18 ++
  include/drm/ttm/ttm_resource.h  |  4 ++
  6 files changed, 84 insertions(+), 133 deletions(-)