Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Randolph Sapp <rs@ti.com>
To: <rs@ti.com>,
	Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>,
	<richard.purdie@linuxfoundation.org>, <ross.burton@arm.com>,
	<alex@linutronix.de>, <otavio@ossystems.com.br>,
	<kexin.hao@windriver.com>, <afd@ti.com>, <detheridge@ti.com>,
	<denis@denix.org>, <reatmon@ti.com>
Cc: <openembedded-core@lists.openembedded.org>, <vijayp@ti.com>
Subject: Re: [oe-core][PATCHv9 0/6] Display manager proposal for x11 and wayland
Date: Mon, 22 Sep 2025 18:16:18 -0500	[thread overview]
Message-ID: <DCZPMZBQPTOP.2OZDXN58O263N@ti.com> (raw)
In-Reply-To: <1867BAF5F353054B.23856@lists.openembedded.org>

On Mon Sep 22, 2025 at 5:21 PM CDT, Randolph Sapp via lists.openembedded.org wrote:
> On Mon Sep 22, 2025 at 4:51 PM CDT, Randolph Sapp via lists.openembedded.org wrote:
>> On Sun Sep 21, 2025 at 7:25 AM CDT, Mathieu Dubois-Briand wrote:
>>> On Thu Sep 18, 2025 at 11:48 PM CEST, rs wrote:
>>>> From: Randolph Sapp <rs@ti.com>
>>>>
>>>> We've recently run into some issues with weston-init attempting to start Weston
>>>> prior to all drm devices being registered. There's not really a good, scriptable
>>>> mechanism to listen in to device registration events that works with the
>>>> existing weston-init package. Well, at least one that doesn't involve polling
>>>> files or introducing more dependency on the init system being used.
>>>>
>>>> I also see there is also a lot of scripting around starting X11,
>>>> xserver-nodm-init, that (from my limited review) should experience the same
>>>> issue.
>>>>
>>>> I'd like to introduce the following display manager for oe-core, emptty [1].
>>>> This display manager is, as described upstream, a "Dead simple CLI Display
>>>> Manager on TTY". It supports both x11 and wayland sessions, with togglable build
>>>> parameters to completely remove x11 and pam dependencies. It's licensed MIT,
>>>> which shouldn't be an issue for any users. (It is written in Go, if you have
>>>> opinions about that.)
>>>>
>>>> With this, both weston-init and the xserver-nodm-init packages can be re-tuned
>>>> to leverage this display manager and simply add a user and emptty config for an
>>>> autologin session. This can resolve the current behavior across init systems
>>>> without additional scripting, and move some development out of this layer.
>>>>
>>>> This lists myself as a maintainer of emptty as well as xserver-nodm-init and
>>>> xuser-account since these are currently unassigned and I've reworked them
>>>> significantly here.
>>>>
>>>> Sorry for the delay on this series. I found a few bugs in emptty that I wanted
>>>> to address before submitting this officially.
>>>>
>>>
>>> Hi Randolph,
>>>
>>> Thanks for your patch.
>>>
>>> I took this series, but it looks like we have a build issue, maybe only
>>> with altcfg distro:
>>>
>>> Traceback (most recent call last):
>>>   File "/srv/pokybuild/yocto-worker/qemux86-alt/build/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f
>>>     return func(*args, **kwargs)
>>>   File "/srv/pokybuild/yocto-worker/qemux86-alt/build/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f
>>>     return func(*args, **kwargs)
>>>   File "/srv/pokybuild/yocto-worker/qemux86-alt/build/meta/lib/oeqa/core/decorator/__init__.py", line 35, in wrapped_f
>>>     return func(*args, **kwargs)
>>>   File "/srv/pokybuild/yocto-worker/qemux86-alt/build/meta/lib/oeqa/runtime/cases/xorg.py", line 31, in test_xorg_running
>>>     self.assertEqual(status, 0, msg=msg)
>>> AssertionError: 1 != 0 : Xorg does not appear to be running   PID USER       VSZ STAT COMMAND
>>> ...
>>> 2025/09/21 11:59:34 Closing PAM auth
>>> 2025/09/21 11:59:34 Failure setting user credentials
>>> 2025/09/21 11:59:34 Authentication failure
>>> ...
>>> RESULTS - xorg.XorgTest.test_xorg_running: FAILED (1.41s)
>>>
>>> https://autobuilder.yoctoproject.org/valkyrie/#/builders/20/builds/2402
>>> https://autobuilder.yoctoproject.org/valkyrie/#/builders/9/builds/2407
>>>
>>> Can you have a look at it please?
>>>
>>> Thanks,
>>> Mathieu
>>
>> In the past auth issues can occur if the user was created after the current
>> system time (of which the default would be the kernel build time if it is unable
>> to get a value from NTP).
>>
>> I've already tested this series with an equivalent config to poky-altcfg, with
>> both root and rootless x11. I reset my cache and did another local build to
>> match that CICD failure and it still works for me.
>>
>> My local.conf for my test build was:
>>
>> MACHINE ??= "qemux86"
>> DISTRO ??= "poky-altcfg"
>> BBMULTICONFIG ?= ""
>> IMAGE_CLASSES += "testimage"
>>
>> Other variables were dictated by templateconf.cfg:
>>
>> meta-poky/conf/templates/default
>>
>> Is that automation setting anything that could possibly play with the build time
>> or system time values for the testimage? I'm not seeing anything in the logs,
>> but I'm also not seeing anything that reports the current config outright.
>>
>> I guess I should add some messages for that as part of the test function
>> regardless.
>
> Whoops, I see the issue now. v10 inbound.

