cleanup: Deadlock checker is no longer called from signal handler

Clean up a few leftovers from when the deadlock checker was called
from signal handler. We stopped doing that in commit 6753333f55, in
year 2015.

- CheckDeadLock can return a return value directly to the caller,
  there's no need to use a global variable for that.

- Remove outdated comments that claimed that CheckDeadLock "signals
  ProcSleep".

- It should be OK to ereport() from DeadLockCheck now. I considered
  getting rid of InitDeadLockChecking() and moving the workspace
  allocations into DeadLockCheck, but it's still good to avoid doing
  the allocations while we're holding all the partition locks. So just
  update the comment to give that as the reason we do the allocations
  up front.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/73d60ac385a93684f68297ae0ccb8f75bc6f23e1

Modified Files
--------------
src/backend/storage/lmgr/deadlock.c | 10 ++++------
src/backend/storage/lmgr/proc.c     | 28 ++++++++++++++++------------
2 files changed, 20 insertions(+), 18 deletions(-)

Reply via email to