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)
Thu, Apr 18, 1:03 PM
Unknown Object (File)
Tue, Apr 16, 9:04 AM
Unknown Object (File)
Sun, Apr 7, 5:07 AM
Unknown Object (File)
Wed, Mar 27, 7:18 PM
Unknown Object (File)
Jan 10 2024, 2:48 AM
Unknown Object (File)
Dec 23 2023, 3:34 AM
Unknown Object (File)
Dec 12 2023, 3:34 PM
Unknown Object (File)
Dec 2 2023, 6:23 PM
View All 23 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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 87033

View Options

sys/arm64/arm64/vm_machdep.c

Loading...