Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Gerd Hoffmann <kraxel@redhat.com>
To:     dri-devel@lists.freedesktop.org
Cc:     gurchetansingh@chromium.org, Gerd Hoffmann <kraxel@redhat.com>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        virtualization@lists.linux-foundation.org (open list:VIRTIO GPU DRIVER),
        linux-kernel@vger.kernel.org (open list)
Subject: [PATCH 1/3] drm/virtio: skip set_scanout if framebuffer didn't change
Date:   Wed, 11 Dec 2019 09:42:14 +0100
Message-Id: <20191211084216.25405-2-kraxel@redhat.com>
In-Reply-To: <20191211084216.25405-1-kraxel@redhat.com>
References: <20191211084216.25405-1-kraxel@redhat.com>
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 drivers/gpu/drm/virtio/virtgpu_plane.c | 31 ++++++++++++++------------
 1 file changed, 17 insertions(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/virtio/virtgpu_plane.c b/drivers/gpu/drm/virti=
o/virtgpu_plane.c
index bc4bc4475a8c..a0f91658c2bc 100644
--- a/drivers/gpu/drm/virtio/virtgpu_plane.c
+++ b/drivers/gpu/drm/virtio/virtgpu_plane.c
@@ -151,20 +151,23 @@ static void virtio_gpu_primary_plane_update(struct dr=
m_plane *plane,
 =09if (bo->dumb)
 =09=09virtio_gpu_update_dumb_bo(vgdev, bo, plane->state);
=20
-=09DRM_DEBUG("handle 0x%x, crtc %dx%d+%d+%d, src %dx%d+%d+%d\n",
-=09=09  bo->hw_res_handle,
-=09=09  plane->state->crtc_w, plane->state->crtc_h,
-=09=09  plane->state->crtc_x, plane->state->crtc_y,
-=09=09  plane->state->src_w >> 16,
-=09=09  plane->state->src_h >> 16,
-=09=09  plane->state->src_x >> 16,
-=09=09  plane->state->src_y >> 16);
-=09virtio_gpu_cmd_set_scanout(vgdev, output->index,
-=09=09=09=09   bo->hw_res_handle,
-=09=09=09=09   plane->state->src_w >> 16,
-=09=09=09=09   plane->state->src_h >> 16,
-=09=09=09=09   plane->state->src_x >> 16,
-=09=09=09=09   plane->state->src_y >> 16);
+=09if (plane->state->fb !=3D old_state->fb) {
+=09=09DRM_DEBUG("handle 0x%x, crtc %dx%d+%d+%d, src %dx%d+%d+%d\n",
+=09=09=09  bo->hw_res_handle,
+=09=09=09  plane->state->crtc_w, plane->state->crtc_h,
+=09=09=09  plane->state->crtc_x, plane->state->crtc_y,
+=09=09=09  plane->state->src_w >> 16,
+=09=09=09  plane->state->src_h >> 16,
+=09=09=09  plane->state->src_x >> 16,
+=09=09=09  plane->state->src_y >> 16);
+=09=09virtio_gpu_cmd_set_scanout(vgdev, output->index,
+=09=09=09=09=09   bo->hw_res_handle,
+=09=09=09=09=09   plane->state->src_w >> 16,
+=09=09=09=09=09   plane->state->src_h >> 16,
+=09=09=09=09=09   plane->state->src_x >> 16,
+=09=09=09=09=09   plane->state->src_y >> 16);
+=09}
+
 =09virtio_gpu_cmd_resource_flush(vgdev, bo->hw_res_handle,
 =09=09=09=09      plane->state->src_x >> 16,
 =09=09=09=09      plane->state->src_y >> 16,
--=20
2.18.1