Launchpad has imported 9 comments from the remote bug at
https://bugs.webkit.org/show_bug.cgi?id=160491.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2016-08-03T06:53:14+00:00 Cgarcia-f wrote:

This is becoming the default in several linux distributions and it makes
WebKitGTK+ unusable in accelerated compositing mode, which is now always
enabled because since we switched to use the threaded compositor. The
problem seems to be an optimization of the intel driver for windows that
are offscreen, and our redirected window is always positioned at
ScreenWidth + 1, 0.

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/0

------------------------------------------------------------------------
On 2016-08-03T06:59:05+00:00 Cgarcia-f wrote:

Created attachment 285205
Patch

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/1

------------------------------------------------------------------------
On 2016-08-03T07:50:07+00:00 Svillar wrote:

I was tempted to r+ but it looks like this is an undesirable side
effect, so I'd prefer not to rely on that. From the freedesktop bug:

"Weird, not sure why that avoids the problem; maybe there's an off-by-
one bug somewhere which causes the window to be incorrectly considered
on-screen. I wouldn't recommend relying on that. Also, again I'm not
sure that negative coordinates can never be visible."

Also

"Assuming a window is really the only possibility, (why) can't it set
the swap interval to 0 via one of the GLX extensions for this, to
prevent SwapBuffers operations from getting throttled?"

Can't we do this instead?

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/2

------------------------------------------------------------------------
On 2016-08-03T08:10:51+00:00 Cgarcia-f wrote:

(In reply to comment #2)
> I was tempted to r+ but it looks like this is an undesirable side effect, so
> I'd prefer not to rely on that. From the freedesktop bug:
> 
> "Weird, not sure why that avoids the problem; maybe there's an off-by-one
> bug somewhere which causes the window to be incorrectly considered
> on-screen. I wouldn't recommend relying on that. Also, again I'm not sure
> that negative coordinates can never be visible."

Yes, but he also said that using WidthOfScreen is bad in any case, so if
the patch fixes that and works as a workaround, we could land it in any
case and then continue working a better solution, but with a workaround
to make WebKitGTK+ usable again.

> Also
> 
> "Assuming a window is really the only possibility, (why) can't it set the
> swap interval to 0 via one of the GLX extensions for this, to prevent
> SwapBuffers operations from getting throttled?"
> 
> Can't we do this instead?

I guess, I'm not a graphics expert, and I don't even know if we really
need to use a window or not. I'll take a look at the swap interval thing
in any case.

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/3

------------------------------------------------------------------------
On 2016-08-03T19:40:48+00:00 Mcatanzaro wrote:

(In reply to comment #3)
> Yes, but he also said that using WidthOfScreen is bad in any case

True. r=me on that basis, but I recommend giving the upstream bug a
couple days to settle before committing, now that the Xorg folks are
actively looking into the issue.

We also should have an answer to their question as to why we use an
offscreen window instead of a pixmap, which is clearly their
recommendation.

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/4

------------------------------------------------------------------------
On 2016-08-03T19:41:51+00:00 Mcatanzaro wrote:

CCing folks who might possibly have an answer to the question in comment
#4.

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/5

------------------------------------------------------------------------
On 2016-08-03T19:44:14+00:00 Mcatanzaro wrote:

(In reply to comment #2)
> I was tempted to r+ but it looks like this is an undesirable side effect, so
> I'd prefer not to rely on that.

My main concern is that it might be a bug that the negative offset works
to avoid throttling.

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/6

------------------------------------------------------------------------
On 2016-08-04T04:55:30+00:00 Cgarcia-f wrote:

(In reply to comment #6)
> (In reply to comment #2)
> > I was tempted to r+ but it looks like this is an undesirable side effect, so
> > I'd prefer not to rely on that.
> 
> My main concern is that it might be a bug that the negative offset works to
> avoid throttling.

Right, that's why I don't consider it a fix, if that's actually a bug
and is fixed eventually the performance will be bad again, but I'm
pretty sure we will not be the only ones affected like now, because most
of the code that creates an offscreen window use negative coords, either
-1, -1 or -100, -100.

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/7

------------------------------------------------------------------------
On 2016-08-08T05:42:25+00:00 Cgarcia-f wrote:

Committed r204250: <http://trac.webkit.org/changeset/204250>

Reply at: https://bugs.launchpad.net/ubuntu/+source/xorg-
server/+bug/1615871/comments/8


** Changed in: webkit-open-source
       Status: Unknown => Fix Released

** Changed in: webkit-open-source
   Importance: Unknown => Medium

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to xorg-server in Ubuntu.
https://bugs.launchpad.net/bugs/1615871

Title:
  Poor performance with WebKit on yakkety with Intel modesetting enabled

Status in WebKit:
  Fix Released
Status in xf86-video-intel:
  Unknown
Status in webkit2gtk package in Ubuntu:
  New
Status in webkitgtk package in Ubuntu:
  New
Status in xorg-server package in Ubuntu:
  New

Bug description:
  https://tjaalton.wordpress.com/2016/07/23/intel-graphics-gen4-and-
  newer-now-defaults-to-modesetting-driver-on-x/

  There are reports that WebKitGTK performance is much worse with
  Intel's modesetting driver (which was enabled a month ago for
  yakkety).

  Upstream bug attached.

To manage notifications about this bug go to:
https://bugs.launchpad.net/webkit-open-source/+bug/1615871/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to