Re: [PATCH v2] drm/vmwgfx: Remove unused vmw_goal_waiter_add/remove

[Maaz Mombasawala]

On 7/22/26 9:36 PM, Saswat Jung Khadka wrote:
> vmw_goal_waiter_add() and vmw_goal_waiter_remove() have had no
> callers since commit db6a94b26354 ("drm/vmwgfx: Implement
> dma_fence_ops properly"), which removed their only call site.
> Their sibling functions vmw_seqno_waiter_add/remove are still
> in active use, confirming these are leftover dead code.
> 
> Signed-off-by: Saswat Jung Khadka <saswatjungkhadka@xxxxxxxxx>
> ---
>  drivers/gpu/drm/vmwgfx/vmwgfx_drv.h |  2 --
>  drivers/gpu/drm/vmwgfx/vmwgfx_irq.c | 12 ------------
>  2 files changed, 14 deletions(-)
> 
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h
> index 38bea8aba..d84e077ed 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h
> @@ -1026,8 +1026,6 @@ extern int vmw_fallback_wait(struct vmw_private *dev_priv,
>  			     unsigned long timeout);
>  bool vmw_seqno_waiter_add(struct vmw_private *dev_priv);
>  bool vmw_seqno_waiter_remove(struct vmw_private *dev_priv);
> -bool vmw_goal_waiter_add(struct vmw_private *dev_priv);
> -bool vmw_goal_waiter_remove(struct vmw_private *dev_priv);
>  bool vmw_generic_waiter_add(struct vmw_private *dev_priv, u32 flag,
>  			    int *waiter_count);
>  bool vmw_generic_waiter_remove(struct vmw_private *dev_priv,
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_irq.c b/drivers/gpu/drm/vmwgfx/vmwgfx_irq.c
> index 05773eb39..88c77226c 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_irq.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_irq.c
> @@ -273,18 +273,6 @@ bool vmw_seqno_waiter_remove(struct vmw_private *dev_priv)
>  					&dev_priv->fence_queue_waiters);
>  }
>  
> -bool vmw_goal_waiter_add(struct vmw_private *dev_priv)
> -{
> -	return vmw_generic_waiter_add(dev_priv, vmw_irqflag_fence_goal(dev_priv),
> -					&dev_priv->goal_queue_waiters);
> -}
> -
> -bool vmw_goal_waiter_remove(struct vmw_private *dev_priv)
> -{
> -	return vmw_generic_waiter_remove(dev_priv, vmw_irqflag_fence_goal(dev_priv),
> -					&dev_priv->goal_queue_waiters);
> -}
> -
>  static void vmw_irq_preinstall(struct drm_device *dev)
>  {
>  	struct vmw_private *dev_priv = vmw_priv(dev);

LGTM!

Reviewed-by: Maaz Mombasawala <maaz.mombasawala@xxxxxxxxxxxx>

-- 
Maaz Mombasawala <maaz.mombasawala@xxxxxxxxxxxx>