Paths

Table of Contentst

Differential D33703

exec: Simplify sv_copyout_strings implementations a bit
ClosedPublic
Actions

Authored by markj on Dec 30 2021, 10:41 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Apr 19, 4:46 PM

	Unknown Object (File)
	Sat, Apr 6, 2:06 AM

	Unknown Object (File)
	Mar 20 2024, 12:43 PM

	Unknown Object (File)
	Mar 2 2024, 2:24 AM

	Unknown Object (File)
	Feb 4 2024, 8:07 AM

	Unknown Object (File)
	Dec 20 2023, 8:26 AM

	Unknown Object (File)
	Dec 10 2023, 11:51 PM

	Unknown Object (File)
	Nov 28 2023, 9:50 AM

View All 57 Files

Subscribers

Details

Reviewers

mw
emaste
kib
dgr_semihalf.com
manu

Commits

rGf04a09604938: exec: Simplify sv_copyout_strings implementations a bit

Summary

Simplify control flow around handling of the execpath length and signal
trampoline. Cache the sysentvec pointer in a local variable.

No functional change intended.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 43653
Build 40541: arc lint + arc unit

Event Timeline

markj created this revision.Dec 30 2021, 10:41 PM

Herald added a reviewer: manu. · View Herald TranscriptDec 30 2021, 10:41 PM

Herald added subscribers: andrew, imp. · View Herald Transcript

markj requested review of this revision.Dec 30 2021, 10:41 PM

Harbormaster completed remote builds in B43653: Diff 100768.Dec 30 2021, 10:41 PM

markj added a child revision: D33704: Rework the stack gap implementation.Dec 30 2021, 10:41 PM

kib accepted this revision.Dec 31 2021, 12:40 AM

kib added inline comments.

sys/kern/kern_exec.c
1634	I suspect this code can be removed, but lets keep it for now. This variant is only for ABIs without shared page, and I think native FreeBSD ABI uses shared page on all arches.
1677	szps can be eliminated. Initialize imgp->pagesizelen in place of szps, and substract it from destp

This revision is now accepted and ready to land.Dec 31 2021, 12:40 AM

emaste added inline comments.Dec 31 2021, 1:07 AM

sys/kern/kern_exec.c
1634	That is these conditions could become `KASSERT`s in the future?

kib added inline comments.Dec 31 2021, 2:16 AM

sys/kern/kern_exec.c
1634	It would naturally traps on NULL access.

Eliminate szps.

This revision now requires review to proceed.Dec 31 2021, 3:21 AM

Harbormaster completed remote builds in B43657: Diff 100781.Dec 31 2021, 3:21 AM

kib accepted this revision.Dec 31 2021, 4:03 PM

This revision is now accepted and ready to land.Dec 31 2021, 4:03 PM

Closed by commit rGf04a09604938: exec: Simplify sv_copyout_strings implementations a bit (authored by markj). · Explain WhyDec 31 2021, 6:09 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rGf04a09604938: exec: Simplify sv_copyout_strings implementations a bit.

Revision Contents
Changeset List

Path

Size

sys/

amd64/

linux/

9 lines

linux32/

linux32_sysvec.c

9 lines

arm64/

linux/

9 lines

compat/

freebsd32/

freebsd32_misc.c

28 lines

i386/

linux/

8 lines

kern/

33 lines

Diff 100768

sys/amd64/linux/linux_sysvec.c

Loading...

sys/amd64/linux32/linux32_sysvec.c

Loading...

sys/arm64/linux/linux_sysvec.c

Loading...

sys/compat/freebsd32/freebsd32_misc.c

Loading...

sys/i386/linux/linux_sysvec.c

Loading...

sys/kern/kern_exec.c

Loading...