Page MenuHomeFreeBSD
Differential D36690

snp(4): properly report detached/revoked ttys and implement detach
ClosedPublic
Actions

Authored by kib on Sep 24 2022, 8:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 18, 1:51 AM
Unknown Object (File)
Fri, Nov 14, 6:44 AM
Unknown Object (File)
Sun, Nov 9, 1:24 AM
Unknown Object (File)
Mon, Nov 3, 11:50 AM
Unknown Object (File)
Mon, Nov 3, 11:50 AM
Unknown Object (File)
Mon, Nov 3, 11:49 AM
Unknown Object (File)
Mon, Nov 3, 11:49 AM
Unknown Object (File)
Mon, Nov 3, 11:49 AM
View All Files
Subscribers
imp
pauamma_gundo.com

Details

Reviewers
markj
imp
Commits
rG550e01c55247: snp(4): implement detach
rGbb4be76cf358: snp(4): properly report detached/revoked ttys
Summary

PR: 257964

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kib created this revision.Sep 24 2022, 8:26 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 24 2022, 8:26 PM
kib requested review of this revision.Sep 24 2022, 8:26 PM
pauamma_gundo.com added a subscriber: pauamma_gundo.com.Sep 25 2022, 1:54 AM
pauamma_gundo.com added inline comments.
share/man/man4/snp.4
89

While here: is "using" missing here? ie, "does not allow using SNPSTTY..."?

imp added a comment.Sep 25 2022, 5:02 AM

I think this is good, but it is a behavior change.
The tty_gone stuff is definitely good, though

sys/dev/snp/snp.c
309

This changes the return value from 0 to -2. That seems correct to me, but I wanted to confirm that's intended.

kib marked 2 inline comments as done.Sep 25 2022, 11:37 AM
kib added inline comments.
share/man/man4/snp.4
89

I implemented the detach and removed BUGS section.

sys/dev/snp/snp.c
309

Yes, it is intended.

kib updated this revision to Diff 110947.Sep 25 2022, 11:38 AM
kib marked 2 inline comments as done.
kib retitled this revision from snp(4): properly report detached/revoked ttys to snp(4): properly report detached/revoked ttys and implement detach.

Implement detach.
I changed my mind and swapped SNP_DETACH/SNP_TTYCLOSE cases.

markj added inline comments.Sep 25 2022, 11:49 AM
sys/dev/snp/snp.c
263

ttyhook_unregister() unlocks the tty.

imp added a comment.Sep 25 2022, 3:45 PM

Modulo the unlock mark found, I think this is good now.

sys/dev/snp/snp.c
263

indeed, it's hiding in tty_rel_free()... nice catch.

kib updated this revision to Diff 110952.Sep 25 2022, 5:54 PM

Fix locking bug

markj accepted this revision.Sep 25 2022, 6:22 PM
This revision is now accepted and ready to land.Sep 25 2022, 6:22 PM
imp accepted this revision.Sep 25 2022, 6:45 PM
Closed by commit rGbb4be76cf358: snp(4): properly report detached/revoked ttys (authored by kib). · Explain WhySep 25 2022, 8:37 PM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rGbb4be76cf358: snp(4): properly report detached/revoked ttys.
kib added a commit: rG550e01c55247: snp(4): implement detach.

Revision Contents
Changeset List

PathSize
share/
man/
man4/
12 lines
sys/
dev/
snp/
22 lines

Diff 110952

View Options

share/man/man4/snp.4

Loading...
View Options

sys/dev/snp/snp.c

Loading...