Ubuntu
compiz package

The larger windows, moving towed.

Bug #864330 reported by Adrián on 2011-10-01

This bug report is a duplicate of: Bug #866752: Sometimes configure events are missed and windows move slow as a result. Edit Remove

This bug affects 2 people

	Status	Importance	Assigned to	Milestone
Compiz Core	Fix Committed	High	Sam Spilsbury	Compiz Core 0.9.7.0
Unity	Fix Committed	High	Sam Spilsbury	Unity 4.26.0 "SRU1"
compiz (Ubuntu)	Fix Released	High	Sam Spilsbury
Oneiric	Fix Released	High	Sam Spilsbury

Bug Description

steps:
1. Open nautilus.
2. Maximize.
3. Move the window.
You will see a slowdown.

EDIT: Also happens when switching between applications with multiple windows on the unity alt-tab

Testcase:

1. Maximize a window, pull it down, you shouldn't see windows jump around
2. Open two of the same application, open another application in the alt-tab, switch from the application with one window to the application with two, you shouldn't see jumping windows

ProblemType: Bug
DistroRelease: Ubuntu 11.10
Package: unity 4.20.0-0ubuntu2
ProcVersionSignature: Ubuntu 3.0.0-12.19-generic 3.0.4
Uname: Linux 3.0.0-12-generic x86_64
.tmp.unity.support.test.0:

ApportVersion: 1.23-0ubuntu2
Architecture: amd64
CompizPlugins: [core,bailer,detection,composite,opengl,decor,resize,place,imgpng,move,session,gnomecompat,compiztoolbox,mousepoll,vpswitch,wall,grid,wobbly,screenshot,regex,thumbnail,animation,workarounds,expo,ezoom,staticswitcher,fade,scale,unityshell]
CompositorRunning: compiz
Date: Sat Oct 1 23:17:36 2011
DistUpgraded: Log time: 2011-08-12 20:00:23.116487
DistroCodename: oneiric
DistroVariant: ubuntu
GraphicsCard:
ATI Technologies Inc RV350 AS [Radeon 9550] [1002:4153] (prog-if 00 [VGA controller])
Subsystem: PC Partner Limited Device [174b:0200]
Subsystem: PC Partner Limited Device [174b:0201]
InstallationMedia: Ubuntu 11.10 "Oneiric Ocelot" - Alpha amd64 (20110803.1)
Lsusb:
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 002 Device 001: ID 1d6b:0001 Linux Foundation 1.1 root hub
Bus 003 Device 001: ID 1d6b:0001 Linux Foundation 1.1 root hub
MachineType: ASUSTek Computer Inc. K8N
ProcEnviron:
LANG=hu_HU.utf8
SHELL=/bin/bash
ProcKernelCmdLine: BOOT_IMAGE=/boot/vmlinuz-3.0.0-12-generic root=UUID=4d145e22-358d-424c-af08-7fee97796b8e ro quiet splash vt.handoff=7
SourcePackage: unity
UpgradeStatus: Upgraded to oneiric on 2011-09-26 (5 days ago)
dmi.bios.date: 02/16/2006
dmi.bios.vendor: American Megatrends Inc.
dmi.bios.version: 1011.005
dmi.board.name: 'K8N'
dmi.board.vendor: ASUSTeK Computer INC.
dmi.board.version: Rev 1.xx
dmi.chassis.asset.tag: Asset-1234567890
dmi.chassis.type: 3
dmi.chassis.vendor: Chassis Manufacture
dmi.chassis.version: Chassis Version
dmi.modalias: dmi:bvnAmericanMegatrendsInc.:bvr1011.005:bd02/16/2006:svnASUSTekComputerInc.:pnK8N:pvrSystemVersion:rvnASUSTeKComputerINC.:rn'K8N':rvrRev1.xx:cvnChassisManufacture:ct3:cvrChassisVersion:
dmi.product.name: K8N
dmi.product.version: System Version
dmi.sys.vendor: ASUSTek Computer Inc.
version.compiz: compiz 1:0.9.6+bzr20110929-0ubuntu1
version.ia32-libs: ia32-libs 20090808ubuntu25
version.libdrm2: libdrm2 2.4.26-1ubuntu1
version.libgl1-mesa-dri: libgl1-mesa-dri 7.11-0ubuntu3
version.libgl1-mesa-dri-experimental: libgl1-mesa-dri-experimental N/A
version.libgl1-mesa-glx: libgl1-mesa-glx 7.11-0ubuntu3
version.xserver-xorg: xserver-xorg 1:7.6+7ubuntu7
version.xserver-xorg-input-evdev: xserver-xorg-input-evdev 1:2.6.0-1ubuntu13
version.xserver-xorg-video-ati: xserver-xorg-video-ati 1:6.14.99~git20110811.g93fc084-0ubuntu1
version.xserver-xorg-video-intel: xserver-xorg-video-intel 2:2.15.901-1ubuntu2
version.xserver-xorg-video-nouveau: xserver-xorg-video-nouveau 1:0.0.16+git20110411+8378443-1

See original description

Tags:

Related branches

lp:~compiz-team/compiz-core/compiz-core.fix_864330

Merged into lp:compiz-core/0.9.5 at revision 2877

Jason Smith (community): Approve on 2011-10-11

lp:~compiz/compiz/ubuntu

lp:ubuntu/oneiric-proposed/compiz

Revision history for this message

Adrián (jaksi) wrote on 2011-10-01:

BootDmesg.txt Edit (44.6 KiB, text/plain; charset="utf-8")
BootLog.gz Edit (631 bytes, application/x-gzip)
CurrentDmesg.txt Edit (2.2 KiB, text/plain; charset="utf-8")
Dependencies.txt Edit (5.9 KiB, text/plain; charset="utf-8")
DpkgLog.txt Edit (129.8 KiB, text/plain; charset="utf-8")
GconfCompiz.txt Edit (37.1 KiB, text/plain; charset="utf-8")
Lspci.txt Edit (10.9 KiB, text/plain; charset="utf-8")
ProcCpuinfo.txt Edit (567 bytes, text/plain; charset="utf-8")
ProcInterrupts.txt Edit (1.2 KiB, text/plain; charset="utf-8")
ProcModules.txt Edit (3.0 KiB, text/plain; charset="utf-8")
UdevDb.txt Edit (96.4 KiB, text/plain; charset="utf-8")
UdevLog.txt Edit (214.6 KiB, text/plain; charset="utf-8")
UnitySupportTest.txt Edit (582 bytes, text/plain; charset="utf-8")
XorgLog.txt Edit (182.0 KiB, text/plain; charset="utf-8")
XorgLogOld.txt Edit (60.9 KiB, text/plain; charset="utf-8")
Xrandr.txt Edit (5.6 KiB, text/plain; charset="utf-8")
dmidecode.txt.txt Edit (18.3 KiB, text/plain; charset="utf-8")
locale.txt Edit (315 bytes, text/plain; charset="utf-8")
monitors.xml.txt Edit (1.2 KiB, text/plain; charset="utf-8")
peripherals.txt Edit (724 bytes, text/plain; charset="utf-8")
setxkbmap.txt Edit (324 bytes, text/plain; charset="utf-8")
xdpyinfo.txt Edit (25.0 KiB, text/plain; charset="utf-8")
xinput.txt Edit (603 bytes, text/plain; charset="utf-8")
xkbcomp.txt Edit (69.2 KiB, text/plain; charset="utf-8")

Sam Spilsbury (smspillaz) on 2011-10-05

Changed in unity:
status:	New → Confirmed
Changed in unity (Ubuntu):
status:	New → Confirmed
Changed in unity:
importance:	Undecided → Low
assignee:	nobody → Sam Spilsbury (smspillaz)
Changed in unity (Ubuntu):
assignee:	nobody → Sam Spilsbury (smspillaz)
Changed in unity:
milestone:	none → 4.24.0

Revision history for this message

Omer Akram (om26er) wrote on 2011-10-08:

Whats the issue exactly? I dont seem to understand :p

Changed in unity (Ubuntu):
importance:	Undecided → Low

Revision history for this message

Adrián (jaksi) wrote on 2011-10-09:

Videó0147.3gp Edit (2.8 MiB, video/3gpp)

