]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/LeiXSearch.pm
lei: keep $lei around until workers are reaped
[public-inbox.git] / lib / PublicInbox / LeiXSearch.pm
index b470c1130f60036d0f4c8d9619934825c03cec83..b4a9b89dc169f9f9f2a0f132cfb8b32c91c8ef52 100644 (file)
@@ -33,7 +33,7 @@ sub attach_external {
        my $srch = $ibxish->search or
                return warn("$desc not indexed for Xapian\n");
        my @shards = $srch->xdb_shards_flat or
-               return warn("$desc has no Xapian shardsXapian\n");
+               return warn("$desc has no Xapian shards\n");
 
        if (delete $self->{xdb}) { # XXX: do we need this?
                # clobber existing {xdb} if amending
@@ -109,9 +109,8 @@ sub wait_startq ($) {
 sub query_thread_mset { # for --thread
        my ($self, $lei, $ibxish) = @_;
        local $0 = "$0 query_thread_mset";
-       my $startq = delete $self->{5};
-       my %sig = $lei->atfork_child_wq($self);
-       local @SIG{keys %sig} = values %sig;
+       $lei->atfork_child_wq($self);
+       my $startq = delete $lei->{startq};
 
        my ($srch, $over) = ($ibxish->search, $ibxish->over);
        unless ($srch && $over) {
@@ -145,9 +144,8 @@ sub query_thread_mset { # for --thread
 sub query_mset { # non-parallel for non-"--thread" users
        my ($self, $lei) = @_;
        local $0 = "$0 query_mset";
-       my $startq = delete $self->{5};
-       my %sig = $lei->atfork_child_wq($self);
-       local @SIG{keys %sig} = values %sig;
+       $lei->atfork_child_wq($self);
+       my $startq = delete $lei->{startq};
        my $mo = { %{$lei->{mset_opt}} };
        my $mset;
        for my $loc (locals($self)) {
@@ -173,7 +171,7 @@ sub each_eml { # callback for MboxReader->mboxrd
        $smsg->parse_references($eml, mids($eml));
        $smsg->{$_} //= '' for qw(from to cc ds subject references mid);
        delete @$smsg{qw(From Subject -ds -ts)};
-       if (my $startq = delete($self->{5})) { wait_startq($startq) }
+       if (my $startq = delete($lei->{startq})) { wait_startq($startq) }
        $each_smsg->($smsg, undef, $eml);
 }
 
@@ -187,8 +185,7 @@ sub kill_reap {
 sub query_remote_mboxrd {
        my ($self, $lei, $uris) = @_;
        local $0 = "$0 query_remote_mboxrd";
-       my %sig = $lei->atfork_child_wq($self); # keep $self->{5} startq
-       local @SIG{keys %sig} = values %sig;
+       $lei->atfork_child_wq($self);
        my ($opt, $env) = @$lei{qw(opt env)};
        my @qform = (q => $lei->{mset_opt}->{qstr}, x => 'm');
        push(@qform, t => 1) if $opt->{thread};
@@ -286,7 +283,7 @@ sub query_done { # EOF callback
        my $has_l2m = exists $lei->{l2m};
        for my $f (qw(lxs l2m)) {
                my $wq = delete $lei->{$f} or next;
-               $wq->wq_wait_old;
+               $wq->wq_wait_old($lei);
        }
        $lei->{ovv}->ovv_end($lei);
        if ($has_l2m) { # close() calls LeiToMail reap_compress
@@ -351,75 +348,63 @@ sub start_query { # always runs in main (lei-daemon) process
 sub query_prepare { # called by wq_do
        my ($self, $lei) = @_;
        local $0 = "$0 query_prepare";
-       my %sig = $lei->atfork_child_wq($self);
-       -p $lei->{0} or die "BUG: \$done pipe expected";
-       local @SIG{keys %sig} = values %sig;
+       $lei->atfork_child_wq($self);
+       delete $lei->{l2m}->{-wq_s1};
        eval { $lei->{l2m}->do_augment($lei) };
        $lei->fail($@) if $@;
-       syswrite($lei->{0}, '.') == 1 or die "do_post_augment trigger: $!";
+       syswrite($lei->{op_pipe}, '.') == 1 or die "do_post_augment trigger: $!"
 }
 
 sub sigpipe_handler { # handles SIGPIPE from l2m/lxs workers
        my ($lei) = @_;
        my $lxs = delete $lei->{lxs};
-       if ($lxs && $lxs->wq_kill_old) {
-               kill 'PIPE', $$;
-               $lxs->wq_wait_old;
+       if ($lxs && $lxs->wq_kill_old) { # is this the daemon?
+               $lxs->wq_wait_old($lei);
        }
        close(delete $lei->{1}) if $lei->{1};
+       $lei->x_it(13);
 }
 
 sub do_query {
-       my ($self, $lei_orig) = @_;
-       my ($lei, @io) = $lei_orig->atfork_parent_wq($self);
-       $io[0] = undef;
-       pipe(my $done, $io[0]) or die "pipe $!";
-       $lei_orig->{1}->autoflush(1);
+       my ($self, $lei) = @_;
+       $lei->{1}->autoflush(1);
+       my ($au_done, $zpipe);
+       my $l2m = $lei->{l2m};
+       if ($l2m) {
+               pipe($lei->{startq}, $au_done) or die "pipe: $!";
+               # 1031: F_SETPIPE_SZ
+               fcntl($lei->{startq}, 1031, 4096) if $^O eq 'linux';
+               $zpipe = $l2m->pre_augment($lei);
+       }
+       pipe(my $done, $lei->{op_pipe}) or die "pipe $!";
+       my ($lei_ipc, @io) = $lei->atfork_parent_wq($self);
+       delete($lei->{op_pipe});
 
-       $lei_orig->event_step_init; # wait for shutdowns
+       $lei->event_step_init; # wait for shutdowns
        my $done_op = {
-               '' => [ \&query_done, $lei_orig ],
-               '!' => [ \&sigpipe_handler, $lei_orig ]
+               '' => [ \&query_done, $lei ],
+               '!' => [ \&sigpipe_handler, $lei ]
        };
-       my $in_loop = exists $lei_orig->{sock};
+       my $in_loop = exists $lei->{sock};
        $done = PublicInbox::OpPipe->new($done, $done_op, $in_loop);
-       my $l2m = $lei->{l2m};
        if ($l2m) {
-               # may redirect $lei->{1} for mbox
-               my $zpipe = $l2m->pre_augment($lei_orig);
-               $io[1] = $lei_orig->{1};
-               pipe(my ($startq, $au_done)) or die "pipe: $!";
-               $done_op->{'.'} = [ \&do_post_augment, $lei_orig,
-                                       $zpipe, $au_done ];
-               local $io[4] = *STDERR{GLOB}; # don't send l2m->{-wq_s1}
-               die "BUG: unexpected \$io[5]: $io[5]" if $io[5];
-               $self->wq_do('query_prepare', \@io, $lei);
-               fcntl($startq, 1031, 4096) if $^O eq 'linux'; # F_SETPIPE_SZ
-               $io[5] = $startq;
+               $done_op->{'.'} = [ \&do_post_augment, $lei, $zpipe, $au_done ];
+               $self->wq_do('query_prepare', \@io, $lei_ipc);
                $io[1] = $zpipe->[1] if $zpipe;
        }
-       start_query($self, \@io, $lei);
+       start_query($self, \@io, $lei_ipc);
        $self->wq_close(1);
        unless ($in_loop) {
-               # for the $lei->atfork_child_wq PIPE handler:
+               # for the $lei_ipc->atfork_child_wq PIPE handler:
                while ($done->{sock}) { $done->event_step }
        }
 }
 
-sub ipc_atfork_prepare {
-       my ($self) = @_;
-       if (exists $self->{remotes}) {
-               require PublicInbox::MboxReader;
-               require IO::Uncompress::Gunzip;
-       }
-       # FDS: (0: done_wr, 1: stdout|mbox, 2: stderr,
-       #       3: sock, 4: $l2m->{-wq_s1}, 5: $startq)
-       $self->SUPER::ipc_atfork_prepare; # PublicInbox::IPC
-}
-
 sub add_uri {
        my ($self, $uri) = @_;
        if (my $curl = $self->{curl} //= which('curl') // 0) {
+               require PublicInbox::MboxReader;
+               require IO::Uncompress::Gunzip;
                push @{$self->{remotes}}, $uri;
        } else {
                warn "curl missing, ignoring $uri\n";