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

On Wed, Apr 09, 2014 at 12:07:56PM +0200, Ruediger Meier wrote:
> Actually I think daily "fstrim -a" IS not a good idea for most of my 
> systems

install service != enable (this is important detail, I'll never agree
with automatically enabled services).

BTW, why IS not a good idea for your systems?

> but I feel no need to discuss this here on util-linux because 
> the decision whether, how, when and on which devices fstrim should be 
> called should not be made here at all.

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).

> If we add scripts with one generic use case for fstrim. Why don't we add 
> the generic boot and maintenance scripts how and when to mount or fsck 
> all filesystems, to activate swap, to get and set hwclock or whatever?

I really don't want to follow this insane direction of the discussion.

> Moreover the portability issue. Why adding scripts for systemd only 
> allthough the same could be done without systemd in a more portable 
> way.

is there any other unified, distribution independent and distribution
supported way to install, but no enable the stuff?

    Karel

-- 
 Karel Zak  <kzak@redhat.com>
 http://karelzak.blogspot.com

  reply	other threads:[~2014-04-09 11:03 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 [this message]
2014-04-09 12:12               ` Ruediger Meier
2014-04-09 12:49                 ` Thomas Bächler
2014-04-09 15:16                   ` Ruediger Meier
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=20140409110251.GT22126@x2.net.home \
    --to=kzak@redhat.com \
    --cc=sweet_f_a@gmx.de \
    --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