Sam Spilsbury (smspillaz) on 2011-10-10

Changed in unity:
milestone:	4.24.0 → 4.26.0

Revision history for this message

Sam Spilsbury (smspillaz) wrote on 2011-10-10:

The issue is that you see jumping around windows because synchronous movement is forced sometimes

Changed in unity:
importance:	Low → High

Revision history for this message

Adrián (jaksi) wrote on 2011-10-10:

I do not understand the question! Google translate does not translate clearly.

Sam Spilsbury (smspillaz) on 2011-10-11

Changed in compiz-core:
importance:	Undecided → High
assignee:	nobody → Sam Spilsbury (smspillaz)
milestone:	none → 0.9.6
status:	New → Confirmed
description:	updated

Sam Spilsbury (smspillaz) on 2011-10-11

Changed in compiz-core:
status:	Confirmed → Fix Committed
Changed in unity:
status:	Confirmed → Fix Committed
Changed in unity (Ubuntu):
status:	Confirmed → Fix Committed

Didier Roche-Tolomelli (didrocks) on 2011-10-12

affects:	unity (Ubuntu) → compiz (Ubuntu)
Changed in compiz (Ubuntu):
importance:	Low → High

Sam Spilsbury (smspillaz) on 2011-10-12

description:

updated

Revision history for this message

Sam Spilsbury (smspillaz) wrote on 2011-10-17:

Download full text (8.1 KiB)

Proposed follow up fix pending verification:

@@ -45,8 +45,8 @@
bool removeIfMatching (const PendingEvent::Ptr &p, XEvent *);
Index: compiz-0.9.6+bzr20110929/src/privatewindow.h
===================================================================
---- compiz-0.9.6+bzr20110929.orig/src/privatewindow.h 2011-10-12 17:00:11.243394758 +0800
-+++ compiz-0.9.6+bzr20110929/src/privatewindow.h 2011-10-12 17:00:22.599451570 +0800
+--- compiz-0.9.6+bzr20110929.orig/src/privatewindow.h 2011-10-17 15:09:58.385653215 +0800
++++ compiz-0.9.6+bzr20110929/src/privatewindow.h 2011-10-17 15:10:07.125696557 +0800
@@ -211,7 +211,7 @@

void readIconHint ();
@@ -66,8 +66,8 @@
struct timeval lastConfigureRequest;

}
@@ -116,7 +116,25 @@

if (serverFrameGeometry.x () == xwc.x)
valueMask &= ~(CWX);
-@@ -1012,7 +1015,6 @@
+@@ -1004,6 +1007,17 @@
+
+ }
+
++ compiz::X11::PendingEvent::Ptr pc =
++ boost::shared_static_cast<compiz::X11::PendingEvent> (compiz::X11::PendingConfigureEvent::Ptr (
++ new compiz::X11::PendingConfigureEvent (
++ screen->dpy (), serverFrame, valueMask, &xwc)));
++
++ pendingConfigures.add (pc);
++ if (priv->mClearCheckTimeout.active ())
++ priv->mClearCheckT...

Proposed follow up fix pending verification:

=== modified file 'debian/patches/fix-864330.patch'
--- debian/patches/fix-864330.patch	2011-10-12 10:06:03 +0000
+++ debian/patches/fix-864330.patch	2011-10-17 07:10:33 +0000
@@ -1,8 +1,8 @@
 === modified file 'plugins/move/src/move.cpp'
 Index: compiz-0.9.6+bzr20110929/plugins/move/src/move.cpp
 ===================================================================
---- compiz-0.9.6+bzr20110929.orig/plugins/move/src/move.cpp	2011-10-12 17:00:11.359395413 +0800
-+++ compiz-0.9.6+bzr20110929/plugins/move/src/move.cpp	2011-10-12 17:00:22.599451570 +0800
+--- compiz-0.9.6+bzr20110929.orig/plugins/move/src/move.cpp	2011-10-17 15:09:58.353653059 +0800
++++ compiz-0.9.6+bzr20110929/plugins/move/src/move.cpp	2011-10-17 15:10:07.081696339 +0800
 @@ -525,7 +525,7 @@
  		Window xid = CompOption::getIntOptionNamed (options, "window", 0);
  		int    lock = CompOption::getIntOptionNamed (options, "active", 0);
