[V2 09/11] drm/amdgpu/virt: enable virtual display
Deucher, Alexander
Alexander.Deucher at amd.com
Tue Jan 10 15:44:08 UTC 2017
> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Xiangliang Yu
> Sent: Tuesday, January 10, 2017 5:01 AM
> To: amd-gfx at lists.freedesktop.org
> Cc: Yu, Xiangliang
> Subject: [V2 09/11] drm/amdgpu/virt: enable virtual display
>
> Virtual display is default setting for virtualization, enable it.
>
> Signed-off-by: Xiangliang Yu <Xiangliang.Yu at amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
> index f32a789..5baca9f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
> @@ -25,6 +25,9 @@
>
> void amdgpu_virt_init_setting(struct amdgpu_device *adev)
> {
> + /* enable virtual display */
> + adev->mode_info.num_crtc = 1;
> +
Should probably also set adev->enable_virtual_display = true here. With that fixed:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
As a follow on patch, it would be nice to rework amdgpu_device_enable_virtual_display() to set some sensible defaults in the sr-iov case, so it's all handled in one place.
Alex
> mutex_init(&adev->virt.lock);
> }
>
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
More information about the amd-gfx
mailing list