From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Sat, 3 Apr 2021 13:59:26 +0200 From: Alexey Gladkov To: make-initrd@lists.altlinux.org Message-ID: <20210403115926.agarfiguaod5rghg@example.org> References: <20210331152230.dshas3odtszxafyv@example.org> <27101e86-3f0a-4612-9ecf-eb46e84ec71a@basealt.ru> <20210331155728.w4x2y6sqnnuykdzf@example.org> <0d538efa-d51b-f3f7-3034-97f2aa56ce2e@gmail.com> <20210331165523.lm27dw6bxnsaux3d@example.org> <20210401090223.nvnaf6nahsm2waij@example.org> <7f508fac-8c89-7d23-5a56-9914ba0aaa9e@gmail.com> <20210403110923.djpovbnytsfafkjm@example.org> <7335da84-8213-c77b-7062-8af41ea46d46@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <7335da84-8213-c77b-7062-8af41ea46d46@gmail.com> Subject: Re: [make-initrd] Release make-initrd 2.14.0 X-BeenThere: make-initrd@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: make-initrd@lists.altlinux.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 03 Apr 2021 11:59:27 -0000 Archived-At: List-Archive: On Sat, Apr 03, 2021 at 02:31:22PM +0300, Leonid Krivoshein wrote: > > $ git grep '\ > 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