@@ -14,8 +14,8 @@
  	}
 Index: compiz-0.9.6+bzr20110929/src/privatescreen.h
 ===================================================================
---- compiz-0.9.6+bzr20110929.orig/src/privatescreen.h	2011-10-12 17:00:11.319395193 +0800
-+++ compiz-0.9.6+bzr20110929/src/privatescreen.h	2011-10-12 17:00:22.599451570 +0800
+--- compiz-0.9.6+bzr20110929.orig/src/privatescreen.h	2011-10-17 15:09:58.369653137 +0800
++++ compiz-0.9.6+bzr20110929/src/privatescreen.h	2011-10-17 15:10:07.081696339 +0800
 @@ -111,6 +111,8 @@
      virtual ~PendingEvent ();
  
@@ -45,8 +45,8 @@
      bool removeIfMatching (const PendingEvent::Ptr &p, XEvent *);
 Index: compiz-0.9.6+bzr20110929/src/privatewindow.h
 ===================================================================
---- compiz-0.9.6+bzr20110929.orig/src/privatewindow.h	2011-10-12 17:00:11.243394758 +0800
-+++ compiz-0.9.6+bzr20110929/src/privatewindow.h	2011-10-12 17:00:22.599451570 +0800
+--- compiz-0.9.6+bzr20110929.orig/src/privatewindow.h	2011-10-17 15:09:58.385653215 +0800
++++ compiz-0.9.6+bzr20110929/src/privatewindow.h	2011-10-17 15:10:07.125696557 +0800
 @@ -211,7 +211,7 @@
  
  	void readIconHint ();
@@ -66,8 +66,8 @@
  	struct timeval lastConfigureRequest;

Index: compiz-0.9.6+bzr20110929/src/window.cpp
 ===================================================================
---- compiz-0.9.6+bzr20110929.orig/src/window.cpp	2011-10-12 17:00:11.283395067 +0800
-+++ compiz-0.9.6+bzr20110929/src/window.cpp	2011-10-12 17:02:55.120207454 +0800
+--- compiz-0.9.6+bzr20110929.orig/src/window.cpp	2011-10-17 15:09:58.357653075 +0800
++++ compiz-0.9.6+bzr20110929/src/window.cpp	2011-10-17 15:10:29.229806152 +0800
 @@ -868,13 +868,16 @@
  
  	    }
@@ -116,7 +116,25 @@
  
  	if (serverFrameGeometry.x () == xwc.x)
  	    valueMask &= ~(CWX);
-@@ -1012,7 +1015,6 @@
+@@ -1004,6 +1007,17 @@
+ 
+ 	    }
+ 
++	    compiz::X11::PendingEvent::Ptr pc =
++		    boost::shared_static_cast<compiz::X11::PendingEvent> (compiz::X11::PendingConfigureEvent::Ptr (
++									      new compiz::X11::PendingConfigureEvent (
++										  screen->dpy (), serverFrame, valueMask, &xwc)));
++
++	    pendingConfigures.add (pc);
++	    if (priv->mClearCheckTimeout.active ())
++		priv->mClearCheckTimeout.stop ();
++	    priv->mClearCheckTimeout.start (boost::bind (&PrivateWindow::checkClear, priv),
++					    2000, 2500);
++
+ 	    XSendEvent (screen->dpy (), screen->root (), false,
+ 			SubstructureNotifyMask, (XEvent *) &xev);
+ 
+@@ -1012,13 +1026,17 @@
  	}
  	else
  	{
@@ -124,7 +142,18 @@
  	    compiz::X11::PendingEvent::Ptr pc =
  		    boost::shared_static_cast<compiz::X11::PendingEvent> (compiz::X11::PendingConfigureEvent::Ptr (
  									      new compiz::X11::PendingConfigureEvent (
-@@ -2051,7 +2053,7 @@
+ 										  screen->dpy (), serverFrame, valueMask, &xwc)));
+ 
+ 	    pendingConfigures.add (pc);
++	    if (priv->mClearCheckTimeout.active ())
++		priv->mClearCheckTimeout.stop ();
++	    priv->mClearCheckTimeout.start (boost::bind (&PrivateWindow::checkClear, priv),
++					    2000, 2500);
++
+ 	    XConfigureWindow (screen->dpy (), serverFrame, valueMask, &xwc);
+ 	}
+

