Skip to content

Conversation

@msekletar
Copy link
Member

@msekletar msekletar commented Nov 4, 2025

No description provided.

People make weird assumptions around state preservation and
expect logind to be stoppable. While this is realistically
not OK we can probably improve things a little.

This complements f01d8658a3a57d05a5156aefd32d8137c3ee3996 and
adds support for deserializing the LEADER_PIDFDID= field.
We still prioritize pidfd if got one from fdstore (as with
service_notify_message_parse_new_pid() in pid1), but otherwise
this should make logind restart more robust when fdstore
gets spuriously cleared.

Fixes #39437

(cherry picked from commit 45eea629e3b3a640bf6a5cd13f4c73c86b426b11)

Resolves: RHEL-53112
(cherry picked from commit ebb730b96ded7674994235340c0d4a39af174f52)

Related: RHEL-53112
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

Commit validation

Tracker - RHEL-53112

The following commits meet all requirements

commit upstream
1d3a395 - logind: support deserializing session leader through pidfdid systemd/systemd@45eea62
fa5bea2 - TEST-35-LOGIN: test coldplug without fdstore on kernels with pidfd id systemd/systemd@ebb730b

Follow-up detection

Failed

🔴 Some follow-up commits for this Pull Request were detected in upstream

Follow-ups

commit follow-up
1d3a395 - logind: support deserializing session leader through pidfdid systemd/systemd@c54112b

Tracker validation

Success

🟢 Tracker RHEL-53112 has set desired product: rhel-10.2
🟢 Tracker RHEL-53112 has set desired component: systemd
🟢 Tracker RHEL-53112 has been approved
🟢 Tracker RHEL-53112 has set severity


Pull Request validation

Failed

🔴 Failed or pending checks:

  • ci (centos, 10, 0, -Og, yes, 1, TEST-21-DFUZZER TEST-60-MOUNT-RATELIMIT, regular-vm)[failure]
  • ci (centos, 10, address,undefined, 1, -Og, yes, 0, sanitizers)[failure]

🔴 Approval - missing or changes were requested

Success

🟢 Review - Reviewed by a member

Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The follow-up is relevant. Please backport it too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants