syscalls: if ptrace(PT_SETREG) occured during syscall, do not do set_syscall_retval()
Needs ReviewPublic
Actions

Authored by kib on Oct 30 2025, 6:41 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Apr 7, 9:09 PM

	Unknown Object (File)
	Sat, Apr 4, 10:24 PM

	Unknown Object (File)
	Sat, Mar 28, 11:16 PM

	Unknown Object (File)
	Sun, Mar 22, 11:11 PM

	Unknown Object (File)
	Sun, Mar 22, 5:19 PM

	Unknown Object (File)
	Sun, Mar 22, 2:01 PM

	Unknown Object (File)
	Wed, Mar 18, 9:14 AM

	Unknown Object (File)
	Mar 3 2026, 5:17 AM

View All 67 Files

Subscribers

imp

olce

Details

Reviewers

andrew
markj
jhb

Commits

rGa6a9d97708ec: kern/subr_syscalls.c: declare 'traced' local variables boolean

Summary

to preserve the register state enforced by debugger.  This is only
relevant for transparent attach to the slow syscalls where sleeps are
not interrupted.

PR:     290008


kern/subr_syscalls.c: declare 'traced' local variables boolean

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

kib created this revision.Oct 30 2025, 6:41 PM

Herald added subscribers: olce, imp. · View Herald TranscriptOct 30 2025, 6:41 PM

That said, I disagree that this is a kernel bug at all.

markj added inline comments.Oct 31 2025, 12:42 PM

sys/kern/subr_syscall.c
119	Don't we need the proc lock here?

Clear TDB_USERWR under proc lock. Do it earlier.

I'm not sure if we should make such a change, but it looks ok to me.

This revision is now accepted and ready to land.Nov 2 2025, 5:48 PM

Closed by commit rGa6a9d97708ec: kern/subr_syscalls.c: declare 'traced' local variables boolean (authored by kib). · Explain WhyNov 2 2025, 9:22 PM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rGa6a9d97708ec: kern/subr_syscalls.c: declare 'traced' local variables boolean.

Rebase

Revision Contents
Changeset List

Path

Size

sys/

kern/

subr_syscall.c

14 lines

Diff 165700

View Options

syscalls: if ptrace(PT_SETREG) occured during syscall, do not do set_syscall_retval()Needs ReviewPublicActions