Received-SPF: pass (google.com: domain of linux-kernel+bounces-173005-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Wed, 8 May 2024 11:56:46 +0200
From: Gerd Hoffmann <kraxel@redhat.com>
To: Timo Lindfors <timo.lindfors@iki.fi>
Cc: David Airlie <airlied@redhat.com>, 
	Linus Torvalds <torvalds@linux-foundation.org>, Maxime Ripard <mripard@kernel.org>, 
	Steven Rostedt <rostedt@goodmis.org>, LKML <linux-kernel@vger.kernel.org>, 
	Alex Constantino <dreaming.about.electric.sheep@gmail.com>, Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, 
	Thomas Zimmermann <tzimmermann@suse.de>, Daniel Vetter <daniel@ffwll.ch>
Subject: Re: [BUG][v6.9-rc6] Deadlock with: Revert "drm/qxl: simplify
 qxl_fence_wait"
Message-ID: <ibl7jsqhbdga64nasmrruh3t2oa7rxqmwxywe6wtdfsqrliyue@tbgmm6uvxsqi>
References: <20240502081641.457aa25f@gandalf.local.home>
 <20240504043957.417aa98c@rorschach.local.home>
 <20240506-cuddly-elated-agouti-be981d@houat>
 <CAHk-=wiS70D1sbhsvNfR0e5YjfG2NV0cVKWz9vp=_F_wkw3j9Q@mail.gmail.com>
 <CAMwc25qMXOFOfKsa7BRi3dz125PDyvVtgTty0Q4bkAFftJDLqQ@mail.gmail.com>
 <alpine.DEB.2.20.2405070933070.20162@mail.home>
 <obume4wdvgxc3zljnelhpwrg2rouae322nak5jy3s4hsruoddd@stoopms6fo2a>
 <alpine.DEB.2.20.2405071833510.21166@mail.home>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <alpine.DEB.2.20.2405071833510.21166@mail.home>

On Tue, May 07, 2024 at 06:46:41PM GMT, Timo Lindfors wrote:
> The qxl + spice + firefox combination scrolls smoothly even if I force
> firefox to use wayland.

scrolling sending 2d accel bitblits works only when running the xorg
xserver with the qxl driver.  xwayland wouldn't do that.  So that
doesn't explain the performance difference.

> I suppose I could write a more synthetic test if it would be useful?

Well, don't work on qemu graphics any more.  So I can't sink much time
into this, and my memories on details are starting to fade ...

> With systemtap I can trace what happens during a single scroll event.

Had a brief look at the driver source code.  One possibly important
difference is damage tracking.

qxl seems to send updates for the list of damaged rectangle to the host
(see qxl_draw_dirty_fb).

virtio-gpu goes call drm_atomic_helper_damage_merged() to get a single
rectangle covering all regions which need updating and works with that.

Which could very well explain the performance difference.  If you want
dig deeper I'd suggest to start there.

HTH & take care,
  Gerd