Launchpad has imported 12 comments from the remote bug at
https://bugs.freedesktop.org/show_bug.cgi?id=54654.

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 2012-09-07T20:53:13+00:00 Nick Bowler wrote:

After upgrading from xserver 1.12.99.905 (the very last 1.13 RC) to xserver
1.13.0, screen crossings have ceased to work correctly, in a 2-screen Zaphod
setup.  The pointer starts on the left screen (0); when attempting to move it
rightwards across the boundary to the right screen (1), instead of actually
changing screens, the pointer teleports all the way to the left of the screen
it's already on.

Bisection pinpoints the following:

bafbd99080be49a17be97d2cc758fbe623369945 is the first bad commit
commit bafbd99080be49a17be97d2cc758fbe623369945
Author: Peter Hutterer <[email protected]>
Date:   Wed Aug 8 11:34:32 2012 +1000

    dix: work around scaling issues during WarpPointer (#53037)
    
    In WarpPointer calls, we get input in screen coordinates. They must be
    scaled to device coordinates, and then back to screen coordinates for screen
    crossing and root coordinates in events.
    
    The rounding errors introduced (and clipping in core/XI 1.x events) can lead
    to the actual position being different to the requested input coordinates.
    e.g. 200 scales to 199.9999, truncated to 199 in the event.
    
    Avoid this by simply overwriting the scaled screen coordinates with the
    input coordinates for the POINTER_SCREEN case.
    
    X.Org Bug 53037 <http://bugs.freedesktop.org/show_bug.cgi?id=53037>
    
    Signed-off-by: Peter Hutterer <[email protected]>
    Reviewed-by: Keith Packard <[email protected]>

:040000 040000 f187f7ca2569a510c2f614cc75f9d5e3f4b09b85
394c22bc56781077b7384804648db21c6354eca8 M      dix

Reverting this commit on top of 1.13.0 resolves the issue.

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

------------------------------------------------------------------------
On 2012-10-13T21:29:34+00:00 High-entropy wrote:

Hi,

Arch (stable) upgraded xorg-server to 1.13.
I use a similar Zaphod configuration as described above and
I'm facing the same annoying bug now.

Any news on this issue?

With best regards,
Till

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

------------------------------------------------------------------------
On 2012-10-13T21:32:58+00:00 James Le Cuirot wrote:

In the meantime, you can still revert the commit without any other
problems but that's probably a little easier to do on Gentoo (my distro)
than it is on Arch. ;)

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

------------------------------------------------------------------------
On 2012-10-14T19:16:18+00:00 High-entropy wrote:

I guess my comment was a bit misleading... :)

While it would be possible to revert the commit (thanks to Nick), I just
wanted to ask if the issue will be addressed in 1.13.1. The status is
still set to NEW.

Shouldn't the "Component" be changed to "Input/*"?
Not sure if Peter Hutterer knows about this regression.

Unfortunately, I'm not familiar with the inner workings of X to provide
a patch.

~Till

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

------------------------------------------------------------------------
On 2012-10-15T16:02:29+00:00 Nick Bowler wrote:

(In reply to comment #3)
> While it would be possible to revert the commit (thanks to Nick), I just
> wanted to ask if the issue will be addressed in 1.13.1. The status is still
> set to NEW.

Yes, it would be nice to have some feedback about this 11th-hour
regression.

What's the point in testing the release candidates if regressions are going to
be introduced (and apparently reports ignored) right before release anyway?

> Shouldn't the "Component" be changed to "Input/*"?

That's what I thought at first, but all the Input/ components seem to be for
specific drivers.  This issue isn't driver-specific, it's a problem with the
server.

That being said, this bugtracker has waaaaaaaaay too many components.

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

------------------------------------------------------------------------
On 2012-10-17T11:45:58+00:00 09rp9wq1q1h0lxrf-tom wrote:

*** Bug 55993 has been marked as a duplicate of this bug. ***

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

------------------------------------------------------------------------
On 2012-10-17T12:46:48+00:00 High-entropy wrote:

Wouldn't it make sense to assign this bug report to Peter Hutterer?

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

------------------------------------------------------------------------
On 2012-10-18T05:19:40+00:00 Peter Hutterer wrote:

Created attachment 68748
0001-dix-fix-zaphod-screen-scrossing-54654.patch

adding the offset of the new screen should be enough, can you please
test this? thanks

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

------------------------------------------------------------------------
On 2012-10-18T14:25:59+00:00 Nick Bowler wrote:

(In reply to comment #7)
> Created attachment 68748 [details] [review]
> 0001-dix-fix-zaphod-screen-scrossing-54654.patch
> 
> adding the offset of the new screen should be enough, can you please test
> this? thanks

Seems to work, thanks!

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

------------------------------------------------------------------------
On 2012-10-19T02:22:09+00:00 High-entropy wrote:

Thanks. I can also confirm this patch to work on two different zaphod
setups.

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

------------------------------------------------------------------------
On 2012-10-19T22:01:17+00:00 Ihadzic wrote:

FWIW, it fixes the problem for me too. Is this going to be merged at
some point? I don't see it in git yet.

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

------------------------------------------------------------------------
On 2012-10-24T20:14:27+00:00 James Le Cuirot wrote:

Looking good here, thanks Peter.

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


** Changed in: xorg-server
       Status: Unknown => In Progress

** Changed in: xorg-server
   Importance: Unknown => Medium

** Bug watch added: freedesktop.org Bugzilla #53037
   https://bugs.freedesktop.org/show_bug.cgi?id=53037

-- 
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/1068920

Title:
  Pointer screen crossings broken in Xorg server 1.13.0 (regression)

Status in X.Org X server:
  In Progress
Status in “xorg-server” package in Ubuntu:
  Triaged

Bug description:
  The Xorg server in quantal suffers from the following upstream
  regression for zaphod setups:

  https://bugs.freedesktop.org/show_bug.cgi?id=54654

  When you move the mouse into the second screen, it gets trapped there.  When 
it gets to the
  edge where it should move back onto the first screen, it wraps around to the 
far edge of the
  second screen again.

  I've rebuilt xserver-xorg-core with the attached patch, which comes from the 
upstream
  bug above, and confirmed it fixes this issue.

To manage notifications about this bug go to:
https://bugs.launchpad.net/xorg-server/+bug/1068920/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to