Package: util-linux
Version: 2.33.1-0.1
Severity: wishlist
File: /lib/systemd/system/fstrim.service

Currently fstrim.service runs "fstrim --fstab --verbose", so it silently ignores

  * Stuff mounted via hand-written systemd.mount units (e.g. 
/usr/share/systemd/tmp.mount);
  * ZFS datasets mounted via -o canmount and -o mountpoint properties.

Why not use "fstrim --all --verbose" instead?

Or, if you keep "fstrim --fstab --verbose", please consider adding

    [Unit]
    ConditionPathExists=/etc/fstab

so that it will NOP instead of erroring when fstab is completely absent:

    systemd[1]: Condition check resulted in Discard unused blocks on 
filesystems from /etc/fstab being skipped.

Whereas currently it does this:

    fstrim[5003]: fstrim: failed to parse /etc/fstab: No such file or directory
    systemd[1]: fstrim.service: Main process exited, code=exited, status=32/n/a
    systemd[1]: fstrim.service: Failed with result 'exit-code'.
    systemd[1]: Failed to start Discard unused blocks on filesystems from 
/etc/fstab.
    systemd[1]: fstrim.service: Consumed 38ms CPU time, no IP traffic.

PS: fstrim.timer is disabled by default, so you only see errors if you
have no /etc/fstab *and* enable fstrim.timer or start fstrim.service.


-- System Information:
Debian Release: 10.0
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'testing-proposed-updates'), (500, 
'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 4.19.0-5-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_AU.UTF-8, LC_CTYPE=en_AU.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_AU.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages util-linux depends on:
ii  fdisk          2.33.1-0.1
ii  libaudit1      1:2.8.4-3
ii  libblkid1      2.33.1-0.1
ii  libc6          2.28-10
ii  libcap-ng0     0.7.9-2
ii  libmount1      2.33.1-0.1
ii  libpam0g       1.3.1-5
ii  libselinux1    2.8-1+b1
ii  libsmartcols1  2.33.1-0.1
ii  libsystemd0    241-3
ii  libtinfo6      6.1+20181013-2
ii  libudev1       241-3
ii  libuuid1       2.33.1-0.1
ii  login          1:4.5-1.1
ii  zlib1g         1:1.2.11.dfsg-1

util-linux recommends no packages.

Versions of packages util-linux suggests:
ii  dosfstools          4.1-2
ii  kbd                 2.0.4-4
ii  util-linux-locales  2.33.1-0.1

-- no debconf information

Reply via email to