public inbox for util-linux@vger.kernel.org
 help / color / mirror / Atom feed
From: Ruediger Meier <sweet_f_a@gmx.de>
To: "Thomas Bächler" <thomas@archlinux.org>
Cc: Karel Zak <kzak@redhat.com>, util-linux@vger.kernel.org
Subject: Re: [PATCH] fstrim: add systemd units
Date: Wed, 9 Apr 2014 17:16:04 +0200	[thread overview]
Message-ID: <201404091716.05131.sweet_f_a@gmx.de> (raw)
In-Reply-To: <534541E3.4080906@archlinux.org>

On Wednesday 09 April 2014, Thomas Bächler wrote:
> Am 09.04.2014 14:12, schrieb Ruediger Meier:
> >> well, "fstrim -a" contains heruistic to select the right filestems
> >> (it really does not call trim for all devices), it has been
> >> implemented to *avoid* sysadmins creativity. If you don't like it,
> >> you can use "fstrim <device>" (for example from crontab).
> >
> > I'd like the documentation more detailed.
> > Does it really run on all mounts or only /etc/fstab?
> > Does it write on automounted devices which are probably not owned
> > by the admin? Does it affect read-only mounts?
>
> The documentation is precise here:
>
> 'Trim all mounted filesystems on devices that support the discard
> operation.'

If this is 100% true then Karel's statement 10 lines above would be 
wrong.
Also this doc line indicates that even read-only mounts could be trimmed 
(if FS supports it) which would be at least dangerous if it runs 
automatically while admin is trying to repair a broken filesystem.

> It says exactly what it will do. Every one of your questions is
> answered.

> > IMO the task itself is already installed since we have fstrim's
> > option -a". You just need to _enable_ it to run whenever you want
> > for example by using crontab. That's trivial.
>
> Because it takes time. I have done it four times already and it cost
> me several minutes each time. Now nobody has to do it again.

> > Nobody would have thought about adding a 1-liner cronjob file to
> > util-linux eventhough any distro has /etc/cron.daily/. But now
> > systemd timer? Thats what I don't understand.
>
> I don't know why nobody thought about doing that in the past, maybe
> nobody cared about the needs of all those other admins. Maybe because
> installing a file into cron.daily automatically enables it. Maybe
> because nobody cared about anything but his own system?

I guess because adding one crontab line is trivial in comparision to add 
two files plus symlinks to get such systemd timer running. 

> > If _creating_ a systemd timer is too complicated for today's admins
>
> It is not complicated, it is just a task that shouldn't be repeated
> countless times.

Now you still need to enable it "countless times" which I would do using 
rsync or git anyway like I would sync self created timers, crontabs or 
whatever config files with same effort.

cu,
Rudi


  reply	other threads:[~2014-04-09 15:16 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-03 21:41 [PATCH] fstrim: add systemd units Thomas Bächler
2014-04-07 10:43 ` Karel Zak
2014-04-08 10:25   ` Ruediger Meier
2014-04-08 10:57     ` Thomas Bächler
2014-04-08 12:07       ` Ruediger Meier
2014-04-08 15:42         ` Dave Reisner
2014-04-08 17:12           ` Ruediger Meier
2014-04-09  7:52         ` Karel Zak
2014-04-09 10:07           ` Ruediger Meier
2014-04-09 11:02             ` Karel Zak
2014-04-09 12:12               ` Ruediger Meier
2014-04-09 12:49                 ` Thomas Bächler
2014-04-09 15:16                   ` Ruediger Meier [this message]
2014-04-09 15:24                     ` Thomas Bächler
2014-04-09 15:44                       ` Ruediger Meier
2014-04-09 14:02               ` Markus Trippelsdorf
2014-04-09 15:48                 ` Ruediger Meier
2014-04-09 15:55                   ` Markus Trippelsdorf
2014-04-09 18:39                     ` Theodore Ts'o
2014-04-10  8:05                       ` Karel Zak
2014-04-10  9:17                         ` Ruediger Meier
2014-04-10 12:49                           ` Karel Zak
2014-04-10 13:16                             ` Ruediger Meier
2014-04-10 13:22                               ` Ruediger Meier
2014-04-08 17:26       ` Karel Zak
2014-04-08 17:30         ` Thomas Bächler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201404091716.05131.sweet_f_a@gmx.de \
    --to=sweet_f_a@gmx.de \
    --cc=kzak@redhat.com \
    --cc=thomas@archlinux.org \
    --cc=util-linux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox