Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paint issues on latest trunk #1225

Closed
totaam opened this issue Jun 8, 2016 · 12 comments
Closed

Paint issues on latest trunk #1225

totaam opened this issue Jun 8, 2016 · 12 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Jun 8, 2016

Issue migrated from trac ticket # 1225

component: server | priority: major | resolution: fixed

2016-06-08 19:25:06: maxmylyn created the issue


This is a spinoff from #1218

I'll attach the relevant xpra info and a quick video with the paint boxes enabled. I'm honestly not sure what's up - It works fine if I roll back to r12730, but somewhere between there and r12758 it breaks. I've tried bisecting but I had no luck even after a full day of incrementally rolling through the build revisions. Somewhere around r12750 it breaks.

Once I get it broken, the repro is interesting. I don't get it with Xterms, and with Chrome it's sporadic. It's most easy to hunt down in Firefox.

  • Open Firefox
  • Try to highlight some text
  • It may or may not work
  • If it highlights fine then try scrolling up and down first
  • I find that's the easiest way to repro this

Start command:

xpra start :13 --bind-tcp=0.0.0.0:2200 --start-new-commands=yes --start-child=xterm --start-child="xterm -bg black -cr white -fg white" --start-child="xterm -bg black -cr white -fg white"

And connecting:

xpra attach tcp:ip:port

and sometimes:

XPRA_OPENGL_PAINT_BOX=1 xpra attach tcp:ip:port


For what it's worth, the server is a Fedora 23 hardware machine running Fedora Server. The client is also hardware, but has an Nvidia 745 GPU, the server only Intel. I also run into this issue with AND without OpenGL enabled.

@totaam
Copy link
Collaborator Author

totaam commented Jun 8, 2016

2016-06-08 19:26:06: maxmylyn uploaded file 1225 paint isues.mp4 (3378.7 KiB)

Video of the paint issue. Note that the pauses in between are me clicking to highlight text.

@totaam
Copy link
Collaborator Author

totaam commented Jun 8, 2016

2016-06-08 19:26:57: maxmylyn uploaded file 1225xprainfo.txt (141.0 KiB)

@totaam
Copy link
Collaborator Author

totaam commented Jun 9, 2016

2016-06-09 06:21:00: antoine changed owner from antoine to maxmylyn

@totaam
Copy link
Collaborator Author

totaam commented Jun 9, 2016

2016-06-09 06:21:00: antoine commented


Please include the bisection data. Even an unsuccessful bisection generates some data which may be useful in hunting down the bug. For an example of bisection data logging see #557#comment:1

Also, please verify that this is a regression by testing the latest 0.17.x (and even maybe 0.14.x), bearing in mind that the bug may be present there and just harder to hit.

Does this also occur if you start the server with XPRA_VIDEO_SUBREGION=0?

Does this patch help? [/attachment/ticket/1218/track-nonvideo.patch]

Of all the commits between r12730 and r12750:

@totaam
Copy link
Collaborator Author

totaam commented Jun 9, 2016

2016-06-09 22:54:57: maxmylyn changed owner from maxmylyn to antoine

@totaam
Copy link
Collaborator Author

totaam commented Jun 9, 2016

2016-06-09 22:54:57: maxmylyn commented


Retested a couple times:

  • Latest trunk with XPRA_VIDEO_SUBREGION=0 doesn't seem to trigger the issue

  • latest trunk being r12771

  • The given patch had no effect on latest trunk

  • As per your comment (comment:1), I zeroed in on r12744, and found that to be the revision where the break is introduced

  • Issue does not show up in the latest 0.17.X branch.

@totaam
Copy link
Collaborator Author

totaam commented Jun 10, 2016

2016-06-10 12:47:38: antoine changed owner from antoine to maxmylyn

@totaam
Copy link
Collaborator Author

totaam commented Jun 10, 2016

2016-06-10 12:47:38: antoine commented


See #1218#comment:10

@totaam
Copy link
Collaborator Author

totaam commented Jun 10, 2016

2016-06-10 20:34:45: maxmylyn changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Jun 10, 2016

2016-06-10 20:34:45: maxmylyn set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented Jun 10, 2016

2016-06-10 20:34:45: maxmylyn commented


As per #1218, it looks like this one is fixed in r12780.

Closing, but will keep a good eye out for this one in case we see it again.

@totaam totaam closed this as completed Jun 10, 2016
@totaam
Copy link
Collaborator Author

totaam commented Jul 12, 2016

2016-07-12 17:52:23: antoine commented


Milestone renamed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant