]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Spawn.pm
No ext_urls
[public-inbox.git] / lib / PublicInbox / Spawn.pm
index 6ca1ca2a0be3538b3cb9dc663422698b6b94952b..826ee508801ea1f9d519b16196c118e535997a02 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (C) 2016-2021 all contributors <meta@public-inbox.org>
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 #
 # This allows vfork to be used for spawning subprocesses if
 # We don't want too many DSOs: https://udrepper.livejournal.com/8790.html
 
 package PublicInbox::Spawn;
-use strict;
+use v5.12;
 use parent qw(Exporter);
 use Symbol qw(gensym);
 use Fcntl qw(LOCK_EX SEEK_SET);
 use IO::Handle ();
 use PublicInbox::ProcessPipe;
-our @EXPORT_OK = qw(which spawn popen_rd run_die nodatacow_dir);
+our @EXPORT_OK = qw(which spawn popen_rd run_die);
 our @RLIMITS = qw(RLIMIT_CPU RLIMIT_CORE RLIMIT_DATA);
 
 BEGIN {
@@ -165,14 +165,14 @@ int pi_fork_exec(SV *redirref, SV *file, SV *cmdref, SV *envref, SV *rlimref,
        return (int)pid;
 }
 
-static int sleep_wait(unsigned *try, int err)
+static int sleep_wait(unsigned *tries, int err)
 {
        const struct timespec req = { 0, 100000000 }; /* 100ms */
        switch (err) {
        case ENOBUFS: case ENOMEM: case ETOOMANYREFS:
-               if (++*try < 50) {
+               if (++*tries < 50) {
                        fprintf(stderr, "sleeping on sendmsg: %s (#%u)\n",
-                               strerror(err), *try);
+                               strerror(err), *tries);
                        nanosleep(&req, NULL);
                        return 1;
                }
@@ -199,7 +199,7 @@ SV *send_cmd4(PerlIO *s, SV *svfds, SV *data, int flags)
        AV *fds = (AV *)SvRV(svfds);
        I32 i, nfds = av_len(fds) + 1;
        int *fdp;
-       unsigned try = 0;
+       unsigned tries = 0;
 
        if (SvOK(data)) {
                iov.iov_base = SvPV(data, dlen);
@@ -229,7 +229,7 @@ SV *send_cmd4(PerlIO *s, SV *svfds, SV *data, int flags)
        }
        do {
                sent = sendmsg(PerlIO_fileno(s), &msg, flags);
-       } while (sent < 0 && sleep_wait(&try, errno));
+       } while (sent < 0 && sleep_wait(&tries, errno));
        return sent >= 0 ? newSViv(sent) : &PL_sv_undef;
 }
 
@@ -268,63 +268,14 @@ void recv_cmd4(PerlIO *s, SV *buf, STRLEN n)
 #endif /* defined(CMSG_SPACE) && defined(CMSG_LEN) */
 ALL_LIBC
 
-# btrfs on Linux is copy-on-write (COW) by default.  As of Linux 5.7,
-# this still leads to fragmentation for SQLite and Xapian files where
-# random I/O happens, so we disable COW just for SQLite files and Xapian
-# directories.  Disabling COW disables checksumming, so we only do this
-# for regeneratable files, and not canonical git storage (git doesn't
-# checksum refs, only data under $GIT_DIR/objects).
-       my $set_nodatacow = $^O eq 'linux' ? <<'SET_NODATACOW' : '';
-#include <sys/ioctl.h>
-#include <sys/vfs.h>
-#include <linux/magic.h>
-#include <linux/fs.h>
-#include <dirent.h>
-
-void nodatacow_fd(int fd)
-{
-       struct statfs buf;
-       int val = 0;
-
-       if (fstatfs(fd, &buf) < 0) {
-               fprintf(stderr, "fstatfs: %s\\n", strerror(errno));
-               return;
-       }
-
-       /* only btrfs is known to have this problem, so skip for non-btrfs */
-       if (buf.f_type != BTRFS_SUPER_MAGIC)
-               return;
-
-       if (ioctl(fd, FS_IOC_GETFLAGS, &val) < 0) {
-               fprintf(stderr, "FS_IOC_GET_FLAGS: %s\\n", strerror(errno));
-               return;
-       }
-       val |= FS_NOCOW_FL;
-       if (ioctl(fd, FS_IOC_SETFLAGS, &val) < 0)
-               fprintf(stderr, "FS_IOC_SET_FLAGS: %s\\n", strerror(errno));
-}
-
-void nodatacow_dir(const char *dir)
-{
-       DIR *dh = opendir(dir);
-       int fd;
-
-       if (!dh) croak("opendir(%s): %s", dir, strerror(errno));
-       fd = dirfd(dh);
-       if (fd >= 0)
-               nodatacow_fd(fd);
-       /* ENOTSUP probably won't happen under Linux... */
-       closedir(dh);
-}
-SET_NODATACOW
-
-       my $inline_dir = $ENV{PERL_INLINE_DIRECTORY} //= (
+       my $inline_dir = $ENV{PERL_INLINE_DIRECTORY} // (
                        $ENV{XDG_CACHE_HOME} //
                        ( ($ENV{HOME} // '/nonexistent').'/.cache' )
                ).'/public-inbox/inline-c';
        warn "$inline_dir exists, not writable\n" if -e $inline_dir && !-w _;
-       $set_nodatacow = $all_libc = undef unless -d _ && -w _;
+       $all_libc = undef unless -d _ && -w _;
        if (defined $all_libc) {
+               local $ENV{PERL_INLINE_DIRECTORY} = $inline_dir;
                my $f = "$inline_dir/.public-inbox.lock";
                open my $oldout, '>&', \*STDOUT or die "dup(1): $!";
                open my $olderr, '>&', \*STDERR or die "dup(2): $!";
@@ -337,17 +288,10 @@ SET_NODATACOW
                # CentOS 7.x ships Inline 0.53, 0.64+ has built-in locking
                flock($fh, LOCK_EX) or die "LOCK_EX($f): $!";
                eval <<'EOM';
-use Inline C => $all_libc.$set_nodatacow, BUILD_NOISY => 1;
+use Inline C => $all_libc, BUILD_NOISY => 1;
 EOM
                my $err = $@;
                my $ndc_err = '';
-               if ($err && $set_nodatacow) { # missing Linux kernel headers
-                       $ndc_err = "with set_nodatacow: <\n$err\n>\n";
-                       undef $set_nodatacow;
-                       eval <<'EOM';
-use Inline C => $all_libc, BUILD_NOISY => 1;
-EOM
-               };
                $err = $@;
                open(STDERR, '>&', $olderr) or warn "restore stderr: $!";
                open(STDOUT, '>&', $oldout) or warn "restore stdout: $!";
@@ -356,22 +300,15 @@ EOM
                        my @msg = <$fh>;
                        warn "Inline::C build failed:\n",
                                $ndc_err, $err, "\n", @msg;
-                       $set_nodatacow = $all_libc = undef;
-               } elsif ($ndc_err) {
-                       warn "Inline::C build succeeded w/o set_nodatacow\n",
-                               "error $ndc_err";
+                       $all_libc = undef;
                }
        }
-       unless ($all_libc) {
+       if (defined $all_libc) { # set for Gcf2
+               $ENV{PERL_INLINE_DIRECTORY} = $inline_dir;
+       } else {
                require PublicInbox::SpawnPP;
                *pi_fork_exec = \&PublicInbox::SpawnPP::pi_fork_exec
        }
-       unless ($set_nodatacow) {
-               require PublicInbox::NDC_PP;
-               no warnings 'once';
-               *nodatacow_fd = \&PublicInbox::NDC_PP::nodatacow_fd;
-               *nodatacow_dir = \&PublicInbox::NDC_PP::nodatacow_dir;
-       }
 } # /BEGIN
 
 sub which ($) {
@@ -428,9 +365,9 @@ sub popen_rd {
        $opt->{1} = fileno($w);
        my $pid = spawn($cmd, $env, $opt);
        return ($r, $pid) if wantarray;
-       my $ret = gensym;
-       tie *$ret, 'PublicInbox::ProcessPipe', $pid, $r, @$opt{qw(cb arg)};
-       $ret;
+       my $s = gensym;
+       tie *$s, 'PublicInbox::ProcessPipe', $pid, $r, @{$opt->{cb_arg} // []};
+       $s;
 }
 
 sub run_die ($;$$) {