Re: [make-initrd] Release make-initrd 2.14.0

[Alexey Gladkov <gladkov.alexey@gmail.com>]

On Sat, Apr 03, 2021 at 02:31:22PM +0300, Leonid Krivoshein wrote:
> > $ git grep '\<get_dev '
> > data/lib/uevent/filters/mountdev:21:            get_dev dev "$fsdev" ||
> > data/lib/uevent/filters/resume:8:get_dev devresume "${RESUME-}" ||
> > features/luks/data/lib/uevent/filters/luks:31:                  get_dev devluks "$dev" &&
> > features/luks/data/lib/uevent/filters/luks:35:  get_dev devluks "$DEVNAME" ||
> > features/luks/data/lib/uevent/filters/lukskeys:26:      get_dev realdev "$keydev" ||
> > features/luks/data/lib/uevent/handlers/085-luks:32:             get_dev realdev "$luksdev"
> > features/luks/data/lib/uevent/handlers/085-luks:70:                     get_dev "$luksdev" ||
> > features/pipeline/data/lib/uevent/filters/pipeline-waitdev:13:  if [ -n "$spec" ] && get_dev dev "$spec"; then
> > 
> > Ты правда хочешь, чтобы во всех этих фичах появилась поддержка cdrom:* ?
> 
> Пока CDROM нужен лишь одной фиче (последняя строка), но, вдруг ещё где
> потребуется? Соответственно, варианта только два:
> 
> - либо предусмотреть в get_dev() возможность указывать префиксы a.k.a CDROM:
> , FSTYPE: для всех, кто запрашивает get_dev().
> - либо сделать обёртку типа get_pipeline_dev() и вызывать из неё get_dev(),
> а обёртку вызывать из pipeline-waitdev:13

Я как раз хотел предложить второй вариант. Если юскейсы появятся
глобального применения, то можно будет этот код перенести в get_dev.

> Мне больше нравится первый вариант, но я же советуюсь. По идее на
> перечисленных "клиентов" эта "возможность" влиять не должна. Префиксы
> позволяют дополнительно ограничить спецификацию, но никто же не заставляет
> использовать эти префиксы там, где они не требуются. Но можно и заюзать,
> например, так: resume=FSTYPE=swap:/dev/sdb3 и в таком случае devresume
> получит значение только в том случае, если /dev/sdb3 является SWAP-разделом.
> Таких префиксов можно и больше напридумывать.

-- 
Rgrds, legion