From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Sat, 6 Nov 2021 13:47:43 +0100 From: Alexey Gladkov To: make-initrd@lists.altlinux.org Message-ID: <20211106124743.nfdo5qhrymgbkugk@example.org> References: <20211024172128.744EFA5E4C@lists.altlinux.org> <20211026111221.2bay4wh2xfvxq2vr@example.org> <5fea746a-25c2-01ee-3403-ed4a4815968f@gmail.com> <20211026135834.wvsbihztce6466sz@example.org> <2f91b485-6744-cd07-b1b8-46fcb250c713@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <2f91b485-6744-cd07-b1b8-46fcb250c713@gmail.com> Subject: Re: [make-initrd] [PATCH v6 10/22] bootchain-core: expanded addressing capabilities 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, 06 Nov 2021 12:47:45 -0000 Archived-At: List-Archive: On Sat, Oct 30, 2021 at 05:13:49AM +0300, Leonid Krivoshein wrote: > > 30.10.2021 2:50, Gleb Fotengauer-Malinovskiy пишет: > > On Tue, Oct 26, 2021 at 03:58:34PM +0200, Alexey Gladkov wrote: > > > On Tue, Oct 26, 2021 at 02:28:53PM +0300, Leonid Krivoshein wrote: > > > > 26.10.2021 14:12, Alexey Gladkov пишет: > > > > > On Sun, Oct 24, 2021 at 08:21:28PM +0300, Leonid Krivoshein wrote: > > > > > > Adds support for reverse addressing relative to the current step, > > > > > > as well as calculating the path to the special device node obtained > > > > > > in the previous step. > > > > > > > > > > > > Signed-off-by: Leonid Krivoshein > > > > > > --- > > > > > > features/bootchain-core/README.md | 6 +++ > > > > > > .../data/bin/bootchain-sh-functions | 37 ++++++++++++++++--- > > > > > > 2 files changed, 38 insertions(+), 5 deletions(-) > > > > > > > > > > > > diff --git a/features/bootchain-core/README.md b/features/bootchain-core/README.md > > > > > > index 8d341e0..ba66259 100644 > > > > > > --- a/features/bootchain-core/README.md > > > > > > +++ b/features/bootchain-core/README.md > > > > > > @@ -49,6 +49,12 @@ us to optimize fill in `initramfs` only which we are need. > > > > > > - Modularity: loading methods are initially separated from the common > > > > > > code and daemon. > > > > > > +- Via resolve_target() supports not only forward, but also reverse addressing, > > > > > > + relative to the current step. For example, a record like `step-3/dir1/dev` > > > > > > + will process the result of `dir1/dev`, made in the third step from the current > > > > > > + one. Together with the overload of the chain of steps, direct addressing is safe > > > > > > + only when storing the numbers of the completed steps in files, whereas reverse > > > > > > + relative addressing it is safe in any case and can often be more convenient. > > > > > На мой взгляд очень непонятный синтаксис. step1 и step-1 кажется > > > > > одинаковым. Хотя бы current-1 > > > > OK. Давай переделаю так: > > > > > > > > step = pipe -- прямая адресация, так первоначально и было. > > > > curr- -- обратная относительная адресация. > > > > > > > > Так будет лучше? > > > Мне будет понятнее. Но я хотел бы спросить помощи зала. > > > > > > Глеб, Антон, как вы думаете, как будет понятнее ? > > Если из предложенного выбирать, то curr- однозначно лучше. > > > > Но мне кажется, что тут - не читается как минус. Я бы скорее предложил > > что-то вроде prev или back. > > Отлично, тогда переделаю на back ! > > Как я понял, по остальным моментам Алексей пока взял паузу... В прошлый раз я вроде всё прокомментировал, что нашёл. -- Rgrds, legion