virtio-gpu: add atomic_commit function
authorGerd Hoffmann <kraxel@redhat.com>
Tue, 31 May 2016 06:50:47 +0000 (08:50 +0200)
committerGerd Hoffmann <kraxel@redhat.com>
Tue, 7 Jun 2016 08:33:12 +0000 (10:33 +0200)
Do modesets first, then call drm_atomic_helper_commit_planes with
active_only = true.  That way the outputs doesn't get disabled
temporarly on atomic commits.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
drivers/gpu/drm/virtio/virtgpu_display.c
drivers/gpu/drm/virtio/virtgpu_drv.h

index d4305da88f44a3d28d8c6db5fad066eac52ebcf9..c5d33e051f5cffc866aac6f0cf1097934a2a0bf0 100644 (file)
@@ -466,10 +466,30 @@ virtio_gpu_user_framebuffer_create(struct drm_device *dev,
        return &virtio_gpu_fb->base;
 }
 
+static int vgdev_atomic_commit(struct drm_device *dev,
+                              struct drm_atomic_state *state,
+                              bool nonblock)
+{
+       if (nonblock)
+               return -EBUSY;
+
+       drm_atomic_helper_swap_state(dev, state);
+       drm_atomic_helper_wait_for_fences(dev, state);
+
+       drm_atomic_helper_commit_modeset_disables(dev, state);
+       drm_atomic_helper_commit_modeset_enables(dev, state);
+       drm_atomic_helper_commit_planes(dev, state, true);
+
+       drm_atomic_helper_wait_for_vblanks(dev, state);
+       drm_atomic_helper_cleanup_planes(dev, state);
+       drm_atomic_state_free(state);
+       return 0;
+}
+
 static const struct drm_mode_config_funcs virtio_gpu_mode_funcs = {
        .fb_create = virtio_gpu_user_framebuffer_create,
        .atomic_check = drm_atomic_helper_check,
-       .atomic_commit = drm_atomic_helper_commit,
+       .atomic_commit = vgdev_atomic_commit,
 };
 
 int virtio_gpu_modeset_init(struct virtio_gpu_device *vgdev)
index 0a54f43f846a9d560d4b626872a86eecda1cdaf7..ecd1c4398e93652ad6e61c45e432869fd031c738 100644 (file)
@@ -33,6 +33,7 @@
 
 #include <drm/drmP.h>
 #include <drm/drm_gem.h>
+#include <drm/drm_atomic.h>
 #include <drm/drm_crtc_helper.h>
 #include <ttm/ttm_bo_api.h>
 #include <ttm/ttm_bo_driver.h>