mirror of
https://github.com/postgres/postgres.git
synced 2025-09-02 04:21:28 +03:00
Fix stale idle flag when IO workers exit.
Otherwise we could choose a worker that has exited and crash while trying to wake it up. Back-patch to 18. Reported-by: Tomas Vondra <tomas@vondra.me> Reported-by: Andres Freund <andres@anarazel.de> Discussion: https://postgr.es/m/t5aqjhkj6xdkido535pds7fk5z4finoxra4zypefjqnlieevbg%40357aaf6u525j
This commit is contained in:
@@ -172,6 +172,7 @@ pgaio_choose_idle_worker(void)
|
||||
/* Find the lowest bit position, and clear it. */
|
||||
worker = pg_rightmost_one_pos64(io_worker_control->idle_worker_mask);
|
||||
io_worker_control->idle_worker_mask &= ~(UINT64_C(1) << worker);
|
||||
Assert(io_worker_control->workers[worker].in_use);
|
||||
|
||||
return worker;
|
||||
}
|
||||
@@ -316,6 +317,7 @@ pgaio_worker_die(int code, Datum arg)
|
||||
Assert(io_worker_control->workers[MyIoWorkerId].in_use);
|
||||
Assert(io_worker_control->workers[MyIoWorkerId].latch == MyLatch);
|
||||
|
||||
io_worker_control->idle_worker_mask &= ~(UINT64_C(1) << MyIoWorkerId);
|
||||
io_worker_control->workers[MyIoWorkerId].in_use = false;
|
||||
io_worker_control->workers[MyIoWorkerId].latch = NULL;
|
||||
LWLockRelease(AioWorkerSubmissionQueueLock);
|
||||
|
Reference in New Issue
Block a user