From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] make pseudo GUI from bootchain-interactime common feature
Date: Tue, 29 Jun 2021 10:28:10 +0200
Message-ID: <20210629082810.4hjqerighdissl3f@example.org> (raw)
In-Reply-To: <1e589277-b594-cc3e-9ff7-b1e90ddb89f4@gmail.com>
On Tue, Jun 29, 2021 at 03:50:21AM +0300, Leonid Krivoshein wrote:
>
> 29.06.2021 2:38, Alexey Gladkov пишет:
> > [...]
> > Так что я не согласен с этими перечисленными юскейсами. Я не вижу иного
> > применения bootchain, кроме как для случаев не связанных с локальным
> > железом... ну почти.
>
> OK, подождём других юскейсов...
Насчёт resume наверное всё-таки стоит запоминать расположение swap'ов и
ждать их появления некоторое время (да, опять таймаут).
У меня уже мысль не городить очередной таймаут "по месту", а считать
разные типы таймаутов централизовано. Есть счётчик, который нужен уже в
двух местах: время с последнего появления диска/раздела. Его можно считать
на уровне udev и использовать в разных ситуациях.
> > > Сейчас, если в bootchain шаг ничего не принимает на входе и не передаёт
> > > на выходе, он вызывает bypass_results(), связывая выход предыдущего шага
> > > со входом следующего. Расходуется при этом лишний каталог в tmpfs.
> > Ты экономишь один dentry в tmpfs ? Да будь из хоть 100 ты не сможешь
> > переплюнуть libcrypto, которая занимает 2,9M. Ты экономишь совсем не то.
>
> Речь конечно не об экономии, там и лишние действия выполняются, и всё только
> для того, чтобы "передать наработки следующему соседу". А оно, может, соседу
> и не потребуется.))
Если результат вообще никому не нужен, то это ошибка в составлении
bootchain= и соответствующих параметров.
> > Кроме того, мне кажется, что можно обойтись и без этого лишнего создания.
>
> Возможно. Нужно смотреть код bypass_results() и bootchain-loop, но я
> подозреваю, что из-за ссылок на номера шагов и возможности их использования
> в resolve_target() без этого не обойтись.
Мы знаем всю информацию и, кажется, можно выявить шаги,
которые никто не использует и не делать их. Правда, я не уверен, что
оптимизация стоит трудозатрат.
--
Rgrds, legion
next prev parent reply other threads:[~2021-06-29 8:28 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-28 14:01 ` Alexey Gladkov
2021-06-28 14:43 ` Leonid Krivoshein
2021-06-28 16:06 ` Alexey Gladkov
2021-06-28 16:34 ` Leonid Krivoshein
2021-06-28 18:48 ` Alexey Gladkov
2021-06-28 19:43 ` Leonid Krivoshein
2021-06-28 23:38 ` Alexey Gladkov
2021-06-29 0:50 ` Leonid Krivoshein
2021-06-29 8:28 ` Alexey Gladkov [this message]
2021-06-28 16:39 ` Leonid Krivoshein
2021-08-22 13:09 ` Leonid Krivoshein
2021-06-28 15:57 ` Leonid Krivoshein
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210629082810.4hjqerighdissl3f@example.org \
--to=gladkov.alexey@gmail.com \
--cc=make-initrd@lists.altlinux.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Make-initrd development discussion
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/make-initrd/0 make-initrd/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 make-initrd make-initrd/ http://lore.altlinux.org/make-initrd \
make-initrd@lists.altlinux.org make-initrd@lists.altlinux.ru make-initrd@lists.altlinux.com
public-inbox-index make-initrd
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.make-initrd
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git