From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM autolearn=ham autolearn_force=no version=3.4.1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=subject:to:references:from:message-id:date:user-agent:mime-version :in-reply-to:content-language; bh=yBMtpnGGBFNQDliZmje6Zxf6v8QmUmYjHaV36KF5h6g=; b=WI+4e6OE4rJcxwqQBDqoNL3onWv+LMlYKaed1qfBC+8xuWLNVhsc0AnQy6AbCw75oM 32hw8PO0ySRJGeei54S0HrsPrzMFSFIcy2x4MyI98NgH/dI69FaCNywk6p6c/Lqi6ou2 7wTx6YQp6kb+OSCEezxI9o/a6Mf6mYsr+5mh8+D+ASAvhNiKfG4e+rWR6deqZkL+wmWi ST8rYazSSrovickkcmKmldPhRnnyVEBKFOjCO4TrM7Si80vRkeishu73oyIkP9TWpvN0 zb+8zUWkx9ibCE0TLeS0uV2nRIPHjmp++9ryTKEK2h5MVF8xy0HsS39zGRdM85gaBf3D 41PQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language; bh=yBMtpnGGBFNQDliZmje6Zxf6v8QmUmYjHaV36KF5h6g=; b=A4p1bH6+gMoMLjwqNNhtnXVnUo+hJxM1vpA7Saqdq+xVfl5VT9fzh7CQ8KMfRsCK9n yAgEHyPMs9mvEPnEIuxCOf69sePmQA5fVQrU5W8wUcZ7K/VMlnOuBMkDLFFq7S4KSJ2d eh9tmKtHwAJ+DS/RCedmImx2o/2FKJaSgm34JeR/ImAwVIJcM9DcD6frmZ00TL71XkF+ Uwe68+691JGZQrY+74R+/JbRX9KBQqnobC+q0YZ7DFCcIMHEm5bH4k0agNiYtpwJq+9C vyWdht/gy7bkmBHCTohFV3v0kUg54sHgs/cVmZQA921IV5OBLARitXhc2t12UuF6CoCL vFKQ== X-Gm-Message-State: AOAM532VbzzMahp+3rrnxCjkNn0IOpqSL7DgAQHsYXtFGM0wxR1jVJUu 0AlczyQnBgOVmyI8YUpHGaRQ7FQrPYQ= X-Google-Smtp-Source: ABdhPJyUGYd/L8A0KkqprBNr/5f+r6HNskFjQxnE8Qjgk+ybQMtgH7tsYOEDTug4zyWCk496YQv+Vw== X-Received: by 2002:a19:ee12:: with SMTP id g18mr2578467lfb.300.1632789636469; Mon, 27 Sep 2021 17:40:36 -0700 (PDT) To: make-initrd@lists.altlinux.org References: <7636585b-202d-95e0-eb0a-d320f69cc77a@gmail.com> <20210926140917.4xky7nfnlemy4jh3@example.org> <94921a86-1043-8273-2c29-957cf53e14e1@gmail.com> <20210927090907.fbre22j7urxlnukt@example.org> <20210927135745.35ejs7bg67mdjxbx@example.org> <245f1b4c-b417-11b7-57c3-6650ade2ef75@gmail.com> From: Leonid Krivoshein Message-ID: <49aa03c4-c94d-53fe-c595-5ce605dca197@gmail.com> Date: Tue, 28 Sep 2021 03:40:34 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: <245f1b4c-b417-11b7-57c3-6650ade2ef75@gmail.com> Content-Type: multipart/mixed; boundary="------------294EABED080940846D77A1DA" Content-Language: ru Subject: Re: [make-initrd] [PATCH v1 21/41] fork pipeline: bootchain sysvinit script now use external hooks 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: Tue, 28 Sep 2021 00:40:39 -0000 Archived-At: List-Archive: This is a multi-part message in MIME format. --------------294EABED080940846D77A1DA Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit 27.09.2021 20:22, Leonid Krivoshein пишет: > > 27.09.2021 16:57, Alexey Gladkov пишет: >> On Mon, Sep 27, 2021 at 04:11:43PM +0300, Leonid Krivoshein wrote: >>> [...] >>>>> Другим компонентам bootchain при расширении (а теперь это уже более >>>>> десятка фич и подпакетов) тоже может потребоваться подобная >>>>> инициализация. >>>>> Здесь в одном коммите видно, куда переехал код функции prepare(). >>>>> Хуки нужны >>>>> для того, чтобы вызывать подобную инициализацию из определённого >>>>> места, так >>>>> как базовый пакет bootchain-core может не знать, с какими >>>>> компонентами >>>>> bootchain собран initramfs. Соответственно, включаемые скрипты >>>>> кладутся в >>>>> /lib/bootchain-prepare.d. Если потребуется, по аналогии можно >>>>> добавить в >>>>> stop() хуки для деинициализации. >>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L32-L38 >>>> >>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L61 >>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L109 >>>> >>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L114 >>>> >>>> >>>> Ты имеешь в виду вот такие хуки ? >>> Ну очень похожие, хотя use_hooks() явно проще -- без блокировки >>> консоли, без >>> бита выполнения, просто соурсим исходники: >> Эти хуки уже давно есть и можно использовать. Не нужно дублировать >> этот же >> функционал. > > Дойдя до этого места не понял, как их можно использовать... > > Делаю скрипт выполняемым, а класть-то его куда? В > /lib/initrd/pre/bootchain-core/ ? > Проверочная сборка показала, что этот путь не работает. Если загрузиться с параметрами waitdev=UUID=1234 waitdev_timeout=180 rdshell и запустить из rdshell руками /lib/initrd/pre/bootchain-core/300-waitdev , только тогда видно, что правильно отрабатывает. Но автоматом хук не запускается. В коде есть механизм запуска, но он не используется ни в одной фиче и в data/ его тоже нет, только сам этот скрипт rc. Вот изменения, на всякий случай: -- Best regards, Leonid Krivoshein. --------------294EABED080940846D77A1DA Content-Type: text/x-patch; name="waitdev.diff" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="waitdev.diff" commit d5340fea7efcf9995b7391a17b9781462556e661 Author: Leonid Krivoshein Date: Tue Sep 28 00:14:28 2021 +0300 core+waitdev: use common way for hooks call instead internal use_hooks() See also: https://lists.altlinux.org/pipermail/make-initrd/2021-September/000685.html diff --git a/bootchain-core/data/etc/rc.d/init.d/bootchain b/bootchain-core/data/etc/rc.d/init.d/bootchain index 7bdb1cf..1538339 100755 --- a/bootchain-core/data/etc/rc.d/init.d/bootchain +++ b/bootchain-core/data/etc/rc.d/init.d/bootchain @@ -17,24 +17,9 @@ NAME=bootchained PIDFILE="/var/run/$NAME.pid" ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME --displayname $NAME" -use_hooks() -{ - local hook hdir="$1" - - if [ -d "$hdir" ]; then - # shellcheck disable=SC2012 - for hook in $(ls -1 -- "$hdir"/* |sort) _; do - [ -s "$hook" ] || - continue - . "$hook" - done - fi -} - start() { RETVAL=0 if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then - use_hooks /lib/bootchain-prepare.d start_daemon --background $ARGS "$NAME" RETVAL=$? fi diff --git a/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev b/bootchain-waitdev/data/lib/initrd/pre/bootchain-core/300-waitdev old mode 100644 new mode 100755 similarity index 71% rename from bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev rename to bootchain-waitdev/data/lib/initrd/pre/bootchain-core/300-waitdev index f7315a3..5e0f040 --- a/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev +++ b/bootchain-waitdev/data/lib/initrd/pre/bootchain-core/300-waitdev @@ -1,10 +1,13 @@ #!/bin/bash -efu +. /.initrd/initenv + dir=/.initrd/bootchain/waitdev mkdir -p -- "$dir" [ -z "${WAITDEV_TIMEOUT-}" ] || - printf '%s\n' "$WAITDEV_TIMEOUT" >"$dir/TIMECNT" + printf '%s\n' "$WAITDEV_TIMEOUT" >"$dir"/TIMECNT + i=0 while [ "$i" -lt "${WAITDEV:-0}" ]; do touch "$dir/$i" --------------294EABED080940846D77A1DA--