Re: [PATCH] drm/amdgpu: Hide xcp partition sysfs under SRIOV

["Lazar, Lijo" <lijo.lazar@xxxxxxx>]

On 8/24/2023 8:53 PM, Rajneesh Bhardwaj wrote:
> XCP partitions should not be visible for the VF for GFXIP 9.4.3.
>
> Signed-off-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@xxxxxxx>

Reviewed-by: Lijo Lazar <lijo.lazar@xxxxxxx>

Thanks,
Lijo

> ---
>   drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c | 11 ++++++++---
>   1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c
> index b4fdb269f856..b1ca3014a9e2 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c
> @@ -863,11 +863,15 @@ static int gfx_v9_4_3_sw_init(void *handle)
>   	if (r)
>   		return r;
>   
> -	r = amdgpu_gfx_sysfs_init(adev);
> +	r = amdgpu_gfx_ras_sw_init(adev);
>   	if (r)
>   		return r;
>   
> -	return amdgpu_gfx_ras_sw_init(adev);
> +
> +	if (!amdgpu_sriov_vf(adev))
> +		r = amdgpu_gfx_sysfs_init(adev);
> +
> +	return r;
>   }
>   
>   static int gfx_v9_4_3_sw_fini(void *handle)
> @@ -888,7 +892,8 @@ static int gfx_v9_4_3_sw_fini(void *handle)
>   	gfx_v9_4_3_mec_fini(adev);
>   	amdgpu_bo_unref(&adev->gfx.rlc.clear_state_obj);
>   	gfx_v9_4_3_free_microcode(adev);
> -	amdgpu_gfx_sysfs_fini(adev);
> +	if (!amdgpu_sriov_vf(adev))
> +		amdgpu_gfx_sysfs_fini(adev);
>   
>   	return 0;
>   }