Page MenuHomePhabricator
Differential D19174

Add a "lock log" for debugging where locks are being held
ClosedPublic
Actions

Authored by epriestley on Mar 5 2018, 11:27 PM.
Tags
None
Referenced Files
F19505767: D19174.diff
Fri, Jan 9, 2:04 PM
F19505765: D19174.diff
Fri, Jan 9, 2:03 PM
F19505407: D19174.diff
Fri, Jan 9, 11:28 AM
F19153130: D19174.id45928.diff
Dec 11 2025, 7:43 AM
F19072675: D19174.diff
Nov 30 2025, 11:11 PM
F19069839: D19174.diff
Nov 30 2025, 3:06 PM
F19069700: D19174.diff
Nov 30 2025, 3:00 PM
F19069691: D19174.diff
Nov 30 2025, 3:00 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13096: Implement a global lock log to make it easier to debug global locks
Commits
rP44f0664d2c6f: Add a "lock log" for debugging where locks are being held
Summary

Depends on D19173. Ref T13096. Adds an optional, disabled-by-default lock log to make it easier to figure out what is acquiring and holding locks.

Test Plan

Ran bin/lock log --enable, --disable, --name, etc. Saw sensible-looking output with log enabled and daemons restarted. Saw no additional output with log disabled and daemons restarted.

Diff Detail

Repository
rP Phabricator
Branch
lock2
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsrc/infrastructure/util/PhabricatorGlobalLock.php:155XHP79Global Variables
Unit
Tests Passed
Build Status
Buildable 19739
Build 26736: Run Core Tests
Build 26735: arc lint + arc unit

Event Timeline

epriestley created this revision.Mar 5 2018, 11:27 PM
Harbormaster failed remote builds in B19739: Diff 45928!Mar 5 2018, 11:27 PM
This revision was not accepted when it landed; it landed in state Draft.Mar 6 2018, 1:55 AM
Closed by commit rP44f0664d2c6f: Add a "lock log" for debugging where locks are being held (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
bin/
1 line
resources/
sql/
autopatches/
9 lines
scripts/
setup/
21 lines
src/
8 lines
applications/
daemon/
garbagecollector/
29 lines
management/
222 lines
4 lines
storage/
32 lines
infrastructure/
daemon/
garbagecollector/
6 lines
storage/
management/
workflow/
12 lines
util/
51 lines

Diff 45928

View Options

bin/lock

Loading...
View Options

resources/sql/autopatches/20180305.lock.01.locklog.sql

Loading...
View Options

scripts/setup/manage_lock.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/daemon/garbagecollector/PhabricatorDaemonLockLogGarbageCollector.php

Loading...
View Options

src/applications/daemon/management/PhabricatorLockLogManagementWorkflow.php

Loading...
View Options

src/applications/daemon/management/PhabricatorLockManagementWorkflow.php

Loading...
View Options

src/applications/daemon/storage/PhabricatorDaemonLockLog.php

Loading...
View Options

src/infrastructure/daemon/garbagecollector/PhabricatorGarbageCollector.php

Loading...
View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php

Loading...
View Options

src/infrastructure/util/PhabricatorGlobalLock.php

Loading...
Phabricator · Built by Phacility