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=20161025; h=subject:to:references:from:message-id:date:user-agent:mime-version :in-reply-to:content-transfer-encoding:content-language; bh=fzkHbtIZ8uztNw7KwyV+Jr0WRtW2rYEiQkAfDNqW57M=; b=EJTn2HQhOHYfcfFzB1vCBTpzCb95e+iVrT/+0Ycej9tQqcBI4RClliHWWimLpyylNq rSYcdJTBRXg2s4y+pUPI/P0Xgv/McSoTMBsTAkB6M1/4U1NKWd96Dxjgq7ubwRuXOBo+ vNjMopdjLz9i4LLYOQyj3iIruR2I8nboMbnjuiL/uI25/HhbM4+fvvD3NfgT1P/GOCuh d8XJrpemyYBNkJPCbOtGQ5HtIHgOeyL4SkyPQVVDvraZ2AD23YW1SmrsO4TTtSCtDjU5 7jRVdSD3PaYlX4NicAt/yjLIJQoYY56LNOfL4zi1vqtAFTBxkHQaE3qQlgpMt/sP8DIT pYKQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; 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=fzkHbtIZ8uztNw7KwyV+Jr0WRtW2rYEiQkAfDNqW57M=; b=SPDC7R1fj5kbXdXfNr+WHD/+Duep3dOEjJg4mfziZykv6wJxwikVEq1O0uGWal1osW ahbSNA7ZzQoOInETBbo/sCBeUXpBQBSVu7NhGjOHgjNrk12x/UCoNiiHInHX2+lLf3PB 6HFT5XEKCNpNF5jhrmOiaMI6uY/IQjg8F3SPu/DndhQLZa0uzMYe8XsoQqtoEzSbwblA P3c8X/UzkQULDR71ycwAXJA8Ui1gFxJSgNU/VxiLvWlmWEg0A3N0WNuwRHd5jyA1PrG4 7Ql+s8qgwF7Qv0xFlTOBJvzSXoiWlJA6gi8KNywbnRfhW69u1iXMNkXEXh/9AzkqwDmC +zyw== X-Gm-Message-State: AOAM530n+JVLfz2ReXRs5awlo5+w1hqTuwwmDuroE5Bh3ivDuZe5AJqk MNtOB59ATe0Kmd5xlcD51aso2LwdYDpV3w== X-Google-Smtp-Source: ABdhPJyLO66axPei/AD+Qoy6x758t3UG8I/U9S08G6VIRtx74kM7pvWE6jPsZfjLm80cDj9RzJ7tYA== X-Received: by 2002:a2e:2e16:: with SMTP id u22mr2567564lju.322.1622041531507; Wed, 26 May 2021 08:05:31 -0700 (PDT) To: make-initrd@lists.altlinux.org References: <20210406082842.pg3rejmmnxuxvddf@example.org> <20210407125739.jeenqyplba3v6itn@example.org> From: Leonid Krivoshein Message-ID: <53dd1676-6f7b-9a75-49a9-e970e9440b1c@gmail.com> Date: Wed, 26 May 2021 18:05:29 +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: <20210407125739.jeenqyplba3v6itn@example.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: ru Subject: Re: [make-initrd] Fwd: [#269003] TESTED make-initrd.git=2.14.1-alt1 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: Wed, 26 May 2021 15:05:38 -0000 Archived-At: List-Archive: Привет! 07.04.2021 15:57, Alexey Gladkov пишет: > On Wed, Apr 07, 2021 at 04:51:15AM +0300, Leonid Krivoshein wrote: >> 06.04.2021 11:28, Alexey Gladkov пишет: >>> Это сразу наложит ограничение на использование запятой в аргументе. А она >>> уже используется как разделитель например в опциях монтирования. Недавно я >>> предлагал вариант передачи дополнительных параметров монтирования: >>> >>> pipeline=waitdev,mountfs \ >>> waitdev=/dev/sda \ >>> mountfs=/dev/sda:nodev,noexec,mode=620 >>> >>> Я на нём не настаиваю, но как будет выглядеть тоже самое в твоём >>> синтаксисе ? >> Вроде как есть два варианта реализации, я склоняюсь ко второму: >> >> 1. Использовать register_pipe с разделителем "|" вместо register_string для >> pipeline=... или парсить токены не по запятым, а по "|", тогда проблемы с >> запятыми и двоеточиями отпадают: >> >> pipeline=waitdev=/dev/sda|mountfs=/dev/sda:nodev,noexec,mode=620 >> >> >> 2. Сохранить полностью нынешний синтаксис, добавив в него возможность в >> простых случаях (где в значении отсутствуют запятые и двоеточия) >> использовать символ "=" для отделения имени шага от его параметров, >> разделяемых символом ":" или ";": >> >> pipeline=waitdev=/dev/sda,mountfs mountfs=/dev/sda:nodev,noexec,mode=620 >> >> >> Первый вариант решает сразу много проблем, но создаёт проблему совместимости >> (которой, впрочем, можно пренебречь), а также создаёт не всегда интуитивно >> соответствующее восприятие происходящего, поскольку запятые отражают >> последовательность запускаемых шагов (их перечисление), тогда как "|" >> показывает кто-кому передаёт сделанное, а это, как мы выяснили, не всегда >> будет соответствовать написанному. >> >> Второй вариант кажется хорошим компромиссом для экономии байтов в >> /proc/cmdline и не ломает совместимость. В приведённом выше примере экономия >> небольшая, но на больших реальных цепочках она будет ощутимей, например: >> >> pipeline=waitdev=/dev/sda,mountfs=:/root,ram=/root/rescue,mountfs=:,live=rw,rootfs >> vs >> pipeline=waitdev,mountfs,ram,mountfs,live,rootfs waitdev=/dev/sda >> mountfs=:/root ram=/root/rescue mountfs=: live=rw > А ещё можно не заниматься метапрограммированием и для сложных случаев > генерировать файлик с массивом и параметрами. Или же совсем иной синтаксис > придумать и прицеплять его к initrd, оставив cmdline для переопределения > того, что сохранено внутри. > > Кстати, так сейчас работает root=. При создании initrd текущие параметры > прописываются в внутренний /etc/fstab, а параметры root= из cmdline просто > переопределяет эту запись. Учёл практически все твои замечания. Всё ещё делаю обещанное, однако дело уже близится к концу. В коде ещё остаются захардкореными некоторые пути и имена, это потом уберётся в альт-специфичный конфиг. #271420 -- предыдущая версия "всё в одном", около 3 тыс. строк. По ней легче понять задумку. Тут предварительно отлажена локальная загрузка. altboot сделан шагом pipeline, который выполняет роль вложенной pipeline. #272587 -- текущая версия разбита на десяток подпакетов, уже более 4200 строк, устранены первичные замечания, но понять проще предыдущий вариант. Тут тоже работает пока только локальная загрузка, но уже с плимутом, liverw надо переделывать, ещё не проверялись: nfs, cifs и overlayroot. Однако оценить задумку можно уже сейчас. Документации пока тоже нет. Здесь отказался от вложенной pipeline, сделал форк в bootchain и значительное расширение его функционала, обратная совместимость с pipeline сохранена. Хотел посоветоваться насчёт setsid, слишком неожиданно для меня его поведение в теле скрипта, запущенного через openvt. Он ведёт себя так, будто бы его нет, приходится использовать амперсанд (&). При этом, когда я делаю то же самое руками в обычном терминале, setsid ведёт себя ожидаемым образом. Т.е. одна и та же команда работает по-разному: setsid sleep 10 работает как: sleep 10 при открытии через openvt, а в консоли работает как: sleep 10 & речь об этом фрагменте: bootchain-interactive/data/bin/interactive-sh-functions ... setsid /bin/bash -c "/bin/activate-interactive-vt $delay" & message "TTY${_IM_VT_number} will be activated after $((1 + delay)) seconds" ... (в нынешнем варианте задумка работает, но я считаю это неправильно). -- Best regards, Leonid Krivoshein.