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-transfer-encoding:content-language; bh=BDFUXGui4rPvoQTf8KG2raufbacIn+NTlWqMvy35pHY=; b=L55ns4eZ5TVdDubSeeAI270eHf3uDYyceNf2tSFsmoV3J0IlggCSqHdgZlZjkOGrS7 kQxs5zTTh64Bq/YJIWwSUG2HqjuJtSUpC66g2QmxBdfIxAidWBwFjE3cfAMWFkvQWUAp 06zP1/8V6Sygt2nqYhdrxDdaR3KfDWVfwArppSvX480wLRm2qVguEIKwYCW35YnxwWfb Nq3WsqnY71N65RUfKyV0CBBAPnt+Vz+YinecPTdxaMxRvc5ykbL07rUz8TXkV323+xoY VNDEdyMzlreuhKrEiAJeFRVVFPwSX3+PgPPSOHb29v6QYF8IN/BXTP+duv4VKKNacSbm L7wQ== 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-transfer-encoding :content-language; bh=BDFUXGui4rPvoQTf8KG2raufbacIn+NTlWqMvy35pHY=; b=NZOj4FTug9kTQ+7FXLgATprQdukYRN0pMzD6DQtUYGrHBfiN/KR3cN/SUZIsZdnSSO 0QzyM5vEezhA1t38hascJ2IhlyFpNGstfjJUg19GBeY6Bp48QaE8WjgLDKGcazVX9ZA1 B8XY1BlJUZr4FGkjUiyRIm9w/C6cn5szY5y6lntUTals7c34STBLhRD4R3FRwkdURjve 2wh7QNr1kH48uy/IvFWfI28WAJ83m3ZuxbUp5q3ylku6mZE/kz5U02Yom+92d3HQtHrb AkyTy0TbLBBW2ENa9Yr2elD/TD0MWrPmAB7a4O+9wLz35P180BDpLgd3gnV4ksQOEEuo r+2g== X-Gm-Message-State: AOAM531lUzDSbfFSG2SYwQiQnoKNUSeqbZdnJ8PgSUTg4eRPoY8gSbzt 3BGNjllsdH0gr9d0EWGwYPss6HcCC88= X-Google-Smtp-Source: ABdhPJxW0PtTaN4pLxTpkh3ZsqvzS348S00EvAy/srdAruFEx6za0HkJ15MyQDzfUKe+IVo6/7XPdA== X-Received: by 2002:a05:6512:3b7:: with SMTP id v23mr7037709lfp.127.1632685989367; Sun, 26 Sep 2021 12:53:09 -0700 (PDT) To: make-initrd@lists.altlinux.org References: <7636585b-202d-95e0-eb0a-d320f69cc77a@gmail.com> <20210926140917.4xky7nfnlemy4jh3@example.org> From: Leonid Krivoshein Message-ID: <94921a86-1043-8273-2c29-957cf53e14e1@gmail.com> Date: Sun, 26 Sep 2021 22:53:08 +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: <20210926140917.4xky7nfnlemy4jh3@example.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit 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: Sun, 26 Sep 2021 19:53:12 -0000 Archived-At: List-Archive: 26.09.2021 17:09, Alexey Gladkov пишет: > On Fri, Sep 24, 2021 at 06:57:45PM +0300, Leonid Krivoshein wrote: > > Что это за хуки и зачем они нужны ? Отлично, уже дошли до первого принципиального и важного изменения! Полагаю, тут больше камень в огород отсутствия описаний, это я поправлю. Стартовый скрипт SysVinit ранее включал prepare() для предварительной подготовки каталогов для компонента waitdev, отделённого теперь в другую фичу. Другим компонентам bootchain при расширении (а теперь это уже более десятка фич и подпакетов) тоже может потребоваться подобная инициализация. Здесь в одном коммите видно, куда переехал код функции prepare(). Хуки нужны для того, чтобы вызывать подобную инициализацию из определённого места, так как базовый пакет bootchain-core может не знать, с какими компонентами bootchain собран initramfs. Соответственно, включаемые скрипты кладутся в /lib/bootchain-prepare.d. Если потребуется, по аналогии можно добавить в stop() хуки для деинициализации. >> --- >> .../data/etc/rc.d/init.d/bootchain | 33 ++++++++++--------- >> .../data/lib/bootchain-prepare.d/300-waitdev | 12 +++++++ >> 2 files changed, 29 insertions(+), 16 deletions(-) >> create mode 100644 make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev >> >> diff --git >> a/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain >> b/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain >> index 89d9de5..7bdb1cf 100755 >> --- a/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain >> +++ b/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain >> @@ -1,6 +1,6 @@ >> #!/bin/bash >> ### BEGIN INIT INFO >> -# Provides: pipeline >> +# Provides: bootchain >> # Required-Start: uevent udev >> # Should-Start: >> # Required-Stop: >> @@ -13,27 +13,28 @@ >> . /.initrd/initenv >> . /etc/init.d/template >> -NAME=pipelined >> +NAME=bootchained >> PIDFILE="/var/run/$NAME.pid" >> ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME --displayname >> $NAME" >> -prepare() { >> - local dir i n >> - >> - dir=/.initrd/pipeline/waitdev >> - mkdir -p -- "$dir" >> - >> - i=0 >> - while [ "$i" -lt "${WAITDEV:-0}" ]; do >> - touch "$dir/$i" >> - i=$(($i + 1)) >> - done >> +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-}" = pipeline ]; then >> - prepare >> + if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then >> + use_hooks /lib/bootchain-prepare.d >> start_daemon --background $ARGS "$NAME" >> RETVAL=$? >> fi >> @@ -43,7 +44,7 @@ start() { >> stop() { >> stop_daemon $ARGS "$NAME" >> RETVAL=$? >> - [ ! -f "$PIDFILE" ] || rm -f -- "$PIDFILE" >> + rm -f -- "$PIDFILE" >> return $RETVAL >> } >> diff --git a/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev b/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev >> new file mode 100644 >> index 0000000..f7315a3 >> --- /dev/null >> +++ b/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev >> @@ -0,0 +1,12 @@ >> +#!/bin/bash -efu >> + >> +dir=/.initrd/bootchain/waitdev >> +mkdir -p -- "$dir" >> + >> +[ -z "${WAITDEV_TIMEOUT-}" ] || >> + printf '%s\n' "$WAITDEV_TIMEOUT" >"$dir/TIMECNT" >> +i=0 >> +while [ "$i" -lt "${WAITDEV:-0}" ]; do >> + touch "$dir/$i" >> + i=$((1 + $i)) >> +done >> -- >> 2.21.0 >> >> >> _______________________________________________ >> Make-initrd mailing list >> Make-initrd@lists.altlinux.org >> https://lists.altlinux.org/mailman/listinfo/make-initrd >> -- Best regards, Leonid Krivoshein.