From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] Upcoming changes
Date: Thu, 4 May 2023 11:05:52 +0200
Message-ID: <ZFN1cAZuWz8M0oD6@example.org> (raw)
In-Reply-To: <a6c019c0-f681-b3bd-2517-e25a48e867f1@gmail.com>
On Thu, May 04, 2023 at 07:05:15AM +0300, Leonid Krivoshein wrote:
> Привет!
>
>
> On 5/3/23 17:35, Alexey Gladkov wrote:
> > Привет!
> >
> > Я хотел бы рассказать, что готовится. Я сейчас тестирую некоторые
> > изменения в архитектуре.
> >
> > * Я переписал ueventd с bash + утилита inotify на нормальный си. Новый код
> > легче читать и понять.
>
> Это же здорово! И в какой-то степени ожидаемо. Только я чур на systemd
> не намекал. :-) Есть ли шанс, что уберутся sleep .1 по всему коду? Даёшь
> тепло в дом другим способом!
sleep'ов стенет меньше, но всё ещё останутся. Чтобы от них избавится, то
нужно придумать сущность, которая бы без поллигинга контролировала условия
загрузки.
Возможно, следующим шагом попробую эту область переделать, если никто
раньше не предложит что-то.
> > * polld перестал быть отдельным демоном (сервис остаётся на месте) и
> > становится просто отдельной очередью в ueventd. Скрипты, которые
> > выполняются в ueventd. Очередь запускается и останавливается из сервиса.
> >
> > Несмотря на то, что изменение большое я не думаю, что это как-то затронет
> > фичи не из репозитория.
> >
> > Пользуясь случаем я также хотел спросить, стоит ли присылать в этот список
> > рассылки патчсеты, которые готовятся для master ?
>
> Тех, кто разберётся в твоём коде, не так уж много, и всё увидят в
> конечном продукте и дадут обратную связь так или иначе.
Я стараюсь писать как можно проще. Судя по всему, мне не удалось писать
понятно ((
> Или ты сам себе не доверяешь и хочешь перестраховаться, чтобы кто-то
> поревьювил?
Я никогда себе не доверял. Тут скорее вопрос в том, есть ли тут желающие
поревьювить. Иначе такие патчи будут просто спамом.
> В принципе хорошая практика даже в Сизиф не отправлять, без тестирования
> на регулярках, а для этого можно договориться с antohami@, но и это
> совершенно необязательно, т.к. в худшем случае забракуется очередная
> партия регулярок. Ну и системы немного поломаются у тех, кто на Сизифе
> сидит, а их не так много, я полагаю.
Я отправляю изменения не прямо уж без тестирования. У меня всё-таки есть
end-to-end тесты для разных схем загрузки и разных вендоров. Там
тестируется как kickstart, так и сама загрузка. Разумеется, это не
настоящее железо и не настоящая система, но всё-таки.
--
Rgrds, legion
next prev parent reply other threads:[~2023-05-04 9:05 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-03 14:35 Alexey Gladkov
2023-05-04 4:05 ` Leonid Krivoshein
2023-05-04 9:05 ` Alexey Gladkov [this message]
2023-05-04 9:39 ` Leonid Krivoshein
2023-05-04 14:15 ` Alexey Gladkov
2023-05-05 3:06 ` 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=ZFN1cAZuWz8M0oD6@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