]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Spawn.pm
favor procedural calls for most private functions
[public-inbox.git] / lib / PublicInbox / Spawn.pm
index aa8d81b36de4832303fe1b923b26b092cd4dff64..23f303fbe182ee7968f6ca6ff6c1486498fe1f61 100644 (file)
@@ -1,10 +1,23 @@
 # Copyright (C) 2016 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
+# PERL_INLINE_DIRECTORY is explicitly defined in the environment.
+# Under Linux, vfork can make a big difference in spawning performance
+# as process size increases (fork still needs to mark pages for CoW use).
+# Currently, we only use this for code intended for long running
+# daemons (inside the PSGI code (-httpd) and -nntpd).  The short-lived
+# scripts (-mda, -index, -learn, -init) either use IPC::run or standard
+# Perl routines.
+
 package PublicInbox::Spawn;
 use strict;
 use warnings;
 use base qw(Exporter);
-our @EXPORT_OK = qw/which spawn/;
+use Symbol qw(gensym);
+use IO::Handle;
+use PublicInbox::ProcessPipe;
+our @EXPORT_OK = qw/which spawn popen_rd/;
 
 my $vfork_spawn = <<'VFORK_SPAWN';
 #include <sys/types.h>
@@ -99,7 +112,9 @@ if (defined $vfork_spawn) {
                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 => $vfork_spawn';
+               my $err = $@;
                flock($fh, LOCK_UN) or die "LOCK_UN failed on $f: $!\n";
+               die $err if $err;
        };
        if ($@) {
                warn "Inline::C failed for vfork: $@\n";
@@ -149,4 +164,18 @@ sub spawn ($;$$) {
        public_inbox_fork_exec($in, $out, $err, $f, $cmd, \@env);
 }
 
+sub popen_rd {
+       my ($cmd, $env, $opts) = @_;
+       pipe(my ($r, $w)) or die "pipe: $!\n";
+       $opts ||= {};
+       my $blocking = $opts->{Blocking};
+       IO::Handle::blocking($r, $blocking) if defined $blocking;
+       $opts->{1} = fileno($w);
+       my $pid = spawn($cmd, $env, $opts);
+       return ($r, $pid) if wantarray;
+       my $ret = gensym;
+       tie *$ret, 'PublicInbox::ProcessPipe', $pid, $r;
+       $ret;
+}
+
 1;