Re: [ANNOUNCE] System Inactivity Monitor v1.0

From: Alessandro Di Marco
Date: Mon Jan 29 2007 - 17:42:29 EST


Pavel Machek <pavel@xxxxxx> writes:

Hi!

> The /proc/bus/input/devices has an extensible structure. You can just
> add an "A:" line (for Activity) instead of adding a new proc file.
>
> I know, but IMO there is too much stuff to parse in there. Activity counters
> are frequently accessed by daemons, and four or five concurrent daemons are the
> norm in a typical X11 linux box...

Syscalls are fast enough, and the file is _very_ easy (=> fast) to parse.

> Also, the activity counters should IMO coincide with the event times
> passed through /dev/input/event, and should not be jiffies based.
> Ideally, both should be based on clock_gettime(CLOCK_MONOTONIC).
>
> In evdev.c do_gettimeofday() is used. Anyway I just need of a monotonic
> counter, so get_jiffies_64() wouldn't be better? It isn't affected by wrapping
> issues and it is probably faster than do_gtod().

Just use same time source rest of inputs already do...

OK, but what about the time-warp problem?. To fix it I need to know when the
system goes to sleep/resumes. In SIN I've solved via the platform driver,
introducing suspend() resume() callbacks...

greets,

--
Technology is dominated by two types of people: those who understand what they
do not manage, and those who manage what they do not understand. - Putt's Law
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/