[PATCH] drm/amd/pm: enable pm sysfs write for one VF mode

Yao, Yiqing(James) yiqinyao at amd.com
Tue Feb 15 04:05:34 UTC 2022


On 2/14/2022 10:54 AM, Liu, Monk wrote:

> [AMD Official Use Only]
>
> Reviewed-by: Monk.liu at amd.com
>
> Thanks
>
> -------------------------------------------------------------------
>
> Monk Liu | Cloud GPU & Virtualization Solution | AMD
>

On 1/24/2022 6:01 PM, Yiqing Yao wrote:
> [why]
> pm sysfs should be writable in one VF mode as is in passthrough
>
> [how]
> do not remove write access on pm sysfs if device is in one VF mode
>
> Fixes: 4436ea3c43a7 ("amdgpu/pm: Make sysfs pm attributes as read-only for VFs")
> Signed-off-by: Yiqing Yao<yiqing.yao at amd.com>
> ---
>   drivers/gpu/drm/amd/pm/amdgpu_pm.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/amdgpu_pm.c b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> index 837a31a46596..5250032a2ffb 100644
> --- a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> +++ b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> @@ -2006,8 +2006,8 @@ static int default_attr_update(struct amdgpu_device *adev, struct amdgpu_device_
>   		}
>   	}
>   
> -	/* setting should not be allowed from VF */
> -	if (amdgpu_sriov_vf(adev)) {
> +	/* setting should not be allowed from VF if not in one VF mode */
> +	if (amdgpu_sriov_vf(adev) && !amdgpu_sriov_is_pp_one_vf(adev)) {
>   		dev_attr->attr.mode &= ~S_IWUGO;
>   		dev_attr->store = NULL;
>   	}
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20220215/e3e4a476/attachment.htm>


More information about the amd-gfx mailing list