This ensures that configure event serials generated by frame updates in the case of unity's undecoration are correctly added to the event queue so that they will be cleared later on when compiz gets the event for them.

+@@ -2051,7 +2069,7 @@
  #ifdef DEBUG
  	abort ();
  #else
@@ -133,7 +162,7 @@
  #endif
      }
  
-@@ -2160,28 +2162,13 @@
+@@ -2160,28 +2178,13 @@
  	{
  	    XWindowChanges xwc;
  	    unsigned int   valueMask = CWX | CWY;
@@ -163,7 +192,7 @@
  	}
      }
  }
-@@ -2192,16 +2179,59 @@
+@@ -2192,16 +2195,59 @@
      return !mEvents.empty ();
  }
  
@@ -224,7 +253,7 @@
  }
  
  bool
-@@ -2227,8 +2257,21 @@
+@@ -2227,8 +2273,21 @@
      return false;
  }
  
@@ -246,7 +275,7 @@
  }
  
  compiz::X11::PendingEventQueue::~PendingEventQueue ()
-@@ -2271,7 +2314,44 @@
+@@ -2271,7 +2330,44 @@
  bool
  compiz::X11::PendingConfigureEvent::matchVM (unsigned int valueMask)
  {
@@ -292,7 +321,7 @@
  }
  
  bool
-@@ -2283,29 +2363,16 @@
+@@ -2283,29 +2379,16 @@
      if (!compiz::X11::PendingEvent::match (event))
  	return false;
  
@@ -332,7 +361,7 @@
  
      /* Remove events from the queue
       * even if they didn't match what
-@@ -2399,7 +2466,6 @@
+@@ -2399,7 +2482,6 @@
  	    xwc.x = priv->serverFrameGeometry.x ();
  	    xwc.y = priv->serverFrameGeometry.y ();
  
@@ -340,7 +369,7 @@
  	    compiz::X11::PendingEvent::Ptr pc =
  		    boost::shared_static_cast<compiz::X11::PendingEvent> (compiz::X11::PendingConfigureEvent::Ptr (
  									      new compiz::X11::PendingConfigureEvent (
-@@ -2407,6 +2473,11 @@
+@@ -2407,6 +2489,11 @@
  
  	    priv->pendingConfigures.add (pc);
  
@@ -352,7 +381,7 @@
  	    XConfigureWindow (screen->dpy (), ROOTPARENT (this), valueMask, &xwc);
  
  	    if (priv->serverFrame)
-@@ -3218,11 +3289,18 @@
+@@ -3218,11 +3305,18 @@
      return pc->matchVM (CWStackMode | CWSibling);
  }
  
@@ -372,7 +401,7 @@
  
      /* Immediately sync window position
       * if plugins were updating w->geometry () directly
-@@ -3232,28 +3310,37 @@
+@@ -3232,28 +3326,37 @@
  
      /* Remove redundant bits */
  
@@ -406,19 +435,19 @@
 +	    bool matchingRequest = priv->pendingConfigures.forEachIf (boost::bind (isExistingRequest, _1, *xwc, valueMask));
 +	    bool restackPending = window->serverPrev->priv->pendingConfigures.forEachIf (boost::bind (isPendingRestack, _1));
 +	    bool remove = matchingRequest;
++
++	    if (!remove)
++		remove = !restackPending;
  
 -	if (window->serverPrev->priv->pendingConfigures.forEachIf (boost::bind (isPendingRestack, _1)))
 -	    valueMask &= ~(CWSibling | CWStackMode);
-+	    if (!remove)
-+		remove = !restackPending;
-+
 +	    if (remove)
 +		valueMask &= ~(CWSibling | CWStackMode);
 +	}
      }
  
      if (valueMask & CWBorderWidth)
-@@ -3289,13 +3376,18 @@
+@@ -3289,13 +3392,18 @@
  	    compLogMessage ("core", CompLogLevelWarn, "restack_mode not Above");
      }
  
