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

[Alexey Gladkov <legion@kernel.org>]

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 выше.

-- 
Rgrds, legion