]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Spawn.pm
No ext_urls
[public-inbox.git] / lib / PublicInbox / Spawn.pm
index 00e6829e9483918c92bccefef153c1477c084b2f..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);
 
-my $all_libc = <<'ALL_LIBC'; # all *nix systems we support
+BEGIN {
+       my $all_libc = <<'ALL_LIBC'; # all *nix systems we support
 #include <sys/resource.h>
 #include <sys/socket.h>
 #include <sys/types.h>
@@ -31,6 +34,9 @@ my $all_libc = <<'ALL_LIBC'; # all *nix systems we support
 #include <unistd.h>
 #include <stdlib.h>
 #include <errno.h>
+#include <time.h>
+#include <stdio.h>
+#include <string.h>
 
 /* some platforms need alloca.h, but some don't */
 #if defined(__GNUC__) && !defined(alloca)
@@ -159,6 +165,22 @@ int pi_fork_exec(SV *redirref, SV *file, SV *cmdref, SV *envref, SV *rlimref,
        return (int)pid;
 }
 
+static int sleep_wait(unsigned *tries, int err)
+{
+       const struct timespec req = { 0, 100000000 }; /* 100ms */
+       switch (err) {
+       case ENOBUFS: case ENOMEM: case ETOOMANYREFS:
+               if (++*tries < 50) {
+                       fprintf(stderr, "sleeping on sendmsg: %s (#%u)\n",
+                               strerror(err), *tries);
+                       nanosleep(&req, NULL);
+                       return 1;
+               }
+       default:
+               return 0;
+       }
+}
+
 #if defined(CMSG_SPACE) && defined(CMSG_LEN)
 #define SEND_FD_CAPA 10
 #define SEND_FD_SPACE (SEND_FD_CAPA * sizeof(int))
@@ -177,6 +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 tries = 0;
 
        if (SvOK(data)) {
                iov.iov_base = SvPV(data, dlen);
@@ -204,7 +227,9 @@ SV *send_cmd4(PerlIO *s, SV *svfds, SV *data, int flags)
                        *fdp++ = SvIV(*fd);
                }
        }
-       sent = sendmsg(PerlIO_fileno(s), &msg, flags);
+       do {
+               sent = sendmsg(PerlIO_fileno(s), &msg, flags);
+       } while (sent < 0 && sleep_wait(&tries, errno));
        return sent >= 0 ? newSViv(sent) : &PL_sv_undef;
 }
 
@@ -243,105 +268,48 @@ 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>
-#include <errno.h>
-#include <stdio.h>
-#include <string.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} //= (
-               $ENV{XDG_CACHE_HOME} //
-               ( ($ENV{HOME} // '/nonexistent').'/.cache' )
-       ).'/public-inbox/inline-c';
-
-$set_nodatacow = $all_libc = undef unless -d $inline_dir && -w _;
-if (defined $all_libc) {
-       # Inline 0.64 or later has locking in multi-process env,
-       # but we support 0.5 on Debian wheezy
-       use Fcntl qw(:flock);
-       eval {
+       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 _;
+       $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 $fh, '>', $f or die "failed to open $f: $!\n";
-               flock($fh, LOCK_EX) or die "LOCK_EX failed on $f: $!\n";
-               eval 'use Inline C => $all_libc.$set_nodatacow';
-                       # . ', BUILD_NOISY => 1';
+               open my $oldout, '>&', \*STDOUT or die "dup(1): $!";
+               open my $olderr, '>&', \*STDERR or die "dup(2): $!";
+               open my $fh, '+>', $f or die "open($f): $!";
+               open STDOUT, '>&', $fh or die "1>$f: $!";
+               open STDERR, '>&', $fh or die "2>$f: $!";
+               STDERR->autoflush(1);
+               STDOUT->autoflush(1);
+
+               # 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, BUILD_NOISY => 1;
+EOM
                my $err = $@;
-               my $ndc_err;
-               if ($err && $set_nodatacow) { # missing Linux kernel headers
-                       $ndc_err = $err;
-                       undef $set_nodatacow;
-                       eval 'use Inline C => $all_libc';
+               my $ndc_err = '';
+               $err = $@;
+               open(STDERR, '>&', $olderr) or warn "restore stderr: $!";
+               open(STDOUT, '>&', $oldout) or warn "restore stdout: $!";
+               if ($err) {
+                       seek($fh, 0, SEEK_SET);
+                       my @msg = <$fh>;
+                       warn "Inline::C build failed:\n",
+                               $ndc_err, $err, "\n", @msg;
+                       $all_libc = undef;
                }
-               flock($fh, LOCK_UN) or die "LOCK_UN failed on $f: $!\n";
-               die $err if $err;
-               warn $ndc_err if $ndc_err;
-       };
-       if ($@) {
-               warn "Inline::C failed for vfork: $@\n";
-               $set_nodatacow = $all_libc = undef;
        }
-}
-
-unless ($all_libc) {
-       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;
-}
-
-undef $set_nodatacow;
-undef $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
+       }
+} # /BEGIN
 
 sub which ($) {
        my ($file) = @_;
@@ -358,10 +326,9 @@ sub spawn ($;$$) {
        my $f = which($cmd->[0]) // die "$cmd->[0]: command not found\n";
        my @env;
        $opts ||= {};
-
-       my %env = $env ? (%ENV, %$env) : %ENV;
+       my %env = (%ENV, $env ? %$env : ());
        while (my ($k, $v) = each %env) {
-               push @env, "$k=$v";
+               push @env, "$k=$v" if defined($v);
        }
        my $redir = [];
        for my $child_fd (0..2) {
@@ -398,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 ($;$$) {