-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 Howdy howdy,
I have a pull request with upstream now: https://github.com/BOINC/boinc/pull/1453 And I've attached a patch against upstream's master as of this morning (HEAD 1ca5f51cc7c993799f27d99bfa871d57567a30e5). I'm not an autotools whiz, so I don't know if the changes I made to configure.ac --mostly pulled from the commit that originally deleted them, but I added a small flag when building the client so that the original commit's checks are actually run-- are ideal, or if the changes made are suitable for an automatic detection of new dependencies. I'm starting to learn about package management a la git-buildpackage, but I've got a ways to go. Cheers, Preston P.S. - It looks like there's another bug from January 2015 that's essentially a dupe of this one. Hopefully both can be closed soon :D On 12/16/2015 01:16 AM, Gianfranco Costamagna wrote: > Hi if you can give me a patch for the source code and submit a pull > request upstream (github), I can find with git log the configure.ac > patches, add the Debian dependencies again and reupload on > unstable. Thanks G. > > Sent from Yahoo Mail on Android > > On Wed, 16 Dec, 2015 at 0:34, Preston > Maness<aspensmons...@riseup.net> wrote: -----BEGIN PGP SIGNED > MESSAGE----- Hash: SHA512 > > An update: > > Re-incorporating the XSS idle-detection code works as expected. I > scrapped the DPMS code and only kept the XSS code. Inclusion of > > #include <X11/extensions/scrnsaver.h> > > in "client/hostinfo_unix.cpp" ultimately pulls in a conflicting > definition of "Always" between X-related code and boinc's > "lib/prefs.h", fixed by renaming boinc's declaration. After this, > the boinc-client and boincmgr compile and idle detection works as > expected (keyboard and mouse inputs are appropriately detected, and > computation is suspended). > > Working on cleaning up the changes, hopefully getting upstream to > reinstate the XSS code, and determining how best to get the fix > into the Debian packages. From what I can tell, an additional > dependency on "libxss1" would be necessary. > > Cheers, Preston > > On 12/14/2015 07:28 AM, Preston Maness wrote: >> Whelp, it looks like the X-based idle detection --with the >> XScreenSaver API?-- was removed in this commit: >> >> ================== >> >> bfae1032e5c1ac73f2d8d92f222293d8383a6cee >> >> https://github.com/BOINC/boinc/commit/bfae1032e5c1ac73f2d8d92f222293d 8 > >> 38 >> >> > 3a6cee >> >> ================== >> >> The commit message indicates that /dev/input/* devices are the >> intended way to determine idle time on Linux now. However, it >> seems that X-based inputs don't update the /dev/input/* devices. >> Or at least, anything the X server is doing isn't updating the >> atime on those files, which is what is being utilized to >> determine if the system is idle or not. Working in a terminal >> updates the atime on the relevant files --correctly triggering >> BOINC to suspend computation-- but just typing this email, or >> wiggling the mouse around the desktop area, doesn't. >> >> Granted, this code was present earlier, so I don't know if simply >> re-introducing it will solve anything. But that's my first plan >> of attack. I've also looked into other varying methods of idle >> detection. The XSS API is the most common, but there is a more >> convoluted (asynchronous?) Xsync option that might also be >> possible too. I don't know too much about X, but I'll be giving a >> few different possibilities a go in the coming days. >> >> Cheers, Preston >> >> On 12/11/2015 09:44 AM, Preston Maness wrote: >>> Howdy howdy, >> >>> I think I deserve a slow clap for such a slow >>> response/addresssing of a bug. Indeed my response is so slow >>> that I've since changed email address and GPG keys. >> >>> Needless to say, when running BOINC 7.6.12 from testing, some >>> progress seems to have been made on idle detection. If I open a >>> terminal and begin working in it, *sometimes*, the idle >>> detection kicks in. I can't tell whether it's working as >>> intended or not. Regardless, keeping boinc running non-stop >>> doesn't pose any issues thanks to the scheduling. I can happily >>> use my machine for just about anything and not even notice that >>> BOINC is running. >> >>> I'm presently unemployed, so really, I have nothing but time >>> and no real excuses not to give this bug another go and see if >>> I can just determine once and for all if things are fixed/how >>> to fix them :P >> >>> It looks like the BOINC project has moved from their own git >>> infrastructure onto github: >> >>> https://boinc.berkeley.edu/trac/wiki/SourceCodeGit >> >>> https://github.com/BOINC/boinc >> >>> Looking at the git log for hostinfo_unix.cpp (a la "git log >>> client/hostinfo_unix.cpp" from the project root), it looks like >>> there was a commit that included changes from Gianfranco: >> >>> ================== >> >>> commit 0811886cbb1d282376e2dfd017ab39e34cd7d8ea Author: David >>> Anderson <da...@ssl.berkeley.edu> Date: Mon Feb 3 12:46:43 >>> 2014 -0800 >> >>> client: fix for idle detection on Linux (from Gianfranco) >> >> >>> ================== >> >>> and some later refactoring of the unix HOSTINFO code: >> >>> ================== >> >>> commit cda6bd334bf036349f471a0c4eb7c25de8098262 Author: David >>> Anderson <da...@ssl.berkeley.edu> Date: Tue Nov 25 14:46:58 >>> 2014 -0800 >> >>> client: get static host info items only on startup >> >>> On Android, the way we were getting os_version (Linux version + >>> Android version) didn't work because get_host_info() was >>> getting called before every scheduler RPC, and it overwrote the >>> Android version part. >> >>> Solution: divide host info into dynamic (disk usage, network >>> info) and static (everything else). Computer the static part >>> only at startup. >> >>> Also factor the Unix HOST_INFO code into multiple functions. >> >>> ================== >> >>> As it stands, BOINC's Event Log does show periodic "Suspending >>> computation - computer is in use" messages. Show I'm operating >>> on the assumption that something is working. >> >>> Cheers, Preston Maness >> >> >> > > -----BEGIN PGP SIGNATURE----- > > iQIcBAEBCgAGBQJWcKN0AAoJEFpzcfOOuHp0vY8QAM21NXoSO0S73rsL5Ci5WEZL > D2RbjefCRb+ojSGCFRtRG8fnLVS88P9Kyj5EfwvQPzQhHIsv2aZCfvl3JkPz49A0 > khjaKr7DxgM2DtCxkTbq6HNeYtyr2YQounyZdhEOn/vYp4SmeOVxgC/2j31WYY95 > 3XsMZ6Pe1SpCmhab5jyRBlVUKVTUT2Ekbq7buvMHkEvLSryM3ax32K3PXC8gNbFL > YuVvPYkiQZUlpFrPRqjjoJTBO/MziTYSWw1U9dIXurD0jB8sBitWH7JLkSAmyAjh > dusx6rKFSR1XmPv0P+4D9qaKsy4lES4erxp2FUcPaX5L+D5fw7qAkTrgH+Vmbz+E > fCtgTiZz91KqpImlYyMjuzKPqpfMl2+O4qBoHTrXoasvFR0z8toP/gLIMibEb+OA > aD4MZO5IOQ+Orc1qPO+G9p6agDVmQePGk6S7jED3u/iklUI+865xYjbkAyiq4iwp > xVplNgaufJHD0dzRghB3IqSGI1EqEBXKQsBuJeShf2b8FdreVBwmgEWhL2uekhMv > 187UBr6j9ERgjWSbY6QLbXb6y4+qjpINmMfAMkMjplDyB8ozUiYf6QvRjMZdBqUG > dYDq8Bt26M4u1MlMUbYQbZB1A9jn1pHC0CF13l2up8GmeRQzj+1emPKpK18H/5bl > BfkujLktRP+FinR3iDV8 =AW/A -----END PGP SIGNATURE----- > -----BEGIN PGP SIGNATURE----- iQIcBAEBCgAGBQJWcaHhAAoJEFpzcfOOuHp0oa0P/jRjDvtMDXgugKRv6MQobhGh WgYERXamT/wW+mfFQeAIkvVHccEA/9Uy/YvspbhV/lIMDZ5i80CqzpfP8tchA6Lu LwcuL59FWmotKP5AwyCW0nfH6zQnP11LrNZBmaRnghfLIcG2/Wh8i6PzKgf06K5K GlsRcTVCKpeZHWqoiKVKdS3g0sUn3J3y7wQ0u+KHIBauTk+mO3/NHElONMb6G7WV R9VNvS8I+ggKuXBILucWHyKHJ4n3bybG0JtvZiSilHZuV6rw3RQ5CyDE/AsxQWum WhYsxAY0xMF5DR2oHLhc7Vr+qU78sLMlWk4zL0IW6LAjEh9u9hsl6YIvLjU/tw0b quUiDjdH41NwUdrV41qwFJPE7R775NMlvy5XalcJV30Ha6dllK0xJs0cMArmjFE2 gmSCNd+zjOZhaiHOcW1c0mYap1wvQWotBX/5qUfHMYnNMOPdtQwyoruQx65R03sb WHRQmzR843Lw01A/agwTmuNT3+ydlYr83XR5LcUz+qhvu61GhTV47XykRyQ03VxF zaf0ODMdqTrj2uhQZQSKLNjpDgy7IRaXdRhZCTuET5jQjkfMcaNaK2rzv75uUeeG JWieNa11TIYJEVWG4zCRUKclPivTSUK+6CNY0fPaE0qxH9+gyaCG+2A2lTEzrpOj LH3yZUaNsC7LY4WZX3eV =57gn -----END PGP SIGNATURE-----
>From 156b0be9f13b3427973f1d83a004947225b2d8b6 Mon Sep 17 00:00:00 2001
From: Preston Maness <aggroska...@gmail.com>
Date: Wed, 16 Dec 2015 10:45:57 -0600
Subject: [PATCH] client (Linux): Reinstate XScreenSaver API-based idle
detection on GNU/Linux.
---
client/hostinfo_unix.cpp | 68 +++++++++++++++++++++++++++++++++++++++++++++++-
configure.ac | 20 ++++++++++++++
lib/prefs.cpp | 2 +-
lib/prefs.h | 6 ++++-
4 files changed, 93 insertions(+), 3 deletions(-)
diff --git a/client/hostinfo_unix.cpp b/client/hostinfo_unix.cpp
index 617f824..c059895 100644
--- a/client/hostinfo_unix.cpp
+++ b/client/hostinfo_unix.cpp
@@ -41,6 +41,10 @@
#include <cstring>
#endif
+#if HAVE_XSS
+#include <X11/extensions/scrnsaver.h> //X-based idle detection
+#endif
+
#include <cstdio>
#include <cstdlib>
#include <cstring>
@@ -1947,6 +1951,62 @@ bool interrupts_idle(time_t t) {
}
return last_irq < t;
}
+
+#if HAVE_XSS
+// Ask the X server for user idle time (using XScreenSaver API)
+// Return true if the idle time exceeds idle_threshold.
+//
+bool xss_idle(long idle_threshold) {
+ static XScreenSaverInfo* xssInfo = NULL;
+ static Display* disp = NULL;
+ static bool error = false;
+ // some X call failed - always return not idle
+
+ if (error) return false;
+
+ long idle_time = 0;
+
+ if (disp == NULL) {
+ disp = XOpenDisplay(NULL);
+ // XOpenDisplay may return NULL if there is no running X
+ // or DISPLAY points to wrong/invalid display
+ //
+ if (disp == NULL) {
+ error = true;
+ //msg_printf(NULL, MSG_INFO, "XDisplay not found.");
+ return false;
+ }
+ int event_base_return, error_base_return;
+ xssInfo = XScreenSaverAllocInfo();
+ if (!XScreenSaverQueryExtension(
+ disp, &event_base_return, &error_base_return
+ )){
+ error = true;
+ //msg_printf(NULL, MSG_INFO, "XScreenSaverQueryExtension() failed.");
+ return false;
+ }
+ }
+
+ XScreenSaverQueryInfo(disp, DefaultRootWindow(disp), xssInfo);
+ idle_time = xssInfo->idle;
+
+ // convert from milliseconds to seconds
+ //
+ idle_time = idle_time / 1000;
+
+ //msg_printf(NULL, MSG_INFO, "XSS idle detection succeeded.");
+ //msg_printf(NULL, MSG_INFO, "idle threshold: %ld", idle_threshold);
+ //msg_printf(NULL, MSG_INFO, "idle_time: %ld", idle_time);
+
+ if ( idle_threshold < idle_time ) {
+ //msg_printf(NULL, MSG_INFO, "System is idle according to X.");
+ } else {
+ //msg_printf(NULL, MSG_INFO, "System is active according to X.");
+ }
+ return idle_threshold < idle_time;
+}
+#endif // HAVE_XSS
+
#endif // LINUX_LIKE_SYSTEM
bool HOST_INFO::users_idle(bool check_all_logins, double idle_time_to_run) {
@@ -1972,6 +2032,12 @@ bool HOST_INFO::users_idle(bool check_all_logins, double idle_time_to_run) {
return false;
}
+#if HAVE_XSS
+ if (!xss_idle((long)(idle_time_to_run * 60))) {
+ return false;
+ }
+#endif // HAVE_XSS
+
// Lets at least check the dev entries which should be correct for
// USB keyboards and mice. If the linux kernel doc is correct it should
// also work for bluetooth input devices as well.
@@ -1990,7 +2056,7 @@ bool HOST_INFO::users_idle(bool check_all_logins, double idle_time_to_run) {
if (!device_idle(idle_time, "/dev/input/mice")) return false;
if (!device_idle(idle_time, "/dev/kbd")) return false;
// solaris
-#endif
+#endif // LINUX_LIKE_SYSTEM
return true;
}
diff --git a/configure.ac b/configure.ac
index 792c6fb..919fd46 100644
--- a/configure.ac
+++ b/configure.ac
@@ -200,6 +200,7 @@ configured_to_build=
if test x$enable_client = xyes ; then
configured_to_build="client"
+ enable_xss="yes"
fi
if test x$enable_manager = xyes ; then
configured_to_build="${configured_to_build} manager"
@@ -628,6 +629,15 @@ fi
AM_CONDITIONAL(BUILD_GRAPHICS_API, [ test "$have_glut" = yes -a "$have_jpeg" = 1])
+dnl check for X screen saver lib (X-based idle detection on Linux)
+if test "$enable_xss" == yes; then
+ AC_CHECK_LIB([Xss], [XScreenSaverAllocInfo], [have_Xss="yes"], [have_Xss="no"])
+ AC_CHECK_HEADER([X11/extensions/scrnsaver.h], [have_Xss="yes"], [have_Xss="no"])
+ if test "$have_Xss" == no; then
+ AC_MSG_WARN([libxss missing, disabling X ScreenSaver user idle detection])
+ fi
+fi
+
dnl --------------------------------------------------------------------------------
dnl put double-inclusion protection into config.h
AH_TOP([
@@ -1194,6 +1204,16 @@ if test "${ac_cv_func_res_init}" != "yes" ; then
fi
LIBS=$svlibs
+if (test "$enable_xss" == yes) && (test "$have_Xss" == yes); then
+ SAH_CHECK_LIB([Xss],[XScreenSaverAllocInfo],[
+ AC_DEFINE([HAVE_XSS],[1],[Define to 1 if you have xss library])
+ CLIENTLIBS="${sah_lib_last} ${CLIENTLIBS}"])
+ SAH_CHECK_LIB([X11],[XOpenDisplay],[
+ AC_DEFINE([HAVE_X11],[1],[Define to 1 if you have X11 library])
+ CLIENTLIBS="${sah_lib_last} ${CLIENTLIBS}"])
+ echo DEBUG: CLIENTLIBS=${CLIENTLIBS} >&5
+fi
+
SAH_CHECK_LIB([resolv],[res_query],[
AC_DEFINE([HAVE_RESOLV],[1],[Define to 1 if you have the resolv library])
CLIENTLIBS="${sah_lib_last} ${CLIENTLIBS}"])
diff --git a/lib/prefs.cpp b/lib/prefs.cpp
index 3a8267e..9bd4160 100644
--- a/lib/prefs.cpp
+++ b/lib/prefs.cpp
@@ -149,7 +149,7 @@ bool TIME_SPAN::suspended(double hour) const {
TIME_SPAN::TimeMode TIME_SPAN::mode() const {
if (end_hour == start_hour || (start_hour == 0 && end_hour == 24)) {
- return Always;
+ return Always2;
} else if (start_hour == 24 && end_hour == 0) {
return Never;
}
diff --git a/lib/prefs.h b/lib/prefs.h
index ebbe3cc..da81ea7 100644
--- a/lib/prefs.h
+++ b/lib/prefs.h
@@ -89,7 +89,11 @@ struct TIME_SPAN {
double end_hour;
enum TimeMode {
- Always = 7000,
+ // "Always" is defined by X.h, which itself is (indirectly) included
+ // from client/hostinfo_unix.cpp when including the XSS API
+ // (<X11/extensions/scrnsaver.h>). Hence, renaming here (and in
+ // prefs.cpp).
+ Always2 = 7000,
Never,
Between
};
--
2.6.2
reinstate_xss_idle_detection_linux.patch.sig
Description: PGP signature
