Re: [PATCH] drm/xe/configfs: Fix is_bound() pci_dev lifetime

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Wed, 21 Jan 2026 09:37:51 -0800, Shuicheng Lin wrote:
>
> Move pci_dev_put() after pci_dbg() to avoid using pdev after dropping its
> reference.
>
> Fixes: 2674f1ef29f46 ("drm/xe/configfs: Block runtime attribute changes")
> Signed-off-by: Shuicheng Lin <shuicheng.lin@intel.com>

Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_configfs.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_configfs.c b/drivers/gpu/drm/xe/xe_configfs.c
> index 9f6251b1008b..82edd0466005 100644
> --- a/drivers/gpu/drm/xe/xe_configfs.c
> +++ b/drivers/gpu/drm/xe/xe_configfs.c
> @@ -347,11 +347,10 @@ static bool is_bound(struct xe_config_group_device *dev)
>		return false;
>
>	ret = pci_get_drvdata(pdev);
> -	pci_dev_put(pdev);
> -
>	if (ret)
>		pci_dbg(pdev, "Already bound to driver\n");
>
> +	pci_dev_put(pdev);
>	return ret;
>  }
>
> --
> 2.50.1
>