Re: [make-initrd] [PATCH v2 00/11] Improve support for atomic actions

[Leonid Krivoshein <klark.devel@gmail.com>]

On 10/21/26 11:03 PM, Alexey Gladkov wrote:
> On Wed, Oct 21, 2026 at 10:31:55PM +0300, Leonid Krivoshein wrote:
>>
>> On 10/21/26 9:52 PM, Leonid Krivoshein wrote:
>>> Можно ещё глянуть эти пару функций (longtermop_start и longtermop_end)
>>> [1] и весь патч [2] на предмет взаимодействия с make-initrd rootdelay
>>> из-вне -- здесь учитывается вложенность вызовов. Такая обёртка меня
>>> вполне устраивает, т.е. на стороне make-initrd вложенность можно не
>>> учитывать. Позже переделаю их с учётом атомарности.
>>>
>>
>> Впрочем, на мой взгляд, в самом make-initrd реализация
>> rootdelay_pause/rootdelay_unpause *ТОЛЬКО* с подсчётом вызовов было бы
>> весьма здравой идеей, поскольку предсказать, какой модуль и в какой момент
>> что из этого вызовет заранее невозможно. Приостанавливать таймер нужно
>> только при первом вызове rootdelay_pause(), а возобновлять его работу только
>> при последнем вызове rootdelay_unpause(). Тогда и оборачивать не придётся.
>> Без этого, как мне кажется, оно уже и сейчас будет не правильно рассчитывать
>> оставшийся таймаут с несколькими параллельно работающими пользователями этих
>> функций.
> 
> Ну в целом да. rootdelay_pause / rootdelay_unpause в текущем виде это не
> тот API, который избавит от коллизий и гонок. Чтобы избавиться от гонок
> нужен fd_lock.
> 
> rootdelay_* функции пусть остаются как API именно для взаимодействия с
> rootdelayd. Синхронизацию пользователей пусть делает API выше.
> 

Обёртка решает проблему только в пределах единой группы модулей. Но, как 
я пояснил выше, оборачивать имеет смысл глобально, так как разные модули 
могут не знать ничего друг о друге, не догадываться, кто и когда вызовет 
эти функции. Следовательно, уже сейчас из-за гонок вычисления могут быть 
ошибочными, это глобальная проблема.


-- 
WBR, Leonid Krivoshein.