Well, I know what the issue is, but I guess I should ask how people want to
address it. Currently, that test expect a root x11 instance to autostart. I have
it attempt to do that currently, and my old test images were being built with
"empty-root-password allow-empty-password allow-root-login" features enabled,
allowing it to proceed as usual. The password auth is what is currently failing.

1. We can do passwordless root auth for these by adjusting the pam rule for
emptty to allow auth as root for that application.

2. I would suggest that we instead make sure the root user is in the
nopasswdlogin group, as it makes it clear that the user has unprotected GUI
access.

I'm not sure either of these should really be allowed in a production
environment though. Technically we can disable this behavior outright when
ROOTLESS_X is set, but it's still a little odd to me.

3. Should the test images explicitly enable the empty-root-password
allow-empty-password? That means the x11 test will have to move from an
arbitrary runtime test to something like the way locales are currently tested.

Seems like solution 2 is the easiest and preserves the expected behavior for
that test, assuming people are okay with that behavior.

- Randolph


  parent reply	other threads:[~2025-09-22 23:16 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-18 21:48 [oe-core][PATCHv9 0/6] Display manager proposal for x11 and wayland rs
2025-09-18 21:48 ` [oe-core][PATCHv9 1/6] libx11: create tmpfile dir for x11 domain socket rs
2025-09-18 21:48 ` [oe-core][PATCHv9 2/6] go: fix sigaction usage on i386 platforms rs
2025-09-18 21:48 ` [oe-core][PATCHv9 3/6] emptty: add version 0.14.0 rs
2025-09-18 21:48 ` [oe-core][PATCHv9 4/6] weston-init: convert to virtual-emptty-conf rs
2025-09-18 22:03   ` Patchtest results for " patchtest
2025-09-18 21:49 ` [oe-core][PATCHv9 5/6] weston: remove deprecated weston-start scripts rs
2025-09-18 21:49 ` [oe-core][PATCHv9 6/6] xserver-nodm-init: convert to virtual-emptty-conf rs
2025-09-18 22:25 ` [oe-core][PATCHv9 0/6] Display manager proposal for x11 and wayland Joshua Watt
2025-09-18 22:31   ` Randolph Sapp
2025-09-18 22:35     ` Joshua Watt
2025-09-18 22:39       ` Randolph Sapp
     [not found]       ` <186681957172BF4C.23626@lists.openembedded.org>
2025-09-18 22:42         ` Randolph Sapp
2025-09-18 23:03           ` Joshua Watt
2025-09-18 23:13             ` Randolph Sapp
2025-09-21 12:25 ` Mathieu Dubois-Briand
2025-09-22 21:51   ` Randolph Sapp
     [not found]   ` <1867B951D10D3F9C.23856@lists.openembedded.org>
2025-09-22 22:21     ` Randolph Sapp
     [not found]     ` <1867BAF5F353054B.23856@lists.openembedded.org>
2025-09-22 23:16       ` Randolph Sapp [this message]
2025-09-23  9:03         ` Richard Purdie
2025-09-23 17:56           ` Randolph Sapp
     [not found]           ` <1867FB133639FF67.16927@lists.openembedded.org>
2025-09-23 18:04             ` Randolph Sapp
2025-09-23 22:41               ` Richard Purdie

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=DCZPMZBQPTOP.2OZDXN58O263N@ti.com \
    --to=rs@ti.com \
    --cc=afd@ti.com \
    --cc=alex@linutronix.de \
    --cc=denis@denix.org \
    --cc=detheridge@ti.com \
    --cc=kexin.hao@windriver.com \
    --cc=mathieu.dubois-briand@bootlin.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=otavio@ossystems.com.br \
    --cc=reatmon@ti.com \
    --cc=richard.purdie@linuxfoundation.org \
    --cc=ross.burton@arm.com \
    --cc=vijayp@ti.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox