Page MenuHomeFreeBSD
Differential D29630

arm64: clear debug register state on fork
ClosedPublic
Actions

Authored by mhorne on Apr 7 2021, 7:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jul 4, 6:14 AM
Unknown Object (File)
Tue, Jun 30, 2:48 AM
Unknown Object (File)
Sun, Jun 28, 12:04 PM
Unknown Object (File)
Sat, Jun 27, 9:36 AM
Unknown Object (File)
Sat, Jun 27, 9:36 AM
Unknown Object (File)
Wed, Jun 24, 5:32 AM
Unknown Object (File)
Tue, Jun 23, 7:58 PM
Unknown Object (File)
Mon, Jun 22, 7:44 PM
View All Files
Subscribers
arichardson
imp
mgorny_gentoo.org

Details

Reviewers
andrew
emaste
kib
manu
Commits
rG1fd001db9c33: arm64: clear debug register state on fork
Summary

Following the analogous change for amd64 and i386 in 8223717ce62c,
ensure that new processes start with these registers inactive.

Reported by: Michał Górny

Test Plan

Check the result of PT_GETDBREGS on a grandchild process before and after this change.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 38385
Build 35274: arc lint + arc unit

Event Timeline

mhorne created this revision.Apr 7 2021, 7:34 PM
Herald added a reviewer: manu. · View Herald TranscriptApr 7 2021, 7:34 PM
Herald added a subscriber: imp. · View Herald Transcript
Harbormaster completed remote builds in B38385: Diff 86973.Apr 7 2021, 7:34 PM
mhorne requested review of this revision.Apr 7 2021, 7:34 PM
kib accepted this revision.Apr 7 2021, 9:13 PM
This revision is now accepted and ready to land.Apr 7 2021, 9:13 PM
emaste accepted this revision.Apr 7 2021, 9:39 PM
Closed by commit rG1fd001db9c33: arm64: clear debug register state on fork (authored by mhorne). · Explain WhyApr 8 2021, 12:43 PM
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: rG1fd001db9c33: arm64: clear debug register state on fork.
arichardson added a subscriber: arichardson.Apr 8 2021, 1:05 PM

Should they also be cleared in exec_setregs()? Or do we expect to keep them if they are changed after fork()?

mhorne added a comment.Apr 8 2021, 4:26 PM
In D29630#664646, @arichardson wrote:

Should they also be cleared in exec_setregs()? Or do we expect to keep them if they are changed after fork()?

Yes, we should clear them to be safe, since some settings will not apply to the new executable. I missed your recently added TODO comment there.

I'll post a new review shortly.

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
3 lines

Diff 86973

View Options

sys/arm64/arm64/vm_machdep.c

Loading...