Re: [PATCH v2] drm/amdkfd: set uuid for each partition in topology

[Lazar, Lijo]

On 8/8/2025 8:30 PM, Eric Huang wrote:
> Currently each kfd compute partition/node is sharing
> the same uuid of AID, which doen't meet the CUDA spec
> for visible device, so corresponding XCD id for each
> partition in smu has been assigned to xcp, and exposed
> to kfd topology.
> 
> v2: add NULL check (Lijo)
> 
> Signed-off-by: Eric Huang <jinhuieric.hu...@amd.com>

Reviewed-by: Lijo Lazar <lijo.la...@amd.com>

Thanks,
Lijo

> ---
>  drivers/gpu/drm/amd/amdkfd/kfd_topology.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_topology.c 
> b/drivers/gpu/drm/amd/amdkfd/kfd_topology.c
> index 4ec73f33535e..82dbd68d8c99 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_topology.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_topology.c
> @@ -530,6 +530,8 @@ static ssize_t node_show(struct kobject *kobj, struct 
> attribute *attr,
>               sysfs_show_32bit_prop(buffer, offs, "sdma_fw_version",
>                                     dev->gpu->kfd->sdma_fw_version);
>               sysfs_show_64bit_prop(buffer, offs, "unique_id",
> +                                   dev->gpu->xcp ?
> +                                   dev->gpu->xcp->unique_id :
>                                     dev->gpu->adev->unique_id);
>               sysfs_show_32bit_prop(buffer, offs, "num_xcc",
>                                     NUM_XCC(dev->gpu->xcc_mask));