ydb/util/thread/pool.cpp, branch stable-23-3Mirror of YDB github reposhttps://code.mastervirt.ru/ydb/atom?h=stable-23-32022-11-10T09:04:05Z[util] Terminate instead of deadlock after fork with running thread pools2022-11-10T09:04:05Zsskvorsskvor@yandex-team.com2022-11-10T09:04:05Zurn:sha1:6d5ed9f0166178823f3a752bfe46dd7ef5042df8
Контекст. В древнем странном коде обнаружили, что если создавать `TThreadPool` часто и конкуретно с запуском `TShellCommand`, то иногда последняя дедлочится: если начали выполнять конструктор `TThreadPool` и регистрацию в `TAtforkQueueRestarter`, захватив `TAtforkQueueRestarter::ActionMutex`, а в другом потоке параллельно сделали `fork`, то последний может форкнуться с залоченым мьютексом. Первым делом происходит вызов коллбеков из `pthread_atfork`, где мьютекс лочится заново.
Конечно, форкаться в сложных программах с тредпулами категорически запрещено, но, подозреваю, много кода в аркадии так написано. Давайте как-то защитимся от такого.IGNIETFERRO-1105 Get rid of TAtomic in thread/pool.cpp2022-05-17T07:48:10Zeeighteeight@yandex-team.ru2022-05-17T07:48:10Zurn:sha1:1e1dfc1b3545147948cc2190bb15161f630507e9
ref:3fa56aa890b75b7301a3193c2ee40106281a71cb
IGNIETFERRO-1105 TAtomic -> std::atomic in util/generic/* and threadpool2022-05-11T07:40:18Zeeighteeight@yandex-team.ru2022-05-11T07:40:18Zurn:sha1:05a6fea78142058210511273b3ece21050ad137b
ref:39a714b781c60dca9e3b946d870971076e14ab7c
Restoring authorship annotation for <seege@yandex-team.ru>. Commit 2 of 2.2022-02-10T13:52:02Zseegeseege@yandex-team.ru2022-02-10T13:52:02Zurn:sha1:e7d460ea3389cb4f573ef5fb09f7072549a17765Restoring authorship annotation for <seege@yandex-team.ru>. Commit 1 of 2.2022-02-10T13:52:02Zseegeseege@yandex-team.ru2022-02-10T13:52:02Zurn:sha1:091a65e2b3147ce957ba692cf07d3790405afe0fRestoring authorship annotation for <sievlev@yandex-team.ru>. Commit 2 of 2.2022-02-10T13:51:45Zsievlevsievlev@yandex-team.ru2022-02-10T13:51:45Zurn:sha1:3288844da9a26e598b08a1f4c2362603bcf1f506Restoring authorship annotation for <sievlev@yandex-team.ru>. Commit 1 of 2.2022-02-10T13:51:45Zsievlevsievlev@yandex-team.ru2022-02-10T13:51:45Zurn:sha1:c38d513b2c4ace7c22019ef4cc24003204b77391Restoring authorship annotation for <onpopov@yandex-team.ru>. Commit 2 of 2.2022-02-10T13:50:38Zonpopovonpopov@yandex-team.ru2022-02-10T13:50:38Zurn:sha1:8773f7661194d4c0bdb1e3937b2ff7ae01dd13f8Restoring authorship annotation for <onpopov@yandex-team.ru>. Commit 1 of 2.2022-02-10T13:50:38Zonpopovonpopov@yandex-team.ru2022-02-10T13:50:38Zurn:sha1:84a29dd4980d5b39615e453f289bd1a81213296dRestoring authorship annotation for <grig@yandex-team.ru>. Commit 2 of 2.2022-02-10T13:50:24Zgriggrig@yandex-team.ru2022-02-10T13:50:24Zurn:sha1:beb63ece3a6872dfbe113104f524ab6fdbec0adc