config: lazy-load coderepos, support extindex

[public-inbox.git] / lib / PublicInbox / Lock.pm

diff --git a/lib/PublicInbox/Lock.pm b/lib/PublicInbox/Lock.pm
index 1d0b4f9cbcbb104741bb6cb09396184d71fcad53..76c3ffb231f8d344196a80da5d574dc028943766 100644 (file)
--- a/lib/PublicInbox/Lock.pm
+++ b/lib/PublicInbox/Lock.pm
@@ -1,4 +1,4 @@
-# Copyright (C) 2018-2020 all contributors <meta@public-inbox.org>
+# Copyright (C) 2018-2021 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
 # Base class for per-inbox locking
@@ -8,7 +8,6 @@ use v5.10.1;
 use Fcntl qw(:flock :DEFAULT);
 use Carp qw(croak);
 use PublicInbox::OnDestroy;
-use File::Temp ();
 
 # we only acquire the flock if creating or reindexing;
 # PublicInbox::Import already has the lock on its own.
@@ -37,14 +36,25 @@ sub lock_release {
 # caller must use return value
 sub lock_for_scope {
        my ($self, @single_pid) = @_;
-       $self->lock_acquire;
+       lock_acquire($self) or return; # lock_path not set
        PublicInbox::OnDestroy->new(@single_pid, \&lock_release, $self);
 }
 
-sub new_tmp {
-       my ($cls, $ident) = @_;
-       my $tmp = File::Temp->new("$ident.lock-XXXXXX", TMPDIR => 1);
-       bless { lock_path => $tmp->filename, tmp => $tmp }, $cls;
+sub lock_acquire_fast {
+       $_[0]->{lockfh} or return lock_acquire($_[0]);
+       flock($_[0]->{lockfh}, LOCK_EX) or croak "lock (fast) failed: $!";
+}
+
+sub lock_release_fast {
+       flock($_[0]->{lockfh} // return, LOCK_UN) or
+                       croak "unlock (fast) $_[0]->{lock_path}: $!";
+}
+
+# caller must use return value
+sub lock_for_scope_fast {
+       my ($self, @single_pid) = @_;
+       lock_acquire_fast($self) or return; # lock_path not set
+       PublicInbox::OnDestroy->new(@single_pid, \&lock_release_fast, $self);
 }
 
 1;