After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 577749 - Confusing warning: "Unable to find watch for alarm XXXXX"
Confusing warning: "Unable to find watch for alarm XXXXX"
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: gnome-session
2.26.x
Other All
: Normal trivial
: ---
Assigned To: Session Maintainers
Session Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-04-02 16:59 UTC by Tomislav Vujec
Modified: 2009-04-08 15:31 UTC
See Also:
GNOME target: ---
GNOME version: 2.25/2.26


Attachments
Proposed patch to disable no watch warning. (731 bytes, patch)
2009-04-02 17:01 UTC, Tomislav Vujec
none Details | Review
Much improved patch proposal that disables useless warning only when it is really useless. (766 bytes, patch)
2009-04-03 02:57 UTC, Tomislav Vujec
none Details | Review

Description Tomislav Vujec 2009-04-02 16:59:07 UTC
Please describe the problem:
gnome-session seems to report the above warning at random times during desktop session. This can be confusing to a user when noticed in either syslog or console output.
The problem is that alarms that are being created to check on session's idle state, are later destroyed when conditions for it are satisfied. However, when an alarm is destroyed, xserver will send an event on it, and gnome-session will try to locate the appropriate watch for that alarm which can't be located. Most correct implementation would keep the watch structure and release it on receipt of the final alarm event, but short of that, switching the message log level from warning to debug removes the confusion.

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Tomislav Vujec 2009-04-02 17:01:50 UTC
Created attachment 131929 [details] [review]
Proposed patch to disable no watch warning.
Comment 2 Tomislav Vujec 2009-04-03 02:57:02 UTC
Created attachment 131966 [details] [review]
Much improved patch proposal that disables useless warning only when it is really useless.

After spending some time on XSync protocol description, I realized that the server does send an event with the state Destroyed after destroying an alarm. This patch ignores such events. As long as alarms are only destroyed from watch remove callback, this should work perfectly. However if that changes, the whole event handler will need to change anyway.
Comment 3 Vincent Untz 2009-04-08 14:40:04 UTC
Matthias actually committed a patch similar to your second one a few days ago :-)
Comment 4 Tomislav Vujec 2009-04-08 15:31:25 UTC
Yup, he got it from me from the fedora bugzilla.