Make-initrd development discussion
 help / color / mirror / Atom feed
From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] Инфраструктура отладки в make-initrd
Date: Thu, 1 Jul 2021 16:12:53 +0200
Message-ID: <20210701141253.nqsetmy74odf4g6b@example.org> (raw)
In-Reply-To: <bf7c443d-ad7c-83df-de12-f6b0efe88861@gmail.com>

On Thu, Jul 01, 2021 at 04:50:21PM +0300, Leonid Krivoshein wrote:
> 
> 
> 01.07.2021 16:39, Alexey Gladkov пишет:
> > On Thu, Jul 01, 2021 at 03:31:07PM +0300, Leonid Krivoshein wrote:
> > > Всем привет!
> > > 
> > > 
> > > Из-за скудности сабжа пришлось городить свой набор функций и в процессе
> > > отладки они сильно выручили. С включенным bc_debug получаются вменяемые
> > > отладочные журналы, не сравнить с использованием set -x. Примеры сейчас
> > > можно увидеть здесь:
> > > http://git.altlinux.org/people/klark/packages/make-initrd-bootchain.git в
> > > подкаталоге boot-logs.
> > > 
> > > В самом make-initrd на эту тему есть немного, например, параметр debug, от
> > > которого мало что меняется в run-time, а раньше он ещё и был включен по
> > > дефолту (я не знал, как отключить, поэтому ввёл свой bc_debug). Было бы
> > > здорово иметь такую инфраструктуру на верхнем уровне make-initrd, доступную
> > > для любых фич. Речь о примерно следующем наборе функций:
> > > 
> > > debug() -- Вывод текстового сообщения при расширенной отладке.
> > > enter() -- Трассировка при расширенной отладке: вход в указанную функцию.
> > > leave() -- Трассировка при расширенной отладке: выход из указанной функции.
> > > run() -- Запуск внешней команды. При расширенной отладке команда попадёт в
> > > журнал.
> > > fdump() -- Вывод в журнал содержимого указанного файла при расширенной
> > > отладке.
> > > assign() -- Присвоение переменной указанного значения, попадающее в журнал.
> > > 
> > > message() -- уже есть, он отправляет в журнал сообщение независимо от debug.
> > > 
> > > Я понимаю, что после отладки enter()/leave() и возможно что-то ещё из кода
> > > может даже лучше убирать. Но в таком виде как сейчас их наличие помогает не
> > > только программисту отлаживать программу, но и пользователю разобраться,
> > > почему не грузится система. Возможно имеет смысл перетащить в
> > > initrd-sh-functions?
> > В качестве отдельной фичи ? Если да, то в ней можно принудительно включать
> > rdlog=console. Если нет, то нужно её держать в памяти.
> 
> А почему в памяти? debug() выводит в >&2 если включен debug, остальные
> функции, на нём основаны. А куда уж скрипт направит свой stderr, туда и
> будет выводиться. Зачем делать это фичей, если тут просто расширение API?

Держать в памяти в смысле в уме, что для отладки этот параметр стоит
указывать в cmdline.

-- 
Rgrds, legion



      reply	other threads:[~2021-07-01 14:12 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-01 12:31 Leonid Krivoshein
2021-07-01 13:39 ` Alexey Gladkov
2021-07-01 13:50   ` Leonid Krivoshein
2021-07-01 14:12     ` Alexey Gladkov [this message]

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=20210701141253.nqsetmy74odf4g6b@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