ydb/util/thread, branch stable-23-2Mirror of YDB github reposhttps://code.mastervirt.ru/ydb/atom?h=stable-23-22022-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`, где мьютекс лочится заново.
Конечно, форкаться в сложных программах с тредпулами категорически запрещено, но, подозреваю, много кода в аркадии так написано. Давайте как-то защитимся от такого.Add ydb changelog2022-07-11T08:24:19Zbazeltsevbazeltsev@ydb.tech2022-07-11T08:24:19Zurn:sha1:8895ca7eeadc7a652558c10eb505ff84270a14b2
updatedIGNIETFERRO-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
IGNIETFERRO-1105 TAtomic -> std::atomic in util/thread/lfqueue.h2022-05-04T09:17:36Zeeighteeight@yandex-team.ru2022-05-04T09:17:36Zurn:sha1:1af8bb8789fdb0ca2927ff097537916a7f297bb1
ref:8cf44e7b3fecd13c3a0c699a8c1c7abe780eab0b
IGNIETFERRO-1105 TAtomic -> std::atomic in util/thread/lfstack.h2022-04-27T07:20:47Zeeighteeight@yandex-team.ru2022-04-27T07:20:47Zurn:sha1:590e169a0a90aafab8247e6f1797b646d897f086
ref:7fac644f716f0423df5d65d6ba11439ca0ca35d1
remove unused includes2022-04-07T05:59:43Ztobotobo@yandex-team.ru2022-04-07T05:59:43Zurn:sha1:8d0bed04edbd809859fe349779c07ef122fdeca2
ref:f8c119cf3a796a59ab5be4424e5364ba6a201d40
intermediate changes2022-03-15T10:35:56Zarcadia-devtoolsarcadia-devtools@yandex-team.ru2022-03-15T10:35:56Zurn:sha1:094638589de6a6c9f91fad0005843fc1c1adc957
ref:ca7a95e8c9a9d780f96497136a152091d54e61b5
clang-tidy - move-constructor and move-assignment operators should be marked noexcept + pass some args by reference2022-02-28T09:04:09Ztobotobo@yandex-team.ru2022-02-28T09:04:09Zurn:sha1:f23d808d6c8b0f50c51b44baad1f919cab389cde
ref:5da43163e6b3c2499597234dc80ad47042259013
Restoring authorship annotation for <gridem@yandex-team.ru>. Commit 2 of 2.2022-02-10T13:52:11Zgridemgridem@yandex-team.ru2022-02-10T13:52:11Zurn:sha1:2cc287f6e8cbdfbab1153770572ad4bd1aee9b37