From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Sat, 13 May 2023 13:50:50 +0200 From: Alexey Gladkov To: make-initrd@lists.altlinux.org Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Subject: Re: [make-initrd] [PATCH 0/3] Reimplement ueventd 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, 13 May 2023 11:50:53 -0000 Archived-At: List-Archive: On Thu, May 04, 2023 at 03:42:49PM +0200, Alexey Gladkov wrote: > # Предыстория. > > Когда-то давно в initramfs был один цикл, в котором обрабатывались все эвенты из > udev и в конце каждой итерации производилась проверка возможности загрузиться. > Это давало предсказуемый порядок, но создавало проблему с производительностью. > Плюс ко всему этому требовался эвент, чтобы дать толчёк для проверки возможности > дальнейшей загрузки. > > Чтобы решить эти проблемы единый цикл был разделён на ueventd (обработчик > эвентов) и polld (периодическое выполнение скриптов). Это позволило развязать > эвенты и условия дальнейшей загрузки. > > С появлением необходимости обработки эвентов от сетевых устройств, конфигурации > терминалов и некоторых других, в ueventd была реализована поддержка нескольких > очередей. Эвенты каждой очереди обрабатываются последовательно, но каждая > очередь обрабатывается параллельно. > > В то время распараллеливание было сделано через использование утилиты inotify из > busybox. Это работало, но поскольку inotify это утилита общего назначения, то > оптимизировать или кастомизировать работу демона было очень затруднительно. > > # Изменения. > > Ueventd был переписан с нуля на си. Это открыло возможность для оптимизаций, > увеличило читабильность кода, дало возможность уйти от передачи эвентов через > файловую систему. > > Как следствие сейчас пропала необходимость в отдельном процессе polld. Его > фунционал может обратно вернуться в ueventd в качестве отдельной очереди. Плюс > проверка rootdelay теперь находится в этой отдельной очереди. > > Возможно стоит пойти дальше и уйти от периодической проверки создав опять же > поток эвентов и "базу данных". Но это справедливо и для текущей архитектуры > эвентов. Смерджено с follow-up исправлениями. Также смерджил патчсет создающий общую библиотеку для утилит на си внутри initramfs. > --- > > Alexey Gladkov (3): > Reimplement ueventd > Replace polld by uevent queue > feature/kickstart: Reset rootdelay timer after kickstart > > data/bin/initrd-sh-functions | 8 + > data/bin/uevent-sh-functions | 3 + > data/etc/rc.d/init.d/poll | 25 +- > data/etc/rc.d/init.d/uevent | 9 +- > data/lib/uevent/each/post/.gitignore | 0 > data/lib/uevent/each/post/system-bootable | 4 - > data/lib/uevent/extenders/.gitignore | 0 > data/lib/uevent/handlers/poll/100-extenders | 22 + > data/lib/uevent/handlers/poll/300-boot-method | 29 + > .../poll/400-rootdelay} | 13 +- > data/sbin/polld | 45 -- > data/sbin/process-boot-method | 16 - > data/sbin/ueventd | 180 ------ > data/sbin/ueventd-queue | 65 +++ > datasrc/ueventd/Makefile.mk | 13 + > datasrc/ueventd/logging.c | 63 ++ > datasrc/ueventd/memory.c | 32 + > datasrc/ueventd/path.c | 80 +++ > datasrc/ueventd/process.c | 17 + > datasrc/ueventd/queue-processor.c | 116 ++++ > datasrc/ueventd/ueventd.c | 551 ++++++++++++++++++ > datasrc/ueventd/ueventd.h | 76 +++ > .../kickstart/data/etc/rc.d/init.d/kickstart | 11 +- > 23 files changed, 1108 insertions(+), 270 deletions(-) > create mode 100644 data/lib/uevent/each/post/.gitignore > delete mode 100755 data/lib/uevent/each/post/system-bootable > create mode 100644 data/lib/uevent/extenders/.gitignore > create mode 100755 data/lib/uevent/handlers/poll/100-extenders > create mode 100755 data/lib/uevent/handlers/poll/300-boot-method > rename data/lib/uevent/{extenders/100-rootdelay => handlers/poll/400-rootdelay} (80%) > delete mode 100755 data/sbin/polld > delete mode 100755 data/sbin/process-boot-method > delete mode 100755 data/sbin/ueventd > create mode 100755 data/sbin/ueventd-queue > create mode 100644 datasrc/ueventd/Makefile.mk > create mode 100644 datasrc/ueventd/logging.c > create mode 100644 datasrc/ueventd/memory.c > create mode 100644 datasrc/ueventd/path.c > create mode 100644 datasrc/ueventd/process.c > create mode 100644 datasrc/ueventd/queue-processor.c > create mode 100644 datasrc/ueventd/ueventd.c > create mode 100644 datasrc/ueventd/ueventd.h > > -- > 2.33.7 > -- Rgrds, legion