ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Arseny Maslennikov <arseny@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] DM (was: RFC: wayland session wrapper script)
Date: Fri, 3 Dec 2021 22:59:43 +0300
Message-ID: <Yap3L9rLL+nufhth@cello> (raw)
In-Reply-To: <20211203180501.azzpphfwubhq2pcc@example.org>

[-- Attachment #1: Type: text/plain, Size: 4386 bytes --]

On Fri, Dec 03, 2021 at 07:05:01PM +0100, Alexey Gladkov wrote:
> On Fri, Dec 03, 2021 at 07:05:12PM +0300, Arseny Maslennikov wrote:
> > (2) автору совсем не пришло в голову, что запускать гритеры лучше из
> >     инит-скриптов / строчек в inittab / autovt@.service (зачем заново
> >     писать startup manager?);
> 
> Потому что он как и я не использует systemd ? На это намекает то, что он
> же является автором seatd :)

Я же не зря _сначала_ написал "инит-скриптов / строчек в inittab". Для
этого же совсем не обязателен systemd и его logind.

В сизифе есть пакет kmscon, который можно запустить так:
  /usr/bin/kmscon --vt=tty5 --seats=seat0 --no-switchvt --login -- /sbin/agetty -o '-p -- \\u' --noclear -- -

Судя по https://tldp.org/LDP/sag/html/config-init.html, команду выше или
скрипт с ней можно положить в четвёртое поле записи inittab, разве нет?

> > (3) он, запускаясь на VT X, _сначала_ вызывает ioctl(*, KDSETMODE, KD_TEXT),
> >     а _потом_ спрашивает у гритера, какой режим VT ему нужен;
> 
> Послали ему исправление ?

Надо ещё разбираться подробнее, но не до того пока.

% grep -nC5 compatibility greetd/src/session/worker.rs
140-            pam.putenv(&format!("XDG_VTNR={}", vt))?;
141-
142-            // Opening our target terminal.
143-            let target_term = terminal::Terminal::open(&path)?;
144-
145:            // Set the target VT mode to text for compatibility. Other login managers
146-            // set this to graphics, but that disallows start of textual applications,
147-            // which greetd aims to support.
148-            target_term.kd_setmode(terminal::KdMode::Text)?;
149-
150-            // Clear TTY so that it will be empty when we switch to it.

Как я понимаю, это не столько исправление, сколько улучшение — оно
потребует изменения протокола greetd-ipc: сначала стартует гритер,
декларирует требования (например, что он текстовый и хочет пользоваться
VT в текстовом режиме), потом происходит уже настройка и выдача VT и
ответ гритеру, что получилось, потом последний открывает VT и начинает работу.

> > (4) он, кажется, не решает проблему "если DM крашнулся, сеансы убиваются
> >     вслед"; (один из путей решения — пункт 2)
> 
> Для меня это не вариант решения. Для автора, кажется, тоже.

Ну почему же: greeter или пользовательский сеанс на VT X порождается условным
/usr/sbin/dm --vt=ttyX в процессе-потомке со сброшенными привилегиями,
он же задействует PAM-стек. Если общий деймон необходим для выполнения
каких-то ещё привилегированных операций, с ним можно по сокету общаться.

Наверное, здесь сказано много слов и мало кода...

> 
> > (5) ну и XDG compliance у него не реализован, кажется, ничем: нет в
> >     комплекте лончера, который бы все эти переменные ставил (т. е.
> >     потребуются костыли, о которых пишет Игорь viy@).
> 
> Я их себе сам выставляю их. Не вижу в этом проблемы. Мне кажется это не
> дело DM.

Да, это дело соотв. прослойки, которую можно и не использовать.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2021-12-03 19:59 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-02 18:35 [devel] RFC: wayland session wrapper script Igor Vlasenko
2021-12-02 19:09 ` Alexey V. Vissarionov
2021-12-03  7:30   ` Sergey V Turchin
2021-12-03 10:58     ` Igor Vlasenko
2021-12-03 10:59   ` Igor Vlasenko
2021-12-03 18:41     ` Alexey V. Vissarionov
2021-12-03 12:12 ` Dmitry V. Levin
2021-12-03 12:52   ` Igor Vlasenko
2021-12-03 16:07   ` [devel] почему ssh-agent не имеет смысла на tty1 Arseny Maslennikov
2021-12-03 16:11     ` Dmitry V. Levin
2021-12-03 13:50 ` [devel] RFC: wayland session wrapper script Alexey Gladkov
2021-12-03 14:49   ` Sergey Bolshakov
2021-12-03 15:10     ` Alexey Gladkov
2021-12-03 16:05       ` [devel] DM (RFC: wayland session wrapper script) Arseny Maslennikov
2021-12-03 18:05         ` Alexey Gladkov
2021-12-03 19:59           ` Arseny Maslennikov [this message]
2021-12-03 23:00             ` [devel] DM (was: RFC: " Alexey Gladkov
2021-12-21 13:19       ` [devel] RFC: wayland session wrapper script Alexey Gladkov
2021-12-03 15:16   ` Arseny Maslennikov
2021-12-03 15:14 ` Arseny Maslennikov
2021-12-03 18:01   ` Igor Vlasenko
2021-12-03 18:16     ` Yuri Sedunov

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=Yap3L9rLL+nufhth@cello \
    --to=arseny@altlinux.org \
    --cc=devel@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

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/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 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git