@@ -440,7 +469,7 @@
  				      + serverInput.left + serverInput.right)
  	frameValueMask &= ~(CWWidth);
  
-@@ -3305,13 +3397,15 @@
+@@ -3305,13 +3413,15 @@
  
      if (shaded)
      {
@@ -458,7 +487,7 @@
  	    + serverInput.top + serverInput.bottom)
  	    frameValueMask &= ~(CWHeight);
      }
-@@ -3354,14 +3448,16 @@
+@@ -3354,14 +3464,16 @@
  	    wc.width  = serverFrameGeometry.width ();
  	    wc.height = serverFrameGeometry.height ();
  
@@ -477,7 +506,7 @@
  
  	    XConfigureWindow (screen->dpy (), serverFrame, frameValueMask, &wc);
  	}
-@@ -4053,13 +4149,16 @@
+@@ -4055,13 +4167,16 @@
  
  		if (serverFrame)
  		{
@@ -495,7 +524,7 @@
  		}
  
  		/* Below with no sibling puts the window at the bottom
-@@ -4072,8 +4171,15 @@
+@@ -4074,8 +4189,15 @@
  	    }
  	    else if (sibling)
  	    {
@@ -512,7 +541,7 @@
  		{
  		    mask |= CWSibling | CWStackMode;
  
-@@ -6150,8 +6256,6 @@
+@@ -6152,8 +6274,6 @@
      if (dbg)
  	dbg->overrideRedirectRestack (priv->id, aboveId);
  
@@ -521,7 +550,7 @@
      priv->attrib = wa;
      priv->serverGeometry.set (priv->attrib.x, priv->attrib.y,
  			      priv->attrib.width, priv->attrib.height,
-@@ -7085,9 +7189,15 @@
+@@ -7087,9 +7207,15 @@
      XUnmapWindow (screen->dpy (), serverFrame);
      XDestroyWindow (screen->dpy (), wrapper);

Revision history for this message

Launchpad Janitor (janitor) wrote on 2011-10-18:

This bug was fixed in the package compiz - 1:0.9.6+bzr20110929-0ubuntu5

---------------
compiz (1:0.9.6+bzr20110929-0ubuntu5) oneiric-proposed; urgency=low

  * debian/patches/fix-864330.patch:
    - fix windows which can have towed moving (LP: #864330)
  * debian/patches/fix-864478.patch:
    - Window shading is broken (LP: #864478)

compiz (1:0.9.6+bzr20110929-0ubuntu4) oneiric-proposed; urgency=low

  * debian/control:
    - don't suggest nvidia-glx, it's part of the old initial packaging
      and probably a bad hint on non nvidia system (LP: #844218)
  * Cherry-pick upstream patches:
    - Windows should not automatically be focused when opened if the focus
      is on another application (LP: #748840)
    - Launcher - If a spread contains minimised windows, when the spread
      exits, the minimised windows momentarily appear on the desktop
      before disappearing (LP: #863328)
    - reproducible stacking bug in compiz (LP: #869316)
    - Click-dragging a window that's stacked above a fullscreen window will
      cause it to go underneath the fullscreen window (LP: #869919)
    - sometimes the keyboard input doesn't go to the apparently focussed
      dialog (LP: #869967)
    - Opening mumble can cause it to be stacked above the dash if you
      open the dash at the same time (LP: #865863)
    - Sometimes configure events are missed and windows move slow as a result
      (LP: #866752)
    - Workaround ubuntu desktop unity. Mouse at the left side doesn't reveal
      launcher (LP: #832150)
-- Didier Roche <email address hidden> Wed, 12 Oct 2011 10:44:49 +0200

Changed in compiz (Ubuntu):
status:	Fix Committed → Fix Released
Changed in compiz (Ubuntu Oneiric):
status:	Fix Committed → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

Duplicate of bug #866752 Remove

You are

Subscribing...

Edit bug mail

Other bug subscribers

Bug attachments

Add attachment

Remote bug watches

Bug watches keep track of this bug in other bug trackers.

Ubuntucompiz package

The larger windows, moving towed.

Bug Description

Related branches

Other bug subscribers

Bug attachments

Remote bug watches

Ubuntu
compiz package