Package: runit
Version: 2.1.2-42
Severity: normal

Hi,

I'm the guy who originally submitted the patch to support a 'nosync' flag
file to avoid syncing on shutdown.

I noted with some amazement in the changelog for version 2.1.2-42 that this
file had been moved to /run sometime in the past, with no mention in the
changelog.

I think this change should be reverted.

The use case of the nosync flag file is when running runit in a container
like LXC or linux-vserver. When you stop one of these (or hundreds of these
simultaneously), you don't want them to sync() because the host itself isn't
stopping, so there is neither a need nor a benefit to invoking sync().

Not wanting to invoke sync() on shutdown is a permanent property of a
system, not an ephemeral one. Thus, the flag file for this behaviour should
be in a permanent location (like /etc/runit, where it originally was), not
under /run, where it would need to be created on every boot.

Best regards,

AndrĂ¡s

-- System Information:
Debian Release: 11.0
  APT prefers unstable
  APT policy: (350, 'unstable'), (350, 'stable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 5.10.0-8-amd64 (SMP w/8 CPU threads)
Kernel taint flags: TAINT_PROPRIETARY_MODULE, TAINT_WARN, TAINT_OOT_MODULE, 
TAINT_UNSIGNED_MODULE
Locale: LANG=en_US.UTF-8, LC_CTYPE=hu_HU.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US.UTF-8
Shell: /bin/sh linked to /bin/bash
Init: runit (via /run/runit.stopit)
LSM: AppArmor: enabled

-- 
          A computer's attention span is as long as it's power cord.

Reply via email to