Hello,
thank you for the answer.
On 01/31/2012 12:28 AM, Jeremy Huddleston wrote:
I really don't see how 2abe83df686ed64c4f4df711ac3c1fd12131c2e4 could be the
culprit. It's trivial.
I made a mistake somewhere so to double check I did the bisection again
starting from 1.11.99.1 (working) to 1.11.99.2 (not working).
The result is that I was unable to bisect the last three steps because
of compilation errors (that are fixed later on). You can find the last
patchs that caused the compilation issue enclosed to the email
(bisect.result). The result from the bisection are in the git.log file.
Can you please create a bug report at http://bugs.freedesktop.org to track this?
I will do that.
Mathieu
On Jan 30, 2012, at 4:59 AM, mathieu.taillefum...@free.fr wrote:
Hello,
i encounter a bug in the last rc of the xserver (present before but I
was using it so far) when I use it in combination with kdm (and only
kdm, I did not try xdm). The server simply crash when kdm is running and
I touch any of the keyboard keys but works perfectly fine when I run it
from a console. The all stack is from the git repository but the problem
originates from the server itself since I can not reproduce it with xorg
1.11.4 for instance (with the same stack almost).
I am able to reproduce it all the time and I have been spending some
time to bisect the problem (changing the server, the rest been fixed).
The result of the bisection gave me this:
git bisect start
# bad: [052ca3f22eadd0aa60dd24ac7d5d76137273926f] Bump version to
1.11.99.902 (1.12 RC2)
git bisect bad 052ca3f22eadd0aa60dd24ac7d5d76137273926f
# good: [e597f0119cd69b6d9edf86d06d941468f90d8e6d] configure.ac: 1.11.4
git bisect good e597f0119cd69b6d9edf86d06d941468f90d8e6d
# good: [4ad271d06c5aa42721c0e2e01e17e34a39825c65] xfree86: Bump
extension ABI version to 6.0
git bisect good 4ad271d06c5aa42721c0e2e01e17e34a39825c65
# good: [34b0e4eee911f8b09a3682a7f1b4c8598ef48b8d] dri2: Register the
DRI2DrawableType after server regeneration
git bisect good 34b0e4eee911f8b09a3682a7f1b4c8598ef48b8d
# bad: [2df539c0bc3300ea858f8bc7d52e95e67ff379b8] glx: Only declare
GlxExtensionInit in one header file
git bisect bad 2df539c0bc3300ea858f8bc7d52e95e67ff379b8
# bad: [2abe83df686ed64c4f4df711ac3c1fd12131c2e4] include: add
BUG_WARN_MSG for custom error message on bug condition
git bisect bad 2abe83df686ed64c4f4df711ac3c1fd12131c2e4
# good: [d26fae246d7c451b4d5ffe24fdb959d4bd00b107] glx: don't leak fbconfigs
git bisect good d26fae246d7c451b4d5ffe24fdb959d4bd00b107
# good: [6acebf9e1298939593b942ec91ae9ec9e74faa19] include: add
list_append()
git bisect good 6acebf9e1298939593b942ec91ae9ec9e74faa19
# good: [4bc2761ad5ec2d0668aec639780ffb136605fbc8] dix: switch the
dev->deviceGrab.activeGrab from GrabRec to GrabPtr
git bisect good 4bc2761ad5ec2d0668aec639780ffb136605fbc8
# good: [d2ebbcdaf6b13d70eee704b1764ff349e1be22a0] Xi: when removing a
device, reset ClientPointers where needed
git bisect good d2ebbcdaf6b13d70eee704b1764ff349e1be22a0
# good: [631516a4aa9858874ee197444cd93d91b97a1089] Xi: check button
mapping value _before_ assigning it
git bisect good 631516a4aa9858874ee197444cd93d91b97a1089
# good: [4fc797f3756611a97767f407e1af0b6a7cf2f1d9] xfree86: include
xorg-config.h from xaalocal.h
git bisect good 4fc797f3756611a97767f407e1af0b6a7cf2f1d9
so the first bad commit is 2abe83df686ed64c4f4df711ac3c1fd12131c2e4.
I also have some information after using gdb from a remote computer. I
attach the log file of the debugging session. Since i can reproduce it
do not hesitate to ask me for more information.
regards
Mathieu
<X.log>_______________________________________________
xorg@lists.x.org: X.Org support
Archives: http://lists.freedesktop.org/archives/xorg
Info: http://lists.x.org/mailman/listinfo/xorg
Your subscription address: jerem...@freedesktop.org
commit 218752bdc5d9323d1e6202e762573a925cf8a4eb
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Thu Dec 8 14:27:01 2011 +1000
input: replace GRABTYPE_* with the InputLevel enums
They achieve the same thing, re-use the more generic InputLevel so we can
convert to/fro easier.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit cd56bd7b3ee34a4b10eb3a57a6e94cac7512167a
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Thu Dec 8 14:59:06 2011 +1000
Add GrabIsPointerGrab and GrabIsKeyboardGrab helpers
No functional changes
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 47734b2ba2e1f9246fd1a3b7059ca60b0e15acb2
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Thu Dec 8 12:00:34 2011 +1000
dix: move delivery stop condition out of event mask
Previously, this was only called if there was a mask match, so even if we
had a no-propagate flag set or a stopAt window specified, if no mask
triggered on the window we would recurse up to the root window and
eventually deliver.
Move this, so that the stopAt and do-not-propagate mask is honoured.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 1d01e861b69d7ca8fe2335270b8286bc115fb6e9
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Thu Dec 8 10:22:26 2011 +1000
dix: split out core state and event state setting into helper functions
no functional changes.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit fb010c543696b781a83b945f72865b7d7b75bf95
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Wed Dec 7 14:41:18 2011 +1000
dix: move grab matching code into a helper function
No functional changes.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit d253a262c2c690357a4db7e235c48ab5dd0b77f8
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Wed Dec 7 13:57:25 2011 +1000
dix: split core grab interference check into helper function
No functional changes.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 97d79dd740ed1ac4eb02b0d4135850f953716278
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Wed Dec 7 13:54:28 2011 +1000
dix: compare the grab type, not the tempGrab type
No functional change. To get here, GrabMatchesSecond() needs to be TRUE and
for that the two grab types must be identical.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit b9d6ae42b686287d7d23c2eefe2b12da98fe6ff8
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Wed Dec 7 15:43:31 2011 +1000
dix: remove event type check
Can't remember why this is there but we'll need to pass in XI2 events soon,
so this check is obsolete.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 80816366aa77f1730a1b6ddabfa55a946d76d494
Author: Chase Douglas <chase.doug...@canonical.com>
Date: Fri Nov 25 13:57:03 2011 -0800
dix: Split ActivatePassiveGrab() from CheckPassiveGrab()
The changed logic means we don't require the explicit grab = NULL setting
and early exit anymore. Not 100% of it, but if we see that message pop up in
a log we know it's broken.
Signed-off-by: Chase Douglas <chase.doug...@canonical.com>
Reviewed-by: Peter Hutterer <peter.hutte...@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
commit 9ee62cd8ce3c3effc3663f3d56b322385ce12fdb
Author: Chase Douglas <chase.doug...@canonical.com>
Date: Thu Nov 17 17:40:24 2011 -0800
dix: Move grab check and activation logic to CheckPassiveGrab()
This is needed for future pointer emulation work.
Signed-off-by: Chase Douglas <chase.doug...@canonical.com>
Reviewed-by: Peter Hutterer <peter.hutte...@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
commit c53651dabc66aeb9882819a404a799a364f00a29
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Tue Dec 6 12:57:38 2011 +1000
dix: move EventDeliveryState into a header file, we'll need it elsewhere
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 86c3137c81e924bacd919e9d65809e07afeabbff
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Tue Dec 6 12:04:11 2011 +1000
Xi: split updating button count and state into helper functions
Functional change: for a button mapped to 0, the motionHintWindow is not
updated to the NullWindow anymore. Before it got updated unconditionally to
the button mapping. I have no idea what the practical effect of this is, but
I guess it's closer to the correct behaviour: pressing a button that's
logically disabled now does not disrupt the motion hint delivery.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit e0f37250ffff5dcb3bc0e8cad63439995ce01a20
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Tue Dec 6 11:40:33 2011 +1000
Xi: deduplicate button motion mask setting
No functional changes
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 6eff14a789341d366b3013c5aa020e959c954651
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Mon Dec 5 18:54:30 2011 +1000
dix: deduplicate callers of DeliverDeviceEvents in DeliverGrabbedEvents
No functional change.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 93945b0a74aa8156a88f52b8ba77f1210042f396
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Mon Dec 5 18:42:05 2011 +1000
dix: split grab event conversion and delivery into a helper function
Effective functional change: XI2 events are checked with XACE now.
DeliverOneGrabbedEvent is exported for future use by touch events.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit c81cdb0862e2184f033b3933e6bacbe0809ef2c0
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Mon Dec 5 18:29:27 2011 +1000
dix: replace conversion errors with BUG_WARN_MSG
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 6368c2aa4613a7c7eb0e8afca8d41f1a9bc4fc4d
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Mon Dec 5 16:46:18 2011 +1000
dix: deduplicate event delivery code
Move all the event delivery code into DeliverOneEvent, based on the
InputLevel we're sending to.
Functional change: we now check XI2 events with XACE too.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 7e2207548b3173afc9accb5ccd532c181a7b94ed
Author: Chase Douglas <chase.doug...@canonical.com>
Date: Wed Dec 7 20:32:28 2011 +1000
include: Add an InputLevel enum
Currently unused, but will be in the future.
Signed-off-by: Chase Douglas <chase.doug...@canonical.com>
Reviewed-by: Peter Hutterer <peter.hutte...@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
commit e5aa00989cda9ebd18063c5e955235123ad37b88
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Wed Dec 7 14:14:10 2011 +1000
Change GetXI2/XI/CoreType to just take a type argument
Avoids the dummy-event dance if we have an event type and need to get the
matching XI2 type.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
commit 6cc0e6a0afa6a87802767d484aa4e68fa935d3eb
Author: Peter Hutterer <peter.hutte...@who-t.net>
Date: Fri Oct 28 12:55:55 2011 +1000
include: add GetXI2MaskByte and GetXI2EventFilterMask to headers
This is needed for touch event processing.
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
Reviewed-by: Chase Douglas <chase.doug...@canonical.com>
git bisect start
# bad: [c065f7b16329e30cb976359e0a807c2adaf50123] Version 1.11.99.2 (1.12
snapshot 2)
git bisect bad c065f7b16329e30cb976359e0a807c2adaf50123
# good: [5952efa903d57ac69070f1477ce46223a2a7893c] Bump version to 1.11.99.1
(1.12 development release 1)
git bisect good 5952efa903d57ac69070f1477ce46223a2a7893c
# good: [2abe83df686ed64c4f4df711ac3c1fd12131c2e4] include: add BUG_WARN_MSG
for custom error message on bug condition
git bisect good 2abe83df686ed64c4f4df711ac3c1fd12131c2e4
# good: [d8eb9b5faa2d2c9350bdaffef8749a9cac3c87b8] XkbFindSrvLedInfo: remove
extraneous name-clashing sli variable
git bisect good d8eb9b5faa2d2c9350bdaffef8749a9cac3c87b8
# bad: [218752bdc5d9323d1e6202e762573a925cf8a4eb] input: replace GRABTYPE_*
with the InputLevel enums
git bisect bad 218752bdc5d9323d1e6202e762573a925cf8a4eb
# skip: [c53651dabc66aeb9882819a404a799a364f00a29] dix: move EventDeliveryState
into a header file, we'll need it elsewhere
git bisect skip c53651dabc66aeb9882819a404a799a364f00a29
# skip: [1d01e861b69d7ca8fe2335270b8286bc115fb6e9] dix: split out core state
and event state setting into helper functions
git bisect skip 1d01e861b69d7ca8fe2335270b8286bc115fb6e9
# good: [9ee62cd8ce3c3effc3663f3d56b322385ce12fdb] dix: Move grab check and
activation logic to CheckPassiveGrab()
git bisect good 9ee62cd8ce3c3effc3663f3d56b322385ce12fdb
# good: [d253a262c2c690357a4db7e235c48ab5dd0b77f8] dix: split core grab
interference check into helper function
git bisect good d253a262c2c690357a4db7e235c48ab5dd0b77f8
# good: [47734b2ba2e1f9246fd1a3b7059ca60b0e15acb2] dix: move delivery stop
condition out of event mask
git bisect good 47734b2ba2e1f9246fd1a3b7059ca60b0e15acb2
# good: [cd56bd7b3ee34a4b10eb3a57a6e94cac7512167a] Add GrabIsPointerGrab and
GrabIsKeyboardGrab helpers
git bisect good cd56bd7b3ee34a4b10eb3a57a6e94cac7512167a
# good: [cd56bd7b3ee34a4b10eb3a57a6e94cac7512167a] Add GrabIsPointerGrab and
GrabIsKeyboardGrab helpers
git bisect good cd56bd7b3ee34a4b10eb3a57a6e94cac7512167a
# good: [cd56bd7b3ee34a4b10eb3a57a6e94cac7512167a] Add GrabIsPointerGrab and
GrabIsKeyboardGrab helpers
git bisect good cd56bd7b3ee34a4b10eb3a57a6e94cac7512167a
_______________________________________________
xorg@lists.x.org: X.Org support
Archives: http://lists.freedesktop.org/archives/xorg
Info: http://lists.x.org/mailman/listinfo/xorg
Your subscription address: arch...@mail-archive.com
