[v2,0/2] drm/vc4: Improve drm_gem_object handling

Message

Maíra Canal Feb. 2, 2023, 11:19 a.m. UTC

Currently, the array of BOs that are lookup up at the start of exec is being
instantiated as drm_gem_dma_object, which is not needed and makes it difficult
to use the drm_gem_objects_lookup() helper. Therefore, replace
drm_gem_dma_object for drm_gem_object and then replace obj lookup steps with
drm_gem_objects_lookup().

v1 -> v2:

* Add André Almeida's Reviewed-by.

Best Regards,
- Maíra Canal

Maíra Canal (2):
  drm/vc4: replace drm_gem_dma_object for drm_gem_object in vc4_exec_info
  drm/vc4: replace obj lookup steps with drm_gem_objects_lookup

 drivers/gpu/drm/vc4/vc4_drv.h      |  2 +-
 drivers/gpu/drm/vc4/vc4_gem.c      | 78 +++++++-----------------------
 drivers/gpu/drm/vc4/vc4_validate.c |  4 +-
 3 files changed, 21 insertions(+), 63 deletions(-)

Comments

Maíra Canal Feb. 23, 2023, 12:43 p.m. UTC | #1

Ping.

Best Regards,
- Maíra Canal

On 2/2/23 08:19, Maíra Canal wrote:
> Currently, the array of BOs that are lookup up at the start of exec is being
> instantiated as drm_gem_dma_object, which is not needed and makes it difficult
> to use the drm_gem_objects_lookup() helper. Therefore, replace
> drm_gem_dma_object for drm_gem_object and then replace obj lookup steps with
> drm_gem_objects_lookup().
> 
> v1 -> v2:
> 
> * Add André Almeida's Reviewed-by.
> 
> Best Regards,
> - Maíra Canal
> 
> Maíra Canal (2):
>    drm/vc4: replace drm_gem_dma_object for drm_gem_object in vc4_exec_info
>    drm/vc4: replace obj lookup steps with drm_gem_objects_lookup
> 
>   drivers/gpu/drm/vc4/vc4_drv.h      |  2 +-
>   drivers/gpu/drm/vc4/vc4_gem.c      | 78 +++++++-----------------------
>   drivers/gpu/drm/vc4/vc4_validate.c |  4 +-
>   3 files changed, 21 insertions(+), 63 deletions(-)
>

Maíra Canal Feb. 28, 2023, 10:51 p.m. UTC | #2

On 2/2/23 08:19, Maíra Canal wrote:
> Currently, the array of BOs that are lookup up at the start of exec is being
> instantiated as drm_gem_dma_object, which is not needed and makes it difficult
> to use the drm_gem_objects_lookup() helper. Therefore, replace
> drm_gem_dma_object for drm_gem_object and then replace obj lookup steps with
> drm_gem_objects_lookup().
> 
> v1 -> v2:
> 
> * Add André Almeida's Reviewed-by.
> 
> Best Regards,
> - Maíra Canal
> 
> Maíra Canal (2):
>    drm/vc4: replace drm_gem_dma_object for drm_gem_object in vc4_exec_info
>    drm/vc4: replace obj lookup steps with drm_gem_objects_lookup
> 
>   drivers/gpu/drm/vc4/vc4_drv.h      |  2 +-
>   drivers/gpu/drm/vc4/vc4_gem.c      | 78 +++++++-----------------------
>   drivers/gpu/drm/vc4/vc4_validate.c |  4 +-
>   3 files changed, 21 insertions(+), 63 deletions(-)
> 

Applied to drm-misc-next.

Best Regards,
- Maíra Canal