diff options
author | Eric Wong <e@80x24.org> | 2021-01-31 22:28:26 -1000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-02-01 11:38:19 +0000 |
commit | 7318712c430bdd15f4fb52cc6c4e10eeb7d0ded2 (patch) | |
tree | 3e905c7cced5e32bd23c14f0df10588187342594 /lib/PublicInbox/Lock.pm | |
parent | b436cf6e4794a32b3331a8727d10bf000ba55de2 (diff) | |
download | public-inbox-7318712c430bdd15f4fb52cc6c4e10eeb7d0ded2.tar.gz |
This allows us to avoid repeated open() and close() syscalls and speeds up the new xt/stress-sharedkv.t maintainer test by roughly 7%.
Diffstat (limited to 'lib/PublicInbox/Lock.pm')
-rw-r--r-- | lib/PublicInbox/Lock.pm | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/lib/PublicInbox/Lock.pm b/lib/PublicInbox/Lock.pm index bb213de4..c0c4c15c 100644 --- a/lib/PublicInbox/Lock.pm +++ b/lib/PublicInbox/Lock.pm @@ -41,6 +41,23 @@ sub lock_for_scope { PublicInbox::OnDestroy->new(@single_pid, \&lock_release, $self); } +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); +} + sub new_tmp { my ($cls, $ident) = @_; my $tmp = File::Temp->new("$ident.lock-XXXXXX", TMPDIR => 1); |