Re: [PATCH 00/23] per device dirty throttling -v8

Ingo Molnar
2007-08-05T12:46:48Z

* Jakob Oestergaard <jakob@unthought.net> wrote:

> > If you can show massive amounts of users that will actually be
> > negatively impacted, please present hard evidence.
> >
> > Otherwise all this is useless hot air.
>
> Peace Jeff :)
>
> In another mail, I gave an example with tmpreaper clearing out unused
> files; if some of those files are only read and never modified,
> tmpreaper would start deleting files which were still frequently used.
>
> That's a regression, the way I see it. As for 'massive amounts of
> users', well, tmpreaper exists in most distros, so it's possible it
> has other users than just me.

you mean tmpwatch? The trivial change below fixes this. And with that
we've come to the end of an extremely short list of atime dependencies.

Ingo

--- /etc/cron.daily/tmpwatch.orig
+++ /etc/cron.daily/tmpwatch
@@ -1,9 +1,9 @@
#! /bin/sh
-/usr/sbin/tmpwatch -x /tmp/.X11-unix -x /tmp/.XIM-unix -x /tmp/.font-unix \
+/usr/sbin/tmpwatch --mtime -x /tmp/.X11-unix -x /tmp/.XIM-unix -x /tmp/.font-unix \
-x /tmp/.ICE-unix -x /tmp/.Test-unix 10d /tmp
-/usr/sbin/tmpwatch 30d /var/tmp
+/usr/sbin/tmpwatch --mtime 30d /var/tmp
for d in /var/{cache/man,catman}/{cat?,X11R6/cat?,local/cat?}; do
if [ -d "$d" ]; then
- /usr/sbin/tmpwatch -f 30d "$d"
+ /usr/sbin/tmpwatch --mtime -f 30d "$d"
fi
done